diff options
| author | Roeland Jago Douma <rullzer@users.noreply.github.com> | 2020-06-05 09:02:01 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-05 09:02:01 +0200 |
| commit | 404223686708745ee4aad49de1224c69c2d2cbe2 (patch) | |
| tree | 2bd573923400accedf74e394789fb760747c211a | |
| parent | a5232d9805eb07c43d21aebcd11fea6dcde7e8b1 (diff) | |
| parent | b61b74e81af2677aa9ad037ba17d02644891fd34 (diff) | |
| download | nextcloud-server-404223686708745ee4aad49de1224c69c2d2cbe2.tar.gz nextcloud-server-404223686708745ee4aad49de1224c69c2d2cbe2.zip | |
Merge pull request #21229 from nextcloud/fix-empty-event-reminder-notifications
Fix empty event UUID reminder notifications
| -rw-r--r-- | apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php b/apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php index c6b796b2acb..688e5c3b106 100644 --- a/apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php +++ b/apps/dav/lib/CalDAV/Reminder/NotificationProvider/PushProvider.php @@ -95,8 +95,10 @@ class PushProvider extends AbstractProvider { $eventDetails = $this->extractEventDetails($vevent); $eventDetails['calendar_displayname'] = $calendarDisplayName; $eventUUID = (string) $vevent->UID; - // Empty Notification ObjectId will be catched by OC\Notification\Notification - $eventUUIDHash = $eventUUID ? hash('sha256', $eventUUID, false) : ''; + if (!$eventUUID) { + return; + }; + $eventUUIDHash = hash('sha256', $eventUUID, false); foreach ($users as $user) { /** @var INotification $notification */ |