2 files changed, 4 insertions, 6 deletions
diff --git a/apps/updatenotification/lib/BackgroundJob/ResetToken.php b/apps/updatenotification/lib/BackgroundJob/ResetToken.php
index 856e179814b..05f8ea639a8 100644
--- a/apps/updatenotification/lib/BackgroundJob/ResetToken.php
+++ b/apps/updatenotification/lib/BackgroundJob/ResetToken.php
@@ -55,7 +55,7 @@ class ResetToken extends TimedJob {
 	 * @param $argument
 	 */
 	protected function run($argument) {
-		if ($this->config->getSystemValueBool('config_is_read_only') !== false) {
+		if ($this->config->getSystemValueBool('config_is_read_only')) {
 			return;
 		}
 
diff --git a/apps/updatenotification/lib/BackgroundJob/UpdateAvailableNotifications.php b/apps/updatenotification/lib/BackgroundJob/UpdateAvailableNotifications.php
index 6afe1ec5e37..6e7cc7928ab 100644
--- a/apps/updatenotification/lib/BackgroundJob/UpdateAvailableNotifications.php
+++ b/apps/updatenotification/lib/BackgroundJob/UpdateAvailableNotifications.php
@@ -171,7 +171,7 @@ class UpdateAvailableNotifications extends TimedJob {
 			return;
 		}
 
-		if ($lastNotification !== false) {
+		if ($lastNotification !== '') {
 			// Delete old updates
 			$this->deleteOutdatedNotifications($app, $lastNotification);
 		}
@@ -213,14 +213,12 @@ class UpdateAvailableNotifications extends TimedJob {
 			$groupToNotify = $this->groupManager->get($group);
 			if ($groupToNotify instanceof IGroup) {
 				foreach ($groupToNotify->getUsers() as $user) {
-					$this->users[$user->getUID()] = true;
+					$this->users[] = $user->getUID();
 				}
 			}
 		}
 
-		$this->users = array_keys($this->users);
-
-		return $this->users;
+		return array_unique($this->users);
 	}
 
 	/**