Merge pull request #93 from skipperbent/development

Development

.gitignore

@@ -1,4 +1,3 @@
 .idea
 composer.lock
-vendor/
-demo-project/vendor
+vendor/

README.md

@@ -18,7 +18,7 @@ The goal of this project is to create a router that is 100% compatible with the
 
 ### Features
 
-- Basic routing (`GET`, `POST`, `PUT`, `DELETE`) with support for custom multiple verbs.
+- Basic routing (get, post, put, delete) with support for custom multiple verbs.
 - Regular Expression Constraints for parameters.
 - Named routes.
 - Generating url to routes.
@@ -32,14 +32,6 @@ The goal of this project is to create a router that is 100% compatible with the
 - Custom boot managers to redirect urls to other routes
 - Input manager; to manage `GET`, `POST` params.
 
-## Installation and demo
-
-We've included a simple demo project for the router which can be found in the `demo-project` folder.
-
-Please refer to the demo-project documentation for further reading on how to setup and install simple-php-router:
-
-[Link to demo documentation](demo-project/README.md)
-
 ## Initialising the router
 
 In your ```index.php``` require your ```routes.php``` and call the ```routeRequest()``` method when all your custom routes has been loaded. This will trigger and do the actual routing of the requests.
@@ -49,18 +41,13 @@ This is an example of a basic ```index.php``` file:
 ```php
 use \Pecee\SimpleRouter\SimpleRouter;
 
-// Load external routes file
-require_once 'routes.php';
+require_once 'routes.php'; // change this to whatever makes sense in your project
 
-/* 
- * The default namespace for route-callbacks, so we don't have to specify it each time.
- * Can be overwritten by using the namespace config option.
- */
- 
-SimpleRouter::setDefaultNamespace('MyWebsite\Controller');
+// The apps default namespace (so we don't have to specify it each time we use MyController@home)
+$defaultControllerNamespace = 'MyWebsite\\Controller';
 
-// Start the routing
-SimpleRouter::start();
+// Do the routing
+SimpleRouter::start($defaultControllerNamespace);
 ```
 
 ## Adding routes
@@ -86,9 +73,7 @@ use Pecee\SimpleRouter\SimpleRouter;
 // Add CSRF support (if needed)
 SimpleRouter::csrfVerifier(new \Pecee\Http\Middleware\BaseCsrfVerifier());
 
-SimpleRouter::get('/page/404', 'ControllerPage@notFound', ['as' => 'page.notfound']);
-
-SimpleRouter::group(['prefix' => '/v1', 'middleware' => '\MyWebsite\Middleware\SomeMiddlewareClass'], function() {
+SimpleRouter::group(['prefix' => 'v1', 'middleware' => '\MyWebsite\Middleware\SomeMiddlewareClass'], function() {
 
     SimpleRouter::group(['prefix' => '/services', 'exceptionHandler' => '\MyProject\Handler\CustomExceptionHandler'], function() {
 
@@ -118,42 +103,6 @@ SimpleRouter::group(['prefix' => '/v1', 'middleware' => '\MyWebsite\Middleware\S
 });
 ```
 
-#### ExceptionHandler example
-
-This is a basic example of an ExceptionHandler implementation:
-
-```php
-namespace Demo\Handlers;
-
-use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
-
-class CustomExceptionHandler implements IExceptionHandler {
-
-    public function handleError( Request $request, RouterEntry $router = null, \Exception $error) {
-
-        // If the error-code is 404; show another route which contains the page-not-found
-        if($error->getCode() === 404) {
-        
-            // Throw your custom 404-page view
-            // - or -
-            // load another route with our 404 page
-            
-            return $request->setUri(url('page.notfound'));
-        }
-
-        // Output error as json if on api path.
-        if(stripos($request->getUri(), '/api') !== false) {
-            response()->json(['error' => $error->getMessage()]);
-        }
-
-        // Otherwise default exception will be thrown by the router.
-
-    }
-
-}
-```
-
 ### Sub-domain routing
 
 Route groups may also be used to route wildcard sub-domains. Sub-domains may be assigned route parameters just like route URIs, allowing you to capture a portion of the sub-domain for usage in your route or controller. The sub-domain may be specified using the ```domain``` key on the group attribute array:
@@ -196,39 +145,61 @@ This is a simple example of an integration into a framework.
 The framework has it's own ```Router``` class which inherits from the ```SimpleRouter``` class. This allows the framework to add custom functionality.
 
 ```php
-namespace Demo;
+namespace MyProject;
 
+use Pecee\Handler\ExceptionHandler;
+use Pecee\SimpleRouter\RouterBase;
 use Pecee\SimpleRouter\SimpleRouter;
 
 class Router extends SimpleRouter {
 
-    public static function start() {
+    protected static $defaultExceptionHandler;
 
-        // change this to whatever makes sense in your project
-        require_once 'routes.php';
-        
-        // change default namespace for all routes
-        parent::setDefaultNamespace('\Demo\Controllers');
+    public static function start($defaultNamespace = null) {
 
-        // Do initial stuff
-        parent::start();
+        // Load routes.php
+        $file = $_ENV['base_path'] . DIRECTORY_SEPARATOR . 'lib' . DIRECTORY_SEPARATOR . 'routes.php';
+        if(file_exists($file)) {
+            require_once $file;
+        }
 
+        // Set default namespace
+        $defaultNamespace = '\\'.$_ENV['app_name'] . '\\Controller';
+
+        // Handle exceptions
+        try {
+            parent::start($defaultNamespace);
+        } catch(\Exception $e) {
+
+            if(self::$defaultExceptionHandler !== null) {
+                $class = new self::$defaultExceptionHandler();
+                $class->handleError(RouterBase::getInstance()->getRequest(), $route, $e);
+            }
+
+            throw $e;
+        }
+    }
+
+    public static function setDefaultExceptionHandler($handler) {
+        self::$defaultExceptionHandler = $handler;
     }
 
 }
 ```
 
-## Helper functions
-
-To simplify to use of simple-router functionality, we recommend you add these helper functions to your project.
+#### Helper functions examples
+**This is a basic example of a helper function for generating urls.**
 
 ```php
-use Pecee\SimpleRouter\SimpleRouter;
-
+use Pecee\SimpleRouter\RouterBase;
 function url($controller, $parameters = null, $getParams = null) {
-    SimpleRouter::getRoute($controller, $parameters, $getParams);
+    RouterBase::getInstance()->getRoute($controller, $parameters, $getParams);
 }
+```
 
+**This is a basic example for getting the current csrf token**
+
+```php
 /**
  * Get current csrf-token
  * @return null|string
@@ -237,30 +208,6 @@ function csrf_token() {
     $token = new \Pecee\CsrfToken();
     return $token->getToken();
 }
-
-/**
- * Get request object
- * @return \Pecee\Http\Request
- */
-function request() {
-    return SimpleRouter::request();
-}
-
-/**
- * Get response object
- * @return \Pecee\Http\Response
- */
-function response() {
-    return SimpleRouter::response();
-}
-
-/**
- * Get input class
- * @return \Pecee\Http\Input\Input
- */
-function input() {
-    return SimpleRouter::request()->getInput();
-}
 ```
 
 ## Getting urls
@@ -296,10 +243,7 @@ use Pecee\Http\Middleware\BaseCsrfVerifier;
 
 class CsrfVerifier extends BaseCsrfVerifier {
 
-    protected $except = [
-        '/companies/*', 
-        '/api'
-    ];
+    protected $except = ['/companies/*', '/user/save'];
 
 }
 ```
@@ -317,6 +261,7 @@ Sometimes it can be necessary to keep urls stored in the database, file or simil
 To interfere with the router, we create a class that inherits from ```RouterBootManager```. This class will be loaded before any other rules in ```routes.php``` and allow us to "change" the current route, if any of our criteria are fulfilled (like coming from the url ```/my-cat-is-beatiful```).
 
 ```php
+
 use Pecee\Http\Request;
 use Pecee\SimpleRouter\RouterBootManager;
 
@@ -353,113 +298,62 @@ By doing this the route will now load the url ```/article/view/1``` instead of `
 
 The last thing we need to do, is to add our custom boot-manager to the ```routes.php``` file. You can create as many bootmanagers as you like and easily add them in your ```routes.php``` file.
 
-## Easily overwrite route about to be loaded
-Sometimes it can be useful to manipulate the route about to be loaded. 
-simple-php-router allows you to easily change the route about to be executed. 
-All information about the current route is stored in the ```\Pecee\SimpleRouter\RouterBase``` instance's `loadedRoute` property. 
-
-For easy access you can use the shortcut method `\Pecee\SimpleRouter\SimpleRouter::router()`.
-
+**routes.php example:**
 
 ```php
-use Pecee\SimpleRouter;
-$route = SimpleRouter::router()->getLoadedRoute();
+// Add new bootmanager
+SimpleRouter::addBootManager(new CustomRouterRules());
+
+// This rule is what our custom bootmanager will use.
+SimpleRouter::get('/article/view/{id}', 'ControllerArticle@view');
+```
+
+## Easily overwrite route about to be loaded
+Sometimes it can be useful to manipulate the route that's about to be loaded, for instance if a user is not authenticated or if an error occurred within your Middleware that requires
+some other route to be initialised. Simple PHP Router allows you to easily change the route about to be executed. All information about the current route is stored in
+the ```\Pecee\SimpleRouter\Http\Request``` object.
+
+**Note:** Please note that it's only possible to change the route BEFORE any route has initially been loaded, so doing this in your custom ExceptionHandler or Middleware is highly recommended.
+
+```php
+$route = Request::getInstance()->getLoadedRoute();
 
 $route->setCallback('Example\MyCustomClass@hello');
 
-// -- or you can rewrite by doing --
+// -- or --
 
 $route->setClass('Example\MyCustomClass');
 $route->setMethod('hello');
 ```
 
-
-### Examples
-
-It's only possible to change the route BEFORE the route has initially been loaded. If you want to redirect to another route, we highly recommend that you 
-modify the `RouterEntry` object from a `Middleware` or `ExceptionHandler`, like the examples below.
-
-#### Faking new route
-
-The example below will cause the router to re-route the request with another url. We are using the `url()` helper function to get the uri to another route added in the `routes.php` file.
- 
-This does require the `$request` object to be returned, otherwise the `request` object will be ignored by the router.
-
-Using the example below will NOT inherit the rules from the other route. This means that IF you are faking a route that is enabled in `post`.
-
-**NOTE: Use this method if you want to fully load a route (middlewares, request-method etc. will be kept).**
-
-
-```php
-namespace demo\Middlewares;
-
-use Pecee\Http\Middleware\IMiddleware;
-use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
-
-class CustomMiddleware implements Middleware {
-
-    public function handle(Request $request, RouterEntry &$route = null) {
-        return $request->setUri(url('home'));
-    }
-    
-}
-
-```
-
-#### Changing callback
-You can also change the callback by modifying the `$route` parameter. This is perfect if you just want to display a view quickly - or change the callback depending
-on some criteria's for the request.
-
-The callback below will fire immediately after the `Middleware` or `ExceptionHandler` has been loaded, as they are loaded before the route is rendered.
-If you wish to change the callback from outside, please have this in mind.
-
-**NOTE: Use this method if you want to load another controller. No additional middlewares or rules will be loaded.**
-
-```php
-namespace demo\Middlewares;
-
-use Pecee\Http\Middleware\IMiddleware;
-use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
-
-class CustomMiddleware implements Middleware {
-
-    public function handle(Request $request, RouterEntry &$route = null) { 
-        $route->callback('DefaultController@home');
-    }
-    
-}
-```
-
 ## Using the Input class to manage parameters
 
 We've added the `Input` class to easy access parameters from your Controller-classes.
 
 **Return single parameter value (matches both GET, POST, FILE):**
 ```php
-$value = input()->get('name');
+$value = Request::getInstance()->getInput()->get('name');
 ```
 
 **Return parameter object (matches both GET, POST, FILE):**
 ```php
-$object = input()->getObject('name');
+$object = Request::getInstance()->getInput()->getObject('name');
 ```
 
 **Return specific GET parameter (where name is the name of your parameter):**
 ```php
-$object = input()->get->name;
-$object = input()->post->name;
-$object = input()->file->name;
+$object = Request::getInstance()->getInput()->get->name;
+$object = Request::getInstance()->getInput()->post->name;
+$object = Request::getInstance()->getInput()->file->name;
 ```
 
 **Return all parameters:**
 ```php
 // Get all
-$values = input()->all();
+$objects = Request::getInstance()->getInput()->all();
 
 // Only match certain keys
-$values = input()->all([
+$objects = Request::getInstance()->getInput()->all([
     'company_name',
     'user_id'
 ]);
@@ -479,13 +373,26 @@ All object inherits from `InputItem` class and will always contain these methods
 - `getError()` - get file upload error.
 
 
-### Easy access to methods
+### Easy access your input
+Create a helper function to easily get access to the input elements.
 
-Below example requires you to have the helper functions added. Please refer to the helper functions section in the documentation.
+Example:
+
+```php
+/**
+ * Get input class
+ * @return \Pecee\Http\Input\Input
+ */
+function input() {
+    return new \Pecee\Http\Input\Input();
+}
+```
+
+Then you can easily do something like this in your controller:
 
 ```php
 // Get parameter site_id or default-value 2
-$siteId = input()->get('site_id', 2);
+$value = input()->get('site_id', '2');
 ```
 
 ## Sites
@@ -493,8 +400,6 @@ This is some sites that uses the simple-router project in production.
 
 - [holla.dk](http://www.holla.dk)
 - [ninjaimg.com](http://ninjaimg.com)
-- [bookandbegin.com](https://bookandbegin.com)
-- [dscuz.com](https://www.dscuz.com)
 
 ## Documentation
 While I work on a better documentation, please refer to the Laravel 5 routing documentation here:

demo-project/README.md

@@ -1,100 +0,0 @@
-# Simple PHP router demo project
-
-This project is here to give you a basic understanding of how to setup and using simple-php-router.
-
-Please note that this demo-project only covers how to integrate the `simple-php-router` in a project without a framework. If you are using some sort of PHP framework in your project the implementation might vary.
-
-**What we won't cover:**
-
-- How to setup a solution that fits your need. This is a basic demo to help you get started.
-- Understanding of MVC; including Controllers, Middlewares or ExceptionHandlers.
-- How to integrate into third party frameworks.
-
-**What we cover:**
-
-- How to get up and running fast - from scratch.
-- How to get ExceptionHandlers, Middlewares and Controllers working.
-- How to setup your webservers.
-
-## Installation
-
-- Navigate to the `demo-project` folder in terminal and run `composer update` to install the latest version.
-- Point your webserver to `demo-project/public`.
-
-### Setting up Nginx
-
-If you are using Nginx please make sure that url-rewriting is enabled.
-
-You can easily enable url-rewriting by adding the following configuration for the Nginx configuration-file for the demo-project.
-
-```
-location / {
-    try_files $uri $uri/ /index.php?$query_string;
-}
-```
-
-### Setting up Apache
-
-Nothing special is required for Apache to work. We've include the `.htaccess` file in the `public` folder. If rewriting is not working for you, please check that the `mod_rewrite` module (htaccess support) is enabled in the Apache configuration.
-
-## Folder structure
-
-| Folder        | Description |
-| ------------- |-------------|
-| app            |Contains projects-specific PHP classes|
-| public         |Public folder which are accessible through the web.|
-
-## Notes
-
-The demo project has it's own `Router` class implementation which extends the `SimpleRouter` class with further functionality. 
-This class can be useful adding additional functionality that are required before and after routing occurs or any extra functionality belonging to the router itself. 
-
-In this project we also use our custom router-class to autoload the `routes.php` file from our custom location (`app/routes.php`).
-
-Please check the `routes.php` file in `demo-project/app` for all the urls/rules available in the project.
-
-### CSRF-verifier
-
-For the purpose of this demo, we've added a custom CSRF-verifier middleware called `CsrfVerifier` and disabled CSRF checks for all calls to `/api/*`. This will ensure that CSRF form-checks are not applied when calling our demo api url.
-
-### Exception handlers
-
-The included `CustomExceptionHandler` class returns a very basic json response for errors received on calls to `/api/*` or otherwise just a simple formatted error response.
-
-### Middlewares
-
-`ApiVerification` class is added to all calls to `/api/*`. This simple class just adds some data to the `Request` object, which is returned in one of the methods in the `ApiController` class. We've added this class to demonstrate that you can use middlewares to ensure that the user has the correct authentication - before router loads the controller itself.
-
-### Urls
-
-Please see `routes.php` for all routes and rules.
-
-| URL        |
-| ------------- |
-| /             |
-| /api/demo       |
-| /companies       |
-| /companies/[id]  |
-| /contact         |
-
-## The MIT License (MIT)
-
-Copyright (c) 2016 Simon Sessingø / simple-php-router
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

demo-project/app/Controllers/ApiController.php

@@ -1,20 +0,0 @@
-<?php
-namespace Demo\Controllers;
-
-use Pecee\SimpleRouter\SimpleRouter;
-
-class ApiController {
-
-    public function index() {
-
-        // The variable authenticated is set to true in the ApiVerification middleware class.
-
-        header('content-type: application/json');
-
-        echo json_encode([
-            'authenticated' => request()->authenticated
-        ]);
-
-    }
-
-}

demo-project/app/Controllers/DefaultController.php

@@ -1,29 +0,0 @@
-<?php
-namespace Demo\Controllers;
-
-class DefaultController {
-
-    public function index() {
-
-        // implement
-        echo sprintf('DefaultController -> index (?fun=%s)', input()->get('fun'));
-
-    }
-
-    public function contact() {
-
-        echo 'DefaultController -> contact';
-
-    }
-
-    public function companies($id = null) {
-
-        echo 'DefaultController -> companies -> id: ' . $id;
-
-    }
-
-    public function notFound() {
-        echo 'Page not found';
-    }
-
-}

demo-project/app/Handlers/CustomExceptionHandler.php

@@ -1,34 +0,0 @@
-<?php
-namespace Demo\Handlers;
-
-use Pecee\Handler\IExceptionHandler;
-use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
-
-class CustomExceptionHandler implements IExceptionHandler {
-
-    public function handleError( Request $request, RouterEntry &$route = null, \Exception $error) {
-
-        // Return json errors if we encounter an error on /api.
-        if(stripos($request->getUri(), '/api') !== false) {
-            header('content-type: application/json');
-            echo json_encode([
-                'error' => $error->getMessage(),
-                'code' => $error->getCode()
-            ]);
-            die();
-        }
-
-        // else we just throw the error
-        if($error->getCode() == 404) {
-
-            // Return 404 path
-            $request->setUri('/404');
-            return $request;
-
-        }
-
-        throw $error;
-    }
-
-}

demo-project/app/Middlewares/ApiVerification.php

@@ -1,17 +0,0 @@
-<?php
-namespace Demo\Middlewares;
-
-use Pecee\Http\Middleware\IMiddleware;
-use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
-
-class ApiVerification implements IMiddleware {
-
-    public function handle(Request $request, RouterEntry &$route = null) {
-
-        // Do authentication
-        $request->authenticated = true;
-
-    }
-
-}

demo-project/app/Middlewares/CsrfVerifier.php

@@ -1,13 +0,0 @@
-<?php
-namespace Demo\Middlewares;
-
-use Pecee\Http\Middleware\BaseCsrfVerifier;
-
-class CsrfVerifier extends BaseCsrfVerifier {
-
-    /**
-     * CSRF validation will be ignored on the following urls.
-     */
-    protected $except = ['/api/*'];
-
-}

demo-project/app/Router.php

@@ -1,29 +0,0 @@
-<?php
-
-/**
- * Custom router which handles default middlewares, default exceptions and things
- * that should be happen before and after the router is initialised.
- */
-
-namespace Demo;
-
-use Pecee\SimpleRouter\SimpleRouter;
-
-class Router extends SimpleRouter {
-
-    public static function start($defaultNamespace = null) {
-
-        // Load our helpers
-        require_once 'helpers.php';
-
-        // Load our custom routes
-        require_once 'routes.php';
-
-        parent::setDefaultNamespace('\Demo\Controllers');
-
-        // Do initial stuff
-        parent::start();
-
-    }
-
-}

demo-project/app/helpers.php

@@ -1,39 +0,0 @@
-<?php
-use Pecee\SimpleRouter\SimpleRouter;
-
-function url($controller, $parameters = null, $getParams = null) {
-    SimpleRouter::getRoute($controller, $parameters, $getParams);
-}
-
-/**
- * Get current csrf-token
- * @return null|string
- */
-function csrf_token() {
-    $token = new \Pecee\CsrfToken();
-    return $token->getToken();
-}
-
-/**
- * Get request object
- * @return \Pecee\Http\Request
- */
-function request() {
-    return SimpleRouter::request();
-}
-
-/**
- * Get response object
- * @return \Pecee\Http\Response
- */
-function response() {
-    return SimpleRouter::response();
-}
-
-/**
- * Get input class
- * @return \Pecee\Http\Input\Input
- */
-function input() {
-    return SimpleRouter::request()->getInput();
-}

demo-project/app/routes.php

@@ -1,23 +0,0 @@
-<?php
-/**
- * This file contains all the routes for the project
- */
-
-use Demo\Router;
-
-Router::csrfVerifier(new \Demo\Middlewares\CsrfVerifier());
-
-Router::group(['exceptionHandler' => 'Demo\Handlers\CustomExceptionHandler'], function() {
-
-    Router::get('/', 'DefaultController@index')->setAlias('home');
-    Router::get('/contact', 'DefaultController@contact')->setAlias('contact');
-    Router::get('/404', 'DefaultController@notFound')->setAlias('404');
-    Router::basic('/companies', 'DefaultController@companies')->setAlias('companies');
-    Router::basic('/companies/{id}', 'DefaultController@companies')->setAlias('companies');
-
-    // Api
-    Router::group(['prefix' => '/api', 'middleware' => 'Demo\Middlewares\ApiVerification'], function() {
-        Router::resource('/demo', 'ApiController');
-    });
-
-});

demo-project/composer.json

@@ -1,26 +0,0 @@
-{
- 	"name": "pecee/simple-router-demo",
- 	"description": "Simple router demo project",
- 	"keywords": [
-		"simple-router",
-		"php",
-		"php-simple-router"
-	],
-	"license": "MIT",
-	"type": "project",
-	"require": {
-		"php": ">=5.4.0",
-		"pecee/simple-router": "1.*"
-	},
-	"require-dev": {
-
-	},
-	"config": {
-		"preferred-install": "dist"
-	},
-    "autoload": {
-      "psr-4": {
-        "Demo\\": "app/"
-      }
-    }
-}

demo-project/public/.htaccess

@@ -1,5 +0,0 @@
-RewriteEngine on
-RewriteCond %{SCRIPT_FILENAME} !-f
-RewriteCond %{SCRIPT_FILENAME} !-d
-RewriteCond %{SCRIPT_FILENAME} !-l
-RewriteRule ^(.*)$ index.php/$1

demo-project/public/index.php

@@ -1,7 +0,0 @@
-<?php
-
-// load composer dependencies
-require '../vendor/autoload.php';
-
-// Start the routing
-\Demo\Router::start();

src/Pecee/Exception/RouterException.php

@@ -1,4 +1,3 @@
 <?php
 namespace Pecee\Exception;
-
 class RouterException extends \Exception { }

src/Pecee/Handler/IExceptionHandler.php

@@ -6,12 +6,6 @@ use Pecee\SimpleRouter\RouterEntry;
 
 interface IExceptionHandler {
 
-    /**
-     * @param Request $request
-     * @param RouterEntry|null $route
-     * @param \Exception $error
-     * @return Request|null
-     */
-	public function handleError(Request $request, RouterEntry &$route = null, \Exception $error);
+	public function handleError(Request $request, RouterEntry $router = null, \Exception $error);
 
 }

src/Pecee/Http/Input/Input.php

@@ -1,8 +1,6 @@
 <?php
 namespace Pecee\Http\Input;
 
-use Pecee\Http\Request;
-
 class Input {
 
     /**
@@ -20,13 +18,7 @@ class Input {
      */
     public $file;
 
-    /**
-     * @var Request
-     */
-    protected $request;
-
-    public function __construct(Request $request) {
-        $this->request = $request;
+    public function __construct() {
         $this->setGet();
         $this->setPost();
         $this->setFile();
@@ -38,58 +30,41 @@ class Input {
      * @return array
      */
     public function all(array $filter = null) {
-
-        $output = $_POST;
-
-        if($this->request->getMethod() === 'post') {
-
-            $contents = file_get_contents('php://input');
-
-            if (stripos(trim($contents), '{') === 0) {
-                $output = json_decode($contents, true);
-                if($output === false) {
-                    $output = array();
-                }
-            }
-        }
-
-        $output = array_merge($_GET, $output);
+        $output = $this->get->getData();
+        $output = array_merge($output, $this->post->getData());
 
         if($filter !== null) {
-            $output = array_filter($output, function ($key) use ($filter) {
-                if (in_array($key, $filter)) {
-                    return true;
+            $tmp = array();
+            foreach($output as $key => $val) {
+                if(in_array($key, $filter)) {
+                    $tmp[$key] = $val;
                 }
-
-                return false;
-            }, ARRAY_FILTER_USE_KEY);
+            }
+            return $tmp;
         }
 
         return $output;
     }
 
     public function getObject($index, $default = null) {
-        $key = (strpos($index, '[') > -1) ? substr($index, strpos($index, '[')+1, strpos($index, ']') - strlen($index)) : null;
         $index = (strpos($index, '[') > -1) ? substr($index, 0, strpos($index, '[')) : $index;
 
         $element = $this->get->findFirst($index);
 
         if($element !== null) {
-            return ($key !== null) ? $element[$key] : $element;
+            return $element;
         }
 
-        if($this->request->getMethod() !== 'get') {
+        $element = $this->post->findFirst($index);
 
-            $element = $this->post->findFirst($index);
+        if($element !== null) {
+            return $element;
+        }
 
-            if ($element !== null) {
-                return ($key !== null) ? $element[$key] : $element;
-            }
+        $element = $this->file->findFirst($index);
 
-            $element = $this->file->findFirst($index);
-            if ($element !== null) {
-                return ($key !== null) ? $element[$key] : $element;
-            }
+        if($element !== null) {
+            return $element;
         }
 
         return $default;
@@ -103,24 +78,23 @@ class Input {
      */
     public function get($index, $default = null) {
 
+        $key = (strpos($index, '[') > -1) ? substr($index, strpos($index, '[')+1, strpos($index, ']') - strlen($index)) : null;
+        $index = (strpos($index, '[') > -1) ? substr($index, 0, strpos($index, '[')) : $index;
+
         $item = $this->getObject($index);
 
         if($item !== null) {
 
-            if($item instanceof InputCollection || $item instanceof InputFile) {
-                return $item;
+            if (is_array($item->getValue())) {
+                return ($key !== null && isset($item->getValue()[$key])) ? $item->getValue()[$key] : $item->getValue();
             }
 
-            return (trim($item->getValue()) === '') ? $default : $item->getValue();
+            return ($item->getValue() === null) ? $default : $item->getValue();
         }
 
         return $default;
     }
 
-    public function exists($index) {
-        return ($this->getObject($index) !== null);
-    }
-
     public function setGet() {
         $this->get = new InputCollection();
 
@@ -131,13 +105,13 @@ class Input {
                     continue;
                 }
 
-                $output = new InputCollection();
+                $output = array();
 
                 foreach($get as $k => $g) {
-                    $output->{$k} = new InputItem($k, $g);
+                    $output[$k] = new InputItem($k, $g);
                 }
 
-                $this->get->{$key} = $output;
+                $this->get->{$key} = new InputItem($key, $output);
             }
         }
     }
@@ -145,10 +119,12 @@ class Input {
     public function setPost() {
         $this->post = new InputCollection();
 
-        $postVars = $_POST;
+        $postVars = array();
 
-        if(in_array($this->request->getMethod(), ['put', 'patch', 'delete'])) {
+        if(in_array($_SERVER['REQUEST_METHOD'], ['PUT', 'PATCH', 'DELETE'])) {
             parse_str(file_get_contents('php://input'), $postVars);
+        } else {
+            $postVars = $_POST;
         }
 
         if(count($postVars)) {
@@ -159,13 +135,13 @@ class Input {
                     continue;
                 }
 
-                $output = new InputCollection();
+                $output = array();
 
-                foreach($post as $k => $p) {
-                    $output->{$k} = new InputItem($k, $p);
+                foreach($post as $k=>$p) {
+                    $output[$k] = new InputItem($k, $p);
                 }
 
-                $this->post->{strtolower($key)} = $output;
+                $this->post->{strtolower($key)} = new InputItem($key, $output);
             }
         }
     }
@@ -190,7 +166,7 @@ class Input {
                     continue;
                 }
 
-                $output = new InputCollection();
+                $output = array();
 
                 foreach($value['name'] as $k=>$val) {
                     // Strip empty values
@@ -201,11 +177,11 @@ class Input {
                         $file->setType($value['type'][$k]);
                         $file->setTmpName($value['tmp_name'][$k]);
                         $file->setError($value['error'][$k]);
-                        $output->{$k} = $file;
+                        $output[$k] = $file;
                     }
                 }
 
-                $this->file->{strtolower($key)} = $output;
+                $this->file->{strtolower($key)} = new InputItem($key, $output);
             }
         }
     }

src/Pecee/Http/Input/InputCollection.php

@@ -10,10 +10,9 @@ class InputCollection implements \IteratorAggregate {
      * Useful for searching for finding items where $index doesn't contain form name.
      *
      * @param string $index
-     * @param string|null $defaultValue
      * @return mixed
      */
-    public function findFirst($index, $defaultValue = null) {
+    public function findFirst($index) {
         if(count($this->data)) {
 
             if(isset($this->data[$index])) {
@@ -27,24 +26,7 @@ class InputCollection implements \IteratorAggregate {
             }
         }
 
-        return $defaultValue;
-    }
-
-    public function getValue($index, $defaultValue = null) {
-        if(count($this->data)) {
-
-            if(isset($this->data[$index])) {
-                return $this->data[$index]->getValue();
-            }
-
-            foreach($this->data as $key => $value) {
-                if(strtolower($index) === strtolower($key)) {
-                    return $value->getValue();
-                }
-            }
-        }
-
-        return $defaultValue;
+        return null;
     }
 
     /**

src/Pecee/Http/Input/InputItem.php

@@ -46,16 +46,6 @@ class InputItem {
         return $this;
     }
 
-    /**
-     * Set input value
-     * @param string $value
-     * @return static $this
-     */
-    public function setValue($value) {
-        $this->value = $value;
-        return $this;
-    }
-
     public function __toString() {
         return (string)$this->getValue();
     }

src/Pecee/Http/Middleware/BaseCsrfVerifier.php

@@ -4,7 +4,6 @@ namespace Pecee\Http\Middleware;
 use Pecee\CsrfToken;
 use Pecee\Exception\TokenMismatchException;
 use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
 
 class BaseCsrfVerifier implements IMiddleware {
 
@@ -50,11 +49,11 @@ class BaseCsrfVerifier implements IMiddleware {
         return false;
     }
 
-    public function handle(Request $request, RouterEntry &$route = null) {
+    public function handle(Request $request) {
 
-        if($request->getMethod() !== 'get' && !$this->skip($request)) {
+        if($request->getMethod() != 'get' && !$this->skip($request)) {
 
-            $token = $request->getInput()->post->getValue(static::POST_KEY);
+            $token = (isset($_POST[static::POST_KEY])) ? $_POST[static::POST_KEY] : null;
 
             // If the token is not posted, check headers for valid x-csrf-token
             if($token === null) {

src/Pecee/Http/Middleware/IMiddleware.php

@@ -2,15 +2,7 @@
 namespace Pecee\Http\Middleware;
 
 use Pecee\Http\Request;
-use Pecee\SimpleRouter\RouterEntry;
 
 interface IMiddleware {
-
-    /**
-     * @param Request $request
-     * @param RouterEntry|null $route
-     * @return Request|null
-     */
-    public function handle(Request $request, RouterEntry &$route = null);
-
+    public function handle(Request $request);
 }

src/Pecee/Http/Request.php

@@ -5,41 +5,45 @@ use Pecee\Http\Input\Input;
 
 class Request {
 
-    protected $data = array();
-    protected $headers;
-    protected $host;
-    protected $uri;
-    protected $method;
-    protected $input;
+    protected static $instance;
 
+    protected $data;
+
+    /**
+     * Return new instance
+     * @return static
+     */
+    public static function getInstance() {
+        if(self::$instance === null) {
+            self::$instance = new static();
+        }
+        return self::$instance;
+    }
 
     public function __construct() {
-        $this->parseHeaders();
-        $this->input = new Input($this);
-
-        $this->host = $this->getHeader('http_host');;
-        $this->uri = $this->getHeader('request_uri');
-        $this->method = strtolower($this->input->post->findFirst('_method', $this->getHeader('request_method')));
+        $this->data = array();
+        $this->host = $_SERVER['HTTP_HOST'];
+        $this->uri = $_SERVER['REQUEST_URI'];
+        $this->method = (isset($_POST['_method'])) ? strtolower($_POST['_method']) : strtolower($_SERVER['REQUEST_METHOD']);
+        $this->headers = $this->getAllHeaders();
+        $this->input = new Input();
     }
 
-    protected function parseHeaders() {
-        $this->headers = array();
-
+    protected function getAllHeaders() {
+        $headers = array();
         foreach ($_SERVER as $name => $value) {
-            $this->headers[strtolower($name)] = $value;
+            if (substr($name, 0, 5) === 'HTTP_') {
+                $headers[strtolower(str_replace('_', '-', substr($name, 5)))] = $value;
+            }
         }
+        return $headers;
     }
 
-    public function isSecure() {
-        if($this->getHeader('http_x_forwarded_proto') === 'https') {
+    public function getIsSecure() {
+        if(isset($_SERVER['HTTP_X_FORWARDED_PROTO']) && strtolower($_SERVER['HTTP_X_FORWARDED_PROTO']) === 'https') {
             return true;
         }
-
-        if($this->getHeader('https') !== null) {
-            return true;
-        }
-
-        return ($this->getHeader('server_port') === 443);
+        return isset($_SERVER['HTTPS']) ? true : (isset($_SERVER['SERVER_PORT']) && $_SERVER['SERVER_PORT'] === 443);
     }
 
     /**
@@ -68,7 +72,7 @@ class Request {
      * @return string|null
      */
     public function getUser() {
-        return $this->getHeader('php_auth_user');
+        return (isset($_SERVER['PHP_AUTH_USER'])) ? $_SERVER['PHP_AUTH_USER']: null;
     }
 
     /**
@@ -76,11 +80,11 @@ class Request {
      * @return string|null
      */
     public function getPassword() {
-        return $this->getHeader('php_auth_pw');
+        return (isset($_SERVER['PHP_AUTH_PW'])) ? $_SERVER['PHP_AUTH_PW']: null;
     }
 
     /**
-     * Get all headers
+     * Get headers
      * @return array
      */
     public function getHeaders() {
@@ -92,15 +96,7 @@ class Request {
      * @return string
      */
     public function getIp() {
-        if($this->getHeader('http_cf_connecting_ip') !== null) {
-            return $this->getHeader('http_cf_connecting_ip');
-        }
-
-        if($this->getHeader('http_x_forwarded_for') !== null && strlen($this->getHeader('http_x_forwarded_for'))) {
-            return $this->getHeader('http_x_forwarded_for');
-        }
-
-        return $this->getHeader('remote_addr');
+        return ((isset($_SERVER['HTTP_X_FORWARDED_FOR']) && strlen($_SERVER['HTTP_X_FORWARDED_FOR'])) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : $_SERVER['REMOTE_ADDR']);
     }
 
     /**
@@ -108,7 +104,7 @@ class Request {
      * @return string
      */
     public function getReferer() {
-        return $this->getHeader('http_referer');
+        return isset($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : '';
     }
 
     /**
@@ -116,17 +112,16 @@ class Request {
      * @return string
      */
     public function getUserAgent() {
-        return $this->getHeader('http_user_agent');
+        return isset($_SERVER['HTTP_USER_AGENT']) ? $_SERVER['HTTP_USER_AGENT'] : '';
     }
 
     /**
      * Get header value by name
      * @param string $name
-     * @param object|null $defaultValue
      * @return string|null
      */
-    public function getHeader($name, $defaultValue = null) {
-        return isset($this->headers[strtolower($name)]) ? $this->headers[strtolower($name)] : $defaultValue;
+    public function getHeader($name) {
+        return (isset($this->headers[strtolower($name)])) ? $this->headers[strtolower($name)] : null;
     }
 
     /**
@@ -137,42 +132,15 @@ class Request {
         return $this->input;
     }
 
-    /**
-     * Is format accepted
-     * @param string $format
-     * @return bool
-     */
     public function isFormatAccepted($format) {
-        return ($this->getHeader('http_accept') !== null && stripos($this->getHeader('http_accept'), $format) > -1);
+        return (isset($_SERVER['HTTP_ACCEPT']) && stripos($_SERVER['HTTP_ACCEPT'], $format) > -1);
     }
 
-    /**
-     * Get accept formats
-     * @return array
-     */
     public function getAcceptFormats() {
-        return explode(',', $this->getHeader('http_accept'));
-    }
-
-    /**
-     * @param string $uri
-     */
-    public function setUri($uri) {
-        $this->uri = $uri;
-    }
-
-    /**
-     * @param string $host
-     */
-    public function setHost($host) {
-        $this->host = $host;
-    }
-
-    /**
-     * @param string $method
-     */
-    public function setMethod($method) {
-        $this->method = $method;
+        if(isset($_SERVER['HTTP_ACCEPT'])) {
+            return explode(',', $_SERVER['HTTP_ACCEPT']);
+        }
+        return array();
     }
 
     public function __set($name, $value = null) {
@@ -183,4 +151,33 @@ class Request {
         return isset($this->data[$name]) ? $this->data[$name] : null;
     }
 
+    /**
+     * Get the currently loaded route.
+     * @return \Pecee\SimpleRouter\RouterEntry
+     */
+    public function getLoadedRoute() {
+        return $this->loadedRoute;
+    }
+
+    /**
+     * @param mixed $uri
+     */
+    public function setUri($uri) {
+        $this->uri = $uri;
+    }
+
+    /**
+     * @param mixed $host
+     */
+    public function setHost($host) {
+        $this->host = $host;
+    }
+
+    /**
+     * @param mixed $method
+     */
+    public function setMethod($method) {
+        $this->method = $method;
+    }
+
 }

src/Pecee/Http/Response.php

@@ -4,17 +4,11 @@ namespace Pecee\Http;
 
 class Response {
 
-    protected $request;
-
-    public function __construct(Request $request) {
-        $this->request = $request;
-    }
-
     /**
      * Set the http status code
      *
      * @param int $code
-     * @return static
+     * @return self $this
      */
     public function httpCode($code) {
         http_response_code($code);
@@ -37,13 +31,13 @@ class Response {
     }
 
     public function refresh() {
-        $this->redirect($this->request->getUri());
+        $this->redirect(Request::getInstance()->getUri());
     }
 
     /**
      * Add http authorisation
      * @param string $name
-     * @return static
+     * @return self $this
      */
     public function auth($name = '') {
         $this->headers([
@@ -87,7 +81,7 @@ class Response {
     /**
      * Add header to response
      * @param string $value
-     * @return static
+     * @return self $this
      */
     public function header($value) {
         header($value);
@@ -97,7 +91,7 @@ class Response {
     /**
      * Add multiple headers to response
      * @param array $headers
-     * @return static
+     * @return self $this
      */
     public function headers(array $headers) {
         foreach($headers as $header) {

src/Pecee/SimpleRouter/IControllerRoute.php

@@ -1,11 +0,0 @@
-<?php
-namespace Pecee\SimpleRouter;
-
-interface IControllerRoute {
-
-    public function getController();
-    public function setController($controller);
-    public function getMethod();
-    public function setMethod($method);
-
-}

src/Pecee/SimpleRouter/ILoadableRoute.php

@@ -1,9 +0,0 @@
-<?php
-namespace Pecee\SimpleRouter;
-
-interface ILoadableRoute {
-
-    public function getUrl();
-    public function setUrl($url);
-
-}

src/Pecee/SimpleRouter/RouterBase.php

@@ -5,133 +5,59 @@ use Pecee\Exception\RouterException;
 use Pecee\Handler\IExceptionHandler;
 use Pecee\Http\Middleware\BaseCsrfVerifier;
 use Pecee\Http\Request;
-use Pecee\Http\Response;
 
 class RouterBase {
 
     protected static $instance;
 
-    /**
-     * Current request
-     * @var Request
-     */
     protected $request;
-
-    /**
-     * Response
-     * @var Response
-     */
-    protected $response;
-
-    /**
-     * Used to keep track of whether to add routes to stack or not.
-     * @var RouterEntry
-     */
     protected $currentRoute;
-
-    /**
-     * All added routes
-     * @var array
-     */
     protected $routes;
-
-    /**
-     * List of
-     * @var array
-     */
+    protected $processedRoutes;
     protected $controllerUrlMap;
-
-    /**
-     * Backstack array used to keep track of sub-routes
-     * @var array
-     */
     protected $backStack;
-
-    /**
-     * The default namespace that all routes will inherit
-     * @var string
-     */
     protected $defaultNamespace;
-
-    /**
-     * List of added bootmanagers
-     * @var array
-     */
     protected $bootManagers;
+    protected $baseCsrfVerifier;
+    protected $middlewaresToLoad;
 
-    /**
-     * Csrf verifier class
-     * @var BaseCsrfVerifier
-     */
-    protected $csrfVerifier;
-
-    /**
-     * Get exception handlers
-     * @var array
-     */
-    protected $exceptionHandlers;
-
-    /**
-     * The current loaded route
-     * @var RouterRoute|null
-     */
-    protected $loadedRoute;
-
-    /**
-     * List over route changes (to avoid looping)
-     * @var array
-     */
-    protected $routeChanges;
+    // TODO: clean up - cut some of the methods down to smaller pieces
 
     public function __construct() {
-        $this->reset();
-    }
-
-    public function reset() {
-        $this->request = new Request();
-        $this->response = new Response($this->request);
+        $this->request = Request::getInstance();
         $this->routes = array();
         $this->backStack = array();
         $this->controllerUrlMap = array();
         $this->bootManagers = array();
-        $this->exceptionHandlers = array();
-        $this->routeChanges = array();
+        $this->middlewaresToLoad = array();
     }
 
-    /**
-     * Add route
-     * @param RouterEntry $route
-     * @return RouterEntry
-     */
     public function addRoute(RouterEntry $route) {
         if($this->currentRoute !== null) {
             $this->backStack[] = $route;
         } else {
             $this->routes[] = $route;
         }
-
-        return $route;
     }
 
-    protected function processRoutes(array $routes, array $settings = array(), array $prefixes = array(), $backStack = false, RouterGroup $group = null) {
+    protected function processRoutes(array $routes, array $settings = array(), array $prefixes = array(), $backStack = false, $group = null) {
         // Loop through each route-request
 
+        $routesCount = count($routes);
         $mergedSettings = array();
 
         /* @var $route RouterEntry */
-        for($i = 0; $i < count($routes); $i++) {
+        for($i = 0; $i < $routesCount; $i++) {
 
             $route = $routes[$i];
 
-            if(count($settings)) {
-                $route->addSettings($settings);
-            }
+            $route->addSettings($settings);
 
-            if($backStack && $group !== null) {
+            if($backStack) {
                 $route->setGroup($group);
             }
 
-            if($route->getNamespace() === null && $this->defaultNamespace !== null) {
+            if($this->defaultNamespace && !$route->getNamespace()) {
                 $namespace = $this->defaultNamespace;
                 if ($route->getNamespace()) {
                     $namespace .= '\\' . $route->getNamespace();
@@ -140,37 +66,32 @@ class RouterBase {
                 $route->setNamespace($namespace);
             }
 
-            if($group !== null && $group->getPrefix() !== null && trim($group->getPrefix(), '/') !== '') {
-                $prefixes[] = trim($group->getPrefix(), '/');
+            $newPrefixes = $prefixes;
+
+            if($route->getPrefix() && trim($route->getPrefix(), '/') !== '') {
+                array_push($newPrefixes, trim($route->getPrefix(), '/'));
             }
 
-            $group = null;
-            $this->currentRoute = $route;
-
-            if($route instanceof ILoadableRoute) {
-                if(is_array($prefixes) && count($prefixes) && $backStack) {
-                    $route->setUrl( '/' . join('/', $prefixes) . $route->getUrl() );
+            if(!($route instanceof RouterGroup)) {
+                if(is_array($newPrefixes) && count($newPrefixes) && $backStack) {
+                    $route->setUrl( '/' . join('/', $newPrefixes) . $route->getUrl() );
                 }
 
+                $group = null;
                 $this->controllerUrlMap[] = $route;
-            } else {
-                if(is_callable($route->getCallback())) {
+            }
 
-                    $route->renderRoute($this->request);
+            $this->currentRoute = $route;
 
-                    if ($route->matchRoute($this->request)) {
+            if($route instanceof RouterGroup && is_callable($route->getCallback())) {
+                $group = $route;
 
-                        /* @var $group RouterGroup */
-                        $group = $route;
+                $route->renderRoute($this->request);
+                $mergedSettings = array_merge($settings, $route->getMergeableSettings());
 
-                        $mergedSettings = array_merge($settings, $group->getMergeableSettings());
-
-                        // Add ExceptionHandler
-                        if ($group->getExceptionHandler() !== null) {
-                            $this->exceptionHandlers[] = $route;
-                        }
-
-                    }
+                // Load middleware on group if route matches
+                if($route->getPrefix() !== null && $route->matchRoute($this->request)) {
+                    $this->middlewaresToLoad[] = $route;
                 }
             }
 
@@ -181,116 +102,109 @@ class RouterBase {
                 $this->backStack = array();
 
                 // Route any routes added to the backstack
-                $this->processRoutes($backStack, $mergedSettings, $prefixes, true, $group);
+                $this->processRoutes($backStack, $mergedSettings, $newPrefixes, true, $group);
             }
         }
     }
 
-    public function routeRequest(Request $newRequest = null) {
+    public function routeRequest() {
+
+        $originalUri = $this->request->getUri();
+
+        // Load group middlewares
+
+        /* @var $middleware RouterEntry */
+        foreach($this->middlewaresToLoad as $middleware) {
+            $middleware->loadMiddleware($this->request);
+        }
+
+        // Initialize boot-managers
+        if(count($this->bootManagers)) {
+            /* @var $manager RouterBootManager */
+            foreach($this->bootManagers as $manager) {
+                $this->request = $manager->boot($this->request);
+
+                if(!($this->request instanceof Request)) {
+                    throw new RouterException('Custom router bootmanager "'. get_class($manager) .'" must return instance of Request.');
+                }
+            }
+        }
+
+        // Verify csrf token for request
+        if($this->baseCsrfVerifier !== null) {
+            $this->baseCsrfVerifier->handle($this->request);
+        }
+
+        // Loop through each route-request
+        $this->processRoutes($this->routes);
 
-        $this->loadedRoute = null;
         $routeNotAllowed = false;
 
-        // Create a fictive request - so it can be changed in the middleware or exceptionhandler later on...
-        $request = clone $this->request;
-
-        try {
-
-            // Initialize boot-managers
-            if(count($this->bootManagers)) {
-                /* @var $manager RouterBootManager */
-                foreach($this->bootManagers as $manager) {
-                    $request = $manager->boot($request);
-
-                    if(!($this->request instanceof Request)) {
-                        throw new RouterException('Custom router bootmanager "'. get_class($manager) .'" must return instance of Request.');
-                    }
-                }
+        // Make sure routes with longer urls are rendered first
+        usort($this->controllerUrlMap, function($a, $b) {
+            if(strlen($a->getUrl()) < strlen($b->getUrl())) {
+                return 1;
             }
+            return -1;
+        });
 
-            if($newRequest === null) {
+        $max = count($this->controllerUrlMap);
 
-                // Loop through each route-request
-                $this->processRoutes($this->routes);
+        /* @var $route RouterEntry */
+        for($i = 0; $i < $max; $i++) {
 
-                if($this->csrfVerifier !== null) {
+            $route = $this->controllerUrlMap[$i];
 
-                    // Verify csrf token for request
-                    $this->csrfVerifier->handle($this->request);
+            $routeMatch = $route->matchRoute($this->request);
+
+            if($routeMatch) {
+
+                if(count($route->getRequestMethods()) && !in_array($this->request->getMethod(), $route->getRequestMethods())) {
+                    $routeNotAllowed = true;
+                    continue;
                 }
-            }
 
-            $request = ($newRequest !== null) ? $newRequest : $request;
+                $routeNotAllowed = false;
 
-            /* @var $route RouterEntry */
-            for ($i = 0; $i < count($this->controllerUrlMap); $i++) {
+                $this->request->rewrite_uri = $this->request->uri;
+                $this->request->setUri($originalUri);
 
-                $route = $this->controllerUrlMap[$i];
+                $this->request->loadedRoute = $route;
+                $route->loadMiddleware($this->request);
 
-                if ($route->matchRoute($request)) {
-
-                    if (count($route->getRequestMethods()) && !in_array($request->getMethod(), $route->getRequestMethods())) {
-                        $routeNotAllowed = true;
-                        continue;
-                    }
-
-                    $routeNotAllowed = false;
-
-                    $this->loadedRoute = $route;
-
-                    $request = $this->loadedRoute->loadMiddleware($request, $this->loadedRoute);
-                    $request = ($request === null) ? $this->request : $request;
-
-                    if($request !== null && $request->getUri() !== $this->request->getUri() && !in_array($request->getUri(), $this->routeChanges)) {
-                        $this->routeChanges[] = $request->getUri();
-                        $this->routeRequest($request);
-                        return;
-                    }
-
-                    $this->loadedRoute->renderRoute($request);
-
-                    break;
+                try {
+                    $this->request->loadedRoute->renderRoute($this->request);
+                } catch(\Exception $e) {
+                    $this->handleException($e);
                 }
-            }
 
-        } catch(\Exception $e) {
-            $this->handleException($e);
+                break;
+            }
         }
 
         if($routeNotAllowed) {
             $this->handleException(new RouterException('Route or method not allowed', 403));
         }
 
-        if($this->loadedRoute === null) {
-            $this->handleException(new RouterException(sprintf('Route not found: %s', $request->getUri()), 404));
+        if(!$this->request->loadedRoute) {
+            throw new RouterException(sprintf('Route not found: %s', $this->request->getUri()), 404);
         }
     }
 
     protected function handleException(\Exception $e) {
-
-        /* @var $route RouterGroup */
-        foreach ($this->exceptionHandlers as $route) {
-            $handler = $route->getExceptionHandler();
-            $handler = new $handler();
-
-            if (!($handler instanceof IExceptionHandler)) {
+        if($this->request->loadedRoute !== null && $this->request->loadedRoute->exceptionHandler !== null) {
+            $handler = new $this->request->loadedRoute->exceptionHandler();
+            if(!($handler instanceof IExceptionHandler)) {
                 throw new RouterException('Exception handler must implement the IExceptionHandler interface.');
             }
 
-            $request = $handler->handleError($this->request, $this->loadedRoute, $e);
-
-            if($request !== null && !in_array($request->getUri(), $this->routeChanges)) {
-                $this->routeChanges[] = $request->getUri();
-                $this->routeRequest($request);
-            }
-
+            $handler->handleError($this->request, $this->request->loadedRoute, $e);
         }
 
         throw $e;
     }
 
     /**
-     * Get default namespace
      * @return string
      */
     public function getDefaultNamespace(){
@@ -298,7 +212,6 @@ class RouterBase {
     }
 
     /**
-     * Set the main default namespace that all routes will inherit
      * @param string $defaultNamespace
      * @return static
      */
@@ -308,7 +221,6 @@ class RouterBase {
     }
 
     /**
-     * Get bootmanagers
      * @return array
      */
     public function getBootManagers() {
@@ -316,21 +228,40 @@ class RouterBase {
     }
 
     /**
-     * Set bootmanagers
      * @param array $bootManagers
      */
     public function setBootManagers(array $bootManagers) {
         $this->bootManagers = $bootManagers;
     }
 
-    /**
-     * Add bootmanager
-     * @param RouterBootManager $bootManager
-     */
     public function addBootManager(RouterBootManager $bootManager) {
         $this->bootManagers[] = $bootManager;
     }
 
+    /**
+     * @return RouterEntry
+     */
+    public function getLoadedRoute() {
+        if(!($this->request->loadedRoute instanceof RouterGroup)) {
+            return $this->request->loadedRoute;
+        }
+        return null;
+    }
+
+    /**
+     * @return array
+     */
+    public function getBackstack() {
+        return $this->backStack;
+    }
+
+    /**
+     * @return RouterEntry
+     */
+    public function getCurrentRoute(){
+        return $this->currentRoute;
+    }
+
     /**
      * @return array
      */
@@ -348,48 +279,37 @@ class RouterBase {
     }
 
     /**
-     * Get response
-     * @return Response
-     */
-    public function getResponse() {
-        return $this->response;
-    }
-
-    /**
-     * Get csrf verifier class
+     * Get base csrf verifier class
      * @return BaseCsrfVerifier
      */
-    public function getCsrfVerifier() {
-        return $this->csrfVerifier;
+    public function getBaseCsrfVerifier() {
+        return $this->baseCsrfVerifier;
     }
 
     /**
-     * Set csrf verifier class
+     * Set base csrf verifier class
      *
-     * @param BaseCsrfVerifier $csrfVerifier
-     * @return static
+     * @param BaseCsrfVerifier $baseCsrfVerifier
+     * @return self
      */
-    public function setCsrfVerifier(BaseCsrfVerifier $csrfVerifier) {
-        $this->csrfVerifier = $csrfVerifier;
+    public function setBaseCsrfVerifier(BaseCsrfVerifier $baseCsrfVerifier) {
+        $this->baseCsrfVerifier = $baseCsrfVerifier;
         return $this;
     }
 
     public function arrayToParams(array $getParams = null, $includeEmpty = true) {
-
-        if(is_array($getParams) && count($getParams)) {
-            if ($includeEmpty === false) {
-                $getParams = array_filter($getParams, function ($item) {
-                    return (!empty($item));
-                });
+        if (is_array($getParams) && count($getParams) > 0) {
+            foreach ($getParams as $key => $val) {
+                if (!empty($val) || $includeEmpty) {
+                    $getParams[$key] = (is_array($val) ? $this->arrayToParams($val, $includeEmpty) : $key . '=' . $val);
+                }
             }
-
-            return '?' . http_build_query($getParams);
+            return join('&', $getParams);
         }
-
         return '';
     }
 
-    protected function processUrl(RouterRoute $route, $method = null, $parameters = null, $getParams = null) {
+    protected function processUrl($route, $method = null, $parameters = null, $getParams = null) {
 
         $domain = '';
 
@@ -406,22 +326,25 @@ class RouterBase {
 
         $url = $domain . '/' . trim($route->getUrl(), '/');
 
-        if($route instanceof IControllerRoute && $method !== null) {
+        if(($route instanceof RouterController || $route instanceof RouterResource) && $method !== null) {
             $url .= $method;
+        }
 
+        if($route instanceof RouterController || $route instanceof RouterResource) {
             if(count($parameters)) {
                 $url .= join('/', $parameters);
             }
         } else {
-            if($parameters !== null && is_array($parameters)) {
+            /* @var $route RouterEntry */
+            if(is_array($parameters)) {
                 $params = array_merge($route->getParameters(), $parameters);
             } else {
                 $params = $route->getParameters();
             }
 
-            $otherParams = array();
-            $i = 0;
+            $otherParams = [];
 
+            $i = 0;
             foreach($params as $param => $value) {
                 $value = (isset($parameters[$param])) ? $parameters[$param] : $value;
                 if(stripos($url, '{' . $param. '}') !== false || stripos($url, '{' . $param . '?}') !== false) {
@@ -437,8 +360,8 @@ class RouterBase {
 
         $url = rtrim($url, '/') . '/';
 
-        if($getParams !== null) {
-            $url .= $this->arrayToParams($getParams);
+        if($getParams !== null && count($getParams)) {
+            $url .= '?' . $this->arrayToParams($getParams);
         }
 
         return $url;
@@ -456,23 +379,25 @@ class RouterBase {
 
         // Return current route if no options has been specified
         if($controller === null && $parameters === null) {
-            $getParams = ($getParams !== null && is_array($getParams)) ? array_merge($_GET, $getParams) : $_GET;
+            $getParams = (is_array($getParams)) ? array_merge($_GET, $getParams) : $_GET;
 
-            $url = parse_url($this->request->getUri(), PHP_URL_PATH);
+            $url = parse_url(Request::getInstance()->getUri());
+            $url = $url['path'];
 
-            if($getParams !== null) {
-                $url .= $this->arrayToParams($getParams);
+            if(count($getParams)) {
+                $url .= '?' . $this->arrayToParams($getParams);
             }
 
             return $url;
         }
 
-        if($controller === null && $this->loadedRoute !== null) {
-            return $this->processUrl($this->loadedRoute, $this->loadedRoute->getMethod(), $parameters, $getParams);
+        if($controller === null && $this->request->loadedRoute !== null) {
+            return $this->processUrl($this->request->loadedRoute, $this->request->loadedRoute->getMethod(), $parameters, $getParams);
         }
 
         $c = '';
         $method = null;
+
         $max = count($this->controllerUrlMap);
 
         /* @var $route RouterRoute */
@@ -481,16 +406,14 @@ class RouterBase {
             $route = $this->controllerUrlMap[$i];
 
             // Check an alias exist, if the matches - use it
-            if($route instanceof IControllerRoute) {
-                $c = $route->getController();
-            } else {
-                if($route->hasAlias($controller)) {
-                    return $this->processUrl($route, $route->getMethod(), $parameters, $getParams);
-                }
+            if($route instanceof RouterRoute && $route->hasAlias($controller)) {
+                return $this->processUrl($route, $route->getMethod(), $parameters, $getParams);
+            }
 
-                if(!is_callable($route->getCallback()) && stripos($route->getCallback(), '@') !== false) {
-                    $c = $route->getCallback();
-                }
+            if($route instanceof RouterRoute && !is_callable($route->getCallback()) && stripos($route->getCallback(), '@') !== false) {
+                $c = $route->getCallback();
+            } else if($route instanceof RouterController || $route instanceof RouterResource) {
+                $c = $route->getController();
             }
 
             if($c === $controller || strpos($c, $controller) === 0) {
@@ -505,10 +428,10 @@ class RouterBase {
 
             $route = $this->controllerUrlMap[$i];
 
-            if($route instanceof IControllerRoute) {
-                $c = $route->getController();
-            } else if(!is_callable($route->getCallback()) && stripos($route->getCallback(), '@') !== false) {
+            if($route instanceof RouterRoute && !is_callable($route->getCallback()) && stripos($route->getCallback(), '@') !== false) {
                 $c = $route->getClass();
+            } else if($route instanceof RouterController || $route instanceof RouterResource) {
+                $c = $route->getController();
             }
 
             if(stripos($controller, '@') !== false) {
@@ -525,29 +448,30 @@ class RouterBase {
         $controller = ($controller === null) ? '/' : $controller;
         $url = array($controller);
 
-        if($parameters !== null && is_array($parameters) && count($parameters)) {
-            $url = array_merge($url, $parameters);
+        if(is_array($parameters)) {
+            foreach($parameters as $key => $value) {
+                array_push($url,$value);
+            }
         }
 
         $url = '/' . trim(join('/', $url), '/') . '/';
 
-        if($getParams !== null) {
-            $url .= $this->arrayToParams($getParams);
+        if($getParams !== null && count($getParams)) {
+            $url .= '?' . $this->arrayToParams($getParams);
         }
 
         return $url;
     }
 
-    /**
-     * Get current router instance
-     * @return static
-     */
     public static function getInstance() {
-        if(static::$instance === null) {
-            static::$instance = new static();
+        if(self::$instance === null) {
+            self::$instance = new static();
         }
+        return self::$instance;
+    }
 
-        return static::$instance;
+    public static function reset() {
+        self::$instance = null;
     }
 
 }

src/Pecee/SimpleRouter/RouterController.php

@@ -4,7 +4,7 @@ namespace Pecee\SimpleRouter;
 use Pecee\Exception\RouterException;
 use Pecee\Http\Request;
 
-class RouterController extends RouterEntry implements ILoadableRoute, IControllerRoute {
+class RouterController extends RouterEntry {
 
     const DEFAULT_METHOD = 'index';
 
@@ -13,6 +13,7 @@ class RouterController extends RouterEntry implements ILoadableRoute, IControlle
     protected $method;
 
     public function __construct($url, $controller) {
+        parent::__construct();
         $this->url = $url;
         $this->controller = $controller;
     }
@@ -43,8 +44,8 @@ class RouterController extends RouterEntry implements ILoadableRoute, IControlle
     }
 
     public function matchRoute(Request $request) {
-        $url = parse_url(urldecode($request->getUri()), PHP_URL_PATH);
-        $url = rtrim($url, '/') . '/';
+        $url = parse_url(urldecode($request->getUri()));
+        $url = rtrim($url['path'], '/') . '/';
 
         if(strtolower($url) == strtolower($this->url) || stripos($url, $this->url) === 0) {
 
@@ -54,11 +55,11 @@ class RouterController extends RouterEntry implements ILoadableRoute, IControlle
 
             if(count($path)) {
 
-                $method = (!isset($path[0]) || trim($path[0]) === '') ? static::DEFAULT_METHOD : $path[0];
+                $method = (!isset($path[0]) || trim($path[0]) === '') ? self::DEFAULT_METHOD : $path[0];
                 $this->method = $method;
 
                 array_shift($path);
-                $this->settings['parameters'] = $path;
+                $this->parameters = $path;
 
                 // Set callback
                 $this->setCallback($this->controller . '@' . $this->method);

src/Pecee/SimpleRouter/RouterEntry.php

@@ -14,26 +14,41 @@ abstract class RouterEntry {
     const REQUEST_TYPE_PATCH = 'patch';
     const REQUEST_TYPE_DELETE = 'delete';
 
-    public static $allowedRequestTypes = [
+    public static $allowedRequestTypes = array(
         self::REQUEST_TYPE_DELETE,
         self::REQUEST_TYPE_GET,
         self::REQUEST_TYPE_POST,
         self::REQUEST_TYPE_PUT,
-        self::REQUEST_TYPE_PATCH,
-    ];
-
-    protected $settings = [
-        'requestMethods' => array(),
-        'where' => array(),
-        'parameters' => array(),
-        'middleware' => array(),
-    ];
+        self::REQUEST_TYPE_PATCH
+    );
 
+    protected $settings;
     protected $callback;
 
+    public function __construct() {
+        $this->settings = array();
+        $this->settings['requestMethods'] = array();
+        $this->settings['where'] = array();
+        $this->settings['parameters'] = array();
+    }
+
+    /**
+     * Returns callback name/identifier for the current route based on the callback.
+     * Useful if you need to get a unique identifier for the loaded route, for instance
+     * when using translations etc.
+     *
+     * @return string
+     */
+    public function getIdentifier() {
+        if(strpos($this->callback, '@') !== false) {
+            return $this->callback;
+        }
+        return 'function_' . md5($this->callback);
+    }
+
     /**
      * @param string $callback
-     * @return static
+     * @return self;
      */
     public function setCallback($callback) {
         $this->callback = $callback;
@@ -73,36 +88,52 @@ abstract class RouterEntry {
         return $this;
     }
 
+    /**
+     * @param string $prefix
+     * @return self
+     */
+    public function setPrefix($prefix) {
+        $this->prefix = '/' . ltrim($prefix, '/');
+        return $this;
+    }
+
     /**
      * @param string $middleware
-     * @return static
+     * @return self
      */
     public function setMiddleware($middleware) {
-        $this->settings['middleware'][] = $middleware;
+        $this->middleware = $middleware;
         return $this;
     }
 
     /**
      * @param string $namespace
-     * @return static
+     * @return self
      */
     public function setNamespace($namespace) {
-        $this->settings['namespace'] = $namespace;
+        $this->namespace = $namespace;
         return $this;
     }
 
+    /**
+     * @return string
+     */
+    public function getPrefix() {
+        return $this->prefix;
+    }
+
     /**
      * @return string|array
      */
     public function getMiddleware() {
-        return $this->settingArray('middleware');
+        return $this->middleware;
     }
 
     /**
      * @return string
      */
     public function getNamespace() {
-        return $this->setting('namespace');
+        return $this->namespace;
     }
 
     /**
@@ -113,18 +144,18 @@ abstract class RouterEntry {
     }
 
     /**
-     * @return array
+     * @return mixed
      */
     public function getParameters(){
-        return $this->setting('parameters', array());
+        return ($this->parameters === null) ? array() : $this->parameters;
     }
 
     /**
      * @param mixed $parameters
-     * @return static
+     * @return self
      */
     public function setParameters($parameters) {
-        $this->settings['parameters'] = $parameters;
+        $this->parameters = $parameters;
         return $this;
     }
 
@@ -132,10 +163,10 @@ abstract class RouterEntry {
      * Add regular expression parameter match
      *
      * @param array $options
-     * @return static
+     * @return self
      */
     public function where(array $options) {
-        $this->settings['where'] = array_merge($this->settings['where'], $options);
+        $this->where = array_merge($this->where, $options);
         return $this;
     }
 
@@ -143,10 +174,10 @@ abstract class RouterEntry {
      * Add regular expression match for url
      *
      * @param string $regex
-     * @return static
+     * @return self
      */
     public function match($regex) {
-        $this->settings['regexMatch'] = $regex;
+        $this->regexMatch = $regex;
         return $this;
     }
 
@@ -156,27 +187,60 @@ abstract class RouterEntry {
      * @return array
      */
     public function getMergeableSettings() {
-        return $this->settings;
+        $settings = $this->settings;
+
+        if(isset($settings['prefix'])) {
+            unset($settings['prefix']);
+        }
+
+        return $settings;
     }
 
     /**
      * @param array $settings
-     * @return static
+     * @return self
      */
-    public function addSettings(array $settings) {
-        $this->settings = array_merge($this->settings, $settings);
+    public function addSettings(array $settings = null) {
+        if(is_array($settings)) {
+            $this->settings = array_merge($this->settings, $settings);
+        }
         return $this;
     }
 
     /**
      * @param array $settings
-     * @return static
+     * @return self
      */
     public function setSettings($settings) {
         $this->settings = $settings;
+
+        if(isset($settings['prefix'])) {
+            $this->setPrefix($settings['prefix']);
+        }
+
         return $this;
     }
 
+    /**
+     * Dynamically access settings value
+     *
+     * @param $name
+     * @return mixed|null
+     */
+    public function __get($name) {
+        return (isset($this->settings[$name]) ? $this->settings[$name] : null);
+    }
+
+    /**
+     * Dynamicially set settings value
+     *
+     * @param string $name
+     * @param mixed|null $value
+     */
+    public function __set($name, $value = null) {
+        $this->settings[$name] = $value;
+    }
+
     protected function loadClass($name) {
         if(!class_exists($name)) {
             throw new RouterException(sprintf('Class %s does not exist', $name));
@@ -185,7 +249,7 @@ abstract class RouterEntry {
         return new $name();
     }
 
-    protected function parseParameters($route, $url, $parameterRegex = '[\w]+') {
+    protected function parseParameters($route, $url, $parameterRegex = '[a-z0-9]+') {
         $parameterNames = array();
         $regex = '';
         $lastCharacter = '';
@@ -197,6 +261,12 @@ abstract class RouterEntry {
 
             $character = $route[$i];
 
+            // Skip "/" if we are at the end of a parameter
+            if($lastCharacter === '}' && $character === '/') {
+                $lastCharacter = $character;
+                continue;
+            }
+
             if($character === '{') {
                 // Remove "/" and "\" from regex
                 if(substr($regex, strlen($regex)-1) === '/') {
@@ -209,23 +279,18 @@ abstract class RouterEntry {
                 // Check for optional parameter
 
                 // Use custom parameter regex if it exists
-                if(is_array($this->setting('where')) && isset($this->settings['where'][$parameter])) {
-                    $parameterRegex = $this->settings['where'][$parameter];
+                if(is_array($this->where) && isset($this->where[$parameter])) {
+                    $parameterRegex = $this->where[$parameter];
                 }
 
                 if($lastCharacter === '?') {
                     $parameter = substr($parameter, 0, strlen($parameter)-1);
-                    $regex .= '(?:\/?(?P<' . $parameter . '>'. $parameterRegex .')[^\/]?)?';
+                    $regex .= '(?:\\/?(?P<'.$parameter.'>[^\/]+)?\\/?)';
                     $required = false;
                 } else {
-                    $regex .= '\/?(?P<' . $parameter . '>'. $parameterRegex .')[^\/]?';
+                    $regex .= '\\/(?P<' . $parameter . '>'. $parameterRegex .')\\/';
                 }
-
-                $parameterNames[] = [
-                    'name' => $parameter,
-                    'required' => $required
-                ];
-
+                $parameterNames[] = array('name' => $parameter, 'required' => $required);
                 $parameter = '';
                 $isParameter = false;
 
@@ -242,25 +307,26 @@ abstract class RouterEntry {
 
         $parameterValues = array();
 
-        if(preg_match('/^'.$regex.'\/?$/is', $url, $parameterValues)) {
-
+        if(preg_match('/^'.$regex.'$/is', $url, $parameterValues)) {
             $parameters = array();
 
             $max = count($parameterNames);
 
-            for($i = 0; $i < $max; $i++) {
-                $name = $parameterNames[$i];
-                $parameterValue = isset($parameterValues[$name['name']]) ? $parameterValues[$name['name']] : null;
+            if($max) {
+                for($i = 0; $i < $max; $i++) {
+                    $name = $parameterNames[$i];
+                    $parameterValue = (isset($parameterValues[$name['name']]) && !empty($parameterValues[$name['name']])) ? $parameterValues[$name['name']] : null;
 
-                if($name['required'] && $parameterValue === null) {
-                    throw new RouterException('Missing required parameter ' . $name['name'], 404);
+                    if($name['required'] && $parameterValue === null) {
+                        throw new RouterException('Missing required parameter ' . $name['name'], 404);
+                    }
+
+                    if(!$name['required'] && $parameterValue === null) {
+                        continue;
+                    }
+
+                    $parameters[$name['name']] = $parameterValue;
                 }
-
-                if(!$name['required'] && $parameterValue === null) {
-                    continue;
-                }
-
-                $parameters[$name['name']] = $parameterValue;
             }
 
             return $parameters;
@@ -269,16 +335,26 @@ abstract class RouterEntry {
         return null;
     }
 
-    public function loadMiddleware(Request $request, RouterEntry &$route) {
-        if(count($this->getMiddleware())) {
-            foreach($this->getMiddleware() as $middleware) {
-                $middleware = $this->loadClass($middleware);
+    public function loadMiddleware(Request $request) {
+        if($this->getMiddleware()) {
+            if(is_array($this->getMiddleware())) {
+                foreach($this->getMiddleware() as $middleware) {
+                    $middleware = $this->loadClass($middleware);
+                    if (!($middleware instanceof IMiddleware)) {
+                        throw new RouterException($middleware . ' must be instance of Middleware');
+                    }
+
+                    /* @var $class IMiddleware */
+                    $middleware->handle($request);
+                }
+            } else {
+                $middleware = $this->loadClass($this->getMiddleware());
                 if (!($middleware instanceof IMiddleware)) {
-                    throw new RouterException($middleware . ' must be instance of Middleware');
+                    throw new RouterException($this->getMiddleware() . ' must be instance of Middleware');
                 }
 
                 /* @var $class IMiddleware */
-                $middleware->handle($request, $route);
+                $middleware->handle($request);
             }
         }
     }
@@ -300,7 +376,7 @@ abstract class RouterEntry {
             }
 
             $parameters = array_filter($this->getParameters(), function($var){
-                return ($var !== null);
+                return !is_null($var);
             });
 
             call_user_func_array(array($class, $method), $parameters);
@@ -315,7 +391,7 @@ abstract class RouterEntry {
      * Set allowed request methods
      *
      * @param array $methods
-     * @return static $this
+     * @return self $this
      */
     public function setRequestMethods(array $methods) {
         $this->settings['requestMethods'] = $methods;
@@ -323,37 +399,27 @@ abstract class RouterEntry {
     }
 
     /**
-     * Get allowed request methods
+     * Get allowed requeset methods
      *
      * @return array
      */
     public function getRequestMethods() {
-        return $this->settingArray('requestMethods');
+        if(!isset($this->settings['requestMethods']) || isset($this->settings['requestMethods']) && !is_array($this->settings['requestMethods'])) {
+            $value = isset($this->settings['requestMethods']) ? $this->settings['requestMethods'] : null;
+            return array($value);
+        }
+        return $this->settings['requestMethods'];
     }
 
     public function getGroup() {
-        return $this->setting('group');
+        return $this->group;
     }
 
     public function setGroup($group) {
-        $this->settings['group'] = $group;
+        $this->group = $group;
         return $this;
     }
 
-    protected function setting($name, $defaultValue = null) {
-        return isset($this->settings[$name]) ? $this->settings[$name] : $defaultValue;
-    }
-
-    protected function settingArray($name) {
-        $value = $this->setting($name);
-
-        if($value === null) {
-            return [];
-        }
-
-        return (!is_array($value)) ? array($value) : $value;
-    }
-
     abstract function matchRoute(Request $request);
 
 }

src/Pecee/SimpleRouter/RouterGroup.php

@@ -8,17 +8,19 @@ use Pecee\Http\Request;
 class RouterGroup extends RouterEntry {
 
     public function matchDomain(Request $request) {
-        if($this->setting('domain') !== null) {
+        if($this->domain !== null) {
 
-            if(is_array($this->setting('domain'))) {
+            if(is_array($this->domain)) {
 
-                for($i = 0; $i < count($this->setting('domain')); $i++) {
-                    $domain = $this->settings['domain'][$i];
+                $max = count($this->domain);
+
+                for($i = 0; $i < $max; $i++) {
+                    $domain = $this->domain[$i];
 
                     $parameters = $this->parseParameters($domain, $request->getHost(), '[^.]*');
 
                     if($parameters !== null) {
-                        $this->settings['parameters'] = $parameters;
+                        $this->parameters = $parameters;
                         return true;
                     }
                 }
@@ -26,10 +28,10 @@ class RouterGroup extends RouterEntry {
                 return false;
             }
 
-            $parameters = $this->parseParameters($this->setting('domain'), $request->getHost(), '[^.]*');
+            $parameters = $this->parseParameters($this->domain, $request->getHost(), '[^.]*');
 
             if ($parameters !== null) {
-                $this->settings['parameters'] = $parameters;
+                $this->parameters = $parameters;
                 return true;
             }
 
@@ -41,10 +43,10 @@ class RouterGroup extends RouterEntry {
 
     public function renderRoute(Request $request) {
         // Check if request method is allowed
-        $hasAccess = true;
+        $hasAccess = (!$this->method);
 
-        if($this->setting('method') !== null) {
-            if(is_array($this->setting('method'))) {
+        if($this->method) {
+            if(is_array($this->method)) {
                 $hasAccess = (in_array($request->getMethod(), $this->getRequestMethods()));
             } else {
                 $hasAccess = strtolower($this->getRequestMethods()) == strtolower($request->getMethod());
@@ -70,69 +72,30 @@ class RouterGroup extends RouterEntry {
     }
 
     public function setExceptionHandler($class) {
-        $this->settings['exceptionHandler'] = $class;
+        $this->exceptionHandler = $class;
         return $this;
     }
 
     public function getExceptionHandler() {
-        return $this->setting('exceptionHandler');
+        return $this->exceptionHandler;
     }
 
     public function getDomain() {
-        return $this->setting('domain');
-    }
-
-    /**
-     * @param string $prefix
-     * @return static
-     */
-    public function setPrefix($prefix) {
-        $this->settings['prefix'] = '/' . ltrim($prefix, '/');
-        return $this;
-    }
-
-    /**
-     * @return string
-     */
-    public function getPrefix() {
-        return $this->setting('prefix');
+        return $this->domain;
     }
 
     /**
      * @param array $settings
-     * @return static
+     * @return self
      */
-    public function addSettings(array $settings) {
-
-        if ($this->getNamespace() !== null && isset($settings['namespace'])) {
+    public function addSettings(array $settings = null) {
+        if($this->getNamespace() !== null && isset($settings['namespace'])) {
             unset($settings['namespace']);
         }
-
-        // Push middleware if multiple
-        if ($this->getMiddleware() !== null && isset($settings['middleware'])) {
-
-            if (!is_array($settings['middleware'])) {
-                $settings['middleware'] = array_merge($this->getMiddleware(), array($settings['middleware']));
-            } else {
-                $settings['middleware'][] = $this->getMiddleware();
-            }
-
-            $settings['middleware'] = array_unique(array_reverse($settings['middleware']));
-
+        if(is_array($settings)) {
+            $this->settings = array_merge($this->settings, $settings);
         }
-
-        $this->settings = array_merge($this->settings, $settings);
         return $this;
     }
 
-    public function getMergeableSettings() {
-        $settings = $this->settings;
-
-        if(isset($settings['prefix'])) {
-            unset($settings['prefix']);
-        }
-
-        return $settings;
-    }
-
 }

src/Pecee/SimpleRouter/RouterResource.php

@@ -4,14 +4,17 @@ namespace Pecee\SimpleRouter;
 use Pecee\Exception\RouterException;
 use Pecee\Http\Request;
 
-class RouterResource extends RouterEntry implements ILoadableRoute, IControllerRoute {
+class RouterResource extends RouterEntry {
 
     protected $url;
     protected $controller;
+    protected $postMethod;
 
     public function __construct($url, $controller) {
+        parent::__construct();
         $this->url = $url;
         $this->controller = $controller;
+        $this->postMethod = strtolower(($_SERVER['REQUEST_METHOD'] != 'GET') ? 'post' : 'get');
     }
 
     public function renderRoute(Request $request) {
@@ -39,54 +42,54 @@ class RouterResource extends RouterEntry implements ILoadableRoute, IControllerR
 
     protected function call($method, $parameters) {
         $this->setCallback($this->controller . '@' . $method);
-        $this->settings['parameters'] = $parameters;
+        $this->parameters = $parameters;
         return true;
     }
 
     public function matchRoute(Request $request) {
-        $url = parse_url(urldecode($request->getUri()), PHP_URL_PATH);
-        $url = rtrim($url, '/') . '/';
+        $url = parse_url(urldecode($request->getUri()));
+        $url = rtrim($url['path'], '/') . '/';
 
         $route = rtrim($this->url, '/') . '/{id?}/{action?}';
 
-        $parameters = $this->parseParameters($route, $url);
+        $parameters = $this->parseParameters($route, $url, '[0-9]+?');
 
         if($parameters !== null) {
 
             if(is_array($parameters)) {
-                $parameters = array_merge($this->settings['parameters'], $parameters);
+                $parameters = array_merge($this->parameters, $parameters);
             }
 
             $action = isset($parameters['action']) ? $parameters['action'] : null;
             unset($parameters['action']);
 
             // Delete
-            if($request->getMethod() === static::REQUEST_TYPE_DELETE && $request->getMethod() === static::REQUEST_TYPE_POST) {
+            if($request->getMethod() === self::REQUEST_TYPE_DELETE && $this->postMethod === self::REQUEST_TYPE_POST) {
                 return $this->call('destroy', $parameters);
             }
 
             // Update
-            if(in_array($request->getMethod(), array(static::REQUEST_TYPE_PATCH, static::REQUEST_TYPE_PUT)) && $request->getMethod() === static::REQUEST_TYPE_POST) {
+            if(in_array($request->getMethod(), array(self::REQUEST_TYPE_PATCH, self::REQUEST_TYPE_PUT)) && $this->postMethod === self::REQUEST_TYPE_POST) {
                 return $this->call('update', $parameters);
             }
 
             // Edit
-            if(isset($action) && strtolower($action) === 'edit' && $request->getMethod() === static::REQUEST_TYPE_GET) {
+            if(isset($action) && strtolower($action) === 'edit' && $this->postMethod === self::REQUEST_TYPE_GET) {
                 return $this->call('edit', $parameters);
             }
 
             // Create
-            if(strtolower($action) === 'create' && $request->getMethod() === static::REQUEST_TYPE_GET) {
+            if(strtolower($action) === 'create' && $request->getMethod() === self::REQUEST_TYPE_GET) {
                 return $this->call('create', $parameters);
             }
 
             // Save
-            if($request->getMethod() === static::REQUEST_TYPE_POST) {
+            if($this->postMethod === self::REQUEST_TYPE_POST) {
                 return $this->call('store', $parameters);
             }
 
             // Show
-            if(isset($parameters['id']) && $request->getMethod() === static::REQUEST_TYPE_GET) {
+            if(isset($parameters['id']) && $this->postMethod === self::REQUEST_TYPE_GET) {
                 return $this->call('show', $parameters);
             }
 
@@ -109,7 +112,8 @@ class RouterResource extends RouterEntry implements ILoadableRoute, IControllerR
      * @return static
      */
     public function setUrl($url) {
-        $this->url = rtrim($url, '/') . '/';
+        $url = rtrim($url, '/') . '/';
+        $this->url = $url;
         return $this;
     }
 

src/Pecee/SimpleRouter/RouterRoute.php

@@ -4,27 +4,28 @@ namespace Pecee\SimpleRouter;
 
 use Pecee\Http\Request;
 
-class RouterRoute extends RouterEntry implements ILoadableRoute {
+class RouterRoute extends RouterEntry {
 
     const PARAMETERS_REGEX_MATCH = '{([A-Za-z\-\_]*?)\?{0,1}}';
 
     protected $url;
 
     public function __construct($url, $callback) {
+        parent::__construct();
         $this->setUrl($url);
         $this->setCallback($callback);
     }
 
     public function matchRoute(Request $request) {
 
-        $url = parse_url(urldecode($request->getUri()), PHP_URL_PATH);
-        $url = rtrim($url, '/') . '/';
+        $url = parse_url(urldecode($request->getUri()));
+        $url = rtrim($url['path'], '/') . '/';
 
         // Match on custom defined regular expression
-        if($this->setting('regexMatch') !== null) {
+        if($this->regexMatch) {
             $parameters = array();
-            if(preg_match('/(' . $this->setting('regexMatch') . ')/is', $request->getHost() . $url, $parameters)) {
-                $this->settings['parameters'] = (!is_array($parameters[0]) ? array($parameters[0]) : $parameters[0]);
+            if(preg_match('/('.$this->regexMatch.')/is', $request->getHost() . $url, $parameters)) {
+                $this->parameters = (!is_array($parameters[0]) ? array($parameters[0]) : $parameters[0]);
                 return true;
             }
             return null;
@@ -36,7 +37,13 @@ class RouterRoute extends RouterEntry implements ILoadableRoute {
         $parameters = $this->parseParameters($route, $url);
 
         if($parameters !== null) {
-            $this->settings['parameters'] = array_merge($this->settingArray('parameters'), $parameters);
+
+            if(is_array($this->parameters)) {
+                $this->parameters = array_merge($this->parameters, $parameters);
+            } else {
+                $this->parameters = $parameters;
+            }
+
             return true;
         }
 
@@ -52,23 +59,22 @@ class RouterRoute extends RouterEntry implements ILoadableRoute {
 
     /**
      * @param string $url
-     * @return static
+     * @return self
      */
     public function setUrl($url) {
         $parameters = array();
         $matches = array();
 
-        if(preg_match_all('/' . static::PARAMETERS_REGEX_MATCH . '/is', $url, $matches)) {
+        if(preg_match_all('/'.self::PARAMETERS_REGEX_MATCH.'/is', $url, $matches)) {
             $parameters = $matches[1];
         }
 
         if(count($parameters)) {
-
-            foreach(array_keys($parameters) as $key) {
-                $parameters[$key] = null;
+            $tmp = array();
+            foreach($parameters as $param) {
+                $tmp[$param] = null;
             }
-
-            $this->settings['parameters'] = $parameters;
+            $this->parameters = $tmp;
         }
 
         $this->url = $url;
@@ -80,48 +86,47 @@ class RouterRoute extends RouterEntry implements ILoadableRoute {
      * @return string|array
      */
     public function getAlias(){
-        return $this->setting('alias');
+        return $this->alias;
     }
 
     /**
      * Check if route has given alias.
      *
-     * @param string $name
+     * @param $name
      * @return bool
      */
     public function hasAlias($name) {
-        if ($this->getAlias() !== null) {
-            if (is_array($this->getAlias())) {
-                foreach ($this->setting('alias') as $alias) {
-                    if (strtolower($alias) === strtolower($name)) {
-                        return true;
-                    }
+        if(is_array($this->alias)) {
+            foreach($this->alias as $alias) {
+                if(strtolower($alias) === strtolower($name)) {
+                    return true;
                 }
             }
+        } else {
             return strtolower($this->getAlias()) === strtolower($name);
         }
-
+        
         return false;
     }
 
     /**
      * Set the url alias for easier getting the url route.
      * @param string|array $alias
-     * @return static
+     * @return self
      */
     public function setAlias($alias){
-        $this->settings['alias'] = $alias;
+        $this->alias = $alias;
         return $this;
     }
 
-    public function addSettings(array $settings) {
+    public function setSettings($settings) {
 
         // Change as to alias
-        if(isset($settings['as'])) {
+        if(isset($settings{'as'})) {
             $this->setAlias($settings['as']);
         }
 
-        return parent::addSettings($settings);
+        return parent::setSettings($settings);
     }
 
 }

src/Pecee/SimpleRouter/SimpleRouter.php

@@ -15,18 +15,11 @@ class SimpleRouter {
 
     /**
      * Start/route request
+     * @param null $defaultNamespace
      * @throws \Pecee\Exception\RouterException
      */
-    public static function start() {
-        RouterBase::getInstance()->routeRequest();
-    }
-
-    /**
-     * Set default namespace for all routes
-     * @param string $defaultNamespace
-     */
-    public static function setDefaultNamespace($defaultNamespace) {
-        RouterBase::getInstance()->setDefaultNamespace($defaultNamespace);
+    public static function start($defaultNamespace = null) {
+        RouterBase::getInstance()->setDefaultNamespace($defaultNamespace)->routeRequest();
     }
 
     /**
@@ -34,7 +27,7 @@ class SimpleRouter {
      * @param BaseCsrfVerifier $baseCsrfVerifier
      */
     public static function csrfVerifier(BaseCsrfVerifier $baseCsrfVerifier) {
-        RouterBase::getInstance()->setCsrfVerifier($baseCsrfVerifier);
+        RouterBase::getInstance()->setBaseCsrfVerifier($baseCsrfVerifier);
     }
 
     public static function addBootManager(RouterBootManager $bootManager) {
@@ -42,19 +35,19 @@ class SimpleRouter {
     }
 
     public static function get($url, $callback, array $settings = null) {
-        return static::match(['get'], $url, $callback, $settings);
+        return self::match(['get'], $url, $callback, $settings);
     }
 
     public static function post($url, $callback, array $settings = null) {
-        return static::match(['post'], $url, $callback, $settings);
+        return self::match(['post'], $url, $callback, $settings);
     }
 
     public static function put($url, $callback, array $settings = null) {
-        return static::match(['put'], $url, $callback, $settings);
+        return self::match(['put'], $url, $callback, $settings);
     }
 
     public static function delete($url, $callback, array $settings = null) {
-        return static::match(['delete'], $url, $callback, $settings);
+        return self::match(['delete'], $url, $callback, $settings);
     }
 
     public static function group($settings = array(), $callback) {
@@ -79,7 +72,7 @@ class SimpleRouter {
      * @return RouterRoute
      */
     public static function basic($url, $callback, array $settings = null) {
-        return static::match(['get', 'post'], $url, $callback, $settings);
+        return self::match(['get', 'post'], $url, $callback, $settings);
     }
 
     public static function match(array $requestMethods, $url, $callback, array $settings = null) {
@@ -126,25 +119,13 @@ class SimpleRouter {
             $route->addSettings($settings);
         }
 
-        static::router()->addRoute($route);
+        RouterBase::getInstance()->addRoute($route);
 
         return $route;
     }
 
     public static function getRoute($controller = null, $parameters = null, $getParams = null) {
-        return static::router()->getRoute($controller, $parameters, $getParams);
-    }
-
-    public static function request() {
-        return static::router()->getRequest();
-    }
-
-    public static function response() {
-        return static::router()->getResponse();
-    }
-
-    protected static function router() {
-        return RouterBase::getInstance();
+        return RouterBase::getInstance()->getRoute($controller, $parameters, $getParams);
     }
 
 }

test/Dummy/DummyController.php

@@ -6,13 +6,4 @@ class DummyController {
         echo static::class . '@' .'start() OK';
     }
 
-    public function param($params = null) {
-        $params = func_get_args();
-        echo 'Params: ' . join(', ', $params);
-    }
-
-    public function notFound() {
-        echo 'not found';
-    }
-
 }

test/Dummy/DummyMiddleware.php

@@ -7,7 +7,7 @@ use Pecee\Http\Request;
 
 class DummyMiddleware implements IMiddleware {
 
-    public function handle(Request $request, \Pecee\SimpleRouter\RouterEntry &$route = null) {
+    public function handle(Request $request) {
        throw new MiddlewareLoadedException('Middleware loaded!');
     }
 

test/Dummy/Handler/ExceptionHandler.php

@@ -1,8 +0,0 @@
-<?php
-class ExceptionHandler implements \Pecee\Handler\IExceptionHandler {
-
-    public function handleError(\Pecee\Http\Request $request, \Pecee\SimpleRouter\RouterEntry &$route = null, \Exception $error){
-        throw $error;
-    }
-
-}

test/GroupTest.php

@@ -7,28 +7,37 @@ class GroupTest extends PHPUnit_Framework_TestCase  {
 
     protected $result;
 
-    public function testGroupLoad() {
+    public function __construct() {
+        // Initial setup
+        $_SERVER['HTTP_HOST'] = 'example.com';
+        $_SERVER['REQUEST_URI'] = '/api/v1/test';
+        $_SERVER['REQUEST_METHOD'] = 'get';
+    }
+
+    protected function group() {
+        $this->result = true;
+    }
+
+    public function testGroup() {
+        \Pecee\SimpleRouter\RouterBase::reset();
 
         $this->result = false;
 
-        \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/group'], function() {
-            $this->result = true;
-        });
+        \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/group'], $this->group());
 
         try {
             \Pecee\SimpleRouter\SimpleRouter::start();
         } catch(Exception $e) {
-            echo $e->getMessage();
+
         }
 
         $this->assertTrue($this->result);
     }
 
     public function testNestedGroup() {
+        \Pecee\SimpleRouter\RouterBase::reset();
 
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/api/v1/test');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
+        \Pecee\Http\Request::getInstance()->setUri('/api/v1/test');
 
         \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/api'], function() {
             \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/v1'], function() {
@@ -39,39 +48,4 @@ class GroupTest extends PHPUnit_Framework_TestCase  {
         \Pecee\SimpleRouter\SimpleRouter::start();
     }
 
-    public function testManyRoutes() {
-
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/match');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-
-        \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/api'], function() {
-            \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/v1'], function() {
-                \Pecee\SimpleRouter\SimpleRouter::get('/test', 'DummyController@start');
-            });
-        });
-
-        \Pecee\SimpleRouter\SimpleRouter::get('/my/match', 'DummyController@start');
-
-        \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/service'], function() {
-            \Pecee\SimpleRouter\SimpleRouter::group(['prefix' => '/v1'], function() {
-                \Pecee\SimpleRouter\SimpleRouter::get('/no-match', 'DummyController@start');
-            });
-        });
-
-        \Pecee\SimpleRouter\SimpleRouter::start();
-    }
-
-    public function testUrls() {
-
-        \Pecee\SimpleRouter\SimpleRouter::get('/my/fancy/url/1', 'DummyController@start', ['as' => 'fancy1']);
-        \Pecee\SimpleRouter\SimpleRouter::get('/my/fancy/url/2', 'DummyController@start')->setAlias('fancy2');
-
-        \Pecee\SimpleRouter\SimpleRouter::start();
-
-        $this->assertTrue((\Pecee\SimpleRouter\SimpleRouter::getRoute('fancy1') === '/my/fancy/url/1/'));
-        $this->assertTrue((\Pecee\SimpleRouter\SimpleRouter::getRoute('fancy2') === '/my/fancy/url/2/'));
-
-    }
-
 }

test/MiddlewareTest.php

@@ -2,30 +2,35 @@
 
 require_once 'Dummy/DummyMiddleware.php';
 require_once 'Dummy/DummyController.php';
-require_once 'Dummy/Handler/ExceptionHandler.php';
 
 class MiddlewareTest extends PHPUnit_Framework_TestCase  {
 
+    public function __construct() {
+        // Initial setup
+        $_SERVER['HTTP_HOST'] = 'example.com';
+        $_SERVER['REQUEST_URI'] = '/my/test/url';
+        $_SERVER['REQUEST_METHOD'] = 'get';
+    }
+
     public function testMiddlewareFound() {
 
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
+        \Pecee\Http\Request::getInstance()->setMethod('get');
 
-        \Pecee\SimpleRouter\SimpleRouter::group(['exceptionHandler' => 'ExceptionHandler'], function() {
-            \Pecee\SimpleRouter\SimpleRouter::get('/my/test/url', 'DummyController@start', ['middleware' => 'DummyMiddleware']);
-        });
+        \Pecee\SimpleRouter\RouterBase::reset();
 
-        $found = false;
+        \Pecee\SimpleRouter\SimpleRouter::get('/my/test/url', 'DummyController@start', ['middleware' => 'DummyMiddleware']);
 
         try {
             \Pecee\SimpleRouter\SimpleRouter::start();
-        }catch(\Exception $e) {
-            $found = ($e instanceof MiddlewareLoadedException);
+        }catch(Exception $e) {
+            $this->assertTrue(($e instanceof MiddlewareLoadedException));
+            return;
         }
 
-        $this->assertTrue($found);
+        throw new Exception('Middleware not loaded');
 
     }
 
+
+
 }

test/RouterRouteTest.php

@@ -2,63 +2,48 @@
 
 require_once 'Dummy/DummyMiddleware.php';
 require_once 'Dummy/DummyController.php';
-require_once 'Dummy/Handler/ExceptionHandler.php';
 
 class RouterRouteTest extends PHPUnit_Framework_TestCase  {
 
-
-    public function testNotFound() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/test-param1-param2');
-
-        \Pecee\SimpleRouter\SimpleRouter::group(['exceptionHandler' => 'ExceptionHandler'], function() {
-            \Pecee\SimpleRouter\SimpleRouter::get('/non-existing-path', 'DummyController@start');
-        });
-
-        $found = false;
-
-        try {
-            \Pecee\SimpleRouter\SimpleRouter::start();
-        }catch(\Exception $e) {
-            $found = ($e instanceof \Pecee\Exception\RouterException && $e->getCode() == 404);
-        }
-
-        $this->assertTrue($found);
-
+    public function __construct() {
+        // Initial setup
+        $_SERVER['HTTP_HOST'] = 'example.com';
+        $_SERVER['REQUEST_URI'] = '/my/test/url';
+        $_SERVER['REQUEST_METHOD'] = 'get';
     }
 
     public function testGet() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
+
+        \Pecee\SimpleRouter\RouterBase::reset();
+
+        \Pecee\Http\Request::getInstance()->setMethod('get');
 
         \Pecee\SimpleRouter\SimpleRouter::get('/my/test/url', 'DummyController@start');
         \Pecee\SimpleRouter\SimpleRouter::start();
     }
 
     public function testPost() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('post');
+        \Pecee\Http\Request::getInstance()->setMethod('post');
+
+        \Pecee\SimpleRouter\RouterBase::reset();
 
         \Pecee\SimpleRouter\SimpleRouter::post('/my/test/url', 'DummyController@start');
         \Pecee\SimpleRouter\SimpleRouter::start();
     }
 
     public function testPut() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('put');
+        \Pecee\Http\Request::getInstance()->setMethod('put');
+
+        \Pecee\SimpleRouter\RouterBase::reset();
 
         \Pecee\SimpleRouter\SimpleRouter::put('/my/test/url', 'DummyController@start');
         \Pecee\SimpleRouter\SimpleRouter::start();
     }
 
     public function testDelete() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('delete');
+        \Pecee\Http\Request::getInstance()->setMethod('delete');
+
+        \Pecee\SimpleRouter\RouterBase::reset();
 
         \Pecee\SimpleRouter\SimpleRouter::delete('/my/test/url', 'DummyController@start');
         \Pecee\SimpleRouter\SimpleRouter::start();
@@ -66,9 +51,10 @@ class RouterRouteTest extends PHPUnit_Framework_TestCase  {
     }
 
     public function testMethodNotAllowed() {
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/my/test/url');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('post');
+
+        \Pecee\SimpleRouter\RouterBase::reset();
+
+        \Pecee\Http\Request::getInstance()->setMethod('post');
 
         \Pecee\SimpleRouter\SimpleRouter::get('/my/test/url', 'DummyController@start');
 
@@ -80,37 +66,4 @@ class RouterRouteTest extends PHPUnit_Framework_TestCase  {
 
     }
 
-    public function testSimpleParam() {
-
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/test-param1');
-
-        \Pecee\SimpleRouter\SimpleRouter::get('/test-{param1}', 'DummyController@param');
-        \Pecee\SimpleRouter\SimpleRouter::start();
-
-    }
-
-    public function testMultiParam() {
-
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/test-param1-param2');
-
-        \Pecee\SimpleRouter\SimpleRouter::get('/test-{param1}-{param2}', 'DummyController@param');
-        \Pecee\SimpleRouter\SimpleRouter::start();
-
-    }
-
-    public function testPathParamRegex() {
-
-        \Pecee\SimpleRouter\RouterBase::getInstance()->reset();
-        \Pecee\SimpleRouter\SimpleRouter::request()->setMethod('get');
-        \Pecee\SimpleRouter\SimpleRouter::request()->setUri('/test/path/123123');
-
-        \Pecee\SimpleRouter\SimpleRouter::get('/test/path/{myParam}', 'DummyController@param', ['where' => ['myParam' => '([0-9]+)']]);
-        \Pecee\SimpleRouter\SimpleRouter::start();
-
-    }
-
 }