Improved error handling - passing original error to the response now.

Author: georgique

Action.php

@@ -136,16 +136,17 @@ public function executeRequest($request) {
         }
         catch (\Exception $exception) {
             if ($exception instanceof InvalidRouteException) {
-                $result = new JsonRpcException($request->id, 'Method not found: ' . $route . '.', JSON_RPC_ERROR_METHOD_NOT_FOUND);
-                return $result;
+                $result = new JsonRpcException($request->id, 'Method not found: ' . $route . '.',
+                    JSON_RPC_ERROR_METHOD_NOT_FOUND, $exception);
             }
             else if ($exception instanceof JsonRpcException) {
-                return $exception;
+                $result = $exception;
             }
             else {
-                $result = new JsonRpcException($request->id, 'Internal error.', JSON_RPC_ERROR_INTERNAL);
-                return $result;
+                $result = new JsonRpcException($request->id, 'Internal error.', JSON_RPC_ERROR_INTERNAL, $exception);
             }
+
+            return $result;
         }
     }
 

JsonRpcException.php

@@ -10,7 +10,7 @@
  */
 class JsonRpcException extends \yii\base\Exception {
 
-    public function __construct($id, $message = "", $code = 0, \Throwable $previous = null)
+    public function __construct($id, $message = "", $code = 0, $previous = null)
     {
         parent::__construct($message, $code, $previous);
         $this->id = $id;