HakimCh / Router

examples/basic/index.php

Indentation +13 added lines, -13 removed lines

@@ -26,19 +26,19 @@
 <pre>
  <?php
  if ($match) {
-     foreach ($match as $key => $value) {
-         echo '<p>' . $key . ': ';
-         if (is_array($value)) {
-             echo '<ul>';
-             foreach ($value as $k => $v) {
-                 echo '<li>'.$k.': '.$v.'</li>';
-             }
-             echo '</ul>';
-         } else {
-             echo $value;
-         }
-         echo '</p>';
-     }
+	 foreach ($match as $key => $value) {
+		 echo '<p>' . $key . ': ';
+		 if (is_array($value)) {
+			 echo '<ul>';
+			 foreach ($value as $k => $v) {
+				 echo '<li>'.$k.': '.$v.'</li>';
+			 }
+			 echo '</ul>';
+		 } else {
+			 echo $value;
+		 }
+		 echo '</p>';
+	 }
  }
  ?>
  </pre>

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
          if (is_array($value)) {
              echo '<ul>';
              foreach ($value as $k => $v) {
-                 echo '<li>'.$k.': '.$v.'</li>';
+                 echo '<li>' . $k . ': ' . $v . '</li>';
              }
              echo '</ul>';
          } else {

src/Router.php

Indentation +195 added lines, -195 removed lines

@@ -6,199 +6,199 @@
 
 class Router
 {
-    protected $routes = [];
-    protected $namedRoutes = [];
-    protected $basePath = '';
-    protected $all = ['get', 'post'];
-    protected $server;
-    /**
-     * @var RouterParser
-     */
-    private $parser;
-
-    /**
-     * Create router in one call from config.
-     *
-     * @param RouterParser $parser
-     * @param array $routes
-     * @param string $basePath
-     * @param array|null $server
-     */
-    public function __construct(RouterParser $parser, $routes = [], $basePath = '', $server = null)
-    {
-        $this->setRoutes($routes);
-        $this->setBasePath($basePath);
-        $this->parser = $parser;
-        $this->server = $server;
-    }
-
-    /**
-     * Map a route to a target
-     *
-     * @param string $method One of 5 HTTP Methods, or a pipe-separated list of multiple HTTP Methods (GET|POST|PATCH|PUT|DELETE)
-     * @param string $route The route regex, custom regex must start with an @. You can use multiple pre-set regex filters, like [i:id]
-     * @param mixed $target The target where this route should point to. Can be anything.
-     * @param string $routeName Optional name of this route. Supply if you want to reverse route this url in your application.
-     */
-    public function map($method, $route, $target, $routeName = null)
-    {
-        if (is_string($routeName)) {
-            $this->handleException($routeName, "Can not redeclare route '%s'", true);
-            $this->namedRoutes[$routeName] = $route;
-        }
-
-        $this->routes[] = array($method, $route, $target, $routeName);
-    }
-
-    /**
-     * Reversed routing
-     * Generate the URL for a named route. Replace regexes with supplied parameters
-     *
-     * @param string $routeName The name of the route.
-     * @param array $params Associative array of parameters to replace placeholders with.
-     *
-     * @return string The URL of the route with named parameters in place.
-     */
-    public function generate($routeName, array $params = [])
-    {
-        $this->handleException($routeName, "Route '%s' does not exist.", false);
-
-        $route = $this->namedRoutes[$routeName];
-
-        return $this->parser->generate($this->basePath, $route, $params);
-    }
-
-    /**
-     * Match a given Request Url against stored routes
-     * @param string $requestUrl
-     * @param string $requestMethod
-     *
-     * @return array|boolean Array with route information on success, false on failure (no match).
-     */
-    public function match($requestUrl = null, $requestMethod = null)
-    {
-        $requestUrl = $this->getRequestUrl($requestUrl);
-
-        // set Request Method if it isn't passed as a parameter
-        if (is_null($requestMethod)) {
-            $requestMethod = $this->server['REQUEST_METHOD'];
-        }
-
-        foreach ($this->routes as $handler) {
-            if (!$this->parser->methodMatch($handler[0], $requestMethod, $handler[1], $requestUrl)) {
-                continue;
-            }
-
-            return array(
-                'target' => $handler[2],
-                'params' => array_filter($this->parser->getParams(), function ($k) {
-                    return !is_numeric($k);
-                }, ARRAY_FILTER_USE_KEY),
-                'name'   => $handler[3]
-            );
-        }
-
-        return false;
-    }
-
-    /**
-     * @param $method
-     * @param $arguments
-     *
-     * @throws RouterException
-     */
-    public function __call($method, $arguments)
-    {
-        if (!in_array($method, array('get', 'post', 'delete', 'put', 'patch', 'update', 'all'))) {
-            throw new RouterException($method . ' not exist in the '. __CLASS__);
-        }
-
-        $methods = $method == 'all' ? implode('|', $this->all) : $method;
-
-        $route = array_merge([$methods], $arguments);
-
-        call_user_func_array([$this, 'map'], $route);
-    }
-
-    /**
-     * Retrieves all routes.
-     * Useful if you want to process or display routes.
-     *
-     * @return array All routes.
-     */
-    public function getRoutes()
-    {
-        return $this->routes;
-    }
-
-    /**
-     * Add multiple routes at once from array in the following format:
-     *
-     *   $routes = array(
-     *      array($method, $route, $target, $name)
-     *   );
-     *
-     * @param array|Traversable $routes
-     *
-     * @return void
-     */
-    public function setRoutes($routes)
-    {
-        if (!is_array($routes) && !$routes instanceof Traversable) {
-            throw new RouterException('Routes should be an array or an instance of Traversable');
-        }
-        if (!empty($routes)) {
-            foreach ($routes as $route) {
-                call_user_func_array(array($this, 'map'), $route);
-            }
-        }
-    }
-
-    /**
-     * Set the base path.
-     * Useful if you are running your application from a subdirectory.
-     *
-     * @param $basePath
-     */
-    public function setBasePath($basePath)
-    {
-        $this->basePath = $basePath;
-    }
-
-    /**
-     * @param $routeName
-     * @param $message
-     * @param bool $cmpTo
-     *
-     * @throws RouterException
-     */
-    private function handleException($routeName, $message, $cmpTo)
-    {
-        if (array_key_exists($routeName, $this->namedRoutes) === $cmpTo) {
-            throw new RouterException(sprintf($message, $routeName));
-        }
-    }
-
-    /**
-     * @param $requestUrl
-     *
-     * @return mixed
-     */
-    private function getRequestUrl($requestUrl = null)
-    {
-        // set Request Url if it isn't passed as parameter
-        if (is_null($requestUrl)) {
-            $requestUrl = parse_url($this->server['REQUEST_URI'], PHP_URL_PATH);
-        }
-
-        return str_replace($this->basePath, '', strtok($requestUrl, '?'));
-    }
-
-    /**
-     * @return RouterParser
-     */
-    public function getParser()
-    {
-        return $this->parser;
-    }
+	protected $routes = [];
+	protected $namedRoutes = [];
+	protected $basePath = '';
+	protected $all = ['get', 'post'];
+	protected $server;
+	/**
+	 * @var RouterParser
+	 */
+	private $parser;
+
+	/**
+	 * Create router in one call from config.
+	 *
+	 * @param RouterParser $parser
+	 * @param array $routes
+	 * @param string $basePath
+	 * @param array|null $server
+	 */
+	public function __construct(RouterParser $parser, $routes = [], $basePath = '', $server = null)
+	{
+		$this->setRoutes($routes);
+		$this->setBasePath($basePath);
+		$this->parser = $parser;
+		$this->server = $server;
+	}
+
+	/**
+	 * Map a route to a target
+	 *
+	 * @param string $method One of 5 HTTP Methods, or a pipe-separated list of multiple HTTP Methods (GET|POST|PATCH|PUT|DELETE)
+	 * @param string $route The route regex, custom regex must start with an @. You can use multiple pre-set regex filters, like [i:id]
+	 * @param mixed $target The target where this route should point to. Can be anything.
+	 * @param string $routeName Optional name of this route. Supply if you want to reverse route this url in your application.
+	 */
+	public function map($method, $route, $target, $routeName = null)
+	{
+		if (is_string($routeName)) {
+			$this->handleException($routeName, "Can not redeclare route '%s'", true);
+			$this->namedRoutes[$routeName] = $route;
+		}
+
+		$this->routes[] = array($method, $route, $target, $routeName);
+	}
+
+	/**
+	 * Reversed routing
+	 * Generate the URL for a named route. Replace regexes with supplied parameters
+	 *
+	 * @param string $routeName The name of the route.
+	 * @param array $params Associative array of parameters to replace placeholders with.
+	 *
+	 * @return string The URL of the route with named parameters in place.
+	 */
+	public function generate($routeName, array $params = [])
+	{
+		$this->handleException($routeName, "Route '%s' does not exist.", false);
+
+		$route = $this->namedRoutes[$routeName];
+
+		return $this->parser->generate($this->basePath, $route, $params);
+	}
+
+	/**
+	 * Match a given Request Url against stored routes
+	 * @param string $requestUrl
+	 * @param string $requestMethod
+	 *
+	 * @return array|boolean Array with route information on success, false on failure (no match).
+	 */
+	public function match($requestUrl = null, $requestMethod = null)
+	{
+		$requestUrl = $this->getRequestUrl($requestUrl);
+
+		// set Request Method if it isn't passed as a parameter
+		if (is_null($requestMethod)) {
+			$requestMethod = $this->server['REQUEST_METHOD'];
+		}
+
+		foreach ($this->routes as $handler) {
+			if (!$this->parser->methodMatch($handler[0], $requestMethod, $handler[1], $requestUrl)) {
+				continue;
+			}
+
+			return array(
+				'target' => $handler[2],
+				'params' => array_filter($this->parser->getParams(), function ($k) {
+					return !is_numeric($k);
+				}, ARRAY_FILTER_USE_KEY),
+				'name'   => $handler[3]
+			);
+		}
+
+		return false;
+	}
+
+	/**
+	 * @param $method
+	 * @param $arguments
+	 *
+	 * @throws RouterException
+	 */
+	public function __call($method, $arguments)
+	{
+		if (!in_array($method, array('get', 'post', 'delete', 'put', 'patch', 'update', 'all'))) {
+			throw new RouterException($method . ' not exist in the '. __CLASS__);
+		}
+
+		$methods = $method == 'all' ? implode('|', $this->all) : $method;
+
+		$route = array_merge([$methods], $arguments);
+
+		call_user_func_array([$this, 'map'], $route);
+	}
+
+	/**
+	 * Retrieves all routes.
+	 * Useful if you want to process or display routes.
+	 *
+	 * @return array All routes.
+	 */
+	public function getRoutes()
+	{
+		return $this->routes;
+	}
+
+	/**
+	 * Add multiple routes at once from array in the following format:
+	 *
+	 *   $routes = array(
+	 *      array($method, $route, $target, $name)
+	 *   );
+	 *
+	 * @param array|Traversable $routes
+	 *
+	 * @return void
+	 */
+	public function setRoutes($routes)
+	{
+		if (!is_array($routes) && !$routes instanceof Traversable) {
+			throw new RouterException('Routes should be an array or an instance of Traversable');
+		}
+		if (!empty($routes)) {
+			foreach ($routes as $route) {
+				call_user_func_array(array($this, 'map'), $route);
+			}
+		}
+	}
+
+	/**
+	 * Set the base path.
+	 * Useful if you are running your application from a subdirectory.
+	 *
+	 * @param $basePath
+	 */
+	public function setBasePath($basePath)
+	{
+		$this->basePath = $basePath;
+	}
+
+	/**
+	 * @param $routeName
+	 * @param $message
+	 * @param bool $cmpTo
+	 *
+	 * @throws RouterException
+	 */
+	private function handleException($routeName, $message, $cmpTo)
+	{
+		if (array_key_exists($routeName, $this->namedRoutes) === $cmpTo) {
+			throw new RouterException(sprintf($message, $routeName));
+		}
+	}
+
+	/**
+	 * @param $requestUrl
+	 *
+	 * @return mixed
+	 */
+	private function getRequestUrl($requestUrl = null)
+	{
+		// set Request Url if it isn't passed as parameter
+		if (is_null($requestUrl)) {
+			$requestUrl = parse_url($this->server['REQUEST_URI'], PHP_URL_PATH);
+		}
+
+		return str_replace($this->basePath, '', strtok($requestUrl, '?'));
+	}
+
+	/**
+	 * @return RouterParser
+	 */
+	public function getParser()
+	{
+		return $this->parser;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
 
             return array(
                 'target' => $handler[2],
-                'params' => array_filter($this->parser->getParams(), function ($k) {
+                'params' => array_filter($this->parser->getParams(), function($k) {
                     return !is_numeric($k);
                 }, ARRAY_FILTER_USE_KEY),
                 'name'   => $handler[3]
@@ -108,7 +108,7 @@ 
     public function __call($method, $arguments)
     {
         if (!in_array($method, array('get', 'post', 'delete', 'put', 'patch', 'update', 'all'))) {
-            throw new RouterException($method . ' not exist in the '. __CLASS__);
+            throw new RouterException($method . ' not exist in the ' . __CLASS__);
         }
 
         $methods = $method == 'all' ? implode('|', $this->all) : $method;

src/RouterParserInterface.php

Indentation +28 added lines, -28 removed lines

@@ -3,34 +3,34 @@
 
 interface RouterParserInterface
 {
-    /**
-     * Get the route parameters
-     *
-     * @return array
-     */
-    public function getParams();
+	/**
+	 * Get the route parameters
+	 *
+	 * @return array
+	 */
+	public function getParams();
 
-    /**
-     * Get the url from a route name
-     *
-     * @param string $basePath
-     * @param string $route
-     * @param array $params
-     *
-     * @return string
-     */
-    public function generate($basePath, $route, array $params);
+	/**
+	 * Get the url from a route name
+	 *
+	 * @param string $basePath
+	 * @param string $route
+	 * @param array $params
+	 *
+	 * @return string
+	 */
+	public function generate($basePath, $route, array $params);
 
-    /**
-     * Check if the request method match the route methods
-     * Update the $params
-     *
-     * @param string $method
-     * @param string $requestMethod
-     * @param string $routeString
-     * @param string $requestUrl
-     *
-     * @return mixed
-     */
-    public function methodMatch($method, $requestMethod, $routeString, $requestUrl);
+	/**
+	 * Check if the request method match the route methods
+	 * Update the $params
+	 *
+	 * @param string $method
+	 * @param string $requestMethod
+	 * @param string $routeString
+	 * @param string $requestUrl
+	 *
+	 * @return mixed
+	 */
+	public function methodMatch($method, $requestMethod, $routeString, $requestUrl);
 }

src/Tests/RouterTest.php

Indentation +386 added lines, -386 removed lines

@@ -10,51 +10,51 @@ 
 
 class RouterDebug extends Router
 {
-    public function getNamedRoutes()
-    {
-        return $this->namedRoutes;
-    }
+	public function getNamedRoutes()
+	{
+		return $this->namedRoutes;
+	}
 
-    public function getBasePath()
-    {
-        return $this->basePath;
-    }
+	public function getBasePath()
+	{
+		return $this->basePath;
+	}
 
-    public function setServer($server)
-    {
-        $this->server = $server;
-    }
+	public function setServer($server)
+	{
+		$this->server = $server;
+	}
 }
 
 class SimpleTraversable implements Iterator
 {
-    protected $_position = 0;
+	protected $_position = 0;
 
-    protected $_data = array(
-        array('GET', '/foo', 'foo_action', null),
-        array('POST', '/bar', 'bar_action', 'second_route')
-    );
+	protected $_data = array(
+		array('GET', '/foo', 'foo_action', null),
+		array('POST', '/bar', 'bar_action', 'second_route')
+	);
 
-    public function current()
-    {
-        return $this->_data[$this->_position];
-    }
-    public function key()
-    {
-        return $this->_position;
-    }
-    public function next()
-    {
-        ++$this->_position;
-    }
-    public function rewind()
-    {
-        $this->_position = 0;
-    }
-    public function valid()
-    {
-        return isset($this->_data[$this->_position]);
-    }
+	public function current()
+	{
+		return $this->_data[$this->_position];
+	}
+	public function key()
+	{
+		return $this->_position;
+	}
+	public function next()
+	{
+		++$this->_position;
+	}
+	public function rewind()
+	{
+		$this->_position = 0;
+	}
+	public function valid()
+	{
+		return isset($this->_data[$this->_position]);
+	}
 }
 
 /**
@@ -62,399 +62,399 @@ 
  */
 class AltoRouterTest extends PHPUnit_Framework_TestCase
 {
-    /**
-     * @var RouterDebug
-     */
-    protected $router;
+	/**
+	 * @var RouterDebug
+	 */
+	protected $router;
 
-    /**
-     * Sets up the fixture, for example, opens a network connection.
-     * This method is called before a test is executed.
-     */
-    protected function setUp()
-    {
-        $parser = new RouterParser();
-        $this->router = new RouterDebug($parser, [], '', $_SERVER);
-    }
+	/**
+	 * Sets up the fixture, for example, opens a network connection.
+	 * This method is called before a test is executed.
+	 */
+	protected function setUp()
+	{
+		$parser = new RouterParser();
+		$this->router = new RouterDebug($parser, [], '', $_SERVER);
+	}
 
-    /**
-     * @covers Router::getRoutes
-     */
-    public function testGetRoutes()
-    {
-        $method = 'POST';
-        $route = '/[:controller]/[:action]';
-        $target = function () {
-        };
+	/**
+	 * @covers Router::getRoutes
+	 */
+	public function testGetRoutes()
+	{
+		$method = 'POST';
+		$route = '/[:controller]/[:action]';
+		$target = function () {
+		};
 
-        $this->assertInternalType('array', $this->router->getRoutes());
-        $this->router->map($method, $route, $target);
-        $this->assertEquals(array(array($method, $route, $target, null)), $this->router->getRoutes());
-    }
+		$this->assertInternalType('array', $this->router->getRoutes());
+		$this->router->map($method, $route, $target);
+		$this->assertEquals(array(array($method, $route, $target, null)), $this->router->getRoutes());
+	}
 
-    /**
-     * @covers Router::setRoutes
-     */
-    public function testAddRoutes()
-    {
-        $method = 'POST';
-        $route = '/[:controller]/[:action]';
-        $target = function () {
-        };
-        
-        $this->router->setRoutes(array(
-            array($method, $route, $target),
-            array($method, $route, $target, 'second_route')
-        ));
-        
-        $routes = $this->router->getRoutes();
-        
-        $this->assertEquals(array($method, $route, $target, null), $routes[0]);
-        $this->assertEquals(array($method, $route, $target, 'second_route'), $routes[1]);
-    }
+	/**
+	 * @covers Router::setRoutes
+	 */
+	public function testAddRoutes()
+	{
+		$method = 'POST';
+		$route = '/[:controller]/[:action]';
+		$target = function () {
+		};
+        
+		$this->router->setRoutes(array(
+			array($method, $route, $target),
+			array($method, $route, $target, 'second_route')
+		));
+        
+		$routes = $this->router->getRoutes();
+        
+		$this->assertEquals(array($method, $route, $target, null), $routes[0]);
+		$this->assertEquals(array($method, $route, $target, 'second_route'), $routes[1]);
+	}
 
-    /**
-     * @covers Router::setRoutes
-     */
-    public function testAddRoutesAcceptsTraverable()
-    {
-        $traversable = new SimpleTraversable();
-        $this->router->setRoutes($traversable);
+	/**
+	 * @covers Router::setRoutes
+	 */
+	public function testAddRoutesAcceptsTraverable()
+	{
+		$traversable = new SimpleTraversable();
+		$this->router->setRoutes($traversable);
         
-        $traversable->rewind();
+		$traversable->rewind();
         
-        $first = $traversable->current();
-        $traversable->next();
-        $second = $traversable->current();
+		$first = $traversable->current();
+		$traversable->next();
+		$second = $traversable->current();
         
-        $routes = $this->router->getRoutes();
+		$routes = $this->router->getRoutes();
         
-        $this->assertEquals($first, $routes[0]);
-        $this->assertEquals($second, $routes[1]);
-    }
+		$this->assertEquals($first, $routes[0]);
+		$this->assertEquals($second, $routes[1]);
+	}
 
-    /**
-     * @covers Router::setRoutes
-     * @expectedException Exception
-     */
-    public function testAddRoutesThrowsExceptionOnInvalidArgument()
-    {
-        $this->router->setRoutes(new stdClass);
-    }
+	/**
+	 * @covers Router::setRoutes
+	 * @expectedException Exception
+	 */
+	public function testAddRoutesThrowsExceptionOnInvalidArgument()
+	{
+		$this->router->setRoutes(new stdClass);
+	}
 
-    /**
-     * @covers AltoRouter::setBasePath
-     */
-    public function testSetBasePath()
-    {
-        $this->router->setBasePath('/some/path');
-        $this->assertEquals('/some/path', $this->router->getBasePath());
-        
-        $this->router->setBasePath('/some/path');
-        $this->assertEquals('/some/path', $this->router->getBasePath());
-    }
+	/**
+	 * @covers AltoRouter::setBasePath
+	 */
+	public function testSetBasePath()
+	{
+		$this->router->setBasePath('/some/path');
+		$this->assertEquals('/some/path', $this->router->getBasePath());
+        
+		$this->router->setBasePath('/some/path');
+		$this->assertEquals('/some/path', $this->router->getBasePath());
+	}
 
-    /**
-     * @covers Router::map
-     */
-    public function testMap()
-    {
-        $method = 'POST';
-        $route = '/[:controller]/[:action]';
-        $target = function () {
-        };
-        
-        $this->router->map($method, $route, $target);
-        
-        $routes = $this->router->getRoutes();
-        
-        $this->assertEquals(array($method, $route, $target, null), $routes[0]);
-    }
+	/**
+	 * @covers Router::map
+	 */
+	public function testMap()
+	{
+		$method = 'POST';
+		$route = '/[:controller]/[:action]';
+		$target = function () {
+		};
+        
+		$this->router->map($method, $route, $target);
+        
+		$routes = $this->router->getRoutes();
+        
+		$this->assertEquals(array($method, $route, $target, null), $routes[0]);
+	}
 
-    /**
-     * @covers Router::map
-     */
-    public function testMapWithName()
-    {
-        $method = 'POST';
-        $route = '/[:controller]/[:action]';
-        $target = function () {
-        };
-        $name = 'myroute';
-        
-        $this->router->map($method, $route, $target, $name);
-        
-        $routes = $this->router->getRoutes();
-        $this->assertEquals(array($method, $route, $target, $name), $routes[0]);
-        
-        $named_routes = $this->router->getNamedRoutes();
-        $this->assertEquals($route, $named_routes[$name]);
-        
-        try {
-            $this->router->map($method, $route, $target, $name);
-            $this->fail('Should not be able to add existing named route');
-        } catch (Exception $e) {
-            $this->assertEquals("Can not redeclare route '{$name}'", $e->getMessage());
-        }
-    }
+	/**
+	 * @covers Router::map
+	 */
+	public function testMapWithName()
+	{
+		$method = 'POST';
+		$route = '/[:controller]/[:action]';
+		$target = function () {
+		};
+		$name = 'myroute';
+        
+		$this->router->map($method, $route, $target, $name);
+        
+		$routes = $this->router->getRoutes();
+		$this->assertEquals(array($method, $route, $target, $name), $routes[0]);
+        
+		$named_routes = $this->router->getNamedRoutes();
+		$this->assertEquals($route, $named_routes[$name]);
+        
+		try {
+			$this->router->map($method, $route, $target, $name);
+			$this->fail('Should not be able to add existing named route');
+		} catch (Exception $e) {
+			$this->assertEquals("Can not redeclare route '{$name}'", $e->getMessage());
+		}
+	}
 
 
-    /**
-     * @covers Router::generate
-     */
-    public function testGenerate()
-    {
-        $params = array(
-            'controller' => 'test',
-            'action' => 'someaction'
-        );
+	/**
+	 * @covers Router::generate
+	 */
+	public function testGenerate()
+	{
+		$params = array(
+			'controller' => 'test',
+			'action' => 'someaction'
+		);
         
-        $this->router->map('GET', '/[:controller]/[:action]', function () {
-        }, 'foo_route');
+		$this->router->map('GET', '/[:controller]/[:action]', function () {
+		}, 'foo_route');
         
-        $this->assertEquals(
+		$this->assertEquals(
         
-            '/test/someaction',
-            $this->router->generate('foo_route', $params)
+			'/test/someaction',
+			$this->router->generate('foo_route', $params)
         
-        );
+		);
         
-        $params = array(
-            'controller' => 'test',
-            'action' => 'someaction',
-            'type' => 'json'
-        );
+		$params = array(
+			'controller' => 'test',
+			'action' => 'someaction',
+			'type' => 'json'
+		);
         
-        $this->assertEquals(
+		$this->assertEquals(
         
-            '/test/someaction',
-            $this->router->generate('foo_route', $params)
+			'/test/someaction',
+			$this->router->generate('foo_route', $params)
         
-        );
-    }
+		);
+	}
 
-    public function testGenerateWithOptionalUrlParts()
-    {
-        $this->router->map('GET', '/[:controller]/[:action].[:type]?', function () {
-        }, 'bar_route');
+	public function testGenerateWithOptionalUrlParts()
+	{
+		$this->router->map('GET', '/[:controller]/[:action].[:type]?', function () {
+		}, 'bar_route');
         
-        $params = array(
-            'controller' => 'test',
-            'action' => 'someaction'
-        );
+		$params = array(
+			'controller' => 'test',
+			'action' => 'someaction'
+		);
         
-        $this->assertEquals(
+		$this->assertEquals(
         
-            '/test/someaction',
-            $this->router->generate('bar_route', $params)
+			'/test/someaction',
+			$this->router->generate('bar_route', $params)
         
-        );
+		);
         
-        $params = array(
-            'controller' => 'test',
-            'action' => 'someaction',
-            'type' => 'json'
-        );
+		$params = array(
+			'controller' => 'test',
+			'action' => 'someaction',
+			'type' => 'json'
+		);
         
-        $this->assertEquals(
+		$this->assertEquals(
         
-            '/test/someaction.json',
-            $this->router->generate('bar_route', $params)
+			'/test/someaction.json',
+			$this->router->generate('bar_route', $params)
         
-        );
-    }
+		);
+	}
     
-    public function testGenerateWithNonexistingRoute()
-    {
-        try {
-            $this->router->generate('nonexisting_route');
-            $this->fail('Should trigger an exception on nonexisting named route');
-        } catch (Exception $e) {
-            $this->assertEquals("Route 'nonexisting_route' does not exist.", $e->getMessage());
-        }
-    }
+	public function testGenerateWithNonexistingRoute()
+	{
+		try {
+			$this->router->generate('nonexisting_route');
+			$this->fail('Should trigger an exception on nonexisting named route');
+		} catch (Exception $e) {
+			$this->assertEquals("Route 'nonexisting_route' does not exist.", $e->getMessage());
+		}
+	}
     
-    /**
-     * @covers Router::match
-     * @covers Router::compileRoute
-     */
-    public function testMatch()
-    {
-        $this->router->map('GET', '/foo/[:controller]/[:action]', 'foo_action', 'foo_route');
-        
-        $this->assertEquals(array(
-            'target' => 'foo_action',
-            'params' => array(
-                'controller' => 'test',
-                'action' => 'do'
-            ),
-            'name' => 'foo_route'
-        ), $this->router->match('/foo/test/do', 'GET'));
-        
-        $this->assertFalse($this->router->match('/foo/test/do', 'POST'));
-        
-        $this->assertEquals(array(
-            'target' => 'foo_action',
-            'params' => array(
-                'controller' => 'test',
-                'action' => 'do'
-            ),
-            'name' => 'foo_route'
-        ), $this->router->match('/foo/test/do?param=value', 'GET'));
-    }
+	/**
+	 * @covers Router::match
+	 * @covers Router::compileRoute
+	 */
+	public function testMatch()
+	{
+		$this->router->map('GET', '/foo/[:controller]/[:action]', 'foo_action', 'foo_route');
+        
+		$this->assertEquals(array(
+			'target' => 'foo_action',
+			'params' => array(
+				'controller' => 'test',
+				'action' => 'do'
+			),
+			'name' => 'foo_route'
+		), $this->router->match('/foo/test/do', 'GET'));
+        
+		$this->assertFalse($this->router->match('/foo/test/do', 'POST'));
+        
+		$this->assertEquals(array(
+			'target' => 'foo_action',
+			'params' => array(
+				'controller' => 'test',
+				'action' => 'do'
+			),
+			'name' => 'foo_route'
+		), $this->router->match('/foo/test/do?param=value', 'GET'));
+	}
     
-    public function testMatchWithFixedParamValues()
-    {
-        $this->router->map('POST', '/users/[i:id]/[delete|update:action]', 'usersController#doAction', 'users_do');
-        
-        $this->assertEquals(array(
-            'target' => 'usersController#doAction',
-            'params' => array(
-                'id' => 1,
-                'action' => 'delete'
-            ),
-            'name' => 'users_do'
-        ), $this->router->match('/users/1/delete', 'POST'));
-        
-        $this->assertFalse($this->router->match('/users/1/delete', 'GET'));
-        $this->assertFalse($this->router->match('/users/abc/delete', 'POST'));
-        $this->assertFalse($this->router->match('/users/1/create', 'GET'));
-    }
+	public function testMatchWithFixedParamValues()
+	{
+		$this->router->map('POST', '/users/[i:id]/[delete|update:action]', 'usersController#doAction', 'users_do');
+        
+		$this->assertEquals(array(
+			'target' => 'usersController#doAction',
+			'params' => array(
+				'id' => 1,
+				'action' => 'delete'
+			),
+			'name' => 'users_do'
+		), $this->router->match('/users/1/delete', 'POST'));
+        
+		$this->assertFalse($this->router->match('/users/1/delete', 'GET'));
+		$this->assertFalse($this->router->match('/users/abc/delete', 'POST'));
+		$this->assertFalse($this->router->match('/users/1/create', 'GET'));
+	}
     
-    public function testMatchWithServerVars()
-    {
-        $this->router->map('GET', '/foo/[:controller]/[:action]', 'foo_action', 'foo_route');
+	public function testMatchWithServerVars()
+	{
+		$this->router->map('GET', '/foo/[:controller]/[:action]', 'foo_action', 'foo_route');
         
-        $_SERVER['REQUEST_URI'] = '/foo/test/do';
-        $_SERVER['REQUEST_METHOD'] = 'GET';
+		$_SERVER['REQUEST_URI'] = '/foo/test/do';
+		$_SERVER['REQUEST_METHOD'] = 'GET';
 
-        $this->router->setServer($_SERVER);
-        
-        $this->assertEquals(array(
-            'target' => 'foo_action',
-            'params' => array(
-                'controller' => 'test',
-                'action' => 'do'
-            ),
-            'name' => 'foo_route'
-        ), $this->router->match());
-    }
+		$this->router->setServer($_SERVER);
+        
+		$this->assertEquals(array(
+			'target' => 'foo_action',
+			'params' => array(
+				'controller' => 'test',
+				'action' => 'do'
+			),
+			'name' => 'foo_route'
+		), $this->router->match());
+	}
     
-    public function testMatchWithOptionalUrlParts()
-    {
-        $this->router->map('GET', '/bar/[:controller]/[:action].[:type]?', 'bar_action', 'bar_route');
-        
-        $this->assertEquals(array(
-            'target' => 'bar_action',
-            'params' => array(
-                'controller' => 'test',
-                'action' => 'do',
-                'type' => 'json'
-            ),
-            'name' => 'bar_route'
-        ), $this->router->match('/bar/test/do.json', 'GET'));
-    }
+	public function testMatchWithOptionalUrlParts()
+	{
+		$this->router->map('GET', '/bar/[:controller]/[:action].[:type]?', 'bar_action', 'bar_route');
+        
+		$this->assertEquals(array(
+			'target' => 'bar_action',
+			'params' => array(
+				'controller' => 'test',
+				'action' => 'do',
+				'type' => 'json'
+			),
+			'name' => 'bar_route'
+		), $this->router->match('/bar/test/do.json', 'GET'));
+	}
     
-    public function testMatchWithWildcard()
-    {
-        $this->router->map('GET', '/a', 'foo_action', 'foo_route');
-        $this->router->map('GET', '*', 'bar_action', 'bar_route');
-        
-        $this->assertEquals(array(
-            'target' => 'bar_action',
-            'params' => array(),
-            'name' => 'bar_route'
-        ), $this->router->match('/everything', 'GET'));
-    }
+	public function testMatchWithWildcard()
+	{
+		$this->router->map('GET', '/a', 'foo_action', 'foo_route');
+		$this->router->map('GET', '*', 'bar_action', 'bar_route');
+        
+		$this->assertEquals(array(
+			'target' => 'bar_action',
+			'params' => array(),
+			'name' => 'bar_route'
+		), $this->router->match('/everything', 'GET'));
+	}
     
-    public function testMatchWithCustomRegexp()
-    {
-        $this->router->map('GET', '@^/[a-z]*$', 'bar_action', 'bar_route');
-        
-        $this->assertEquals(array(
-            'target' => 'bar_action',
-            'params' => array(),
-            'name' => 'bar_route'
-        ), $this->router->match('/everything', 'GET'));
-        
-        $this->assertFalse($this->router->match('/some-other-thing', 'GET'));
-    }
+	public function testMatchWithCustomRegexp()
+	{
+		$this->router->map('GET', '@^/[a-z]*$', 'bar_action', 'bar_route');
+        
+		$this->assertEquals(array(
+			'target' => 'bar_action',
+			'params' => array(),
+			'name' => 'bar_route'
+		), $this->router->match('/everything', 'GET'));
+        
+		$this->assertFalse($this->router->match('/some-other-thing', 'GET'));
+	}
 
-    public function testMatchWithUnicodeRegex()
-    {
-        $pattern = '/(?<path>[^';
-        // Arabic characters
-        $pattern .= '\x{0600}-\x{06FF}';
-        $pattern .= '\x{FB50}-\x{FDFD}';
-        $pattern .= '\x{FE70}-\x{FEFF}';
-        $pattern .= '\x{0750}-\x{077F}';
-        // Alphanumeric, /, _, - and space characters
-        $pattern .= 'a-zA-Z0-9\/_\-\s';
-        // 'ZERO WIDTH NON-JOINER'
-        $pattern .= '\x{200C}';
-        $pattern .= ']+)';
-        
-        $this->router->map('GET', '@' . $pattern, 'unicode_action', 'unicode_route');
-        
-        $this->assertEquals(array(
-            'target' => 'unicode_action',
-            'name' => 'unicode_route',
-            'params' => array(
-                'path' => '大家好'
-            )
-        ), $this->router->match('/大家好', 'GET'));
-        
-        $this->assertFalse($this->router->match('/﷽‎', 'GET'));
-    }
+	public function testMatchWithUnicodeRegex()
+	{
+		$pattern = '/(?<path>[^';
+		// Arabic characters
+		$pattern .= '\x{0600}-\x{06FF}';
+		$pattern .= '\x{FB50}-\x{FDFD}';
+		$pattern .= '\x{FE70}-\x{FEFF}';
+		$pattern .= '\x{0750}-\x{077F}';
+		// Alphanumeric, /, _, - and space characters
+		$pattern .= 'a-zA-Z0-9\/_\-\s';
+		// 'ZERO WIDTH NON-JOINER'
+		$pattern .= '\x{200C}';
+		$pattern .= ']+)';
+        
+		$this->router->map('GET', '@' . $pattern, 'unicode_action', 'unicode_route');
+        
+		$this->assertEquals(array(
+			'target' => 'unicode_action',
+			'name' => 'unicode_route',
+			'params' => array(
+				'path' => '大家好'
+			)
+		), $this->router->match('/大家好', 'GET'));
+        
+		$this->assertFalse($this->router->match('/﷽‎', 'GET'));
+	}
 
-    /**
-     * @covers Router::setMatchTypes
-     */
-    public function testMatchWithCustomNamedRegex()
-    {
-        $this->router->getParser()->setMatchTypes(array('cId' => '[a-zA-Z]{2}[0-9](?:_[0-9]++)?'));
-        $this->router->map('GET', '/bar/[cId:customId]', 'bar_action', 'bar_route');
-        
-        $this->assertEquals(array(
-            'target' => 'bar_action',
-            'params' => array(
-                'customId' => 'AB1',
-            ),
-            'name' => 'bar_route'
-        ), $this->router->match('/bar/AB1', 'GET'));
+	/**
+	 * @covers Router::setMatchTypes
+	 */
+	public function testMatchWithCustomNamedRegex()
+	{
+		$this->router->getParser()->setMatchTypes(array('cId' => '[a-zA-Z]{2}[0-9](?:_[0-9]++)?'));
+		$this->router->map('GET', '/bar/[cId:customId]', 'bar_action', 'bar_route');
+        
+		$this->assertEquals(array(
+			'target' => 'bar_action',
+			'params' => array(
+				'customId' => 'AB1',
+			),
+			'name' => 'bar_route'
+		), $this->router->match('/bar/AB1', 'GET'));
 
-        $this->assertEquals(array(
-            'target' => 'bar_action',
-            'params' => array(
-                'customId' => 'AB1_0123456789',
-            ),
-            'name' => 'bar_route'
-        ), $this->router->match('/bar/AB1_0123456789', 'GET'));
-        
-        $this->assertFalse($this->router->match('/some-other-thing', 'GET'));
-    }
+		$this->assertEquals(array(
+			'target' => 'bar_action',
+			'params' => array(
+				'customId' => 'AB1_0123456789',
+			),
+			'name' => 'bar_route'
+		), $this->router->match('/bar/AB1_0123456789', 'GET'));
+        
+		$this->assertFalse($this->router->match('/some-other-thing', 'GET'));
+	}
 
-    public function testMatchWithCustomNamedUnicodeRegex()
-    {
-        $pattern = '[^';
-        // Arabic characters
-        $pattern .= '\x{0600}-\x{06FF}';
-        $pattern .= '\x{FB50}-\x{FDFD}';
-        $pattern .= '\x{FE70}-\x{FEFF}';
-        $pattern .= '\x{0750}-\x{077F}';
-        $pattern .= ']+';
-        
-        $this->router->getParser()->setMatchTypes(array('nonArabic' => $pattern));
-        $this->router->map('GET', '/bar/[nonArabic:string]', 'non_arabic_action', 'non_arabic_route');
+	public function testMatchWithCustomNamedUnicodeRegex()
+	{
+		$pattern = '[^';
+		// Arabic characters
+		$pattern .= '\x{0600}-\x{06FF}';
+		$pattern .= '\x{FB50}-\x{FDFD}';
+		$pattern .= '\x{FE70}-\x{FEFF}';
+		$pattern .= '\x{0750}-\x{077F}';
+		$pattern .= ']+';
+        
+		$this->router->getParser()->setMatchTypes(array('nonArabic' => $pattern));
+		$this->router->map('GET', '/bar/[nonArabic:string]', 'non_arabic_action', 'non_arabic_route');
 
-        $this->assertEquals(array(
-            'target' => 'non_arabic_action',
-            'name'   => 'non_arabic_route',
-            'params' => array(
-                'string' => 'some-path'
-            )
-        ), $this->router->match('/bar/some-path', 'GET'));
-        
-        $this->assertFalse($this->router->match('/﷽‎', 'GET'));
-    }
+		$this->assertEquals(array(
+			'target' => 'non_arabic_action',
+			'name'   => 'non_arabic_route',
+			'params' => array(
+				'string' => 'some-path'
+			)
+		), $this->router->match('/bar/some-path', 'GET'));
+        
+		$this->assertFalse($this->router->match('/﷽‎', 'GET'));
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -84,7 +84,7 @@ 
     {
         $method = 'POST';
         $route = '/[:controller]/[:action]';
-        $target = function () {
+        $target = function() {
         };
 
         $this->assertInternalType('array', $this->router->getRoutes());
@@ -99,7 +99,7 @@ 
     {
         $method = 'POST';
         $route = '/[:controller]/[:action]';
-        $target = function () {
+        $target = function() {
         };
         
         $this->router->setRoutes(array(
@@ -161,7 +161,7 @@ 
     {
         $method = 'POST';
         $route = '/[:controller]/[:action]';
-        $target = function () {
+        $target = function() {
         };
         
         $this->router->map($method, $route, $target);
@@ -178,7 +178,7 @@ 
     {
         $method = 'POST';
         $route = '/[:controller]/[:action]';
-        $target = function () {
+        $target = function() {
         };
         $name = 'myroute';
         
@@ -209,7 +209,7 @@ 
             'action' => 'someaction'
         );
         
-        $this->router->map('GET', '/[:controller]/[:action]', function () {
+        $this->router->map('GET', '/[:controller]/[:action]', function() {
         }, 'foo_route');
         
         $this->assertEquals(
@@ -235,7 +235,7 @@ 
 
     public function testGenerateWithOptionalUrlParts()
     {
-        $this->router->map('GET', '/[:controller]/[:action].[:type]?', function () {
+        $this->router->map('GET', '/[:controller]/[:action].[:type]?', function() {
         }, 'bar_route');
         
         $params = array(

src/RouterParser.php

Indentation +208 added lines, -208 removed lines

@@ -3,212 +3,212 @@
 
 class RouterParser implements RouterParserInterface
 {
-    protected $params = [];
-    protected $matchTypes = [
-        'i'  => '[0-9]++',
-        'a'  => '[0-9A-Za-z]++',
-        'h'  => '[0-9A-Fa-f]++',
-        '*'  => '.+?',
-        '**' => '.++',
-        ''   => '[^/\.]++'
-    ];
-
-    /**
-     * Create router in one call from config.
-     *
-     * @param array $matchTypes
-     */
-    public function __construct($matchTypes = [])
-    {
-        $this->setMatchTypes($matchTypes);
-    }
-
-    /**
-     * Add named match types. It uses array_merge so keys can be overwritten.
-     *
-     * @param array $matchTypes The key is the name and the value is the regex.
-     */
-    public function setMatchTypes($matchTypes)
-    {
-        $this->matchTypes = array_merge($this->matchTypes, $matchTypes);
-    }
-
-    /**
-     * Get the url from a route name
-     *
-     * @param string $basePath
-     * @param string $route
-     * @param array $params
-     *
-     * @return string
-     */
-    public function generate($basePath, $route, array $params)
-    {
-        $url = $basePath . $route;
-
-        if (preg_match_all('`(/|\.|)\[([^:\]]*+)(?::([^:\]]*+))?\](\?|)`', $route, $matches, PREG_SET_ORDER)) {
-            foreach ($matches as $match) {
-                $pre   = $match[1];
-                $param = $match[3];
-                $block = $pre ? substr($match[0], 1) : $match[0];
-
-                if (isset($params[$param])) {
-                    $url = str_replace($block, $params[$param], $url);
-                } elseif ($match[4]) {
-                    $url = str_replace($pre . $block, '', $url);
-                }
-            }
-        }
-
-        return $url;
-    }
-
-    /**
-     * @param string $method
-     * @param string $requestMethod
-     * @param string $routeString
-     * @param string $requestUrl
-     *
-     * @return mixed
-     */
-    public function methodMatch($method, $requestMethod, $routeString, $requestUrl)
-    {
-        $methods = explode('|', $method);
-
-        if (preg_grep("/{$requestMethod}/i", $methods)) {
-            if ($routeString == '*') {
-                return true;
-            } elseif (isset($routeString[0]) && $routeString[0] == '@') {
-                return preg_match('`' . substr($routeString, 1) . '`u', $requestUrl, $this->params);
-            } elseif (($position = strpos($routeString, '[')) === false) {
-                return strcmp($requestUrl, $routeString) === 0;
-            }
-            if (strncmp($requestUrl, $routeString, $position) !== 0) {
-                return false;
-            }
-
-            return preg_match($this->compileRoute($routeString, $requestUrl), $requestUrl, $this->params);
-        }
-
-        return false;
-    }
-
-    /**
-     * Compile the regex for a given route (EXPENSIVE)
-     *
-     * @param $routeString
-     * @param $requestUrl
-     *
-     * @return string
-     */
-    private function compileRoute($routeString, $requestUrl)
-    {
-        $route = $this->getRoute($routeString, $requestUrl);
-
-        if (preg_match_all('`(/|\.|)\[([^:\]]*+)(?::([^:\]]*+))?\](\?|)`', $route, $matches, PREG_SET_ORDER)) {
-            $matchTypes = $this->matchTypes;
-            foreach ($matches as $match) {
-                $pattern = $this->getRoutePattern($matchTypes, $match[1], $match[2], $match[3], $match[4]);
-                $route   = str_replace($match[0], $pattern, $route);
-            }
-        }
-
-        return "`^$route$`u";
-    }
-
-    /**
-     * @param $matchTypes
-     * @param $pre
-     * @param $type
-     * @param $param
-     * @param $optional
-     *
-     * @return string
-     */
-    private function getRoutePattern($matchTypes, $pre, $type, $param, $optional)
-    {
-        if (isset($matchTypes[$type])) {
-            $type = $matchTypes[$type];
-        }
-        if ($pre === '.') {
-            $pre = '\.';
-        }
-
-        //Older versions of PCRE require the 'P' in (?P<named>)
-        return '(?:'
-            . (!empty($pre) ? $pre : null)
-            . '('
-            . (!empty($param) ? "?P<$param>" : null)
-            . $type
-            . '))'
-            . (!empty($optional) ? '?' : null);
-    }
-
-    /**
-     * @param $routeString
-     * @param $requestUrl
-     *
-     * @return bool|string
-     */
-    private function getRoute($routeString, $requestUrl)
-    {
-        $iPointer = $jPointer = 0;
-        $nPointer = isset($routeString[0]) ? $routeString[0] : null;
-        $regex = $route = false;
-
-        // Find the longest non-regex substring and match it against the URI
-        while (true) {
-            if (!isset($routeString[$iPointer])) {
-                break;
-            }
-            if ($regex === false) {
-                if (!$this->isValidRouteRegex($nPointer, $jPointer, $iPointer, $routeString, $requestUrl)) {
-                    continue;
-                }
-                $jPointer++;
-            }
-            $route .= $routeString[$iPointer++];
-        }
-
-        return $route;
-    }
-
-    /**
-     * @param $nPointer
-     * @param $jPointer
-     * @param $iPointer
-     * @param $routeString
-     * @param $requestUrl
-     *
-     * @return bool
-     */
-    private function isValidRouteRegex($nPointer, $jPointer, $iPointer, $routeString, $requestUrl)
-    {
-        $cPointer = $nPointer;
-        $regex = in_array($cPointer, array('[', '(', '.'));
-        if (!$regex && isset($routeString[$iPointer+1])) {
-            $nPointer = $routeString[$iPointer + 1];
-            $regex = in_array($nPointer, array('?', '+', '*', '{'));
-        }
-        if (!$regex && $cPointer !== '/' && (!isset($requestUrl[$jPointer]) || $cPointer !== $requestUrl[$jPointer])) {
-            return false;
-        }
-        return true;
-    }
-
-    /**
-     * @return array
-     */
-    public function getParams()
-    {
-        return $this->params;
-    }
-
-    /**
-     * @return array
-     */
-    public function getMatchTypes()
-    {
-        return $this->matchTypes;
-    }
+	protected $params = [];
+	protected $matchTypes = [
+		'i'  => '[0-9]++',
+		'a'  => '[0-9A-Za-z]++',
+		'h'  => '[0-9A-Fa-f]++',
+		'*'  => '.+?',
+		'**' => '.++',
+		''   => '[^/\.]++'
+	];
+
+	/**
+	 * Create router in one call from config.
+	 *
+	 * @param array $matchTypes
+	 */
+	public function __construct($matchTypes = [])
+	{
+		$this->setMatchTypes($matchTypes);
+	}
+
+	/**
+	 * Add named match types. It uses array_merge so keys can be overwritten.
+	 *
+	 * @param array $matchTypes The key is the name and the value is the regex.
+	 */
+	public function setMatchTypes($matchTypes)
+	{
+		$this->matchTypes = array_merge($this->matchTypes, $matchTypes);
+	}
+
+	/**
+	 * Get the url from a route name
+	 *
+	 * @param string $basePath
+	 * @param string $route
+	 * @param array $params
+	 *
+	 * @return string
+	 */
+	public function generate($basePath, $route, array $params)
+	{
+		$url = $basePath . $route;
+
+		if (preg_match_all('`(/|\.|)\[([^:\]]*+)(?::([^:\]]*+))?\](\?|)`', $route, $matches, PREG_SET_ORDER)) {
+			foreach ($matches as $match) {
+				$pre   = $match[1];
+				$param = $match[3];
+				$block = $pre ? substr($match[0], 1) : $match[0];
+
+				if (isset($params[$param])) {
+					$url = str_replace($block, $params[$param], $url);
+				} elseif ($match[4]) {
+					$url = str_replace($pre . $block, '', $url);
+				}
+			}
+		}
+
+		return $url;
+	}
+
+	/**
+	 * @param string $method
+	 * @param string $requestMethod
+	 * @param string $routeString
+	 * @param string $requestUrl
+	 *
+	 * @return mixed
+	 */
+	public function methodMatch($method, $requestMethod, $routeString, $requestUrl)
+	{
+		$methods = explode('|', $method);
+
+		if (preg_grep("/{$requestMethod}/i", $methods)) {
+			if ($routeString == '*') {
+				return true;
+			} elseif (isset($routeString[0]) && $routeString[0] == '@') {
+				return preg_match('`' . substr($routeString, 1) . '`u', $requestUrl, $this->params);
+			} elseif (($position = strpos($routeString, '[')) === false) {
+				return strcmp($requestUrl, $routeString) === 0;
+			}
+			if (strncmp($requestUrl, $routeString, $position) !== 0) {
+				return false;
+			}
+
+			return preg_match($this->compileRoute($routeString, $requestUrl), $requestUrl, $this->params);
+		}
+
+		return false;
+	}
+
+	/**
+	 * Compile the regex for a given route (EXPENSIVE)
+	 *
+	 * @param $routeString
+	 * @param $requestUrl
+	 *
+	 * @return string
+	 */
+	private function compileRoute($routeString, $requestUrl)
+	{
+		$route = $this->getRoute($routeString, $requestUrl);
+
+		if (preg_match_all('`(/|\.|)\[([^:\]]*+)(?::([^:\]]*+))?\](\?|)`', $route, $matches, PREG_SET_ORDER)) {
+			$matchTypes = $this->matchTypes;
+			foreach ($matches as $match) {
+				$pattern = $this->getRoutePattern($matchTypes, $match[1], $match[2], $match[3], $match[4]);
+				$route   = str_replace($match[0], $pattern, $route);
+			}
+		}
+
+		return "`^$route$`u";
+	}
+
+	/**
+	 * @param $matchTypes
+	 * @param $pre
+	 * @param $type
+	 * @param $param
+	 * @param $optional
+	 *
+	 * @return string
+	 */
+	private function getRoutePattern($matchTypes, $pre, $type, $param, $optional)
+	{
+		if (isset($matchTypes[$type])) {
+			$type = $matchTypes[$type];
+		}
+		if ($pre === '.') {
+			$pre = '\.';
+		}
+
+		//Older versions of PCRE require the 'P' in (?P<named>)
+		return '(?:'
+			. (!empty($pre) ? $pre : null)
+			. '('
+			. (!empty($param) ? "?P<$param>" : null)
+			. $type
+			. '))'
+			. (!empty($optional) ? '?' : null);
+	}
+
+	/**
+	 * @param $routeString
+	 * @param $requestUrl
+	 *
+	 * @return bool|string
+	 */
+	private function getRoute($routeString, $requestUrl)
+	{
+		$iPointer = $jPointer = 0;
+		$nPointer = isset($routeString[0]) ? $routeString[0] : null;
+		$regex = $route = false;
+
+		// Find the longest non-regex substring and match it against the URI
+		while (true) {
+			if (!isset($routeString[$iPointer])) {
+				break;
+			}
+			if ($regex === false) {
+				if (!$this->isValidRouteRegex($nPointer, $jPointer, $iPointer, $routeString, $requestUrl)) {
+					continue;
+				}
+				$jPointer++;
+			}
+			$route .= $routeString[$iPointer++];
+		}
+
+		return $route;
+	}
+
+	/**
+	 * @param $nPointer
+	 * @param $jPointer
+	 * @param $iPointer
+	 * @param $routeString
+	 * @param $requestUrl
+	 *
+	 * @return bool
+	 */
+	private function isValidRouteRegex($nPointer, $jPointer, $iPointer, $routeString, $requestUrl)
+	{
+		$cPointer = $nPointer;
+		$regex = in_array($cPointer, array('[', '(', '.'));
+		if (!$regex && isset($routeString[$iPointer+1])) {
+			$nPointer = $routeString[$iPointer + 1];
+			$regex = in_array($nPointer, array('?', '+', '*', '{'));
+		}
+		if (!$regex && $cPointer !== '/' && (!isset($requestUrl[$jPointer]) || $cPointer !== $requestUrl[$jPointer])) {
+			return false;
+		}
+		return true;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getParams()
+	{
+		return $this->params;
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getMatchTypes()
+	{
+		return $this->matchTypes;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -172,7 +172,7 @@
     {
         $cPointer = $nPointer;
         $regex = in_array($cPointer, array('[', '(', '.'));
-        if (!$regex && isset($routeString[$iPointer+1])) {
+        if (!$regex && isset($routeString[$iPointer + 1])) {
             $nPointer = $routeString[$iPointer + 1];
             $regex = in_array($nPointer, array('?', '+', '*', '{'));
         }