Update PHPStan configuration, fix some issues

Author: leonardfischer

phpstan.neon

@@ -1,11 +1,4 @@
 parameters:
     level: 5
-    checkMissingIterableValueType: false
     paths:
         - src
-        - tests
-    ignoreErrors:
-        # Ignore skipped tests:
-        - message: "#Unreachable statement#"
-          paths:
-              - tests/Idoit/APIClient/*

src/API.php

@@ -622,7 +622,7 @@ protected function execute(array $data): array {
         $lastResponse = json_decode(trim($body), true);
 
         if (!is_array($lastResponse)) {
-            if (is_string($body) && strlen($body) > 0) {
+            if (strlen($body) > 0) {
                 throw new RuntimeException(sprintf(
                     'i-doit responded with an unknown message: %s',
                     $body

src/CMDBCategoryInfo.php

@@ -131,13 +131,7 @@ public function readAll(): array {
 
         $categories = $this->batchRead($cleanCategoryConstants);
 
-        $combinedArray = array_combine($cleanCategoryConstants, $categories);
-
-        if (!is_array($combinedArray)) {
-            throw new RuntimeException('Unable to restructure result');
-        }
-
-        return $combinedArray;
+        return array_combine($cleanCategoryConstants, $categories);
     }
 
     /**