From 148de0549574abb2f3ab23275094c5a49e585467 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Edi=20Modri=C4=87?= <edi@netgen.hr>
Date: Thu, 9 Feb 2017 10:08:38 +0100
Subject: [PATCH] Use NullLogger if no logger available

---
 EventListener/RouterListener.php |  7 +++----
 Routing/TagRouter.php            | 13 ++++++-------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/EventListener/RouterListener.php b/EventListener/RouterListener.php
index 9f929f0e..b347e6c7 100644
--- a/EventListener/RouterListener.php
+++ b/EventListener/RouterListener.php
@@ -5,6 +5,7 @@
 use eZ\Bundle\EzPublishLegacyBundle\Routing\FallbackRouter;
 use Netgen\TagsBundle\Routing\Generator\TagUrlGenerator;
 use Psr\Log\LoggerInterface;
+use Psr\Log\NullLogger;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpKernel\Event\GetResponseEvent;
 use Symfony\Component\HttpKernel\KernelEvents;
@@ -28,7 +29,7 @@ class RouterListener implements EventSubscriberInterface
      */
     public function __construct(LoggerInterface $logger = null)
     {
-        $this->logger = $logger;
+        $this->logger = $logger ?: new NullLogger();
     }
 
     /**
@@ -71,9 +72,7 @@ public function onKernelRequest(GetResponseEvent $event)
             return;
         }
 
-        if ($this->logger instanceof LoggerInterface) {
-            $this->logger->info('Falling back to tag legacy route as specified in config.');
-        }
+        $this->logger->info('Falling back to tag legacy route as specified in config.');
 
         $request->attributes->set('_route', FallbackRouter::ROUTE_NAME);
         $request->attributes->set('_controller', 'ezpublish_legacy.controller:indexAction');
diff --git a/Routing/TagRouter.php b/Routing/TagRouter.php
index efb9e1cb..1173f333 100644
--- a/Routing/TagRouter.php
+++ b/Routing/TagRouter.php
@@ -9,6 +9,7 @@
 use Netgen\TagsBundle\API\Repository\Values\Tags\Tag;
 use Netgen\TagsBundle\Routing\Generator\TagUrlGenerator;
 use Psr\Log\LoggerInterface;
+use Psr\Log\NullLogger;
 use RuntimeException;
 use Symfony\Cmf\Component\Routing\ChainedRouterInterface;
 use Symfony\Cmf\Component\Routing\RouteObjectInterface;
@@ -68,8 +69,8 @@ public function __construct(
     ) {
         $this->tagsService = $tagsService;
         $this->generator = $generator;
-        $this->requestContext = $requestContext !== null ? $requestContext : new RequestContext();
-        $this->logger = $logger;
+        $this->requestContext = $requestContext ?: new RequestContext();
+        $this->logger = $logger ?: new NullLogger();
     }
 
     /**
@@ -132,11 +133,9 @@ function (TagsService $tagsService) use ($requestedPath, $languages) {
 
         $request->attributes->set('tagId', $tag->id);
 
-        if ($this->logger !== null) {
-            $this->logger->info(
-                "TagRouter matched tag #{$tag->id}. Forwarding to tag view controller"
-            );
-        }
+        $this->logger->info(
+            "TagRouter matched tag #{$tag->id}. Forwarding to tag view controller"
+        );
 
         return $params;
     }