fix: error response when its not json

Author: deviddev

src/Services/BillingoApiV3Service.php

@@ -125,7 +125,7 @@ protected function createResponse(string $methodName, array $params, bool $metho
                     $params
                 );
         } catch (\Throwable $th) {
-            echo ($this->error($th->getMessage())->getJson());
+            echo ($this->error($th->getMessage())->response());
             exit;
         }
 

src/Traits/ProcessErrorsTrait.php

@@ -27,13 +27,17 @@ public function error(string $error): self
     }
 
     /**
-     * Get error message json response
+     * Get error message response
      *
      * @return string
      */
-    public function getJson(): string
+    public function response(): string
     {
-        return \json_encode(\json_decode($this->getJsonFromErrorString(), true), JSON_FORCE_OBJECT);
+        if ($this->isJson($this->error)) {
+            return \json_encode(\json_decode($this->getJsonFromErrorString(), true), JSON_FORCE_OBJECT);
+        }
+
+        return $this->error;
     }
 
     /**
@@ -49,4 +53,17 @@ public function getJsonFromErrorString(): string
             strpos($this->error, "}")
         );
     }
+
+    /**
+     * Check that error string is json encoded
+     *
+     * @param string $error
+     *
+     * @return bool
+     */
+    public function isJson(string $error): bool
+    {
+        json_decode($error);
+        return json_last_error() === JSON_ERROR_NONE;
+    }
 }