[stable28] fix(UpdateNotifications): Handle numeric user ids

manual backport of #44093 to fix #44051 for <=v28 Signed-off-by: Josh <josh.t.richards@gmail.com>
nextcloud · Mar 8, 2024 · 4fedc74 · 4fedc74
1 parent e5b2c0f
commit 4fedc74
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/apps/updatenotification/lib/Notification/BackgroundJob.php b/apps/updatenotification/lib/Notification/BackgroundJob.php
@@ -118,7 +118,7 @@ protected function sendErrorNotifications($numDays) {
 				->setSubject('connection_error', ['days' => $numDays]);
 
 			foreach ($this->getUsersToNotify() as $uid) {
-				$notification->setUser($uid);
+				$notification->setUser((string) $uid);
 				$this->notificationManager->notify($notification);
 			}
 		} catch (\InvalidArgumentException $e) {
@@ -186,7 +186,7 @@ protected function createNotifications($app, $version, $visibleVersion = '') {
 			}
 
 			foreach ($this->getUsersToNotify() as $uid) {
-				$notification->setUser($uid);
+				$notification->setUser((string) $uid);
 				$this->notificationManager->notify($notification);
 			}
 		} catch (\InvalidArgumentException $e) {