mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #14744 from nextcloud/bugfix/noid/fix-push-notifications-for-updatenotifications 

Always register the Notifier
This commit is contained in:
Christoph Wurst 2019-03-19 16:42:54 +01:00 committed by GitHub
parent 04bc272c72 5eae69bd2d
commit af03d3a491
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
apps/updatenotification/lib/AppInfo/Application.php
View File

@ -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) {