diff --git a/apps/dav/tests/unit/Connector/Sabre/ExceptionLoggerPluginTest.php b/apps/dav/tests/unit/Connector/Sabre/ExceptionLoggerPluginTest.php index 9cfb2c465e..ba28c39e09 100644 --- a/apps/dav/tests/unit/Connector/Sabre/ExceptionLoggerPluginTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/ExceptionLoggerPluginTest.php @@ -41,7 +41,7 @@ class TestLogger extends Log { //disable original constructor } - public function log($level, $message, array $context = array()) { + public function log($level, string $message, array $context = array()) { $this->level = $level; $this->message = $message; } diff --git a/lib/private/Log.php b/lib/private/Log.php index bed0321bef..d59d8b5ff0 100644 --- a/lib/private/Log.php +++ b/lib/private/Log.php @@ -1,4 +1,5 @@ config->getValue('log_type', 'file'); $this->logger = static::getLogClass($logType); - call_user_func(array($this->logger, 'init')); + call_user_func([$this->logger, 'init']); } else { $this->logger = $logger; } @@ -148,7 +149,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function emergency($message, array $context = array()) { + public function emergency(string $message, array $context = []) { $this->log(Util::FATAL, $message, $context); } @@ -162,7 +163,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function alert($message, array $context = array()) { + public function alert(string $message, array $context = []) { $this->log(Util::ERROR, $message, $context); } @@ -175,7 +176,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function critical($message, array $context = array()) { + public function critical(string $message, array $context = []) { $this->log(Util::ERROR, $message, $context); } @@ -187,7 +188,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function error($message, array $context = array()) { + public function error(string $message, array $context = []) { $this->log(Util::ERROR, $message, $context); } @@ -201,7 +202,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function warning($message, array $context = array()) { + public function warning(string $message, array $context = []) { $this->log(Util::WARN, $message, $context); } @@ -212,7 +213,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function notice($message, array $context = array()) { + public function notice(string $message, array $context = []) { $this->log(Util::INFO, $message, $context); } @@ -225,7 +226,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function info($message, array $context = array()) { + public function info(string $message, array $context = []) { $this->log(Util::INFO, $message, $context); } @@ -236,7 +237,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function debug($message, array $context = array()) { + public function debug(string $message, array $context = []) { $this->log(Util::DEBUG, $message, $context); } @@ -249,7 +250,7 @@ class Log implements ILogger { * @param array $context * @return void */ - public function log($level, $message, array $context = array()) { + public function log($level, string $message, array $context = []) { $minLevel = min($this->config->getValue('loglevel', Util::WARN), Util::FATAL); $logCondition = $this->config->getValue('log.condition', []); @@ -272,7 +273,7 @@ class Log implements ILogger { $app = 'no app in context'; } // interpolate $message as defined in PSR-3 - $replace = array(); + $replace = []; foreach ($context as $key => $val) { $replace['{' . $key . '}'] = $val; } @@ -318,7 +319,7 @@ class Log implements ILogger { if ($level >= $minLevel) { $logger = $this->logger; - call_user_func(array($logger, 'write'), $app, $message, $level); + call_user_func([$logger, 'write'], $app, $message, $level); } } @@ -330,20 +331,20 @@ class Log implements ILogger { * @return void * @since 8.2.0 */ - public function logException($exception, array $context = array()) { + public function logException($exception, array $context = []) { $level = Util::ERROR; if (isset($context['level'])) { $level = $context['level']; unset($context['level']); } - $data = array( + $data = [ 'Exception' => get_class($exception), 'Message' => $exception->getMessage(), 'Code' => $exception->getCode(), 'Trace' => $exception->getTraceAsString(), 'File' => $exception->getFile(), 'Line' => $exception->getLine(), - ); + ]; $data['Trace'] = preg_replace('!(' . implode('|', $this->methodsWithSensitiveParameters) . ')\(.*\)!', '$1(*** sensitive parameters replaced ***)', $data['Trace']); if ($exception instanceof HintException) { $data['Hint'] = $exception->getHint(); @@ -362,7 +363,7 @@ class Log implements ILogger { * @return string * @internal */ - public static function getLogClass($logType) { + public static function getLogClass(string $logType): string { switch (strtolower($logType)) { case 'errorlog': return \OC\Log\Errorlog::class; diff --git a/lib/public/ILogger.php b/lib/public/ILogger.php index 6ee850ee8a..2d44244075 100644 --- a/lib/public/ILogger.php +++ b/lib/public/ILogger.php @@ -1,4 +1,5 @@