Merge pull request #614 from nextcloud/fix/notifier-logs

nickvergessen · web-flow · commit 74eee63fafca · 2025-10-06T16:41:48.000+02:00
fix(ExAppNotifier): First check if any exapps enabled
diff --git a/lib/Notifications/ExAppNotifier.php b/lib/Notifications/ExAppNotifier.php
@@ -36,6 +36,9 @@ public function getName(): string {
 	}
 
 	public function prepare(INotification $notification, string $languageCode): INotification {
+		if (empty($this->service->getExAppsList())) {
+			throw new UnknownNotificationException();
+		}
 		$exApp = $this->service->getExApp($notification->getApp());
 		if ($exApp === null) {
 			throw new UnknownNotificationException();

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,9 @@ public function getName(): string {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`public function prepare(INotification $notification, string $languageCode): INotification {`
	`39`	`+ if (empty($this->service->getExAppsList())) {`
	`40`	`+ throw new UnknownNotificationException();`
	`41`	`+ }`
`39`	`42`	`$exApp = $this->service->getExApp($notification->getApp());`
`40`	`43`	`if ($exApp === null) {`
`41`	`44`	`throw new UnknownNotificationException();`