From d81edede4741b00fba6b8ababcb112fc6f882e05 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jorge=20Gonz=C3=A1lez?= Date: Thu, 20 Apr 2017 15:55:00 +0100 Subject: [PATCH] cleanup duplication --- src/Http/ControllerFactory.php | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/src/Http/ControllerFactory.php b/src/Http/ControllerFactory.php index b1585404927..fd71b6667b3 100644 --- a/src/Http/ControllerFactory.php +++ b/src/Http/ControllerFactory.php @@ -33,11 +33,7 @@ class ControllerFactory */ public function create(ServerRequest $request, Response $response) { - $controller = null; - if ($request->getParam('controller')) { - $controller = $request->getParam('controller'); - } - $className = $this->getControllerClass($request, $controller); + $className = $this->getControllerClass($request); if (!$className) { $this->missingController($request); } @@ -46,22 +42,20 @@ public function create(ServerRequest $request, Response $response) $this->missingController($request); } - return $reflection->newInstance($request, $response, $controller); + return $reflection->newInstance($request, $response); } /** * Determine the controller class name based on current request and controller param * * @param \Cake\Http\ServerRequest $request The request to build a controller for. - * @param string|null $controllerName The controller name present in the request params * @return string|null */ - public function getControllerClass(ServerRequest $request, $controllerName = null) + public function getControllerClass(ServerRequest $request) { - $pluginPath = null; - $controller = $controllerName; + $pluginPath = $controller = null; $namespace = 'Controller'; - if (!$controller && $request->getParam('controller')) { + if ($request->getParam('controller')) { $controller = $request->getParam('controller'); } if ($request->getParam('plugin')) {