@@ -15,7 +15,7 @@ discard block |
||
| 15 | 15 | use chillerlan\Database\Connection; |
| 16 | 16 | use chillerlan\Logger\LogOptions; |
| 17 | 17 | |
| 18 | -class DatabaseLog extends LogOutputAbstract{ |
|
| 18 | +class DatabaseLog extends LogOutputAbstract { |
|
| 19 | 19 | |
| 20 | 20 | /** |
| 21 | 21 | * @var \chillerlan\Database\Connection |
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | * @param \chillerlan\Logger\LogOptions $options |
| 30 | 30 | * @param \chillerlan\Database\Connection $db |
| 31 | 31 | */ |
| 32 | - public function __construct(LogOptions $options, Connection $db){ |
|
| 32 | + public function __construct(LogOptions $options, Connection $db) { |
|
| 33 | 33 | $this->options = $options; |
| 34 | 34 | $this->db = $db; |
| 35 | 35 | |
@@ -39,7 +39,7 @@ discard block |
||
| 39 | 39 | /** |
| 40 | 40 | * @inheritdoc |
| 41 | 41 | */ |
| 42 | - protected function __log(string $level, string $message, array $context = []){ |
|
| 42 | + protected function __log(string $level, string $message, array $context = []) { |
|
| 43 | 43 | |
| 44 | 44 | // TODO: $context, tests |
| 45 | 45 | $this->db->insert |
@@ -18,7 +18,7 @@ discard block |
||
| 18 | 18 | /** |
| 19 | 19 | * |
| 20 | 20 | */ |
| 21 | -abstract class LogOutputAbstract implements LogOutputInterface{ |
|
| 21 | +abstract class LogOutputAbstract implements LogOutputInterface { |
|
| 22 | 22 | |
| 23 | 23 | /** |
| 24 | 24 | * @var \chillerlan\Logger\LogOptions |
@@ -30,14 +30,14 @@ discard block |
||
| 30 | 30 | * |
| 31 | 31 | * @param \chillerlan\Traits\ContainerInterface|null $options |
| 32 | 32 | */ |
| 33 | - public function __construct(ContainerInterface $options = null){ |
|
| 33 | + public function __construct(ContainerInterface $options = null) { |
|
| 34 | 34 | $this->options = $options ?? new LogOptions; |
| 35 | 35 | } |
| 36 | 36 | |
| 37 | 37 | /** |
| 38 | 38 | * @inheritdoc |
| 39 | 39 | */ |
| 40 | - public function __destruct(){ |
|
| 40 | + public function __destruct() { |
|
| 41 | 41 | $this->close(); |
| 42 | 42 | } |
| 43 | 43 | |
@@ -60,8 +60,8 @@ discard block |
||
| 60 | 60 | /** |
| 61 | 61 | * @inheritdoc |
| 62 | 62 | */ |
| 63 | - public function log(string $level, string $message, array $context = []){ // @todo: loglevel bitmask |
|
| 64 | - if((array_key_exists($level, $this::LEVELS) && $this::LEVELS[$level] >= $this::LEVELS[$this->options->minLogLevel]) || (!array_key_exists($level, $this::LEVELS) && !empty($level))){ |
|
| 63 | + public function log(string $level, string $message, array $context = []) { // @todo: loglevel bitmask |
|
| 64 | + if ((array_key_exists($level, $this::LEVELS) && $this::LEVELS[$level] >= $this::LEVELS[$this->options->minLogLevel]) || (!array_key_exists($level, $this::LEVELS) && !empty($level))) { |
|
| 65 | 65 | $this->__log($level, $message, $context); |
| 66 | 66 | } |
| 67 | 67 | } |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | * |
| 74 | 74 | * @return string |
| 75 | 75 | */ |
| 76 | - protected function message(string $level, string $message, array $context = null){ |
|
| 76 | + protected function message(string $level, string $message, array $context = null) { |
|
| 77 | 77 | return sprintf($this->options->consoleFormat, date($this->options->consoleDateFormat), $level, $message).PHP_EOL; |
| 78 | 78 | } |
| 79 | 79 | |
@@ -14,12 +14,12 @@ |
||
| 14 | 14 | |
| 15 | 15 | /** |
| 16 | 16 | */ |
| 17 | -class ConsoleLog extends LogOutputAbstract{ |
|
| 17 | +class ConsoleLog extends LogOutputAbstract { |
|
| 18 | 18 | |
| 19 | 19 | /** |
| 20 | 20 | * @inheritdoc |
| 21 | 21 | */ |
| 22 | - protected function __log(string $level, string $message, array $context = null){ |
|
| 22 | + protected function __log(string $level, string $message, array $context = null) { |
|
| 23 | 23 | echo $this->message($level, $message, $context); |
| 24 | 24 | } |
| 25 | 25 | |
@@ -17,7 +17,7 @@ discard block |
||
| 17 | 17 | |
| 18 | 18 | /** |
| 19 | 19 | */ |
| 20 | -class Logfile extends LogOutputAbstract{ |
|
| 20 | +class Logfile extends LogOutputAbstract { |
|
| 21 | 21 | |
| 22 | 22 | /** |
| 23 | 23 | * @var bool|resource |
@@ -31,10 +31,10 @@ discard block |
||
| 31 | 31 | * |
| 32 | 32 | * @throws \chillerlan\Logger\LogException |
| 33 | 33 | */ |
| 34 | - public function __construct(ContainerInterface $options = null){ |
|
| 34 | + public function __construct(ContainerInterface $options = null) { |
|
| 35 | 35 | parent::__construct($options); |
| 36 | 36 | |
| 37 | - if(!is_writable($this->options->logfileDir)){ |
|
| 37 | + if (!is_writable($this->options->logfileDir)) { |
|
| 38 | 38 | throw new LogException('log file directory inaccessible'); |
| 39 | 39 | } |
| 40 | 40 | |
@@ -46,10 +46,10 @@ discard block |
||
| 46 | 46 | /** |
| 47 | 47 | * @inheritdoc |
| 48 | 48 | */ |
| 49 | - public function __destruct(){ |
|
| 49 | + public function __destruct() { |
|
| 50 | 50 | $this->close(); |
| 51 | 51 | |
| 52 | - if(gettype($this->fh) === 'resource'){ |
|
| 52 | + if (gettype($this->fh) === 'resource') { |
|
| 53 | 53 | fclose($this->fh); |
| 54 | 54 | } |
| 55 | 55 | } |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | /** |
| 58 | 58 | * @inheritdoc |
| 59 | 59 | */ |
| 60 | - protected function __log(string $level, string $message, array $context = null){ |
|
| 60 | + protected function __log(string $level, string $message, array $context = null) { |
|
| 61 | 61 | fwrite($this->fh, $this->message($level, $message, $context)); |
| 62 | 62 | } |
| 63 | 63 | |
@@ -23,6 +23,6 @@ |
||
| 23 | 23 | * @property string $logfileName |
| 24 | 24 | * @property string $logfileExt |
| 25 | 25 | */ |
| 26 | -class LogOptions extends ContainerAbstract{ |
|
| 26 | +class LogOptions extends ContainerAbstract { |
|
| 27 | 27 | use LogOptionsTrait; |
| 28 | 28 | } |
@@ -19,7 +19,7 @@ discard block |
||
| 19 | 19 | |
| 20 | 20 | /** |
| 21 | 21 | */ |
| 22 | -class Log implements LoggerInterface{ |
|
| 22 | +class Log implements LoggerInterface { |
|
| 23 | 23 | use LoggerTrait; |
| 24 | 24 | /** |
| 25 | 25 | * @var \chillerlan\Logger\LogOptions |
@@ -34,7 +34,7 @@ discard block |
||
| 34 | 34 | /** |
| 35 | 35 | * |
| 36 | 36 | */ |
| 37 | - public function __destruct(){ |
|
| 37 | + public function __destruct() { |
|
| 38 | 38 | $this->close(); |
| 39 | 39 | } |
| 40 | 40 | |
@@ -45,8 +45,8 @@ discard block |
||
| 45 | 45 | */ |
| 46 | 46 | public function close():Log{ |
| 47 | 47 | |
| 48 | - if(!empty($this->logOutputInterfaces)){ |
|
| 49 | - foreach($this->logOutputInterfaces as $instanceName => $instance){ |
|
| 48 | + if (!empty($this->logOutputInterfaces)) { |
|
| 49 | + foreach ($this->logOutputInterfaces as $instanceName => $instance) { |
|
| 50 | 50 | unset($this->logOutputInterfaces[$instanceName]); |
| 51 | 51 | } |
| 52 | 52 | } |
@@ -74,7 +74,7 @@ discard block |
||
| 74 | 74 | */ |
| 75 | 75 | public function getInstance(string $instanceName):LogOutputInterface{ |
| 76 | 76 | |
| 77 | - if(!array_key_exists($instanceName, $this->logOutputInterfaces)){ |
|
| 77 | + if (!array_key_exists($instanceName, $this->logOutputInterfaces)) { |
|
| 78 | 78 | throw new LogException('invalid logger instance: '.$instanceName); |
| 79 | 79 | } |
| 80 | 80 | |
@@ -89,7 +89,7 @@ discard block |
||
| 89 | 89 | */ |
| 90 | 90 | public function removeInstance(string $instanceName):Log{ |
| 91 | 91 | |
| 92 | - if(!array_key_exists($instanceName, $this->logOutputInterfaces)){ |
|
| 92 | + if (!array_key_exists($instanceName, $this->logOutputInterfaces)) { |
|
| 93 | 93 | throw new LogException('invalid logger instance: '.$instanceName); |
| 94 | 94 | } |
| 95 | 95 | |
@@ -109,9 +109,9 @@ discard block |
||
| 109 | 109 | * |
| 110 | 110 | * @return void |
| 111 | 111 | */ |
| 112 | - public function log($level, $message, array $context = [], string $instance = null){ |
|
| 112 | + public function log($level, $message, array $context = [], string $instance = null) { |
|
| 113 | 113 | |
| 114 | - foreach($this->logOutputInterfaces as $logger){ |
|
| 114 | + foreach ($this->logOutputInterfaces as $logger) { |
|
| 115 | 115 | $logger->log($level, $message, $context); |
| 116 | 116 | } |
| 117 | 117 | |