formatted

Author: phossa

.codeclimate.yml

@@ -4,8 +4,8 @@ engines:
     enabled: true
     config:
       languages:
-      - javascript
-      - php
+        - javascript
+        - php
   eslint:
     enabled: true
   fixme:
@@ -23,8 +23,8 @@ engines:
         enabled: false
 ratings:
   paths:
-  - "**.js"
-  - "**.php"
+    - "**.js"
+    - "**.php"
 exclude_paths:
   - tests/
   - build/

.scrutinizer.yml

@@ -1,18 +1,18 @@
 filter:
-    excluded_paths: [tests/*]
+  excluded_paths: [tests/*]
 
 checks:
-    php:
-        code_rating: true
-        remove_extra_empty_lines: true
-        remove_php_closing_tag: true
-        remove_trailing_whitespace: true
-        fix_use_statements:
-            remove_unused: true
-            preserve_multiple: false
-            preserve_blanklines: true
-        fix_php_opening_tag: true
-        fix_linefeed: true
-        fix_line_ending: true
-        fix_identation_4spaces: true
-        fix_doc_comments: true
+  php:
+    code_rating: true
+    remove_extra_empty_lines: true
+    remove_php_closing_tag: true
+    remove_trailing_whitespace: true
+    fix_use_statements:
+      remove_unused: true
+      preserve_multiple: false
+      preserve_blanklines: true
+    fix_php_opening_tag: true
+    fix_linefeed: true
+    fix_line_ending: true
+    fix_identation_4spaces: true
+    fix_doc_comments: true

composer.json

@@ -1,41 +1,48 @@
 {
-    "name": "phoole/logger",
-    "type": "library",
-    "license": "Apache-2.0",
-    "version": "1.0.3",
-    "description": "Simple & slim PSR-3 logger for PHP",
-    "keywords": ["phoole", "library", "php", "logger", "psr-3", "swoole"],
-    "authors": [
-        {
-            "name": "Hong Zhang",
-            "homepage": "https://github.com/phoole"
-        }
-    ],
-    "require": {
-        "php": ">=7.2.0",
-        "phoole/base": "^1.0.20",
-        "psr/log": "^1.0.1"
-    },
-    "require-dev": {
-        "phpunit/phpunit": "^8"
-    },
-    "autoload": {
-        "psr-4": {
-            "Phoole\\Logger\\": "src/"
-        }
-    },
-    "autoload-dev": {
-        "psr-4": {
-            "Phoole\\Tests\\": "tests/"
-        }
-    },
-    "provide": {
-        "psr/log-implementation": "1.0.0"
-    },
-    "minimum-stability": "dev",
-    "scripts": {
-        "test": "phpunit",
-        "check": "phpcs --standard=psr12 src/",
-        "format": "phpcbf --standard=psr12 src/"
+  "name": "phoole/logger",
+  "type": "library",
+  "license": "Apache-2.0",
+  "version": "1.0.5",
+  "description": "Simple & slim PSR-3 logger for PHP",
+  "keywords": [
+    "phoole",
+    "library",
+    "php",
+    "logger",
+    "psr-3",
+    "swoole"
+  ],
+  "authors": [
+    {
+      "name": "Hong Zhang",
+      "homepage": "https://github.com/phoole"
     }
+  ],
+  "require": {
+    "php": ">=7.2.0",
+    "phoole/base": "^1.0.20",
+    "psr/log": "^1.0.1"
+  },
+  "require-dev": {
+    "phpunit/phpunit": "^8"
+  },
+  "autoload": {
+    "psr-4": {
+      "Phoole\\Logger\\": "src/"
+    }
+  },
+  "autoload-dev": {
+    "psr-4": {
+      "Phoole\\Tests\\": "tests/"
+    }
+  },
+  "provide": {
+    "psr/log-implementation": "1.0.0"
+  },
+  "minimum-stability": "dev",
+  "scripts": {
+    "test": "phpunit",
+    "check": "phpcs --standard=psr12 src/",
+    "format": "phpcbf --standard=psr12 src/"
+  }
 }

src/Entry/LogEntry.php

@@ -74,7 +74,7 @@ public function getLevel(): string
     /**
      * {@inheritDoc}
      */
-    public function setLevel(string $level): LogEntryInterface
+    public function setLevel(string $level)
     {
         if (!isset($this->convert[$level])) {
             throw new InvalidArgumentException("unknown log level");
@@ -130,9 +130,9 @@ public function getContext(): array
     /**
      * {@inheritDoc}
      */
-    public function setContext(array $context): LogEntryInterface
+    public function setContext(array $context)
     {
         $this->context = $context;
         return $this;
     }
-}
+}

src/Entry/LogEntryInterface.php

@@ -43,18 +43,18 @@ public function getContext(): array;
 
     /**
      * @param  string $level
-     * @return LogEntryInterface $this
+     * @return $this
      * @throws InvalidArgumentException
      */
-    public function setLevel(string $level): LogEntryInterface;
+    public function setLevel(string $level);
 
     /**
      * Set context
      *
      * @param  array $context
-     * @return LogEntryInterface $this
+     * @return $this
      */
-    public function setContext(array $context): LogEntryInterface;
+    public function setContext(array $context);
 
     /**
      * array of processor classname
@@ -67,4 +67,4 @@ public function getProcessors(): array;
      * @return string
      */
     public function __toString(): string;
-}
+}

src/Entry/LogLevelTrait.php

@@ -24,13 +24,13 @@ trait LogLevelTrait
      * @var array
      */
     protected $convert = [
-        LogLevel::DEBUG     => 100,
-        LogLevel::INFO      => 200,
-        LogLevel::NOTICE    => 300,
-        LogLevel::WARNING   => 400,
-        LogLevel::ERROR     => 500,
-        LogLevel::CRITICAL  => 600,
-        LogLevel::ALERT     => 700,
+        LogLevel::DEBUG => 100,
+        LogLevel::INFO => 200,
+        LogLevel::NOTICE => 300,
+        LogLevel::WARNING => 400,
+        LogLevel::ERROR => 500,
+        LogLevel::CRITICAL => 600,
+        LogLevel::ALERT => 700,
         LogLevel::EMERGENCY => 800
     ];
-}
+}

src/Entry/SystemLog.php

@@ -45,4 +45,4 @@ public function getProcessors(): array
             parent::getProcessors(), [MemoryProcessor::class]
         );
     }
-}
+}

src/Formatter/AnsiFormatter.php

@@ -70,13 +70,13 @@ class AnsiFormatter extends DefaultFormatter
      * @access  protected
      */
     protected $colors = array(
-        LogLevel::DEBUG     => [self::FGCOLOR_GRAY, '', ''],
-        LogLevel::INFO      => ['', '', ''],
-        LogLevel::NOTICE    => [self::FGCOLOR_BRIGHT_GREEN, '', ''],
-        LogLevel::WARNING   => [self::FGCOLOR_BRIGHT_YELLOW, '', ''],
-        LogLevel::ERROR     => [self::FGCOLOR_BRIGHT_RED, '', ''],
-        LogLevel::CRITICAL  => [self::FGCOLOR_BRIGHT_RED, '', self::DECO_UNDERLINE],
-        LogLevel::ALERT     => [self::FGCOLOR_BRIGHT_RED, self::BGCOLOR_WHITE, ''],
+        LogLevel::DEBUG => [self::FGCOLOR_GRAY, '', ''],
+        LogLevel::INFO => ['', '', ''],
+        LogLevel::NOTICE => [self::FGCOLOR_BRIGHT_GREEN, '', ''],
+        LogLevel::WARNING => [self::FGCOLOR_BRIGHT_YELLOW, '', ''],
+        LogLevel::ERROR => [self::FGCOLOR_BRIGHT_RED, '', ''],
+        LogLevel::CRITICAL => [self::FGCOLOR_BRIGHT_RED, '', self::DECO_UNDERLINE],
+        LogLevel::ALERT => [self::FGCOLOR_BRIGHT_RED, self::BGCOLOR_WHITE, ''],
         LogLevel::EMERGENCY => [self::FGCOLOR_BRIGHT_RED, self::BGCOLOR_WHITE, self::DECO_BLINK],
     );
 
@@ -103,4 +103,4 @@ protected function addColor(string $level, string $text): string
         $suffix = $prefix ? self::DECO_END : '';
         return $prefix . $text . $suffix;
     }
-}
+}

src/Formatter/DefaultFormatter.php

@@ -47,4 +47,4 @@ protected function getEol(): string
             return "\n";
         }
     }
-}
+}

src/Formatter/FormatterAwareInterface.php

@@ -28,4 +28,4 @@ public function setFormatter(FormatterInterface $formatter);
      * @return FormatterInterface
      */
     public function getFormatter(): FormatterInterface;
-}
+}

src/Formatter/FormatterAwareTrait.php

@@ -39,4 +39,4 @@ public function setFormatter(FormatterInterface $formatter)
     {
         $this->formatter = $formatter;
     }
-}
+}

src/Formatter/FormatterInterface.php

@@ -27,4 +27,4 @@ interface FormatterInterface
      * @return string
      */
     public function format(LogEntryInterface $entry): string;
-}
+}

src/Handler/EchoHandler.php

@@ -27,4 +27,4 @@ protected function write(LogEntryInterface $entry)
     {
         echo $this->getFormatter()->format($entry);
     }
-}
+}

src/Handler/HandlerAbstract.php

@@ -76,4 +76,4 @@ protected function isHandling(): bool
      * @param  LogEntryInterface $entry
      */
     abstract protected function write(LogEntryInterface $entry);
-}
+}

src/Handler/HandlerAwareInterface.php

@@ -46,4 +46,4 @@ public function addHandler(
      * @return Traversable
      */
     public function getHandlers(LogEntryInterface $entry): Traversable;
-}
+}

src/Handler/HandlerAwareTrait.php

@@ -88,4 +88,4 @@ protected function matchLevel(string $entryLevel, string $handlerLevel): bool
         }
         return FALSE;
     }
-}
+}

src/Handler/HandlerInterface.php

@@ -27,4 +27,4 @@ interface HandlerInterface
      * @return LogEntryInterface
      */
     public function handle(LogEntryInterface $entry): LogEntryInterface;
-}
+}

src/Handler/LogfileHandler.php

@@ -107,4 +107,4 @@ protected function rotateByDate(string $path, string $format = 'Ymd'): bool
             return FALSE;
         }
     }
-}
+}

src/Handler/StreamHandler.php

@@ -78,4 +78,4 @@ protected function write(LogEntryInterface $entry)
             flock($this->stream, LOCK_UN);
         }
     }
-}
+}

src/Handler/SyslogHandler.php

@@ -54,13 +54,13 @@ class SyslogHandler extends HandlerAbstract
      * @access protected
      */
     protected $priorities = [
-        LogLevel::DEBUG     => LOG_DEBUG,
-        LogLevel::INFO      => LOG_INFO,
-        LogLevel::NOTICE    => LOG_NOTICE,
-        LogLevel::WARNING   => LOG_WARNING,
-        LogLevel::ERROR     => LOG_ERR,
-        LogLevel::CRITICAL  => LOG_CRIT,
-        LogLevel::ALERT     => LOG_ALERT,
+        LogLevel::DEBUG => LOG_DEBUG,
+        LogLevel::INFO => LOG_INFO,
+        LogLevel::NOTICE => LOG_NOTICE,
+        LogLevel::WARNING => LOG_WARNING,
+        LogLevel::ERROR => LOG_ERR,
+        LogLevel::CRITICAL => LOG_CRIT,
+        LogLevel::ALERT => LOG_ALERT,
         LogLevel::EMERGENCY => LOG_EMERG,
     ];
 
@@ -95,4 +95,4 @@ protected function write(LogEntryInterface $entry)
         );
         closelog();
     }
-}
+}

src/Handler/TerminalHandler.php

@@ -43,4 +43,4 @@ protected function isHandling(): bool
     {
         return 'cli' === php_sapi_name();
     }
-}
+}

src/Logger.php

@@ -49,6 +49,7 @@ public function log($level, $message, array $context = array())
     {
         // init the log entry
         $entry = $this->initEntry($message, $level, $context);
+
         /** @var HandlerInterface $handler */
         foreach ($this->getHandlers($entry) as $handler) {
             $entry = $handler->handle($entry);
@@ -69,6 +70,7 @@ protected function initEntry($message, string $level, array $context): LogEntryI
             $entry = new LogEntry($message);
         }
         $this->setChannel($context);
+
         return $entry
             ->setLevel($level)
             ->setContext(array_merge($entry->getContext(), $context));
@@ -83,4 +85,4 @@ protected function setChannel(array &$context)
             $context['__channel'] = $this->channel;
         }
     }
-}
+}

src/Processor/MemoryProcessor.php

@@ -27,4 +27,4 @@ protected function updateContext(array $context): array
         $context['memory_peak'] = number_format(memory_get_peak_usage(TRUE) / 1048575, 2);
         return $context;
     }
-}
+}