- added type-hints
- added missing documentation
- renamed Logger::getWorker() to Logger::getLogger() as there is no worker class
  returned but the actual (inner) logger
This commit is contained in:
Roland Häder 2022-06-21 18:40:26 +02:00
parent 1c57ea7f75
commit cdcfb4064b
No known key found for this signature in database
GPG Key ID: C82EDE5DDFA0BA77
2 changed files with 43 additions and 46 deletions

View File

@ -47,7 +47,7 @@ class Logger
/** /**
* @return LoggerInterface * @return LoggerInterface
*/ */
private static function getWorker() private static function getLogger()
{ {
if (self::$type === self::TYPE_LOGGER) { if (self::$type === self::TYPE_LOGGER) {
return DI::logger(); return DI::logger();
@ -66,7 +66,7 @@ class Logger
public static function enableWorker(string $functionName) public static function enableWorker(string $functionName)
{ {
self::$type = self::TYPE_WORKER; self::$type = self::TYPE_WORKER;
self::getWorker()->setFunctionName($functionName); self::getLogger()->setFunctionName($functionName);
} }
/** /**
@ -82,15 +82,14 @@ class Logger
* *
* @see LoggerInterface::emergency() * @see LoggerInterface::emergency()
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function emergency($message, $context = []) public static function emergency(string $message, array $context = [])
{ {
self::getWorker()->emergency($message, $context); self::getLogger()->emergency($message, $context);
} }
/** /**
@ -100,15 +99,14 @@ class Logger
* Example: Entire website down, database unavailable, etc. This should * Example: Entire website down, database unavailable, etc. This should
* trigger the SMS alerts and wake you up. * trigger the SMS alerts and wake you up.
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function alert($message, $context = []) public static function alert(string $message, array $context = [])
{ {
self::getWorker()->alert($message, $context); self::getLogger()->alert($message, $context);
} }
/** /**
@ -117,15 +115,14 @@ class Logger
* *
* Example: Application component unavailable, unexpected exception. * Example: Application component unavailable, unexpected exception.
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function critical($message, $context = []) public static function critical(string $message, array $context = [])
{ {
self::getWorker()->critical($message, $context); self::getLogger()->critical($message, $context);
} }
/** /**
@ -133,15 +130,14 @@ class Logger
* be logged and monitored. * be logged and monitored.
* @see LoggerInterface::error() * @see LoggerInterface::error()
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function error($message, $context = []) public static function error(string $message, array $context = [])
{ {
self::getWorker()->error($message, $context); self::getLogger()->error($message, $context);
} }
/** /**
@ -151,30 +147,28 @@ class Logger
* Example: Use of deprecated APIs, poor use of an API, undesirable things * Example: Use of deprecated APIs, poor use of an API, undesirable things
* that are not necessarily wrong. * that are not necessarily wrong.
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function warning($message, $context = []) public static function warning(string $message, array $context = [])
{ {
self::getWorker()->warning($message, $context); self::getLogger()->warning($message, $context);
} }
/** /**
* Normal but significant events. * Normal but significant events.
* @see LoggerInterface::notice() * @see LoggerInterface::notice()
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function notice($message, $context = []) public static function notice(string $message, array $context = [])
{ {
self::getWorker()->notice($message, $context); self::getLogger()->notice($message, $context);
} }
/** /**
@ -189,24 +183,23 @@ class Logger
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function info($message, $context = []) public static function info(string $message, array $context = [])
{ {
self::getWorker()->info($message, $context); self::getLogger()->info($message, $context);
} }
/** /**
* Detailed debug information. * Detailed debug information.
* @see LoggerInterface::debug() * @see LoggerInterface::debug()
* *
* @param string $message * @param string $message Message to log
* @param array $context * @param array $context Optional variables
*
* @return void * @return void
* @throws \Exception * @throws \Exception
*/ */
public static function debug($message, $context = []) public static function debug(string $message, array $context = [])
{ {
self::getWorker()->debug($message, $context); self::getLogger()->debug($message, $context);
} }
/** /**
@ -216,12 +209,13 @@ class Logger
* to isolate particular elements they are targetting * to isolate particular elements they are targetting
* personally without background noise * personally without background noise
* *
* @param string $msg * @param string $message Message to log
* @param string $level * @param string $level Logging level
* @return void
* @throws \Exception * @throws \Exception
*/ */
public static function devLog($msg, $level = LogLevel::DEBUG) public static function devLog(string $message, string $level = LogLevel::DEBUG)
{ {
DI::devLogger()->log($level, $msg); DI::devLogger()->log($level, $message);
} }
} }

View File

@ -129,6 +129,9 @@ class StaticDatabase extends Database
*/ */
public static function statConnect(array $server) public static function statConnect(array $server)
{ {
// Init variables
$db_host = $db_user = $db_data = $db_pw = '';
// Use environment variables for mysql if they are set beforehand // Use environment variables for mysql if they are set beforehand
if (!empty($server['MYSQL_HOST']) if (!empty($server['MYSQL_HOST'])
&& (!empty($server['MYSQL_USERNAME']) || !empty($server['MYSQL_USER'])) && (!empty($server['MYSQL_USERNAME']) || !empty($server['MYSQL_USER']))
@ -158,14 +161,14 @@ class StaticDatabase extends Database
$serverdata = explode(':', $serveraddr); $serverdata = explode(':', $serveraddr);
$server = $serverdata[0]; $server = $serverdata[0];
if (count($serverdata) > 1) { if (count($serverdata) > 1) {
$port = trim($serverdata[1]); $port = (int) trim($serverdata[1]);
} }
$server = trim($server); $server = trim($server);
$user = trim($db_user); $user = trim($db_user);
$pass = trim($db_pw ?? ''); $pass = trim($db_pw);
$db = trim($db_data); $db = trim($db_data);
if (!(strlen($server) && strlen($user))) { if (!(strlen($server) && strlen($user) && strlen($db))) {
return; return;
} }