From 1eb89421cce61a9a3d1b2c4c494fe6b3db16e681 Mon Sep 17 00:00:00 2001 From: Côme Chilliet Date: Thu, 8 Feb 2024 14:38:03 +0100 Subject: fix(cron.php): Avoid pulling configuration twice MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit IConfig is a small wrapper around SystemConfig, no need to pull both of them Signed-off-by: Côme Chilliet --- cron.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'cron.php') diff --git a/cron.php b/cron.php index e599f460c13..4e95481deb6 100644 --- a/cron.php +++ b/cron.php @@ -44,7 +44,6 @@ declare(strict_types=1); require_once __DIR__ . '/lib/versioncheck.php'; -use OC\SystemConfig; use OCP\App\IAppManager; use OCP\BackgroundJob\IJobList; use OCP\IAppConfig; @@ -62,13 +61,14 @@ try { Server::get(LoggerInterface::class)->debug('Update required, skipping cron', ['app' => 'cron']); exit; } - if ((bool) Server::get(SystemConfig::class)->getValue('maintenance', false)) { + + $config = Server::get(IConfig::class); + + if ($config->getSystemValueBool('maintenance', false)) { Server::get(LoggerInterface::class)->debug('We are in maintenance mode, skipping cron', ['app' => 'cron']); exit; } - $config = Server::get(IConfig::class); - // Don't do anything if Nextcloud has not been installed if (!$config->getSystemValueBool('installed', false)) { exit(0); -- cgit v1.2.3