diff options
| author | Andy Scherzinger <info@andy-scherzinger.de> | 2025-04-28 14:51:25 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-04-28 14:51:25 +0200 |
| commit | 0d19a1333fba5e9109f26b2f073fe2dd199d38fb (patch) | |
| tree | e2259602b92b8a36cb1ebbc2488a64b19f07563c | |
| parent | 1cb1f39279cea852f04b3611265fea617be6cc4d (diff) | |
| parent | 3d36625af6e9311cc62a2d6d41b0aa86d93d93e3 (diff) | |
| download | nextcloud-server-0d19a1333fba5e9109f26b2f073fe2dd199d38fb.tar.gz nextcloud-server-0d19a1333fba5e9109f26b2f073fe2dd199d38fb.zip | |
Merge pull request #52443 from nextcloud/reminder-dont-validiate-node-dav
fix: don't check node access again for listing reminders in dav
| -rw-r--r-- | apps/files_reminders/lib/Dav/PropFindPlugin.php | 2 | ||||
| -rw-r--r-- | apps/files_reminders/lib/Service/ReminderService.php | 6 |
2 files changed, 5 insertions, 3 deletions
diff --git a/apps/files_reminders/lib/Dav/PropFindPlugin.php b/apps/files_reminders/lib/Dav/PropFindPlugin.php index 0f3b53c4861..014e636eb2d 100644 --- a/apps/files_reminders/lib/Dav/PropFindPlugin.php +++ b/apps/files_reminders/lib/Dav/PropFindPlugin.php @@ -62,7 +62,7 @@ class PropFindPlugin extends ServerPlugin { } $fileId = $node->getId(); - $reminder = $this->reminderService->getDueForUser($user, $fileId); + $reminder = $this->reminderService->getDueForUser($user, $fileId, false); if ($reminder === null) { return ''; } diff --git a/apps/files_reminders/lib/Service/ReminderService.php b/apps/files_reminders/lib/Service/ReminderService.php index 45b89a272a3..6ee39562076 100644 --- a/apps/files_reminders/lib/Service/ReminderService.php +++ b/apps/files_reminders/lib/Service/ReminderService.php @@ -64,8 +64,10 @@ class ReminderService { /** * @throws NodeNotFoundException */ - public function getDueForUser(IUser $user, int $fileId): ?RichReminder { - $this->checkNode($user, $fileId); + public function getDueForUser(IUser $user, int $fileId, bool $checkNode = true): ?RichReminder { + if ($checkNode) { + $this->checkNode($user, $fileId); + } /** @var null|false|Reminder $cachedReminder */ $cachedReminder = $this->cache->get("{$user->getUID()}-$fileId"); if ($cachedReminder === false) { |