+ body params demo controller

+ tests + postman for body params controller
+ PSR-2 fixes + unused code cleanup

Author: xzag

Action.php

@@ -33,11 +33,6 @@ class Action extends \yii\base\Action
      */
     public $paramsPassMethod;
 
-    /**
-     * @var array Whether JSON parse should parse objects in `params` as associate arrays or objects
-     */
-    public $requestParseAsArray;
-
     /**
      * Parses json body.
      * @param $rawBody
@@ -83,25 +78,6 @@ protected function restoreYiiRequest()
         return $this;
     }
 
-    /**
-     * Recursively converts object to an associative array.
-     * @param $obj
-     * @return array
-     */
-    protected function objToAssoc($obj) {
-        $result = (array) $obj;
-
-        if (is_array($result)) {
-            foreach ($result as $key => $value) {
-                if (is_object($value) || is_array($value)) {
-                    $result[$key] = $this->objToAssoc($value);
-                }
-            }
-        }
-
-        return $result;
-    }
-
     /**
      * @inheritdoc
      */
@@ -126,32 +102,21 @@ public function runWithParams($params)
                     $request = new JsonRpcRequest();
                     $request->paramsPassMethod = $this->paramsPassMethod;
 
-                    // Handling params
-                    $params = ($this->requestParseAsArray)
-                        ? (isset($request->params) ? $this->objToAssoc($request->params) : [])
-                        : (isset($request->params) ? $request->params : new \stdClass());
-
-                    $requestData = ArrayHelper::toArray($requestData);
-                    $requestData['params'] = $params;
-
-                    $request->load($requestData, '');
+                    $request->load(ArrayHelper::toArray($requestData), '');
                     if ($request->validate()) {
                         $result = $request->execute();
                         if (!is_null($request->id)) {
                             $batchResponse[] = new SuccessResponse($request, $result);
                         }
-                    }
-                    else {
+                    } else {
                         foreach ($request->getFirstErrors() as $attribute => $error) {
                             $request->$attribute = null;
                         }
                         throw new InvalidRequestException();
                     }
-                }
-                catch (InvalidRequestException $e) {
+                } catch (InvalidRequestException $e) {
                     $batchResponse[] = new ErrorResponse($e, $request ?: null);
-                }
-                catch (JsonRpcException $e) {
+                } catch (JsonRpcException $e) {
                     // We do not return response to notifications
                     if ($request && !is_null($request->id)) {
                         $batchResponse[] = new ErrorResponse($e, $request ?: null);
@@ -160,8 +125,7 @@ public function runWithParams($params)
 
                 $this->restoreYiiRequest();
             }
-        }
-        catch (JsonRpcException $e) {
+        } catch (JsonRpcException $e) {
             return new ErrorResponse($e);
         }
 

Controller.php

@@ -23,18 +23,12 @@ class Controller extends \yii\web\Controller
      */
     public $paramsPassMethod = self::JSON_RPC_PARAMS_PASS_FUNCARGS;
 
-    /**
-     * @var array Whether JSON parse should parse objects in `params` as associate arrays or objects
-     */
-    public $requestParseAsArray = true;
-  
     public function actions()
     {
         return [
             'index' => [
                 'class' => Action::class,
                 'paramsPassMethod' => $this->paramsPassMethod,
-                'requestParseAsArray' => $this->requestParseAsArray,
             ]
         ];
     }

demo/controllers/BodyParamsJsonRpcController.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace app\controllers;
+
+use \georgique\yii2\jsonrpc\Controller;
+
+class BodyParamsJsonRpcController extends Controller
+{
+    // Disable CSRF validation for JSON-RPC POST requests
+    public $enableCsrfValidation = false;
+
+    public $paramsPassMethod = self::JSON_RPC_PARAMS_PASS_BODY;
+}

demo/controllers/DemoController.php

@@ -53,12 +53,30 @@ public function actionSum($a, $b)
         return $a + $b;
     }
 
-    public function actionSumList($params)
+    /**
+     * @return mixed
+     * @throws \yii\base\InvalidConfigException
+     */
+    public function actionSumIntegerList()
     {
-        //var_dump($params);
+        $params = \Yii::$app->request->getBodyParams();
         return array_reduce($params, function ($acc, $item) {
-            $acc += $item;
+            $acc += is_int($item) ? $item : 0;
             return $acc;
         }, 0);
     }
+
+    /**
+     * @return string
+     * @throws \yii\base\InvalidConfigException
+     */
+    public function actionDumpRequest()
+    {
+        $output = "Params received: ";
+        $output_chunks = array();
+        foreach (\Yii::$app->request->getBodyParams() as $name => $value) {
+            $output_chunks[] = "$name = $value\n";
+        }
+        return $output . implode(', ', $output_chunks) . '.';
+    }
 }

demo/controllers/JsonRpcController.php

@@ -11,9 +11,4 @@ class JsonRpcController extends Controller
 
     // Disable CSRF validation for JSON-RPC POST requests
     public $enableCsrfValidation = false;
-
-    public function actionVariadic(... $params)
-    {
-        var_dump($params);
-    }
 }