caffeina-core / core

classes/Deferred.php

Indentation +7 added lines, -7 removed lines

@@ -12,12 +12,12 @@ 
 
 class Deferred {
 
-	protected $callback,
+  protected $callback,
             $enabled = true;
 
-	public function __construct( callable $callback ) {
-		$this->callback = $callback;
-	}
+  public function __construct( callable $callback ) {
+    $this->callback = $callback;
+  }
 
   public function disarm() {
     $this->enabled = false;
@@ -27,8 +27,8 @@ 
     $this->enabled = true;
   }
 
-	public function __destruct() {
-		if ( $this->enabled ) call_user_func( $this->callback );
-	}
+  public function __destruct() {
+    if ( $this->enabled ) call_user_func( $this->callback );
+  }
 
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 	protected $callback,
             $enabled = true;
 
-	public function __construct( callable $callback ) {
+	public function __construct(callable $callback) {
 		$this->callback = $callback;
 	}
 
@@ -28,7 +28,7 @@ 
   }
 
 	public function __destruct() {
-		if ( $this->enabled ) call_user_func( $this->callback );
+		if ($this->enabled) call_user_func($this->callback);
 	}
 
 }
\ No newline at end of file

Braces +3 added lines, -1 removed lines

@@ -28,7 +28,9 @@
   }
 
 	public function __destruct() {
-		if ( $this->enabled ) call_user_func( $this->callback );
+		if ( $this->enabled ) {
+		  call_user_func( $this->callback );
+		}
 	}
 
 }
\ No newline at end of file

classes/Route.php

Spacing +74 added lines, -74 removed lines

@@ -38,9 +38,9 @@ 
      * @param string $method The HTTP method for which the route must respond.
      * @return Route
      */
-    public function __construct($URLPattern, $callback = null, $method='get'){
-        $this->URLPattern = rtrim(implode('',static::$prefix),'/') . '/' . trim($URLPattern,'/')?:'/';
-        $this->URLPattern = $this->URLPattern != '/' ? rtrim($this->URLPattern,'/') : $this->URLPattern;
+    public function __construct($URLPattern, $callback = null, $method = 'get') {
+        $this->URLPattern = rtrim(implode('', static::$prefix), '/').'/'.trim($URLPattern, '/') ?: '/';
+        $this->URLPattern = $this->URLPattern != '/' ? rtrim($this->URLPattern, '/') : $this->URLPattern;
         $this->dynamic    = $this->isDynamic($this->URLPattern);
         $this->pattern    = $this->dynamic ? $this->compilePatternAsRegex($this->URLPattern, $this->rules) : $this->URLPattern;
         $this->callback   = $callback;
@@ -56,19 +56,19 @@ 
      * @param  string $method The HTTP Method to check against.
      * @return boolean
      */
-    public function match($URL,$method='get'){
+    public function match($URL, $method = 'get') {
         $method = strtolower($method);
 
         // * is an http method wildcard
         if (empty($this->methods[$method]) && empty($this->methods['*'])) return false;
-        $URL = rtrim($URL,'/');
+        $URL = rtrim($URL, '/');
         $args = [];
-        if ( $this->dynamic
-               ? preg_match($this->pattern,$URL,$args)
-               : $URL == rtrim($this->pattern,'/')
-        ){
+        if ($this->dynamic
+               ? preg_match($this->pattern, $URL, $args)
+               : $URL == rtrim($this->pattern, '/')
+        ) {
             foreach ($args as $key => $value) {
-              if ( false === is_string($key) ) unset($args[$key]);
+              if (false === is_string($key)) unset($args[$key]);
             }
             return $args;
         }
@@ -81,23 +81,23 @@ 
      * @param  string $method The HTTP Method requested.
      * @return array The callback response.
      */
-    public function run(array $args,$method='get'){
+    public function run(array $args, $method = 'get') {
         $method = strtolower($method);
-        $this->response 			 		= '';
-        $this->response_object 		= null;
+        $this->response = '';
+        $this->response_object = null;
        	$this->response_is_object = false;
 
         // Call direct befores
-        if ( $this->befores ) {
+        if ($this->befores) {
           // Reverse befores order
           foreach (array_reverse($this->befores) as $mw) {
 	        	ob_start();
             $mw_result = call_user_func($mw->bindTo($this));
           	$this->response .= ob_get_clean();
-            if ( false  === $mw_result ) {
+            if (false === $mw_result) {
             	return [''];
-            } else if (is_a($mw_result,'View') || is_string($mw_result)) {
-              $this->response .= (string)$mw_result;
+            } else if (is_a($mw_result, 'View') || is_string($mw_result)) {
+              $this->response .= (string) $mw_result;
           	}
           }
         }
@@ -112,45 +112,45 @@ 
 					Response::type(Response::TYPE_HTML);
 	        ob_start();
 	        // Silence "Cannot bind an instance to a static closure" warnings
-	        $view_results 	 = call_user_func_array(@$callback->bindTo($this), $args);
+	        $view_results = call_user_func_array(@$callback->bindTo($this), $args);
 	        $this->response .= ob_get_clean();
 
 	        // Render View if returned, else echo string or encode json response
-	        if ( null !== $view_results ) {
-	          if (is_a($view_results,'View') || is_string($view_results)) {
-	              $this->response .= (string)$view_results;
+	        if (null !== $view_results) {
+	          if (is_a($view_results, 'View') || is_string($view_results)) {
+	              $this->response .= (string) $view_results;
 	          } else {
 			        	$this->response_is_object = true;
-	              $this->response_object 		= $view_results;
+	              $this->response_object = $view_results;
 	          }
 	        }
 
-        } else if (is_a($callback,'View') || is_string($callback)) {
+        } else if (is_a($callback, 'View') || is_string($callback)) {
           // return rendered View or direct string
-        	$this->response .= (string)$callback;
+        	$this->response .= (string) $callback;
         } else {
           // JSON encode returned value
         	$this->response_is_object = true;
-        	$this->response_object 		= $callback;
+        	$this->response_object = $callback;
         }
 
         // Apply afters
-        if ( $this->afters ) {
+        if ($this->afters) {
           foreach ($this->afters as $mw) {
 	        	ob_start();
             $mw_result = call_user_func($mw->bindTo($this));
           	$this->response .= ob_get_clean();
-            if ( false  === $mw_result ) {
+            if (false === $mw_result) {
             	return [''];
-            } else if (is_a($mw_result,'View') || is_string($mw_result)) {
-              $this->response .= (string)$mw_result;
+            } else if (is_a($mw_result, 'View') || is_string($mw_result)) {
+              $this->response .= (string) $mw_result;
           	}
           }
         }
 
         Event::trigger('core.route.after', $this);
 
-        if ( $this->response_is_object ){
+        if ($this->response_is_object) {
 					$this->response = Response::json($this->response_object);
         } else {
 					Response::add($this->response);
@@ -167,8 +167,8 @@ 
      * @param  string $method The HTTP Method to check against.
      * @return array The callback response.
      */
-    public function runIfMatch($URL, $method='get'){
-        return ($args = $this->match($URL,$method)) ? $this->run($args,$method) : null;
+    public function runIfMatch($URL, $method = 'get') {
+        return ($args = $this->match($URL, $method)) ? $this->run($args, $method) : null;
     }
 
     /**
@@ -177,8 +177,8 @@ 
      * @param  $callback The callback to be invoked (with variables extracted from the route if present) when the route match the request URI.
      * @return Route
      */
-    public static function on($URLPattern, $callback = null){
-        return new Route($URLPattern,$callback);
+    public static function on($URLPattern, $callback = null) {
+        return new Route($URLPattern, $callback);
     }
 
     /**
@@ -187,8 +187,8 @@ 
      * @param  $callback The callback to be invoked (with variables extracted from the route if present) when the route match the request URI.
      * @return Route
      */
-    public static function any($URLPattern, $callback = null){
-        return (new Route($URLPattern,$callback))->via('*');
+    public static function any($URLPattern, $callback = null) {
+        return (new Route($URLPattern, $callback))->via('*');
     }
 
     /**
@@ -233,7 +233,7 @@ 
      */
     public function & via(){
         $this->methods = [];
-        foreach (func_get_args() as $method){
+        foreach (func_get_args() as $method) {
             $this->methods[strtolower($method)] = true;
         }
         return $this;
@@ -255,10 +255,10 @@ 
      * @return Route
      */
     public function & rules(array $rules){
-        foreach ((array)$rules as $varname => $rule){
+        foreach ((array) $rules as $varname => $rule) {
             $this->rules[$varname] = $rule;
         }
-        $this->pattern = $this->compilePatternAsRegex($this->URLPattern,$this->rules);
+        $this->pattern = $this->compilePatternAsRegex($this->URLPattern, $this->rules);
         return $this;
     }
 
@@ -296,10 +296,10 @@ 
      * @param  string $pattern The URL schema.
      * @return string The compiled PREG RegEx.
      */
-    protected static function compilePatternAsRegex($pattern, $rules=[]){
-        return '#^'.preg_replace_callback('#:([a-zA-Z]\w*)#S',function($g) use (&$rules){
-            return '(?<' . $g[1] . '>' . (isset($rules[$g[1]])?$rules[$g[1]]:'[^/]+') .')';
-        },str_replace(['.',')','*'],['\.',')?','.+'],$pattern)).'$#';
+    protected static function compilePatternAsRegex($pattern, $rules = []) {
+        return '#^'.preg_replace_callback('#:([a-zA-Z]\w*)#S', function($g) use (&$rules){
+            return '(?<'.$g[1].'>'.(isset($rules[$g[1]]) ? $rules[$g[1]] : '[^/]+').')';
+        },str_replace(['.', ')', '*'], ['\.', ')?', '.+'], $pattern)).'$#';
     }
 
     /**
@@ -309,10 +309,10 @@ 
      * @param  boolean $cut If true don't limit the matching to the whole URL (used for group pattern extraction)
      * @return array The extracted variables
      */
-    protected static function extractVariablesFromURL($pattern, $URL=null, $cut=false){
+    protected static function extractVariablesFromURL($pattern, $URL = null, $cut = false) {
         $URL     = $URL ?: Request::URI();
-        $pattern = $cut ? str_replace('$#','',$pattern).'#' : $pattern;
-        if ( !preg_match($pattern,$URL,$args) ) return false;
+        $pattern = $cut ? str_replace('$#', '', $pattern).'#' : $pattern;
+        if (!preg_match($pattern, $URL, $args)) return false;
         foreach ($args as $key => $value) {
             if (false === is_string($key)) unset($args[$key]);
         }
@@ -324,8 +324,8 @@ 
      * @param  string  $pattern The URL schema.
      * @return boolean
      */
-    protected static function isDynamic($pattern){
-      return strlen($pattern) != strcspn($pattern,':(?[*+');
+    protected static function isDynamic($pattern) {
+      return strlen($pattern) != strcspn($pattern, ':(?[*+');
     }
 
     /**
@@ -333,9 +333,9 @@ 
      * @param Route $r
      * @return Route
      */
-    public static function add($r){
-        if ( isset(static::$group[0]) ) static::$group[0]->add($r);
-        return static::$routes[implode('',static::$prefix)][] = $r;
+    public static function add($r) {
+        if (isset(static::$group[0])) static::$group[0]->add($r);
+        return static::$routes[implode('', static::$prefix)][] = $r;
     }
 
     /**
@@ -343,16 +343,16 @@ 
      * @param  string $prefix The url prefix for the internal route definitions.
      * @param  string $callback This callback is invoked on $prefix match of the current request URI.
      */
-    public static function group($prefix,$callback=null){
+    public static function group($prefix, $callback = null) {
 
         // Skip definition if current request doesn't match group.
 
-        $prefix_complete = rtrim(implode('',static::$prefix),'/') . $prefix;
+        $prefix_complete = rtrim(implode('', static::$prefix), '/').$prefix;
 
-        if ( static::isDynamic($prefix) ){
+        if (static::isDynamic($prefix)) {
 
             // Dynamic group, capture vars
-            $vars = static::extractVariablesFromURL(static::compilePatternAsRegex($prefix_complete),null,true);
+            $vars = static::extractVariablesFromURL(static::compilePatternAsRegex($prefix_complete), null, true);
 
             // Errors in compile pattern or variable extraction, aborting.
             if (false === $vars) return;
@@ -360,14 +360,14 @@ 
             static::$prefix[] = $prefix;
             if (empty(static::$group)) static::$group = [];
             array_unshift(static::$group, new RouteGroup());
-            if ($callback) call_user_func_array($callback,$vars);
+            if ($callback) call_user_func_array($callback, $vars);
             $group = static::$group[0];
             array_shift(static::$group);
             array_pop(static::$prefix);
-            if (empty(static::$prefix)) static::$prefix=[''];
+            if (empty(static::$prefix)) static::$prefix = [''];
             return $group;
 
-        } else if ( 0 === strpos(Request::URI(), $prefix_complete) ){
+        } else if (0 === strpos(Request::URI(), $prefix_complete)) {
 
             // Static group
             static::$prefix[] = $prefix;
@@ -377,7 +377,7 @@ 
             $group = static::$group[0];
             array_shift(static::$group);
             array_pop(static::$prefix);
-            if (empty(static::$prefix)) static::$prefix=[''];
+            if (empty(static::$prefix)) static::$prefix = [''];
             return $group;
         } else {
 
@@ -387,8 +387,8 @@ 
 
     }
 
-    public static function exitWithError($code,$message="Application Error"){
-    	Response::error($code,$message);
+    public static function exitWithError($code, $message = "Application Error") {
+    	Response::error($code, $message);
     	Response::send();
     	exit;
     }
@@ -398,18 +398,18 @@ 
      * @param  string $URL The URL to match onto.
      * @return boolean true if a route callback was executed.
      */
-    public static function dispatch($URL=null,$method=null){
+    public static function dispatch($URL = null, $method = null) {
         if (!$URL)     $URL     = Request::URI();
         if (!$method)  $method  = Request::method();
 
-        if (Options::get('core.response.autosend',true)){
+        if (Options::get('core.response.autosend', true)) {
           $__deferred_send = new Deferred('Response::send');
         }
 
-        foreach ((array)static::$routes as $group => $routes){
+        foreach ((array) static::$routes as $group => $routes) {
             foreach ($routes as $route) {
-                if (is_a($route, 'Route') && false !== ($args = $route->match($URL,$method))){
-                    $route->run($args,$method);
+                if (is_a($route, 'Route') && false !== ($args = $route->match($URL, $method))) {
+                    $route->run($args, $method);
                     return true;
                 }
             }
@@ -424,34 +424,34 @@ 
 class RouteGroup {
   protected $routes;
 
-  public function __construct(){
+  public function __construct() {
     $this->routes = new SplObjectStorage;
     return Route::add($this);
   }
 
-  public function has($r){
+  public function has($r) {
     return $this->routes->contains($r);
   }
 
-  public function add($r){
+  public function add($r) {
     $this->routes->attach($r);
     return $this;
   }
 
-  public function remove($r){
+  public function remove($r) {
     if ($this->routes->contains($r)) $this->routes->detach($r);
     return $this;
   }
 
-  public function before($callbacks){
-    foreach ($this->routes as $route){
+  public function before($callbacks) {
+    foreach ($this->routes as $route) {
       $route->before($callbacks);
     }
     return $this;
   }
 
-  public function after($callbacks){
-    foreach ($this->routes as $route){
+  public function after($callbacks) {
+    foreach ($this->routes as $route) {
       $route->after($callbacks);
     }
     return $this;

Braces +79 added lines, -47 removed lines

@@ -38,7 +38,7 @@ 
      * @param string $method The HTTP method for which the route must respond.
      * @return Route
      */
-    public function __construct($URLPattern, $callback = null, $method='get'){
+    public function __construct($URLPattern, $callback = null, $method='get') {
         $this->URLPattern = rtrim(implode('',static::$prefix),'/') . '/' . trim($URLPattern,'/')?:'/';
         $this->URLPattern = $this->URLPattern != '/' ? rtrim($this->URLPattern,'/') : $this->URLPattern;
         $this->dynamic    = $this->isDynamic($this->URLPattern);
@@ -56,19 +56,23 @@ 
      * @param  string $method The HTTP Method to check against.
      * @return boolean
      */
-    public function match($URL,$method='get'){
+    public function match($URL,$method='get') {
         $method = strtolower($method);
 
         // * is an http method wildcard
-        if (empty($this->methods[$method]) && empty($this->methods['*'])) return false;
+        if (empty($this->methods[$method]) && empty($this->methods['*'])) {
+          return false;
+        }
         $URL = rtrim($URL,'/');
         $args = [];
         if ( $this->dynamic
                ? preg_match($this->pattern,$URL,$args)
                : $URL == rtrim($this->pattern,'/')
-        ){
+        ) {
             foreach ($args as $key => $value) {
-              if ( false === is_string($key) ) unset($args[$key]);
+              if ( false === is_string($key) ) {
+                unset($args[$key]);
+              }
             }
             return $args;
         }
@@ -81,7 +85,7 @@ 
      * @param  string $method The HTTP Method requested.
      * @return array The callback response.
      */
-    public function run(array $args,$method='get'){
+    public function run(array $args,$method='get') {
         $method = strtolower($method);
         $this->response 			 		= '';
         $this->response_object 		= null;
@@ -150,7 +154,7 @@ 
 
         Event::trigger('core.route.after', $this);
 
-        if ( $this->response_is_object ){
+        if ( $this->response_is_object ) {
 					$this->response = Response::json($this->response_object);
         } else {
 					Response::add($this->response);
@@ -167,7 +171,7 @@ 
      * @param  string $method The HTTP Method to check against.
      * @return array The callback response.
      */
-    public function runIfMatch($URL, $method='get'){
+    public function runIfMatch($URL, $method='get') {
         return ($args = $this->match($URL,$method)) ? $this->run($args,$method) : null;
     }
 
@@ -177,7 +181,7 @@ 
      * @param  $callback The callback to be invoked (with variables extracted from the route if present) when the route match the request URI.
      * @return Route
      */
-    public static function on($URLPattern, $callback = null){
+    public static function on($URLPattern, $callback = null) {
         return new Route($URLPattern,$callback);
     }
 
@@ -187,7 +191,7 @@ 
      * @param  $callback The callback to be invoked (with variables extracted from the route if present) when the route match the request URI.
      * @return Route
      */
-    public static function any($URLPattern, $callback = null){
+    public static function any($URLPattern, $callback = null) {
         return (new Route($URLPattern,$callback))->via('*');
     }
 
@@ -233,7 +237,7 @@ 
      */
     public function & via(){
         $this->methods = [];
-        foreach (func_get_args() as $method){
+        foreach (func_get_args() as $method) {
             $this->methods[strtolower($method)] = true;
         }
         return $this;
@@ -255,7 +259,7 @@ 
      * @return Route
      */
     public function & rules(array $rules){
-        foreach ((array)$rules as $varname => $rule){
+        foreach ((array)$rules as $varname => $rule) {
             $this->rules[$varname] = $rule;
         }
         $this->pattern = $this->compilePatternAsRegex($this->URLPattern,$this->rules);
@@ -284,7 +288,9 @@ 
         $route->callback = [];
         foreach ($callbacks as $method => $callback) {
            $method = strtolower($method);
-           if (Request::method() !== $method) continue;
+           if (Request::method() !== $method) {
+             continue;
+           }
            $route->callback[$method] = $callback;
            $route->methods[$method]  = 1;
         }
@@ -296,8 +302,8 @@ 
      * @param  string $pattern The URL schema.
      * @return string The compiled PREG RegEx.
      */
-    protected static function compilePatternAsRegex($pattern, $rules=[]){
-        return '#^'.preg_replace_callback('#:([a-zA-Z]\w*)#S',function($g) use (&$rules){
+    protected static function compilePatternAsRegex($pattern, $rules=[]) {
+        return '#^'.preg_replace_callback('#:([a-zA-Z]\w*)#S',function($g) use (&$rules) {
             return '(?<' . $g[1] . '>' . (isset($rules[$g[1]])?$rules[$g[1]]:'[^/]+') .')';
         },str_replace(['.',')','*'],['\.',')?','.+'],$pattern)).'$#';
     }
@@ -309,12 +315,16 @@ 
      * @param  boolean $cut If true don't limit the matching to the whole URL (used for group pattern extraction)
      * @return array The extracted variables
      */
-    protected static function extractVariablesFromURL($pattern, $URL=null, $cut=false){
+    protected static function extractVariablesFromURL($pattern, $URL=null, $cut=false) {
         $URL     = $URL ?: Request::URI();
         $pattern = $cut ? str_replace('$#','',$pattern).'#' : $pattern;
-        if ( !preg_match($pattern,$URL,$args) ) return false;
+        if ( !preg_match($pattern,$URL,$args) ) {
+          return false;
+        }
         foreach ($args as $key => $value) {
-            if (false === is_string($key)) unset($args[$key]);
+            if (false === is_string($key)) {
+              unset($args[$key]);
+            }
         }
         return $args;
     }
@@ -324,7 +334,7 @@ 
      * @param  string  $pattern The URL schema.
      * @return boolean
      */
-    protected static function isDynamic($pattern){
+    protected static function isDynamic($pattern) {
       return strlen($pattern) != strcspn($pattern,':(?[*+');
     }
 
@@ -333,8 +343,10 @@ 
      * @param Route $r
      * @return Route
      */
-    public static function add($r){
-        if ( isset(static::$group[0]) ) static::$group[0]->add($r);
+    public static function add($r) {
+        if ( isset(static::$group[0]) ) {
+          static::$group[0]->add($r);
+        }
         return static::$routes[implode('',static::$prefix)][] = $r;
     }
 
@@ -343,41 +355,55 @@ 
      * @param  string $prefix The url prefix for the internal route definitions.
      * @param  string $callback This callback is invoked on $prefix match of the current request URI.
      */
-    public static function group($prefix,$callback=null){
+    public static function group($prefix,$callback=null) {
 
         // Skip definition if current request doesn't match group.
 
         $prefix_complete = rtrim(implode('',static::$prefix),'/') . $prefix;
 
-        if ( static::isDynamic($prefix) ){
+        if ( static::isDynamic($prefix) ) {
 
             // Dynamic group, capture vars
             $vars = static::extractVariablesFromURL(static::compilePatternAsRegex($prefix_complete),null,true);
 
             // Errors in compile pattern or variable extraction, aborting.
-            if (false === $vars) return;
+            if (false === $vars) {
+              return;
+            }
 
             static::$prefix[] = $prefix;
-            if (empty(static::$group)) static::$group = [];
+            if (empty(static::$group)) {
+              static::$group = [];
+            }
             array_unshift(static::$group, new RouteGroup());
-            if ($callback) call_user_func_array($callback,$vars);
+            if ($callback) {
+              call_user_func_array($callback,$vars);
+            }
             $group = static::$group[0];
             array_shift(static::$group);
             array_pop(static::$prefix);
-            if (empty(static::$prefix)) static::$prefix=[''];
+            if (empty(static::$prefix)) {
+              static::$prefix=[''];
+            }
             return $group;
 
-        } else if ( 0 === strpos(Request::URI(), $prefix_complete) ){
+        } else if ( 0 === strpos(Request::URI(), $prefix_complete) ) {
 
             // Static group
             static::$prefix[] = $prefix;
-            if (empty(static::$group)) static::$group = [];
+            if (empty(static::$group)) {
+              static::$group = [];
+            }
             array_unshift(static::$group, new RouteGroup());
-            if ($callback) call_user_func($callback);
+            if ($callback) {
+              call_user_func($callback);
+            }
             $group = static::$group[0];
             array_shift(static::$group);
             array_pop(static::$prefix);
-            if (empty(static::$prefix)) static::$prefix=[''];
+            if (empty(static::$prefix)) {
+              static::$prefix=[''];
+            }
             return $group;
         } else {
 
@@ -387,7 +413,7 @@ 
 
     }
 
-    public static function exitWithError($code,$message="Application Error"){
+    public static function exitWithError($code,$message="Application Error") {
     	Response::error($code,$message);
     	Response::send();
     	exit;
@@ -398,17 +424,21 @@ 
      * @param  string $URL The URL to match onto.
      * @return boolean true if a route callback was executed.
      */
-    public static function dispatch($URL=null,$method=null){
-        if (!$URL)     $URL     = Request::URI();
-        if (!$method)  $method  = Request::method();
+    public static function dispatch($URL=null,$method=null) {
+        if (!$URL) {
+          $URL     = Request::URI();
+        }
+        if (!$method) {
+          $method  = Request::method();
+        }
 
-        if (Options::get('core.response.autosend',true)){
+        if (Options::get('core.response.autosend',true)) {
           $__deferred_send = new Deferred('Response::send');
         }
 
-        foreach ((array)static::$routes as $group => $routes){
+        foreach ((array)static::$routes as $group => $routes) {
             foreach ($routes as $route) {
-                if (is_a($route, 'Route') && false !== ($args = $route->match($URL,$method))){
+                if (is_a($route, 'Route') && false !== ($args = $route->match($URL,$method))) {
                     $route->run($args,$method);
                     return true;
                 }
@@ -424,34 +454,36 @@ 
 class RouteGroup {
   protected $routes;
 
-  public function __construct(){
+  public function __construct() {
     $this->routes = new SplObjectStorage;
     return Route::add($this);
   }
 
-  public function has($r){
+  public function has($r) {
     return $this->routes->contains($r);
   }
 
-  public function add($r){
+  public function add($r) {
     $this->routes->attach($r);
     return $this;
   }
 
-  public function remove($r){
-    if ($this->routes->contains($r)) $this->routes->detach($r);
+  public function remove($r) {
+    if ($this->routes->contains($r)) {
+      $this->routes->detach($r);
+    }
     return $this;
   }
 
-  public function before($callbacks){
-    foreach ($this->routes as $route){
+  public function before($callbacks) {
+    foreach ($this->routes as $route) {
       $route->before($callbacks);
     }
     return $this;
   }
 
-  public function after($callbacks){
-    foreach ($this->routes as $route){
+  public function after($callbacks) {
+    foreach ($this->routes as $route) {
       $route->after($callbacks);
     }
     return $this;