Merge pull request #7255 from nextcloud/fix-missing-push-notifications

luka-nextcloud · web-flow · commit 094efdaa067a · 2025-09-25T11:16:21.000+02:00
fix: missing push notifications
diff --git a/lib/Notification/Notifier.php b/lib/Notification/Notifier.php
@@ -77,7 +77,7 @@ public function getName(): string {
 	 */
 	public function prepare(INotification $notification, string $languageCode): INotification {
 		$l = $this->l10nFactory->get('deck', $languageCode);
-		if ($notification->getApp() !== 'deck') {
+		if ($notification->getApp() !== 'deck' || $notification->getObjectType() === 'activity_notification') {
 			throw new UnknownNotificationException();
 		}
 		$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath('deck', 'deck-dark.svg')));
diff --git a/tests/integration/base-query-count.txt b/tests/integration/base-query-count.txt
@@ -1 +1 @@
-81373
+82773

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ public function getName(): string {`
`77`	`77`	`*/`
`78`	`78`	`public function prepare(INotification $notification, string $languageCode): INotification {`
`79`	`79`	`$l = $this->l10nFactory->get('deck', $languageCode);`
`80`		`- if ($notification->getApp() !== 'deck') {`
	`80`	`+ if ($notification->getApp() !== 'deck' \|\| $notification->getObjectType() === 'activity_notification') {`
`81`	`81`	`throw new UnknownNotificationException();`
`82`	`82`	`}`
`83`	`83`	`$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath('deck', 'deck-dark.svg')));`