move phpError() function to trait

Author: samsonasik

src/HeroTrait.php

@@ -6,6 +6,8 @@
 
 use ErrorException;
 use ErrorHeroModule\Handler\Logging;
+use Webmozart\Assert\Assert;
+use Zend\Mvc\MvcEvent;
 
 trait HeroTrait
 {
@@ -22,6 +24,29 @@ trait HeroTrait
     /** @var string */
     private $result = '';
 
+    public function phpError() : void
+    {
+        if ($this instanceof Listener\Mvc && \func_get_args()) {
+            Assert::isInstanceOf(
+                $this->mvcEvent = \func_get_arg(0),
+                MvcEvent::class
+            );
+        }
+
+        if (! $this->errorHeroModuleConfig['display-settings']['display_errors']) {
+            \error_reporting(\E_ALL | \E_STRICT);
+            \ini_set('display_errors', '0');
+        }
+
+        while (\ob_get_level() > 0) {
+            \ob_end_flush();
+        }
+
+        \ob_start([$this, 'phpFatalErrorHandler']);
+        \register_shutdown_function([$this, 'execOnShutdown']);
+        \set_error_handler([$this, 'phpErrorHandler']);
+    }
+
     private static function isUncaught(array $error)
     {
         return 0 === strpos($error['message'], 'Uncaught');

src/Listener/Mvc.php

@@ -57,24 +57,6 @@ public function attach(EventManagerInterface $events, $priority = 1) : void
         $this->listeners[] = $events->attach(MvcEvent::EVENT_BOOTSTRAP, [$this, 'phpError']);
     }
 
-    public function phpError(MvcEvent $e) : void
-    {
-        $this->mvcEvent = $e;
-
-        if (! $this->errorHeroModuleConfig['display-settings']['display_errors']) {
-            \error_reporting(\E_ALL | \E_STRICT);
-            \ini_set('display_errors', '0');
-        }
-
-        while (\ob_get_level() > 0) {
-            \ob_end_flush();
-        }
-
-        \ob_start([$this, 'phpFatalErrorHandler']);
-        \register_shutdown_function([$this, 'execOnShutdown']);
-        \set_error_handler([$this, 'phpErrorHandler']);
-    }
-
     public function exceptionError(MvcEvent $e) : void
     {
         $exception = $e->getParam('exception');

src/Middleware/Expressive.php

@@ -63,22 +63,6 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
         return $this->exceptionError($t);
     }
 
-    public function phpError() : void
-    {
-        if (! $this->errorHeroModuleConfig['display-settings']['display_errors']) {
-            \error_reporting(\E_ALL | \E_STRICT);
-            \ini_set('display_errors', '0');
-        }
-
-        while (\ob_get_level() > 0) {
-            \ob_end_flush();
-        }
-
-        \ob_start([$this, 'phpFatalErrorHandler']);
-        \register_shutdown_function([$this, 'execOnShutdown']);
-        \set_error_handler([$this, 'phpErrorHandler']);
-    }
-
     /**
      * @throws Error      when 'display_errors' config is 1 and Error has thrown
      * @throws Exception  when 'display_errors' config is 1 and Exception has thrown