Always register the Notifier

... so background jobs without a user can prepare push notifications

Signed-off-by: Joas Schilling <coding@schilljs.com>

1 files changed, 5 insertions, 5 deletions

diff --git a/apps/updatenotification/lib/AppInfo/Application.php b/apps/updatenotification/lib/AppInfo/Application.php
index 369b6e3f07d..14512bae838 100644
--- a/apps/updatenotification/lib/AppInfo/Application.php
+++ b/apps/updatenotification/lib/AppInfo/Application.php
@@ -44,17 +44,17 @@ class Application extends App {
 			return;
 		}
 
+		// Always register the notifier, so background jobs (without a user) can send push notifications
+		$this->registerNotifier();
+
 		$user = $server->getUserSession()->getUser();
 		if (!$user instanceof IUser) {
 			// Nothing to do for guests
 			return;
 		}
 
-		if ($server->getAppManager()->isEnabledForUser('notifications')) {
-			// Notifications app is available, so we register.
-			// Since notifications also work for non-admins we don't check this here.
-			$this->registerNotifier();
-		} else if ($server->getGroupManager()->isAdmin($user->getUID())) {
+		if (!$server->getAppManager()->isEnabledForUser('notifications') &&
+			$server->getGroupManager()->isAdmin($user->getUID())) {
 			try {
 				$updateChecker = $this->getContainer()->query(UpdateChecker::class);
 			} catch (QueryException $e) {