URI strategy implementation MAY update the requestParameters

phramework · Jan 6, 2016 · 379e9ec · 379e9ec
1 parent 62b3be5
commit 379e9ec
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 10 deletions.
diff --git a/composer.json b/composer.json
@@ -23,13 +23,10 @@
     "ext-json": "*",
     "phramework/exceptions": "0.*",
     "phramework/validate": "^0.3",
-    "phramework/database": "dev-master"
+    "phramework/database": "0.*"
   },
   "suggest": {
-    "ext-curl": "*",
-    "ext-pdo": "Required when database is used",
-    "ext-pdo_pgsql": "Required when postgreSQL database is used",
-    "ext-pdo_mysql": "Required when MySQL or MariaDB database is used"
+    "ext-curl": "*"
   },
   "require-dev": {
     "phpunit/phpunit": "5.*",

diff --git a/src/URIStrategy/ClassBased.php b/src/URIStrategy/ClassBased.php
@@ -96,7 +96,7 @@ public function __construct(
      * @return string[2] This method should return `[$class, $method]` on success
      */
     public function invoke(
-        $requestParameters,
+        &$requestParameters,
         $requestMethod,
         $requestHeaders,
         $requestUser

diff --git a/src/URIStrategy/IURIStrategy.php b/src/URIStrategy/IURIStrategy.php
@@ -43,7 +43,7 @@ interface IURIStrategy
      * @throws Phramework\Exceptions\UnauthorizedException
      */
     public function invoke(
-        $requestParameters,
+        &$requestParameters,
         $requestMethod,
         $requestHeaders,
         $requestUser

diff --git a/src/URIStrategy/URITemplate.php b/src/URIStrategy/URITemplate.php
@@ -159,7 +159,7 @@ public static function URI()
      * @return string[2] This method should return `[$class, $method]` on success
      */
     public function invoke(
-        $requestParameters,
+        &$requestParameters,
         $requestMethod,
         $requestHeaders,
         $requestUser
@@ -194,7 +194,7 @@ public function invoke(
                 list($URI_parameters) = $test;
 
                 //Merge all available parameters
-                $parameters = (object)array_merge(
+                $requestParameters = (object)array_merge(
                     (array)$requestParameters,
                     $URI_parameters,
                     $test[0]
@@ -215,7 +215,7 @@ public function invoke(
                     [$class, $method],
                     array_merge(
                         [
-                            $parameters,
+                            $requestParameters,
                             $requestMethod,
                             $requestHeaders
                         ],