upstream/nextcloud
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-06-09 08:44:07 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

Merge pull request #44095 from nextcloud/fix/updatenotifications/numeric-ids-v28-and-lower

Browse source 
[stable28] fix(UpdateNotifications): Handle numeric user ids
This commit is contained in:
Joas Schilling 2024-03-08 23:14:43 +01:00 committed by GitHub
commit 996f3593ac
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
apps/updatenotification/lib/Notification/BackgroundJob.php
View file

@ -118,7 +118,7 @@ class BackgroundJob extends TimedJob {
->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 @@ class BackgroundJob extends TimedJob {
}
foreach ($this->getUsersToNotify() as $uid) {
$notification->setUser($uid);
$notification->setUser((string) $uid);
$this->notificationManager->notify($notification);
}
} catch (\InvalidArgumentException $e) {