Monolog 2 compatibility.

derrabus · derrabus · commit 0f05439248f4 · 2019-07-01T11:43:48.000+02:00
diff --git a/composer.json b/composer.json
@@ -108,7 +108,7 @@
         "doctrine/reflection": "~1.0",
         "doctrine/doctrine-bundle": "^1.5|^2.0",
         "masterminds/html5": "^2.6",
-        "monolog/monolog": "~1.11",
+        "monolog/monolog": "~1.11 || ^2",
         "nyholm/psr7": "^1.0",
         "ocramius/proxy-manager": "^2.1",
         "php-http/httplug": "^1.0|^2.0",
diff --git a/src/Symfony/Bridge/Monolog/CHANGELOG.md b/src/Symfony/Bridge/Monolog/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 -----
 
  * The methods `DebugProcessor::getLogs()`, `DebugProcessor::countErrors()`, `Logger::getLogs()` and `Logger::countErrors()` have a new `$request` argument.
+ * Added support for Monolog 2.
 
 4.4.0
 -----
diff --git a/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php b/src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php
@@ -57,7 +57,7 @@ public function onKernelResponse(ResponseEvent $event)
     /**
      * {@inheritdoc}
      */
-    protected function sendHeader($header, $content)
+    protected function sendHeader($header, $content): void
     {
         if (!$this->sendHeaders) {
             return;
diff --git a/src/Symfony/Bridge/Monolog/Handler/ConsoleHandler.php b/src/Symfony/Bridge/Monolog/Handler/ConsoleHandler.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\Monolog\Handler;
 
+use Monolog\Formatter\FormatterInterface;
 use Monolog\Formatter\LineFormatter;
 use Monolog\Handler\AbstractProcessingHandler;
 use Monolog\Logger;
@@ -74,15 +75,15 @@ public function __construct(OutputInterface $output = null, bool $bubble = true,
     /**
      * {@inheritdoc}
      */
-    public function isHandling(array $record)
+    public function isHandling(array $record): bool
     {
         return $this->updateLevel() && parent::isHandling($record);
     }
 
     /**
      * {@inheritdoc}
      */
-    public function handle(array $record)
+    public function handle(array $record): bool
     {
         // we have to update the logging level each time because the verbosity of the
         // console output might have changed in the meantime (it is not immutable)
@@ -100,7 +101,7 @@ public function setOutput(OutputInterface $output)
     /**
      * Disables the output.
      */
-    public function close()
+    public function close(): void
     {
         $this->output = null;
 
@@ -143,7 +144,7 @@ public static function getSubscribedEvents()
     /**
      * {@inheritdoc}
      */
-    protected function write(array $record)
+    protected function write(array $record): void
     {
         // at this point we've determined for sure that we want to output the record, so use the output's own verbosity
         $this->output->write((string) $record['formatted'], false, $this->output->getVerbosity());
@@ -152,7 +153,7 @@ protected function write(array $record)
     /**
      * {@inheritdoc}
      */
-    protected function getDefaultFormatter()
+    protected function getDefaultFormatter(): FormatterInterface
     {
         if (!class_exists(CliDumper::class)) {
             return new LineFormatter();
diff --git a/src/Symfony/Bridge/Monolog/Handler/FingersCrossed/HttpCodeActivationStrategy.php b/src/Symfony/Bridge/Monolog/Handler/FingersCrossed/HttpCodeActivationStrategy.php
@@ -45,7 +45,7 @@ public function __construct(RequestStack $requestStack, array $exclusions, $acti
         $this->exclusions = $exclusions;
     }
 
-    public function isHandlerActivated(array $record)
+    public function isHandlerActivated(array $record): bool
     {
         $isActivated = parent::isHandlerActivated($record);
 
diff --git a/src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategy.php b/src/Symfony/Bridge/Monolog/Handler/FingersCrossed/NotFoundActivationStrategy.php
@@ -34,7 +34,7 @@ public function __construct(RequestStack $requestStack, array $excludedUrls, $ac
         $this->blacklist = '{('.implode('|', $excludedUrls).')}i';
     }
 
-    public function isHandlerActivated(array $record)
+    public function isHandlerActivated(array $record): bool
     {
         $isActivated = parent::isHandlerActivated($record);
 
diff --git a/src/Symfony/Bridge/Monolog/Handler/FirePHPHandler.php b/src/Symfony/Bridge/Monolog/Handler/FirePHPHandler.php
@@ -59,7 +59,7 @@ public function onKernelResponse(ResponseEvent $event)
     /**
      * {@inheritdoc}
      */
-    protected function sendHeader($header, $content)
+    protected function sendHeader($header, $content): void
     {
         if (!self::$sendHeaders) {
             return;
diff --git a/src/Symfony/Bridge/Monolog/Handler/ServerLogHandler.php b/src/Symfony/Bridge/Monolog/Handler/ServerLogHandler.php
@@ -39,7 +39,7 @@ public function __construct(string $host, int $level = Logger::DEBUG, bool $bubb
     /**
      * {@inheritdoc}
      */
-    public function handle(array $record)
+    public function handle(array $record): bool
     {
         if (!$this->isHandling($record)) {
             return false;
diff --git a/src/Symfony/Bridge/Monolog/Handler/SwiftMailerHandler.php b/src/Symfony/Bridge/Monolog/Handler/SwiftMailerHandler.php
@@ -52,7 +52,7 @@ public function onCliTerminate(ConsoleTerminateEvent $event)
     /**
      * {@inheritdoc}
      */
-    protected function send($content, array $records)
+    protected function send($content, array $records): void
     {
         parent::send($content, $records);
 
diff --git a/src/Symfony/Bridge/Monolog/Logger.php b/src/Symfony/Bridge/Monolog/Logger.php
@@ -59,7 +59,7 @@ public function clear()
     /**
      * {@inheritdoc}
      */
-    public function reset()
+    public function reset(): void
     {
         $this->clear();
 
diff --git a/src/Symfony/Bridge/Monolog/composer.json b/src/Symfony/Bridge/Monolog/composer.json
@@ -17,7 +17,7 @@
     ],
     "require": {
         "php": "^7.2.9",
-        "monolog/monolog": "~1.19",
+        "monolog/monolog": "^1.19|^2",
         "symfony/service-contracts": "^1.1",
         "symfony/http-kernel": "^4.4|^5.0"
     },

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ public function onKernelResponse(ResponseEvent $event)`
`57`	`57`	`/**`
`58`	`58`	`* {@inheritdoc}`
`59`	`59`	`*/`
`60`		`- protected function sendHeader($header, $content)`
	`60`	`+ protected function sendHeader($header, $content): void`
`61`	`61`	`{`
`62`	`62`	`if (!$this->sendHeaders) {`
`63`	`63`	`return;`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public function __construct(RequestStack $requestStack, array $exclusions, $acti`
`45`	`45`	`$this->exclusions = $exclusions;`
`46`	`46`	`}`
`47`	`47`
`48`		`- public function isHandlerActivated(array $record)`
	`48`	`+ public function isHandlerActivated(array $record): bool`
`49`	`49`	`{`
`50`	`50`	`$isActivated = parent::isHandlerActivated($record);`
`51`	`51`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function __construct(RequestStack $requestStack, array $excludedUrls, $ac`
`34`	`34`	`$this->blacklist = '{('.implode('\|', $excludedUrls).')}i';`
`35`	`35`	`}`
`36`	`36`
`37`		`- public function isHandlerActivated(array $record)`
	`37`	`+ public function isHandlerActivated(array $record): bool`
`38`	`38`	`{`
`39`	`39`	`$isActivated = parent::isHandlerActivated($record);`
`40`	`40`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public function onKernelResponse(ResponseEvent $event)`
`59`	`59`	`/**`
`60`	`60`	`* {@inheritdoc}`
`61`	`61`	`*/`
`62`		`- protected function sendHeader($header, $content)`
	`62`	`+ protected function sendHeader($header, $content): void`
`63`	`63`	`{`
`64`	`64`	`if (!self::$sendHeaders) {`
`65`	`65`	`return;`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function __construct(string $host, int $level = Logger::DEBUG, bool $bubb`
`39`	`39`	`/**`
`40`	`40`	`* {@inheritdoc}`
`41`	`41`	`*/`
`42`		`- public function handle(array $record)`
	`42`	`+ public function handle(array $record): bool`
`43`	`43`	`{`
`44`	`44`	`if (!$this->isHandling($record)) {`
`45`	`45`	`return false;`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function onCliTerminate(ConsoleTerminateEvent $event)`
`52`	`52`	`/**`
`53`	`53`	`* {@inheritdoc}`
`54`	`54`	`*/`
`55`		`- protected function send($content, array $records)`
	`55`	`+ protected function send($content, array $records): void`
`56`	`56`	`{`
`57`	`57`	`parent::send($content, $records);`
`58`	`58`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public function clear()`
`59`	`59`	`/**`
`60`	`60`	`* {@inheritdoc}`
`61`	`61`	`*/`
`62`		`- public function reset()`
	`62`	`+ public function reset(): void`
`63`	`63`	`{`
`64`	`64`	`$this->clear();`
`65`	`65`