Fix unclear types & PHPStan issues

Author: iquito

composer.json

@@ -39,10 +39,10 @@
     },
     "scripts": {
         "phpstan": "vendor/bin/phpstan analyse",
+        "phpstan_full": "vendor/bin/phpstan clear-result-cache && vendor/bin/phpstan analyse",
         "phpstan_base": "vendor/bin/phpstan analyse --configuration phpstan_base.neon --error-format baselineNeon > phpstan-baseline.neon",
-        "phpstan_clear": "vendor/bin/phpstan clear-result-cache",
         "psalm": "vendor/bin/psalm --show-info=false",
-        "psalm_full": "vendor/bin/psalm --show-info=false --clear-cache",
+        "psalm_full": "vendor/bin/psalm --clear-cache && vendor/bin/psalm --show-info=false",
         "psalm_base": "vendor/bin/psalm --set-baseline=psalm-baseline.xml",
         "phpunit": "vendor/bin/phpunit --colors=always",
         "phpunit_clover": "vendor/bin/phpunit --coverage-text --coverage-clover build/logs/clover.xml",

src/Debug.php

@@ -46,11 +46,18 @@ public static function createException(
 
             // Replace backtrace instance if we find a valid class insance
             foreach ($backtraceClasses as $backtraceClass) {
+                $classImplements = \class_implements($backtrace['class']);
+                $classParents = \class_parents($backtrace['class']);
+
+                if ($classImplements === false || $classParents === false) {
+                    continue;
+                }
+
                 // Check if the class or interface we are looking for is implemented or used
                 // by the current backtrace class
                 if (
-                    \in_array($backtraceClass, \class_implements($backtrace['class']), true) ||
-                    \in_array($backtraceClass, \class_parents($backtrace['class']), true) ||
+                    \in_array($backtraceClass, $classImplements, true) ||
+                    \in_array($backtraceClass, $classParents, true) ||
                     $backtraceClass === $backtrace['class']
                 ) {
                     $lastInstance = $backtrace;
@@ -70,13 +77,24 @@ public static function createException(
         $parts = \explode('\\', $assignedBacktraceClass);
         $shownClass = \array_pop($parts);
 
+        $classImplements = \class_implements($exceptionClass);
+        $classParents = \class_parents($exceptionClass);
+
+        if ($classImplements === false) {
+            $classImplements = [];
+        }
+
+        if ($classParents === false) {
+            $classParents = [];
+        }
+
         // Make sure the provided exception class inherits from Throwable, otherwise replace it with Exception
-        if (!\in_array(\Throwable::class, \class_implements($exceptionClass), true)) {
+        if (!\in_array(\Throwable::class, $classImplements, true)) {
             $exceptionClass = \Exception::class;
         }
 
         // If we have no OriginException child class, we assume the default Exception class constructor is used
-        if (!\in_array(OriginException::class, \class_parents($exceptionClass), true) && $exceptionClass !== OriginException::class) {
+        if (!\in_array(OriginException::class, $classParents, true) && $exceptionClass !== OriginException::class) {
             /**
              * @var \Throwable $exception At this point we know that $exceptionClass inherits from \Throwable for sure
              */

tests/DebugTest.php

@@ -26,7 +26,7 @@ public function testCreateException()
             $this->assertEquals(__FILE__, $e->getFile());
             $this->assertEquals(__LINE__ - 8, $e->getLine());
             $this->assertEquals($debugClassPath, $e->getExceptionFile());
-            $this->assertEquals(95, $e->getExceptionLine());
+            $this->assertEquals(113, $e->getExceptionLine());
             $this->assertEquals('SomeClass->someFunction()', $e->getOriginCall());
         }
     }