From 89ea220f5e6e909b93f3a12d3728f21df7ece996 Mon Sep 17 00:00:00 2001
From: fiftydinar <65243233+fiftydinar@users.noreply.github.com>
Date: Wed, 23 Oct 2024 11:42:06 +0200
Subject: [PATCH] chore(default-flatpaks): No need to query `DISPLAY` variable
 for `notify-send`

Notifications work without it & custom images without XWayland don't have `DISPLAY` variable at all (eg. secureblue & custom images based on it)
---
 modules/default-flatpaks/v1/system-flatpak-setup | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/modules/default-flatpaks/v1/system-flatpak-setup b/modules/default-flatpaks/v1/system-flatpak-setup
index e43c4b3..478c6e9 100755
--- a/modules/default-flatpaks/v1/system-flatpak-setup
+++ b/modules/default-flatpaks/v1/system-flatpak-setup
@@ -157,32 +157,28 @@ function notify-send-pre-install {
      user_name=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .user')
      uid=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .uid')
      xdg_runtime_path="/run/user/$uid"
-     display_var=$(printenv DISPLAY)
-     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus DISPLAY="$display_var" notify-send "Flatpak Installer" "Started install of system flatpaks" --app-name="Flatpak Installer" -u NORMAL
+     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus notify-send "Flatpak Installer" "Started install of system flatpaks" --app-name="Flatpak Installer" -u NORMAL
 }
 
 function notify-send-install {
      user_name=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .user')
      uid=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .uid')
      xdg_runtime_path="/run/user/$uid"
-     display_var=$(printenv DISPLAY)
-     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus DISPLAY="$display_var" notify-send "Flatpak Installer" "Finished install of system flatpaks:\n$INSTALL_LIST" --app-name="Flatpak Installer" -u NORMAL
+     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus notify-send "Flatpak Installer" "Finished install of system flatpaks:\n$INSTALL_LIST" --app-name="Flatpak Installer" -u NORMAL
 }
 
 function notify-send-pre-uninstall {
      user_name=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .user')
      uid=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .uid')
      xdg_runtime_path="/run/user/$uid"
-     display_var=$(printenv DISPLAY)      
-     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus DISPLAY="$display_var" notify-send "Flatpak Installer" "Started uninstall of some system flatpaks" --app-name="Flatpak Installer" -u NORMAL
+     sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus notify-send "Flatpak Installer" "Started uninstall of some system flatpaks" --app-name="Flatpak Installer" -u NORMAL
 }
 
 function notify-send-uninstall {
       user_name=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .user')
       uid=$(loginctl list-users --output=json | jq -r '.[] | select(.state == "active") | .uid')
       xdg_runtime_path="/run/user/$uid"
-      display_var=$(printenv DISPLAY)      
-      sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus DISPLAY="$display_var" notify-send "Flatpak Installer" "Finished uninstall of system flatpaks:\n$REMOVE_LIST" --app-name="Flatpak Installer" -u NORMAL
+      sudo -u "$user_name" DBUS_SESSION_BUS_ADDRESS=unix:path="$xdg_runtime_path"/bus notify-send "Flatpak Installer" "Finished uninstall of system flatpaks:\n$REMOVE_LIST" --app-name="Flatpak Installer" -u NORMAL
 }
 
 # Install flatpaks in list