Merge pull request #2 from Deveodk/fixes

Fix: Fixed exception on non handled exception, and updated reporter l…
Deveodk · Jan 20, 2018 · 9e9da19 · 9e9da19
2 parents b1360d2 + 95799a5
commit 9e9da19
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/Exceptions/BaseException.php b/src/Exceptions/BaseException.php
@@ -204,7 +204,13 @@ private function parseCoreExceptionCode()
 
         $errorCodes = $config['error_codes'];
 
-        return $this->findExceptionCode($errorCodes);
+        $code = $this->findExceptionCode($errorCodes);
+
+        if ($code === null) {
+            return 0;
+        }
+
+        return $code;
     }
 
     /**
@@ -268,9 +274,9 @@ public function setStatusCode(int $statusCode)
     }
 
     /**
-     * @return string
+     * @return string|null
      */
-    public function getCoreExceptionCode(): string
+    public function getCoreExceptionCode(): ?string
     {
         return $this->coreExceptionCode;
     }

diff --git a/src/Handlers/ExceptionHandler.php b/src/Handlers/ExceptionHandler.php
@@ -86,6 +86,10 @@ public function report(Exception $exception)
             return parent::report($exception);
         }
 
+        if (env('APP_DEBUG', false)) {
+            return null;
+        }
+
         if (method_exists($exception, 'getSeverity')) {
             $severity = $exception->getSeverity();
         }