fixup! fix(CapabilitiesManager): only log execution time if debug mode is enabled

Signed-off-by: Simon L. <szaimen@e.mail.de>
Co-Authored-By: Anna <anna@nextcloud.com>

Author: szaimen

lib/private/CapabilitiesManager.php

@@ -64,25 +64,23 @@ public function getCapabilities(bool $public = false, bool $initialState = false
 
 					// Only check execution time if debug mode is enabled
 					$debugMode = \OCP\Server::get(IConfig::class)->getSystemValueBool('debug', false);
-					if ($debugMode) {
-						$timeSpent = $endTime - $startTime;
-						if ($timeSpent > self::ACCEPTABLE_LOADING_TIME) {
-							$logLevel = match (true) {
-								$timeSpent > self::ACCEPTABLE_LOADING_TIME * 16 => ILogger::FATAL,
-								$timeSpent > self::ACCEPTABLE_LOADING_TIME * 8 => ILogger::ERROR,
-								$timeSpent > self::ACCEPTABLE_LOADING_TIME * 4 => ILogger::WARN,
-								$timeSpent > self::ACCEPTABLE_LOADING_TIME * 2 => ILogger::INFO,
-								default => ILogger::DEBUG,
-							};
-							$this->logger->log(
-								$logLevel,
-								'Capabilities of {className} took {duration} seconds to generate.',
-								[
-									'className' => get_class($c),
-									'duration' => round($timeSpent, 2),
-								]
-							);
-						}
+					$timeSpent = $endTime - $startTime;
+					if ($debugMode && $timeSpent > self::ACCEPTABLE_LOADING_TIME) {
+						$logLevel = match (true) {
+							$timeSpent > self::ACCEPTABLE_LOADING_TIME * 16 => ILogger::FATAL,
+							$timeSpent > self::ACCEPTABLE_LOADING_TIME * 8 => ILogger::ERROR,
+							$timeSpent > self::ACCEPTABLE_LOADING_TIME * 4 => ILogger::WARN,
+							$timeSpent > self::ACCEPTABLE_LOADING_TIME * 2 => ILogger::INFO,
+							default => ILogger::DEBUG,
+						};
+						$this->logger->log(
+							$logLevel,
+							'Capabilities of {className} took {duration} seconds to generate.',
+							[
+								'className' => get_class($c),
+								'duration' => round($timeSpent, 2),
+							]
+						);
 					}
 				}
 			} else {