fix(notifications): Make the notification icons absolute URLs so they… (

#268) … work in clients Signed-off-by: Joas Schilling <[email protected]>
nextcloud · Apr 9, 2024 · 5acfac7 · 5acfac7
1 parent 1df064b
commit 5acfac7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/Notifications/ExAppAdminNotifier.php b/lib/Notifications/ExAppAdminNotifier.php
@@ -44,7 +44,7 @@ public function prepare(INotification $notification, string $languageCode): INot
 		$parameters = $notification->getSubjectParameters();
 
 		$notification->setLink($this->url->getAbsoluteURL('/index.php/settings/admin/' . Application::APP_ID));
-		$notification->setIcon($this->url->imagePath(Application::APP_ID, 'app-dark.svg'));
+		$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath(Application::APP_ID, 'app-dark.svg')));
 
 		if (isset($parameters['rich_subject']) && isset($parameters['rich_subject_params'])) {
 			$notification->setRichSubject($parameters['rich_subject'], $parameters['rich_subject_params']);

diff --git a/lib/Notifications/ExAppNotifier.php b/lib/Notifications/ExAppNotifier.php
@@ -47,7 +47,7 @@ public function prepare(INotification $notification, string $languageCode): INot
 		if (isset($parameters['link']) && $parameters['link'] !== '') {
 			$notification->setLink($parameters['link']);
 		}
-		$notification->setIcon($this->url->imagePath(Application::APP_ID, 'app-dark.svg'));
+		$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath(Application::APP_ID, 'app-dark.svg')));
 
 		if (isset($parameters['rich_subject']) && isset($parameters['rich_subject_params'])) {
 			$notification->setRichSubject($l->t($parameters['rich_subject']), $parameters['rich_subject_params']);