diff --git a/src/Pecee/Http/Response.php b/src/Pecee/Http/Response.php index b8dd628..66be015 100644 --- a/src/Pecee/Http/Response.php +++ b/src/Pecee/Http/Response.php @@ -26,7 +26,7 @@ class Response { } public function refresh() { - $this->redirect(url()); + $this->redirect(Request::getInstance()->getUri()); } /** diff --git a/src/Pecee/SimpleRouter/RouterBase.php b/src/Pecee/SimpleRouter/RouterBase.php index 5a7de95..bd94efb 100644 --- a/src/Pecee/SimpleRouter/RouterBase.php +++ b/src/Pecee/SimpleRouter/RouterBase.php @@ -220,10 +220,10 @@ class RouterBase { } public function arrayToParams(array $getParams = null, $includeEmpty = true) { - if(is_array($getParams) && count($getParams) > 0) { - foreach($getParams as $key=>$val) { - if(!empty($val) || empty($val) && $includeEmpty) { - $getParams[$key] = $key.'='.$val; + if (is_array($getParams) && count($getParams) > 0) { + foreach ($getParams as $key => $val) { + if (!empty($val) || empty($val) && $includeEmpty) { + $getParams[$key] = $key . '=' . $val; } } return join('&', $getParams); @@ -345,6 +345,7 @@ class RouterBase { $url = '/' . trim(join('/', $url), '/') . '/'; + if($getParams !== null && count($getParams)) { $url .= '?' . $this->arrayToParams($getParams); } diff --git a/src/Pecee/SimpleRouter/RouterController.php b/src/Pecee/SimpleRouter/RouterController.php index 1bde2c4..a6fef1b 100644 --- a/src/Pecee/SimpleRouter/RouterController.php +++ b/src/Pecee/SimpleRouter/RouterController.php @@ -63,7 +63,7 @@ class RouterController extends RouterEntry { // Set callback $this->setCallback($this->controller . '@' . $this->method); - return $this; + return true; } } return null; diff --git a/src/Pecee/SimpleRouter/RouterEntry.php b/src/Pecee/SimpleRouter/RouterEntry.php index 934c270..67740bd 100644 --- a/src/Pecee/SimpleRouter/RouterEntry.php +++ b/src/Pecee/SimpleRouter/RouterEntry.php @@ -304,7 +304,7 @@ abstract class RouterEntry { $max = count($parameterNames); - if(count($max)) { + if($max) { for($i = 0; $i < $max; $i++) { $name = $parameterNames[$i]; $parameterValue = (isset($parameterValues[$name['name']]) && !empty($parameterValues[$name['name']])) ? $parameterValues[$name['name']] : null; @@ -332,7 +332,7 @@ abstract class RouterEntry { throw new RouterException($middleware . ' must be instance of Middleware'); } - /* @var $class Middleware */ + /* @var $class IMiddleware */ $middleware->handle($request); } } else { @@ -341,7 +341,7 @@ abstract class RouterEntry { throw new RouterException($this->getMiddleware() . ' must be instance of Middleware'); } - /* @var $class Middleware */ + /* @var $class IMiddleware */ $middleware->handle($request); } } diff --git a/src/Pecee/SimpleRouter/RouterGroup.php b/src/Pecee/SimpleRouter/RouterGroup.php index 7db11f2..71de05d 100644 --- a/src/Pecee/SimpleRouter/RouterGroup.php +++ b/src/Pecee/SimpleRouter/RouterGroup.php @@ -10,7 +10,7 @@ class RouterGroup extends RouterEntry { parent::__construct(); } - protected function matchDomain() { + protected function matchDomain(Request $request) { if($this->domain !== null) { if(is_array($this->domain)) { @@ -20,7 +20,7 @@ class RouterGroup extends RouterEntry { for($i = 0; $i < $max; $i++) { $domain = $this->domain[$i]; - $parameters = $this->parseParameters($domain, request()->getHost(), '[^.]*'); + $parameters = $this->parseParameters($domain, $request->getHost(), '[^.]*'); if($parameters !== null) { $this->parameters = $parameters; @@ -31,7 +31,7 @@ class RouterGroup extends RouterEntry { return null; } - $parameters = $this->parseParameters($this->domain, request()->getHost(), '[^.]*'); + $parameters = $this->parseParameters($this->domain, $request->getHost(), '[^.]*'); if ($parameters !== null) { $this->parameters = $parameters; @@ -60,7 +60,7 @@ class RouterGroup extends RouterEntry { throw new RouterException('Method not allowed'); } - if($this->matchDomain() === null) { + if($this->matchDomain($request) === null) { return null; } diff --git a/src/Pecee/SimpleRouter/RouterResource.php b/src/Pecee/SimpleRouter/RouterResource.php index c10435e..de70b17 100644 --- a/src/Pecee/SimpleRouter/RouterResource.php +++ b/src/Pecee/SimpleRouter/RouterResource.php @@ -45,7 +45,7 @@ class RouterResource extends RouterEntry { protected function call($method, $parameters) { $this->setCallback($this->controller . '@' . $method); $this->parameters = $parameters; - return $this; + return true; } public function matchRoute(Request $request) { diff --git a/src/Pecee/SimpleRouter/RouterRoute.php b/src/Pecee/SimpleRouter/RouterRoute.php index 5e6e34b..ec30640 100644 --- a/src/Pecee/SimpleRouter/RouterRoute.php +++ b/src/Pecee/SimpleRouter/RouterRoute.php @@ -26,9 +26,9 @@ class RouterRoute extends RouterEntry { // Match on custom defined regular expression if($this->regexMatch) { $parameters = array(); - if(preg_match('/('.$this->regexMatch.')/is', request()->getHost() . $url, $parameters)) { + if(preg_match('/('.$this->regexMatch.')/is', $request->getHost() . $url, $parameters)) { $this->parameters = (!is_array($parameters[0]) ? array($parameters[0]) : $parameters[0]); - return $this; + return true; } return null; } @@ -46,7 +46,7 @@ class RouterRoute extends RouterEntry { $this->parameters = $parameters; } - return $this; + return true; } return null; diff --git a/src/Pecee/SimpleRouter/SimpleRouter.php b/src/Pecee/SimpleRouter/SimpleRouter.php index 55a9b22..c244d95 100644 --- a/src/Pecee/SimpleRouter/SimpleRouter.php +++ b/src/Pecee/SimpleRouter/SimpleRouter.php @@ -19,7 +19,7 @@ class SimpleRouter { * @throws RouterException */ public static function start($defaultNamespace = null) { - $router = RouterBase::GetInstance(); + $router = RouterBase::getInstance(); $router->setDefaultNamespace($defaultNamespace); $router->routeRequest(); }