caffeina-core / core

classes/Route.php

Spacing +75 added lines, -75 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,20 +398,20 @@ 
      * @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();
 
-        $__deferred_send = new Deferred(function(){
-          if (Options::get('core.response.autosend',false)){
+        $__deferred_send = new Deferred(function() {
+          if (Options::get('core.response.autosend', false)) {
             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;
                 }
             }
@@ -426,34 +426,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 +80 added lines, -48 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,19 +424,23 @@ 
      * @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();
+        }
 
-        $__deferred_send = new Deferred(function(){
-          if (Options::get('core.response.autosend',false)){
+        $__deferred_send = new Deferred(function() {
+          if (Options::get('core.response.autosend',false)) {
             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;
                 }
@@ -426,34 +456,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;

classes/Map.php

Spacing +23 added lines, -23 removed lines

@@ -29,11 +29,11 @@ 
      * @param  mixed $default (optional) The default value. If is a callable it will executed and the return value will be used.
      * @return mixed The value of the key or the default (resolved) value if the key not existed.
      */
-    public function get($key, $default=null){
-        if ($ptr =& $this->find($key,false)){
+    public function get($key, $default = null) {
+        if ($ptr = & $this->find($key, false)) {
             return $ptr;
         } else {
-            if ($default !== null){
+            if ($default !== null) {
                 return $this->set($key, is_callable($default) ? call_user_func($default) : $default);
             } else {
                 return null;
@@ -47,11 +47,11 @@ 
      * @param  mixed $value (optional) The value. If is a callable it will executed and the return value will be used.
      * @return mixed The value of the key or the default (resolved) value if the key not existed.
      */
-    public function set($key, $value=null){
+    public function set($key, $value = null) {
         if (is_array($key)) {
             return $this->merge($key);
         } else {
-            $ptr =& $this->find($key, true);
+            $ptr = & $this->find($key, true);
             return $ptr = $value;
         }
     }
@@ -61,7 +61,7 @@ 
      * @param  string $key The key path in dot notation
      * @param  boolean $compact (optional) Compact map removing empty paths.
      */
-    public function delete($key, $compact=true){
+    public function delete($key, $compact = true) {
         $this->set($key, null);
         if ($compact) $this->compact();
     }
@@ -71,18 +71,18 @@ 
      * @param  string $key The key path in dot notation
      * @return boolean
      */
-    public function exists($key){
+    public function exists($key) {
         return null !== $this->find($key, false);
     }
 
     /**
      * Clear all key path in map.
      */
-    public function clear(){
+    public function clear() {
         $this->fields = [];
     }
 
-    public function __construct($fields=null){
+    public function __construct($fields = null) {
         $this->load($fields);
     }
 
@@ -90,8 +90,8 @@ 
      * Load an associative array/object as the map source.
      * @param  string $fields The array to merge
      */
-    public function load($fields){
-        if ($fields) $this->fields = (array)$fields;
+    public function load($fields) {
+        if ($fields) $this->fields = (array) $fields;
     }
 
     /**
@@ -99,17 +99,17 @@ 
      * @param  array   $array The array to merge
      * @param  boolean $merge_back If `true` merge the map over the $array, if `false` (default) the reverse.
      */
-    public function merge($array, $merge_back=false){
+    public function merge($array, $merge_back = false) {
         $this->fields = $merge_back
-            ? array_replace_recursive((array)$array, $this->fields)
-            : array_replace_recursive($this->fields, (array)$array);
+            ? array_replace_recursive((array) $array, $this->fields)
+            : array_replace_recursive($this->fields, (array) $array);
     }
 
     /**
      * Compact map removing empty paths
      */
 
-    public function compact(){
+    public function compact() {
 
         $array_filter_rec = function($input, $callback = null) use (&$array_filter_rec) {
             foreach ($input as &$value) {
@@ -120,7 +120,7 @@ 
             return array_filter($input, $callback);
         };
 
-        $this->fields = $array_filter_rec($this->fields,function($a){ return $a !== null; });
+        $this->fields = $array_filter_rec($this->fields, function($a) { return $a !== null;});
     }
 
     /**
@@ -130,17 +130,17 @@ 
      * @param  callable  If passed this callback will be applied to the founded value.
      * @return mixed The founded value.
      */
-    public function & find($path, $create=false, callable $operation=null) {
-        $tok = strtok($path,'.');
+    public function & find($path, $create = false, callable $operation = null) {
+        $tok = strtok($path, '.');
 
-        if($create){
-            $value =& $this->fields;
+        if ($create) {
+            $value = & $this->fields;
         } else {
             $value = $this->fields;
         }
 
-        while($tok !== false){
-            $value =& $value[$tok];
+        while ($tok !== false) {
+            $value = & $value[$tok];
             $tok = strtok('.');
         }
         if (is_callable($operation)) $operation($value);
@@ -154,7 +154,7 @@ 
      *
      * @return string        The json object
      */
-    public function jsonSerialize(){
+    public function jsonSerialize() {
       return $this->fields;
     }
 

Braces +23 added lines, -17 removed lines

@@ -29,11 +29,11 @@ 
      * @param  mixed $default (optional) The default value. If is a callable it will executed and the return value will be used.
      * @return mixed The value of the key or the default (resolved) value if the key not existed.
      */
-    public function get($key, $default=null){
-        if ($ptr =& $this->find($key,false)){
+    public function get($key, $default=null) {
+        if ($ptr =& $this->find($key,false)) {
             return $ptr;
         } else {
-            if ($default !== null){
+            if ($default !== null) {
                 return $this->set($key, is_callable($default) ? call_user_func($default) : $default);
             } else {
                 return null;
@@ -47,7 +47,7 @@ 
      * @param  mixed $value (optional) The value. If is a callable it will executed and the return value will be used.
      * @return mixed The value of the key or the default (resolved) value if the key not existed.
      */
-    public function set($key, $value=null){
+    public function set($key, $value=null) {
         if (is_array($key)) {
             return $this->merge($key);
         } else {
@@ -61,9 +61,11 @@ 
      * @param  string $key The key path in dot notation
      * @param  boolean $compact (optional) Compact map removing empty paths.
      */
-    public function delete($key, $compact=true){
+    public function delete($key, $compact=true) {
         $this->set($key, null);
-        if ($compact) $this->compact();
+        if ($compact) {
+          $this->compact();
+        }
     }
 
     /**
@@ -71,18 +73,18 @@ 
      * @param  string $key The key path in dot notation
      * @return boolean
      */
-    public function exists($key){
+    public function exists($key) {
         return null !== $this->find($key, false);
     }
 
     /**
      * Clear all key path in map.
      */
-    public function clear(){
+    public function clear() {
         $this->fields = [];
     }
 
-    public function __construct($fields=null){
+    public function __construct($fields=null) {
         $this->load($fields);
     }
 
@@ -90,8 +92,10 @@ 
      * Load an associative array/object as the map source.
      * @param  string $fields The array to merge
      */
-    public function load($fields){
-        if ($fields) $this->fields = (array)$fields;
+    public function load($fields) {
+        if ($fields) {
+          $this->fields = (array)$fields;
+        }
     }
 
     /**
@@ -99,7 +103,7 @@ 
      * @param  array   $array The array to merge
      * @param  boolean $merge_back If `true` merge the map over the $array, if `false` (default) the reverse.
      */
-    public function merge($array, $merge_back=false){
+    public function merge($array, $merge_back=false) {
         $this->fields = $merge_back
             ? array_replace_recursive((array)$array, $this->fields)
             : array_replace_recursive($this->fields, (array)$array);
@@ -109,7 +113,7 @@ 
      * Compact map removing empty paths
      */
 
-    public function compact(){
+    public function compact() {
 
         $array_filter_rec = function($input, $callback = null) use (&$array_filter_rec) {
             foreach ($input as &$value) {
@@ -133,17 +137,19 @@ 
     public function & find($path, $create=false, callable $operation=null) {
         $tok = strtok($path,'.');
 
-        if($create){
+        if($create) {
             $value =& $this->fields;
         } else {
             $value = $this->fields;
         }
 
-        while($tok !== false){
+        while($tok !== false) {
             $value =& $value[$tok];
             $tok = strtok('.');
         }
-        if (is_callable($operation)) $operation($value);
+        if (is_callable($operation)) {
+          $operation($value);
+        }
         return $value;
     }
 
@@ -154,7 +160,7 @@ 
      *
      * @return string        The json object
      */
-    public function jsonSerialize(){
+    public function jsonSerialize() {
       return $this->fields;
     }
 

classes/Response.php

Indentation +10 added lines, -10 removed lines

@@ -23,13 +23,13 @@ 
           TYPE_BIN                = 'application/octet-stream';
 
     protected static $payload     = [],
-                     $status      = 200,
-                     $charset     = "utf-8",
-                     $headers     = ['Content-Type' => ['text/html; charset=utf-8']],
-                     $buffer      = null,
-                     $force_dl    = false,
-                     $link        = null,
-                     $sent        = false;
+                      $status      = 200,
+                      $charset     = "utf-8",
+                      $headers     = ['Content-Type' => ['text/html; charset=utf-8']],
+                      $buffer      = null,
+                      $force_dl    = false,
+                      $link        = null,
+                      $sent        = false;
 
 
     public static function charset($charset){
@@ -187,12 +187,12 @@ 
       if ($setBody) static::$payload = [$setBody];
       return Filter::with('core.response.body',
                 is_array(static::$payload) ? implode('',static::$payload) : static::$payload
-             );
+              );
     }
 
     public static function headers($setHeaders=null){
-       if ($setHeaders) static::$headers = $setHeaders;
-       return static::$headers;
+        if ($setHeaders) static::$headers = $setHeaders;
+        return static::$headers;
     }
 
     /**

Spacing +43 added lines, -43 removed lines

@@ -32,12 +32,12 @@ 
                      $sent        = false;
 
 
-    public static function charset($charset){
+    public static function charset($charset) {
         static::$charset = $charset;
     }
 
-    public static function type($mime){
-        static::header('Content-Type',$mime . (static::$charset ? '; charset='.static::$charset : ''));
+    public static function type($mime) {
+        static::header('Content-Type', $mime.(static::$charset ? '; charset='.static::$charset : ''));
     }
 
     /**
@@ -45,39 +45,39 @@ 
      * @param  string/bool $filename Pass a falsy value to disable download or pass a filename for exporting content
      * @return [type]        [description]
      */
-    public static function download($filename){
+    public static function download($filename) {
         static::$force_dl = $filename;
     }
 
     /**
      * Start capturing output
      */
-    public static function start(){
+    public static function start() {
         static::$buffer = ob_start();
     }
 
     /**
      * Enable CORS HTTP headers.
      */
-    public static function enableCORS(){
+    public static function enableCORS() {
 
         // Allow from any origin
-        if ($origin = filter_input(INPUT_SERVER,'HTTP_ORIGIN')) {
+        if ($origin = filter_input(INPUT_SERVER, 'HTTP_ORIGIN')) {
           static::header('Access-Control-Allow-Origin', $origin);
           static::header('Access-Control-Allow-Credentials', 'true');
           static::header('Access-Control-Max-Age', 86400);
         }
 
         // Access-Control headers are received during OPTIONS requests
-        if (filter_input(INPUT_SERVER,'REQUEST_METHOD') == 'OPTIONS') {
+        if (filter_input(INPUT_SERVER, 'REQUEST_METHOD') == 'OPTIONS') {
             static::clean();
 
-            if (filter_input(INPUT_SERVER,'HTTP_ACCESS_CONTROL_REQUEST_METHOD')) {
+            if (filter_input(INPUT_SERVER, 'HTTP_ACCESS_CONTROL_REQUEST_METHOD')) {
               static::header('Access-Control-Allow-Methods',
                 'GET, POST, PUT, DELETE, OPTIONS, HEAD, CONNECT, PATCH, TRACE');
             }
-            if ($req_h = filter_input(INPUT_SERVER,'HTTP_ACCESS_CONTROL_REQUEST_HEADERS')) {
-              static::header('Access-Control-Allow-Headers',$req_h);
+            if ($req_h = filter_input(INPUT_SERVER, 'HTTP_ACCESS_CONTROL_REQUEST_HEADERS')) {
+              static::header('Access-Control-Allow-Headers', $req_h);
             }
 
             static::send();
@@ -93,8 +93,8 @@ 
      * Finish the output buffer capturing.
      * @return string The captured buffer
      */
-    public static function end(){
-        if (static::$buffer){
+    public static function end() {
+        if (static::$buffer) {
             static::$payload[] = ob_get_contents();
             ob_end_clean();
             static::$buffer = null;
@@ -106,14 +106,14 @@ 
      * Check if an response output buffering is active.
      * @return boolean
      */
-    public static function isBuffering(){
+    public static function isBuffering() {
         return static::$buffer;
     }
 
     /**
      * Clear the response body
      */
-    public static function clean(){
+    public static function clean() {
         static::$payload = [];
     }
 
@@ -121,76 +121,76 @@ 
      * Append a JSON object to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function json($payload){
+    public static function json($payload) {
         static::type(static::TYPE_JSON);
-        static::$payload[] = json_encode($payload, Options::get('core.response.json_flags',JSON_NUMERIC_CHECK));
+        static::$payload[] = json_encode($payload, Options::get('core.response.json_flags', JSON_NUMERIC_CHECK));
     }
 
     /**
      * Append a text to the buffer.
      * @param  mixed $payload Text to append to the response buffer
      */
-    public static function text(){
+    public static function text() {
         static::type(static::TYPE_TEXT);
-        static::$payload[] = implode('',func_get_args());
+        static::$payload[] = implode('', func_get_args());
     }
 
     /**
      * Append an XML string to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function xml(){
+    public static function xml() {
         static::type(static::TYPE_XML);
-        static::$payload[] = implode('',func_get_args());
+        static::$payload[] = implode('', func_get_args());
     }
 
     /**
      * Append a SVG string to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function svg(){
+    public static function svg() {
         static::type(static::TYPE_SVG);
-        static::$payload[] = implode('',func_get_args());
+        static::$payload[] = implode('', func_get_args());
     }
 
     /**
      * Append an HTML string to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function html(){
+    public static function html() {
         static::type(static::TYPE_HTML);
-        static::$payload[] = implode('',func_get_args());
+        static::$payload[] = implode('', func_get_args());
     }
 
     /**
      * Append a raw string to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function add(){
-        static::$payload[] = implode('',func_get_args());
+    public static function add() {
+        static::$payload[] = implode('', func_get_args());
     }
 
-    public static function status($code,$message=''){
-        static::header('Status',$message?:$code,$code);
+    public static function status($code, $message = '') {
+        static::header('Status', $message ?: $code, $code);
     }
 
-    public static function header($name,$value,$code=null){
-        static::$headers[$name] = [$value,$code];
+    public static function header($name, $value, $code = null) {
+        static::$headers[$name] = [$value, $code];
     }
 
-    public static function error($code=500,$message='Application Error'){
-        Event::trigger('core.response.error',$code,$message);
-        static::status($code,$message);
+    public static function error($code = 500, $message = 'Application Error') {
+        Event::trigger('core.response.error', $code, $message);
+        static::status($code, $message);
     }
 
-    public static function body($setBody=null){
+    public static function body($setBody = null) {
       if ($setBody) static::$payload = [$setBody];
       return Filter::with('core.response.body',
-                is_array(static::$payload) ? implode('',static::$payload) : static::$payload
+                is_array(static::$payload) ? implode('', static::$payload) : static::$payload
              );
     }
 
-    public static function headers($setHeaders=null){
+    public static function headers($setHeaders = null) {
        if ($setHeaders) static::$headers = $setHeaders;
        return static::$headers;
     }
@@ -202,7 +202,7 @@ 
      *
      * @return array Headers and body of the response
      */
-    public static function save(){
+    public static function save() {
         return [
           'head' => static::$headers,
           'body' => static::body(),
@@ -216,13 +216,13 @@ 
      *
      * @param  array $data head/body saved state
      */
-    public static function load($data){
-      $data = (object)$data;
+    public static function load($data) {
+      $data = (object) $data;
       if (isset($data->head)) static::headers($data->head);
       if (isset($data->body)) static::body($data->body);
     }
 
-    public static function send($force = false){
+    public static function send($force = false) {
       if (!static::$sent || $force) {
         static::$sent = true;
         Event::trigger('core.response.send');
@@ -235,8 +235,8 @@ 
                 $code  = null;
             }
 
-            if ($value == 'Status'){
-              if (function_exists('http_response_code')){
+            if ($value == 'Status') {
+              if (function_exists('http_response_code')) {
                 http_response_code($code);
               } else {
                 header("Status: $code", true, $code);