Merge pull request #3 from FVSoftwareDeveloper/master

upgrade to support Symfony 5
ntidev · Oct 13, 2022 · 6239616 · 6239616
2 parents 4785460 + 227e57d
commit 6239616
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/EventListener/KernelExceptionListener.php b/EventListener/KernelExceptionListener.php
@@ -4,7 +4,7 @@
 namespace NTI\LogBundle\EventListener;
 
 use Symfony\Component\DependencyInjection\ContainerInterface;
-use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
+use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
 
@@ -24,10 +24,10 @@ function __construct($container) {
     /**
      * @param GetResponseForExceptionEvent $event
      */
-    public function onKernelException(GetResponseForExceptionEvent $event)
+    public function onKernelException(ExceptionEvent $event)
     {
         // You get the exception object from the received event
-        $exception = $event->getException();
+        $exception = $event;
         $this->container->get('nti.logger')->logException($exception);
     }
-}
+}
diff --git a/Services/Logger.php b/Services/Logger.php
@@ -10,6 +10,7 @@
 use NTI\LogBundle\Exception\SlackException;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use NTI\LogBundle\Entity\Log;
+use Symfony\Component\HttpKernel\Event\ExceptionEvent;
 
 class Logger {
 
@@ -60,8 +61,8 @@ public function logError($message, $action = Log::ACTION_INFO, $entity = null) {
         $this->log($message, $action, $entity, Log::LEVEL_ERROR);
     }
 
-    public function logException(\Exception $ex) {
-        $this->log($ex->getMessage(), Log::ACTION_EXCEPTION, null, Log::LEVEL_ERROR, $ex);
+    public function logException(ExceptionEvent $ex) {
+        $this->log($ex->getThrowable(), Log::ACTION_EXCEPTION, null, Log::LEVEL_ERROR, $ex->getResponse());
     }
 
     public function logSlack($message, $level = Log::LEVEL_NOTICE, $entity = null) {
@@ -185,4 +186,4 @@ private function log($message, $action = Log::ACTION_INFO, $entity = null, $leve
             $this->logSlack($message, $level, $entity);
         }
     }
-}
+}