diff options
author | Christopher Ng <chrng8@gmail.com> | 2023-07-31 18:40:13 -0700 |
---|---|---|
committer | Christopher Ng <chrng8@gmail.com> | 2023-08-03 15:30:11 -0700 |
commit | 9e8354e0ac35090995a0451ccd1166f178815ea1 (patch) | |
tree | dc707b921fa6edbfe9bc3882fd786dd36c497c22 /apps/files_reminders/lib/Notification | |
parent | f865c3ad21e03483a6f16b90a4383faef0356499 (diff) | |
download | nextcloud-server-9e8354e0ac35090995a0451ccd1166f178815ea1.tar.gz nextcloud-server-9e8354e0ac35090995a0451ccd1166f178815ea1.zip |
fix: exit on reminder not found
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Diffstat (limited to 'apps/files_reminders/lib/Notification')
-rw-r--r-- | apps/files_reminders/lib/Notification/Notifier.php | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/apps/files_reminders/lib/Notification/Notifier.php b/apps/files_reminders/lib/Notification/Notifier.php index fb1d3264dbd..c7384b43224 100644 --- a/apps/files_reminders/lib/Notification/Notifier.php +++ b/apps/files_reminders/lib/Notification/Notifier.php @@ -30,6 +30,7 @@ use InvalidArgumentException; use OCA\FilesReminders\AppInfo\Application; use OCA\FilesReminders\Exception\NodeNotFoundException; use OCA\FilesReminders\Service\ReminderService; +use OCP\AppFramework\Db\DoesNotExistException; use OCP\Files\FileInfo; use OCP\IURLGenerator; use OCP\L10N\IFactory; @@ -66,8 +67,13 @@ class Notifier implements INotifier { switch ($notification->getSubject()) { case 'reminder-due': $reminderId = (int)$notification->getObjectId(); - $node = $this->reminderService->get($reminderId)->getNode(); + try { + $reminder = $this->reminderService->get($reminderId); + } catch (DoesNotExistException $e) { + throw new InvalidArgumentException(); + } + $node = $reminder->getNode(); $path = rtrim($node->getPath(), '/'); if (strpos($path, '/' . $notification->getUser() . '/files/') === 0) { // Remove /user/files/... |