#244 Implemented passing route parameters to controller methods

letsdrink · Nov 29, 2016 · f1d9fe6 · bbankowski · Nov 30, 2016 · bbankowski
1 parent a92505d
commit f1d9fe6
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/src/Ouzo/Core/FrontController.php b/src/Ouzo/Core/FrontController.php
@@ -147,8 +147,9 @@ private function _invokeAfterMethods()
 
     private function _invokeAction()
     {
-        $currentAction = $this->currentControllerObject->currentAction;
-        $this->currentControllerObject->$currentAction();
+        $controller = $this->currentControllerObject;
+        $currentAction = $controller->currentAction;
+        call_user_func_array(array($controller, $currentAction), $controller->params);
         $this->_logRequestIfDebugEnabled();
     }
 

diff --git a/test/src/Ouzo/Core/ControllerTest.php b/test/src/Ouzo/Core/ControllerTest.php
@@ -83,6 +83,12 @@ public function string_output()
         $this->layout->renderAjax('ONLY OUTPUT');
         $this->layout->unsetLayout();
     }
+
+    public function receive_params($user, $page)
+    {
+        $this->layout->renderAjax("Param1: $user Param2: $page");
+        $this->layout->unsetLayout();
+    }
 }
 
 class ControllerTest extends ControllerTestCase
@@ -169,6 +175,21 @@ public function shouldDownloadFile()
         $this->assertDownloadsFile('file.txt');
     }
 
+    /**
+     * @test
+     */
+    public function shouldPassUrlParametersToControllerAction()
+    {
+        // given
+        Route::get('/simple_test/receive_params/:user/:page', 'simple_test#receive_params');
+
+        // when
+        $this->get('/simple_test/receive_params/Cersei/about-us');
+
+        // then
+        $this->assertRenderedContent()->isEqualTo('Param1: Cersei Param2: about-us');
+    }
+
     /**
      * @test
      */