From d3068f5db54da56692133e3ee6fe7bc9184681ce Mon Sep 17 00:00:00 2001 From: Thomas Citharel Date: Wed, 20 Nov 2024 10:05:13 +0100 Subject: feat(dav): increase default calendar subscription refresh rate to one day With the performance benefits from #43541 it makes sense Reference https://github.com/nextcloud/server/issues/46171#issuecomment-2487910923 Signed-off-by: Thomas Citharel --- apps/dav/lib/BackgroundJob/RefreshWebcalJob.php | 4 ++-- apps/dav/tests/unit/BackgroundJob/RefreshWebcalJobTest.php | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/apps/dav/lib/BackgroundJob/RefreshWebcalJob.php b/apps/dav/lib/BackgroundJob/RefreshWebcalJob.php index f9d6de9db59..e96735ca50b 100644 --- a/apps/dav/lib/BackgroundJob/RefreshWebcalJob.php +++ b/apps/dav/lib/BackgroundJob/RefreshWebcalJob.php @@ -41,8 +41,8 @@ class RefreshWebcalJob extends Job { $this->fixSubscriptionRowTyping($subscription); - // if no refresh rate was configured, just refresh once a week - $defaultRefreshRate = $this->config->getAppValue('dav', 'calendarSubscriptionRefreshRate', 'P1W'); + // if no refresh rate was configured, just refresh once a day + $defaultRefreshRate = $this->config->getAppValue('dav', 'calendarSubscriptionRefreshRate', 'P1D'); $refreshRate = $subscription[RefreshWebcalService::REFRESH_RATE] ?? $defaultRefreshRate; $subscriptionId = $subscription['id']; diff --git a/apps/dav/tests/unit/BackgroundJob/RefreshWebcalJobTest.php b/apps/dav/tests/unit/BackgroundJob/RefreshWebcalJobTest.php index f97626a6a73..2b11223210e 100644 --- a/apps/dav/tests/unit/BackgroundJob/RefreshWebcalJobTest.php +++ b/apps/dav/tests/unit/BackgroundJob/RefreshWebcalJobTest.php @@ -78,7 +78,7 @@ class RefreshWebcalJobTest extends TestCase { $this->config->expects($this->once()) ->method('getAppValue') - ->with('dav', 'calendarSubscriptionRefreshRate', 'P1W') + ->with('dav', 'calendarSubscriptionRefreshRate', 'P1D') ->willReturn('P1W'); $this->timeFactory->method('getTime') -- cgit v1.2.3