caffeina-core / core

classes/CLI.php

Doc Comments +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
     /**
      * Dispatch the router
      * @param  string[] $args The arguments array.
-     * @return boolean  True if route was correctly dispatched.
+     * @return null|boolean  True if route was correctly dispatched.
      */
     public static function run($args=null){
       if($args) {

Indentation +43 added lines, -43 removed lines

@@ -102,15 +102,15 @@ 
      * @return array   The commands and their description.
      */
     public static function commands(){
-       $results = [];
-       foreach(static::$commands as $name => $cmd){
+        $results = [];
+        foreach(static::$commands as $name => $cmd){
           $results[] = [
             'name'        => $name,
             'params'      => preg_replace('/:(\w+)/','[$1]',implode(' ',$cmd[0])),
             'description' => $cmd[2],
           ];
-       }
-       return $results;
+        }
+        return $results;
     }
 
     /**
@@ -122,7 +122,7 @@ 
       if($args) {
         $_args = $args;
         static::$file = basename(isset($_SERVER['PHP_SELF'])?$_SERVER['PHP_SELF']:__FILE__);
-       } else {
+        } else {
         $_args = $_SERVER['argv'];
         static::$file = basename(array_shift($_args));
       }
@@ -143,7 +143,7 @@ 
           if ($segment[0]==':'){
             // Extract paramenter
             if (isset(static::$arguments[$_idx])){
-               $pars_vector[] = static::$arguments[$_idx];
+                $pars_vector[] = static::$arguments[$_idx];
             } else return static::triggerError("Command [".$command."] needs more parameters");
           } else {
             // Match command
@@ -159,15 +159,15 @@ 
       }
     }
 
-   /**
-    * Prints a message to the console with color formatting.
-    * @param  string $message The html-like encoded message
-    * @return void
-    */
-   public static function write($message){
+    /**
+     * Prints a message to the console with color formatting.
+     * @param  string $message The html-like encoded message
+     * @return void
+     */
+    public static function write($message){
       if( preg_match('~<[^>]+>~',$message)) {
-         // Use preg_replace_callback for fast regex matches navigation
-         preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm',function($m){
+          // Use preg_replace_callback for fast regex matches navigation
+          preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm',function($m){
             static::write($m[1]);
             $color = strtoupper(trim($m[2]));
             if( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
@@ -177,37 +177,37 @@ 
             $back_color = array_pop(static::$color_stack) ?: static::$color_stack[]='NORMAL';
             if( isset(static::$shell_colors[$back_color]) ) echo static::$shell_colors[$back_color];
             static::write($m[4]);
-         },$message);
+          },$message);
       } else {
-         echo $message;
+          echo $message;
       }
-   }
+    }
    
-   /**
-    * Like CLI::write, but appends a newline at the end.
-    * @param  string $message The html-like encoded message
-    * @return void
-    */
-   public static function writeln($message){
-       static::write($message . PHP_EOL);
-   }
-
-   /**
-    * Set output ANSI color
-    * @param string $color The color name constant.
-    * @return void
-    */   
-   public static function color($color){
-       if ( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
-   }
-
-   /**
-    * Edit a temporary block of text with $EDITOR (or nano as fallback)
-    * @param string $text The initial text of the document.
-    * @param string $filename The (fake) filename passed to the editor (for syntax highlighting hint).
-    * @return string The edited contents
-    */
-   public static function edit($text,$filename=''){
+    /**
+     * Like CLI::write, but appends a newline at the end.
+     * @param  string $message The html-like encoded message
+     * @return void
+     */
+    public static function writeln($message){
+        static::write($message . PHP_EOL);
+    }
+
+    /**
+     * Set output ANSI color
+     * @param string $color The color name constant.
+     * @return void
+     */   
+    public static function color($color){
+        if ( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
+    }
+
+    /**
+     * Edit a temporary block of text with $EDITOR (or nano as fallback)
+     * @param string $text The initial text of the document.
+     * @param string $filename The (fake) filename passed to the editor (for syntax highlighting hint).
+     * @return string The edited contents
+     */
+    public static function edit($text,$filename=''){
       $EDITOR = getenv('EDITOR')?:'nano';
       $tmp = tempnam(sys_get_temp_dir(), "E-").strtr($filename,'/','_');
       file_put_contents($tmp, $text);
@@ -224,7 +224,7 @@ 
 // Standard Help Message
 CLI::help(function(){
   echo 'Usage: ', CLI::name(),' [commands]', PHP_EOL,
-       'Commands:',PHP_EOL;
+        'Commands:',PHP_EOL;
   foreach( CLI::commands() as $cmd ){
     echo "\t", $cmd['name'], ' ' ,$cmd['params'], PHP_EOL;    
     if($cmd['description'])

Spacing +58 added lines, -58 removed lines

@@ -21,18 +21,18 @@ 
     protected static $error        = null;
     
     protected static $shell_colors = [
-      'BLACK'     =>"\033[0;30m",      'DARKGRAY'     =>"\033[1;30m",
-      'BLUE'      =>"\033[0;34m",      'LIGHTBLUE'    =>"\033[1;34m",
-      'GREEN'     =>"\033[0;32m",      'LIGHTGREEN'   =>"\033[1;32m",
-      'CYAN'      =>"\033[0;36m",      'LIGHTCYAN'    =>"\033[1;36m",
-      'RED'       =>"\033[0;31m",      'LIGHTRED'     =>"\033[1;31m",
-      'PURPLE'    =>"\033[0;35m",      'LIGHTPURPLE'  =>"\033[1;35m",
-      'BROWN'     =>"\033[0;33m",      'YELLOW'       =>"\033[1;33m",
-      'LIGHTGRAY' =>"\033[0;37m",      'WHITE'        =>"\033[1;37m",
-      'NORMAL'    =>"\033[0;37m",      'B'            =>"\033[1m",
-      'ERROR'     =>"\033[1;31m",      'INFO'         =>"\033[0;36m",
-      'I'         =>"\033[0;30;104m",  'IB'           =>"\033[1;30;104m",
-      'U'         =>"\033[4m",         'D'            =>"\033[2m",
+      'BLACK'     =>"\033[0;30m", 'DARKGRAY'     =>"\033[1;30m",
+      'BLUE'      =>"\033[0;34m", 'LIGHTBLUE'    =>"\033[1;34m",
+      'GREEN'     =>"\033[0;32m", 'LIGHTGREEN'   =>"\033[1;32m",
+      'CYAN'      =>"\033[0;36m", 'LIGHTCYAN'    =>"\033[1;36m",
+      'RED'       =>"\033[0;31m", 'LIGHTRED'     =>"\033[1;31m",
+      'PURPLE'    =>"\033[0;35m", 'LIGHTPURPLE'  =>"\033[1;35m",
+      'BROWN'     =>"\033[0;33m", 'YELLOW'       =>"\033[1;33m",
+      'LIGHTGRAY' =>"\033[0;37m", 'WHITE'        =>"\033[1;37m",
+      'NORMAL'    =>"\033[0;37m", 'B'            =>"\033[1m",
+      'ERROR'     =>"\033[1;31m", 'INFO'         =>"\033[0;36m",
+      'I'         =>"\033[0;30;104m", 'IB'           =>"\033[1;30;104m",
+      'U'         =>"\033[4m", 'D'            =>"\033[2m",
     ];
     protected static $color_stack = ['NORMAL'];
     
@@ -42,16 +42,16 @@ 
      * @param  string   $command  The command route, use ":" before a parameter for extraction. 
      * @param  callable $callback The callback to be binded to the route.
      */
-    public static function on($command,callable $callback,$description=''){
-      $parts = preg_split('/\s+/',$command);
-      static::$commands[array_shift($parts)] = [$parts,$callback,$description];
+    public static function on($command, callable $callback, $description = '') {
+      $parts = preg_split('/\s+/', $command);
+      static::$commands[array_shift($parts)] = [$parts, $callback, $description];
     }
 
     /**
      * Bind a callback to the "help" route.
      * @param  callable $callback The callback to be binded to the route. If omitted triggers the callback.
      */
-    public static function help(callable $callback = null){
+    public static function help(callable $callback = null) {
         $callback
           ? is_callable($callback) && static::$help = $callback
           : static::$help && call_user_func(static::$help);
@@ -61,7 +61,7 @@ 
      * Bind a callback when an error occurs.
      * @param  callable $callback The callback to be binded to the route. If omitted triggers the callback.
      */
-    public static function error(callable $callback = null){
+    public static function error(callable $callback = null) {
         $callback
           ? is_callable($callback) && static::$error = $callback
           : static::$error && call_user_func(static::$error);
@@ -71,7 +71,7 @@ 
      * Returns the script name.
      * @return string
      */
-    public static function name(){
+    public static function name() {
         return static::$file;
     }
 
@@ -79,8 +79,8 @@ 
      * Triggers an error and exit
      * @param string $message 
      */
-    protected static function triggerError($message){
-        is_callable(static::$error) && call_user_func(static::$error,$message);
+    protected static function triggerError($message) {
+        is_callable(static::$error) && call_user_func(static::$error, $message);
         exit -1;
     }
 
@@ -90,8 +90,8 @@ 
      * @param mixed $default The default value if parameter is omitted. (if a callable it will be evaluated) 
      * @return mixed
      */
-    public static function input($key=null,$default=null){
-      return $key ? (isset(static::$options[$key]) ? static::$options[$key] : (is_callable($default)?call_user_func($default):$default)) : static::$options;
+    public static function input($key = null, $default = null) {
+      return $key ? (isset(static::$options[$key]) ? static::$options[$key] : (is_callable($default) ? call_user_func($default) : $default)) : static::$options;
     }
 
     /**
@@ -101,12 +101,12 @@ 
      *
      * @return array   The commands and their description.
      */
-    public static function commands(){
+    public static function commands() {
        $results = [];
-       foreach(static::$commands as $name => $cmd){
+       foreach (static::$commands as $name => $cmd) {
           $results[] = [
             'name'        => $name,
-            'params'      => preg_replace('/:(\w+)/','[$1]',implode(' ',$cmd[0])),
+            'params'      => preg_replace('/:(\w+)/', '[$1]', implode(' ', $cmd[0])),
             'description' => $cmd[2],
           ];
        }
@@ -118,40 +118,40 @@ 
      * @param  string[] $args The arguments array.
      * @return boolean  True if route was correctly dispatched.
      */
-    public static function run($args=null){
-      if($args) {
+    public static function run($args = null) {
+      if ($args) {
         $_args = $args;
-        static::$file = basename(isset($_SERVER['PHP_SELF'])?$_SERVER['PHP_SELF']:__FILE__);
+        static::$file = basename(isset($_SERVER['PHP_SELF']) ? $_SERVER['PHP_SELF'] : __FILE__);
        } else {
         $_args = $_SERVER['argv'];
         static::$file = basename(array_shift($_args));
       }
-      foreach($_args as $e) if(strpos($e,'-')===0) {
-        $h = explode('=',$e);
-        static::$options[ltrim(current($h),'-')] = isset($h[1])?$h[1]:true; 
+      foreach ($_args as $e) if (strpos($e, '-') === 0) {
+        $h = explode('=', $e);
+        static::$options[ltrim(current($h), '-')] = isset($h[1]) ? $h[1] : true; 
       } else {
         static::$arguments[] = $e;
       }
       
-      if(isset(static::$arguments[0])){
+      if (isset(static::$arguments[0])) {
         $command = array_shift(static::$arguments);
         if (empty(static::$commands[$command])) 
           return static::triggerError("Unknown command [".$command."].");
         $cmd = static::$commands[$command];
         $pars_vector = [];
         foreach ($cmd[0] as $_idx => $segment) {
-          if ($segment[0]==':'){
+          if ($segment[0] == ':') {
             // Extract paramenter
-            if (isset(static::$arguments[$_idx])){
+            if (isset(static::$arguments[$_idx])) {
                $pars_vector[] = static::$arguments[$_idx];
             } else return static::triggerError("Command [".$command."] needs more parameters");
           } else {
             // Match command
-            if (empty(static::$arguments[$_idx]) || $segment!=static::$arguments[$_idx]) 
+            if (empty(static::$arguments[$_idx]) || $segment != static::$arguments[$_idx]) 
               return static::triggerError("Command [".$command."] is incomplete.");
           }
         }
-        call_user_func_array($cmd[1],$pars_vector);
+        call_user_func_array($cmd[1], $pars_vector);
         return true;
       } else {
         static::help();
@@ -164,18 +164,18 @@ 
     * @param  string $message The html-like encoded message
     * @return void
     */
-   public static function write($message){
-      if( preg_match('~<[^>]+>~',$message)) {
+   public static function write($message) {
+      if (preg_match('~<[^>]+>~', $message)) {
          // Use preg_replace_callback for fast regex matches navigation
-         preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm',function($m){
+         preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm', function($m) {
             static::write($m[1]);
             $color = strtoupper(trim($m[2]));
-            if( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
+            if (isset(static::$shell_colors[$color])) echo static::$shell_colors[$color];
             static::$color_stack[] = $color;
             static::write($m[3]);
             array_pop(static::$color_stack);
-            $back_color = array_pop(static::$color_stack) ?: static::$color_stack[]='NORMAL';
-            if( isset(static::$shell_colors[$back_color]) ) echo static::$shell_colors[$back_color];
+            $back_color = array_pop(static::$color_stack) ?: static::$color_stack[] = 'NORMAL';
+            if (isset(static::$shell_colors[$back_color])) echo static::$shell_colors[$back_color];
             static::write($m[4]);
          },$message);
       } else {
@@ -188,8 +188,8 @@ 
     * @param  string $message The html-like encoded message
     * @return void
     */
-   public static function writeln($message){
-       static::write($message . PHP_EOL);
+   public static function writeln($message) {
+       static::write($message.PHP_EOL);
    }
 
    /**
@@ -197,8 +197,8 @@ 
     * @param string $color The color name constant.
     * @return void
     */   
-   public static function color($color){
-       if ( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
+   public static function color($color) {
+       if (isset(static::$shell_colors[$color])) echo static::$shell_colors[$color];
    }
 
    /**
@@ -207,9 +207,9 @@ 
     * @param string $filename The (fake) filename passed to the editor (for syntax highlighting hint).
     * @return string The edited contents
     */
-   public static function edit($text,$filename=''){
-      $EDITOR = getenv('EDITOR')?:'nano';
-      $tmp = tempnam(sys_get_temp_dir(), "E-").strtr($filename,'/','_');
+   public static function edit($text, $filename = '') {
+      $EDITOR = getenv('EDITOR') ?: 'nano';
+      $tmp = tempnam(sys_get_temp_dir(), "E-").strtr($filename, '/', '_');
       file_put_contents($tmp, $text);
       passthru("$EDITOR $tmp");
       $result = file_get_contents($tmp);
@@ -222,17 +222,17 @@ 
 
 
 // Standard Help Message
-CLI::help(function(){
-  echo 'Usage: ', CLI::name(),' [commands]', PHP_EOL,
-       'Commands:',PHP_EOL;
-  foreach( CLI::commands() as $cmd ){
-    echo "\t", $cmd['name'], ' ' ,$cmd['params'], PHP_EOL;    
-    if($cmd['description'])
-      echo "\t\t- ", str_replace("\n","\n\t\t  ",$cmd['description']), PHP_EOL, PHP_EOL;    
+CLI::help(function() {
+  echo 'Usage: ', CLI::name(), ' [commands]', PHP_EOL,
+       'Commands:', PHP_EOL;
+  foreach (CLI::commands() as $cmd) {
+    echo "\t", $cmd['name'], ' ', $cmd['params'], PHP_EOL;    
+    if ($cmd['description'])
+      echo "\t\t- ", str_replace("\n", "\n\t\t  ", $cmd['description']), PHP_EOL, PHP_EOL;    
   }
 });
 
 // Standard Error Message
-CLI::error(function($message){
-  echo 'Error: ',$message,PHP_EOL;
+CLI::error(function($message) {
+  echo 'Error: ', $message, PHP_EOL;
 });

Braces +41 added lines, -30 removed lines

@@ -42,7 +42,7 @@ 
      * @param  string   $command  The command route, use ":" before a parameter for extraction. 
      * @param  callable $callback The callback to be binded to the route.
      */
-    public static function on($command,callable $callback,$description=''){
+    public static function on($command,callable $callback,$description='') {
       $parts = preg_split('/\s+/',$command);
       static::$commands[array_shift($parts)] = [$parts,$callback,$description];
     }
@@ -51,7 +51,7 @@ 
      * Bind a callback to the "help" route.
      * @param  callable $callback The callback to be binded to the route. If omitted triggers the callback.
      */
-    public static function help(callable $callback = null){
+    public static function help(callable $callback = null) {
         $callback
           ? is_callable($callback) && static::$help = $callback
           : static::$help && call_user_func(static::$help);
@@ -61,7 +61,7 @@ 
      * Bind a callback when an error occurs.
      * @param  callable $callback The callback to be binded to the route. If omitted triggers the callback.
      */
-    public static function error(callable $callback = null){
+    public static function error(callable $callback = null) {
         $callback
           ? is_callable($callback) && static::$error = $callback
           : static::$error && call_user_func(static::$error);
@@ -71,7 +71,7 @@ 
      * Returns the script name.
      * @return string
      */
-    public static function name(){
+    public static function name() {
         return static::$file;
     }
 
@@ -79,7 +79,7 @@ 
      * Triggers an error and exit
      * @param string $message 
      */
-    protected static function triggerError($message){
+    protected static function triggerError($message) {
         is_callable(static::$error) && call_user_func(static::$error,$message);
         exit -1;
     }
@@ -90,7 +90,7 @@ 
      * @param mixed $default The default value if parameter is omitted. (if a callable it will be evaluated) 
      * @return mixed
      */
-    public static function input($key=null,$default=null){
+    public static function input($key=null,$default=null) {
       return $key ? (isset(static::$options[$key]) ? static::$options[$key] : (is_callable($default)?call_user_func($default):$default)) : static::$options;
     }
 
@@ -101,9 +101,9 @@ 
      *
      * @return array   The commands and their description.
      */
-    public static function commands(){
+    public static function commands() {
        $results = [];
-       foreach(static::$commands as $name => $cmd){
+       foreach(static::$commands as $name => $cmd) {
           $results[] = [
             'name'        => $name,
             'params'      => preg_replace('/:(\w+)/','[$1]',implode(' ',$cmd[0])),
@@ -118,7 +118,7 @@ 
      * @param  string[] $args The arguments array.
      * @return boolean  True if route was correctly dispatched.
      */
-    public static function run($args=null){
+    public static function run($args=null) {
       if($args) {
         $_args = $args;
         static::$file = basename(isset($_SERVER['PHP_SELF'])?$_SERVER['PHP_SELF']:__FILE__);
@@ -133,22 +133,26 @@ 
         static::$arguments[] = $e;
       }
       
-      if(isset(static::$arguments[0])){
+      if(isset(static::$arguments[0])) {
         $command = array_shift(static::$arguments);
-        if (empty(static::$commands[$command])) 
-          return static::triggerError("Unknown command [".$command."].");
+        if (empty(static::$commands[$command])) {
+                  return static::triggerError("Unknown command [".$command."].");
+        }
         $cmd = static::$commands[$command];
         $pars_vector = [];
         foreach ($cmd[0] as $_idx => $segment) {
-          if ($segment[0]==':'){
+          if ($segment[0]==':') {
             // Extract paramenter
-            if (isset(static::$arguments[$_idx])){
+            if (isset(static::$arguments[$_idx])) {
                $pars_vector[] = static::$arguments[$_idx];
-            } else return static::triggerError("Command [".$command."] needs more parameters");
+            } else {
+              return static::triggerError("Command [".$command."] needs more parameters");
+            }
           } else {
             // Match command
-            if (empty(static::$arguments[$_idx]) || $segment!=static::$arguments[$_idx]) 
-              return static::triggerError("Command [".$command."] is incomplete.");
+            if (empty(static::$arguments[$_idx]) || $segment!=static::$arguments[$_idx]) {
+                          return static::triggerError("Command [".$command."] is incomplete.");
+            }
           }
         }
         call_user_func_array($cmd[1],$pars_vector);
@@ -164,18 +168,22 @@ 
     * @param  string $message The html-like encoded message
     * @return void
     */
-   public static function write($message){
+   public static function write($message) {
       if( preg_match('~<[^>]+>~',$message)) {
          // Use preg_replace_callback for fast regex matches navigation
-         preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm',function($m){
+         preg_replace_callback('~^(.*)<([^>]+)>(.+)</\2>(.*)$~USm',function($m) {
             static::write($m[1]);
             $color = strtoupper(trim($m[2]));
-            if( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
+            if( isset(static::$shell_colors[$color]) ) {
+              echo static::$shell_colors[$color];
+            }
             static::$color_stack[] = $color;
             static::write($m[3]);
             array_pop(static::$color_stack);
             $back_color = array_pop(static::$color_stack) ?: static::$color_stack[]='NORMAL';
-            if( isset(static::$shell_colors[$back_color]) ) echo static::$shell_colors[$back_color];
+            if( isset(static::$shell_colors[$back_color]) ) {
+              echo static::$shell_colors[$back_color];
+            }
             static::write($m[4]);
          },$message);
       } else {
@@ -188,7 +196,7 @@ 
     * @param  string $message The html-like encoded message
     * @return void
     */
-   public static function writeln($message){
+   public static function writeln($message) {
        static::write($message . PHP_EOL);
    }
 
@@ -197,8 +205,10 @@ 
     * @param string $color The color name constant.
     * @return void
     */   
-   public static function color($color){
-       if ( isset(static::$shell_colors[$color]) ) echo static::$shell_colors[$color];
+   public static function color($color) {
+       if ( isset(static::$shell_colors[$color]) ) {
+         echo static::$shell_colors[$color];
+       }
    }
 
    /**
@@ -207,7 +217,7 @@ 
     * @param string $filename The (fake) filename passed to the editor (for syntax highlighting hint).
     * @return string The edited contents
     */
-   public static function edit($text,$filename=''){
+   public static function edit($text,$filename='') {
       $EDITOR = getenv('EDITOR')?:'nano';
       $tmp = tempnam(sys_get_temp_dir(), "E-").strtr($filename,'/','_');
       file_put_contents($tmp, $text);
@@ -222,17 +232,18 @@ 
 
 
 // Standard Help Message
-CLI::help(function(){
+CLI::help(function() {
   echo 'Usage: ', CLI::name(),' [commands]', PHP_EOL,
        'Commands:',PHP_EOL;
-  foreach( CLI::commands() as $cmd ){
+  foreach( CLI::commands() as $cmd ) {
     echo "\t", $cmd['name'], ' ' ,$cmd['params'], PHP_EOL;    
-    if($cmd['description'])
-      echo "\t\t- ", str_replace("\n","\n\t\t  ",$cmd['description']), PHP_EOL, PHP_EOL;    
+    if($cmd['description']) {
+          echo "\t\t- ", str_replace("\n","\n\t\t  ",$cmd['description']), PHP_EOL, PHP_EOL;
+    }
   }
 });
 
 // Standard Error Message
-CLI::error(function($message){
+CLI::error(function($message) {
   echo 'Error: ',$message,PHP_EOL;
 });

classes/Errors.php

Doc Comments +3 added lines

@@ -57,6 +57,9 @@
       return true;
     }
 
+    /**
+     * @param ErrorException $e
+     */
     public static function traceException($e){
       switch(self::$mode){
           case self::HTML :

Spacing +21 added lines, -21 removed lines

@@ -20,20 +20,20 @@ 
 
     static $mode = self::SILENT;
 
-    public static function capture($tracing_level=null){
-      if($tracing_level!==null) error_reporting($tracing_level);
+    public static function capture($tracing_level = null) {
+      if ($tracing_level !== null) error_reporting($tracing_level);
       set_error_handler(__CLASS__.'::traceError');
       set_exception_handler(__CLASS__.'::traceException');
     }
 
-    public static function mode($mode=null){
-      return $mode ? self::$mode=$mode : self::$mode;
+    public static function mode($mode = null) {
+      return $mode ? self::$mode = $mode : self::$mode;
     }
 
-    public static function traceError($errno,$errstr,$errfile=null,$errline=null){
+    public static function traceError($errno, $errstr, $errfile = null, $errline = null) {
       // This error code is not included in error_reporting
       if (!(error_reporting() & $errno)) return;
-      switch ( $errno ) {
+      switch ($errno) {
         case E_USER_ERROR:
             $type = 'Fatal';
         break;
@@ -51,16 +51,16 @@ 
         break;
       }
       $e = new \ErrorException($type.': '.$errstr, 0, $errno, $errfile, $errline);
-      $chk_specific = array_filter((array)Event::trigger('core.error.'.strtolower($type),$e));
-      $chk_general  = array_filter((array)Event::trigger('core.error',$e));
-      if (! ($chk_specific || $chk_general) ) static::traceException($e);
+      $chk_specific = array_filter((array) Event::trigger('core.error.'.strtolower($type), $e));
+      $chk_general  = array_filter((array) Event::trigger('core.error', $e));
+      if (!($chk_specific || $chk_general)) static::traceException($e);
       return true;
     }
 
-    public static function traceException($e){
-      switch(self::$mode){
+    public static function traceException($e) {
+      switch (self::$mode) {
           case self::HTML :
-              echo '<pre class="app error"><code>',$e->getMessage(),'</code></pre>',PHP_EOL;
+              echo '<pre class="app error"><code>', $e->getMessage(), '</code></pre>', PHP_EOL;
               break;
           case self::JSON :
               echo json_encode(['error' => $e->getMessage()]);
@@ -70,26 +70,26 @@ 
               break;
           case self::SIMPLE :
           default:
-              echo $e->getMessage(),PHP_EOL;
+              echo $e->getMessage(), PHP_EOL;
               break;
       }
       return true;
     }
 
-    public static function onFatal(callable $listener){
-      Event::on('core.error.fatal',$listener);
+    public static function onFatal(callable $listener) {
+      Event::on('core.error.fatal', $listener);
     }
 
-    public static function onWarning(callable $listener){
-      Event::on('core.error.warning',$listener);
+    public static function onWarning(callable $listener) {
+      Event::on('core.error.warning', $listener);
     }
 
-    public static function onNotice(callable $listener){
-      Event::on('core.error.notice',$listener);
+    public static function onNotice(callable $listener) {
+      Event::on('core.error.notice', $listener);
     }
 
-    public static function onAny(callable $listener){
-      Event::on('core.error',$listener);
+    public static function onAny(callable $listener) {
+      Event::on('core.error', $listener);
     }
 
 }

Braces +18 added lines, -12 removed lines

@@ -20,19 +20,23 @@ 
 
     static $mode = self::SILENT;
 
-    public static function capture($tracing_level=null){
-      if($tracing_level!==null) error_reporting($tracing_level);
+    public static function capture($tracing_level=null) {
+      if($tracing_level!==null) {
+        error_reporting($tracing_level);
+      }
       set_error_handler(__CLASS__.'::traceError');
       set_exception_handler(__CLASS__.'::traceException');
     }
 
-    public static function mode($mode=null){
+    public static function mode($mode=null) {
       return $mode ? self::$mode=$mode : self::$mode;
     }
 
-    public static function traceError($errno,$errstr,$errfile=null,$errline=null){
+    public static function traceError($errno,$errstr,$errfile=null,$errline=null) {
       // This error code is not included in error_reporting
-      if (!(error_reporting() & $errno)) return;
+      if (!(error_reporting() & $errno)) {
+        return;
+      }
       switch ( $errno ) {
         case E_USER_ERROR:
             $type = 'Fatal';
@@ -53,12 +57,14 @@ 
       $e = new \ErrorException($type.': '.$errstr, 0, $errno, $errfile, $errline);
       $chk_specific = array_filter((array)Event::trigger('core.error.'.strtolower($type),$e));
       $chk_general  = array_filter((array)Event::trigger('core.error',$e));
-      if (! ($chk_specific || $chk_general) ) static::traceException($e);
+      if (! ($chk_specific || $chk_general) ) {
+        static::traceException($e);
+      }
       return true;
     }
 
-    public static function traceException($e){
-      switch(self::$mode){
+    public static function traceException($e) {
+      switch(self::$mode) {
           case self::HTML :
               echo '<pre class="app error"><code>',$e->getMessage(),'</code></pre>',PHP_EOL;
               break;
@@ -76,19 +82,19 @@ 
       return true;
     }
 
-    public static function onFatal(callable $listener){
+    public static function onFatal(callable $listener) {
       Event::on('core.error.fatal',$listener);
     }
 
-    public static function onWarning(callable $listener){
+    public static function onWarning(callable $listener) {
       Event::on('core.error.warning',$listener);
     }
 
-    public static function onNotice(callable $listener){
+    public static function onNotice(callable $listener) {
       Event::on('core.error.notice',$listener);
     }
 
-    public static function onAny(callable $listener){
+    public static function onAny(callable $listener) {
       Event::on('core.error',$listener);
     }
 

classes/FileSystem/Native.php

Doc Comments +3 added lines

@@ -27,6 +27,9 @@
         return $this->exists($path) ? file_get_contents($this->realPath($path)) : false;
     }
 
+    /**
+     * @param string|false $data
+     */
     public function write($path, $data){
         $r_path = $this->realPath($path);
         if ( ! is_dir($r_dir = dirname($r_path)) ) @mkdir($r_dir,0775,true);

Spacing +18 added lines, -18 removed lines

@@ -16,42 +16,42 @@ 
     protected $root;
 
     public function __construct(array $options = []) {
-        $this->root = empty($options['root'])?'/':(rtrim($options['root'],'/').'/');
+        $this->root = empty($options['root']) ? '/' : (rtrim($options['root'], '/').'/');
     }
 
-    public function exists($path){
+    public function exists($path) {
         return file_exists($this->realPath($path));
     }
 
-    public function read($path){
+    public function read($path) {
         return $this->exists($path) ? file_get_contents($this->realPath($path)) : false;
     }
 
-    public function write($path, $data){
+    public function write($path, $data) {
         $r_path = $this->realPath($path);
-        if ( ! is_dir($r_dir = dirname($r_path)) ) @mkdir($r_dir,0775,true);
+        if (!is_dir($r_dir = dirname($r_path))) @mkdir($r_dir, 0775, true);
         return file_put_contents($r_path, $data);
     }
 
-    public function append($path, $data){
+    public function append($path, $data) {
         return file_put_contents($this->realPath($path), $data, FILE_APPEND);
     }
 
-    public function move($old, $new){
+    public function move($old, $new) {
         // Atomic
-        if($this->exists($old)){
-            return $this->write($new,$this->read($old)) && $this->delete($old);
+        if ($this->exists($old)) {
+            return $this->write($new, $this->read($old)) && $this->delete($old);
         } else return false;
     }
 
-    public function delete($path){
+    public function delete($path) {
         return $this->exists($path) ? unlink($this->realPath($path)) : false;
     }
 
-    public function search($pattern, $recursive=true){
+    public function search($pattern, $recursive = true) {
         $results    = [];
         $root_len   = strlen($this->root);
-        $rx_pattern = '('.strtr($pattern,['.'=>'\.','*'=>'.*','?'=>'.']).')Ai';
+        $rx_pattern = '('.strtr($pattern, ['.'=>'\.', '*'=>'.*', '?'=>'.']).')Ai';
 
         /*
         $files = new \RegexIterator(new \RecursiveDirectoryIterator($this->root,
@@ -67,12 +67,12 @@ 
                 new \RecursiveDirectoryIterator(
                     $this->root,
                     \RecursiveDirectoryIterator::SKIP_DOTS)),
-                    $rx_pattern,\RegexIterator::GET_MATCH);
+                    $rx_pattern, \RegexIterator::GET_MATCH);
 
         $fileList = [];
-        foreach($tree as $group) {
-            foreach($group as $path) {
-                $results[] = trim(substr($path, $root_len),'/');
+        foreach ($tree as $group) {
+            foreach ($group as $path) {
+                $results[] = trim(substr($path, $root_len), '/');
             }
         }
 
@@ -80,8 +80,8 @@ 
 
     }
 
-    protected function realPath($path){
-        return $this->root . $path;
+    protected function realPath($path) {
+        return $this->root.$path;
     }
 
 }

Braces +15 added lines, -11 removed lines

@@ -19,36 +19,40 @@ 
         $this->root = empty($options['root'])?'/':(rtrim($options['root'],'/').'/');
     }
 
-    public function exists($path){
+    public function exists($path) {
         return file_exists($this->realPath($path));
     }
 
-    public function read($path){
+    public function read($path) {
         return $this->exists($path) ? file_get_contents($this->realPath($path)) : false;
     }
 
-    public function write($path, $data){
+    public function write($path, $data) {
         $r_path = $this->realPath($path);
-        if ( ! is_dir($r_dir = dirname($r_path)) ) @mkdir($r_dir,0775,true);
+        if ( ! is_dir($r_dir = dirname($r_path)) ) {
+          @mkdir($r_dir,0775,true);
+        }
         return file_put_contents($r_path, $data);
     }
 
-    public function append($path, $data){
+    public function append($path, $data) {
         return file_put_contents($this->realPath($path), $data, FILE_APPEND);
     }
 
-    public function move($old, $new){
+    public function move($old, $new) {
         // Atomic
-        if($this->exists($old)){
+        if($this->exists($old)) {
             return $this->write($new,$this->read($old)) && $this->delete($old);
-        } else return false;
+        } else {
+          return false;
+        }
     }
 
-    public function delete($path){
+    public function delete($path) {
         return $this->exists($path) ? unlink($this->realPath($path)) : false;
     }
 
-    public function search($pattern, $recursive=true){
+    public function search($pattern, $recursive=true) {
         $results    = [];
         $root_len   = strlen($this->root);
         $rx_pattern = '('.strtr($pattern,['.'=>'\.','*'=>'.*','?'=>'.']).')Ai';
@@ -80,7 +84,7 @@ 
 
     }
 
-    protected function realPath($path){
+    protected function realPath($path) {
         return $this->root . $path;
     }
 

classes/Route.php

Doc Comments +1 added lines, -2 removed lines

@@ -198,7 +198,6 @@ 
 
     /**
      * Bind a middleware callback to invoked before the route definition
-     * @param  callable $before The callback to be invoked ($this is binded to the route object).
      * @return Route
      */
     public function & before($callback){
@@ -318,7 +317,7 @@ 
      * @param  string  $pattern The URL schema with the named parameters
      * @param  string  $URL The URL to process, if omitted the current request URI will be used.
      * @param  boolean $cut If true don't limit the matching to the whole URL (used for group pattern extraction)
-     * @return array The extracted variables
+     * @return callable The extracted variables
      */
     protected static function extractVariablesFromURL($pattern,$URL=null,$cut=false){
         $URL     = $URL ?: Request::URI();

Indentation +39 added lines, -39 removed lines

@@ -14,9 +14,9 @@ 
     use Module;
 
     protected static $routes,
-                     $base    = '',
-                     $prefix  = [],
-                     $group   = [];
+                      $base    = '',
+                      $prefix  = [],
+                      $group   = [];
 
     protected $URLPattern     = '',
               $pattern        = '',
@@ -85,20 +85,20 @@ 
         $method = strtolower($method);
         $this->response 			 		= '';
         $this->response_object 		= null;
-       	$this->response_is_object = false;
+          $this->response_is_object = false;
 
         // Call direct befores
         if ( $this->befores ) {
           // Reverse befores order
           foreach (array_reverse($this->befores) as $mw) {
-	        	ob_start();
+            ob_start();
             $mw_result = call_user_func($mw->bindTo($this));
-          	$this->response .= ob_get_clean();
+            $this->response .= ob_get_clean();
             if ( false  === $mw_result ) {
-            	return [''];
+              return [''];
             } else if (is_a($mw_result,'View') || is_string($mw_result)) {
               $this->response .= (string)$mw_result;
-          	}
+            }
           }
         }
 
@@ -109,51 +109,51 @@ 
 
         if (is_callable($callback)) {
           // Capure callback output
-					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);
-	        $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;
-	          } else {
-			        	$this->response_is_object = true;
-	              $this->response_object 		= $view_results;
-	          }
-	        }
+          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);
+          $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;
+            } else {
+                $this->response_is_object = true;
+                $this->response_object 		= $view_results;
+            }
+          }
 
         } 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_is_object = true;
+          $this->response_object 		= $callback;
         }
 
         // Apply afters
         if ( $this->afters ) {
           foreach ($this->afters as $mw) {
-	        	ob_start();
+            ob_start();
             $mw_result = call_user_func($mw->bindTo($this));
-          	$this->response .= ob_get_clean();
+            $this->response .= ob_get_clean();
             if ( false  === $mw_result ) {
-            	return [''];
+              return [''];
             } 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 ){
-					$this->response = Response::json($this->response_object);
+          $this->response = Response::json($this->response_object);
         } else {
-					Response::add($this->response);
+          Response::add($this->response);
         }
 
         return [$this->response];
@@ -281,10 +281,10 @@ 
         $route           = new static($URLPattern);
         $route->callback = [];
         foreach ($callbacks as $method => $callback) {
-           $method = strtolower($method);
-           if (Request::method() !== $method) continue;
-           $route->callback[$method] = $callback;
-           $route->methods[$method]  = 1;
+            $method = strtolower($method);
+            if (Request::method() !== $method) continue;
+            $route->callback[$method] = $callback;
+            $route->methods[$method]  = 1;
         }
         return $route;
     }
@@ -386,9 +386,9 @@ 
     }
 
     public static function exitWithError($code,$message="Application Error"){
-    	Response::error($code,$message);
-    	Response::send();
-    	exit;
+      Response::error($code,$message);
+      Response::send();
+      exit;
     }
 
     /**

Spacing +73 added lines, -73 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);
@@ -165,8 +165,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;
     }
 
     /**
@@ -175,8 +175,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);
     }
 
     /**
@@ -185,8 +185,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('*');
     }
 
     /**
@@ -231,7 +231,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;
@@ -253,10 +253,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;
     }
 
@@ -294,10 +294,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)).'$#';
     }
 
     /**
@@ -307,10 +307,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]);
         }
@@ -322,8 +322,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, ':(?[*+');
     }
 
     /**
@@ -331,9 +331,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;
     }
 
     /**
@@ -341,16 +341,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;
@@ -358,14 +358,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;
@@ -375,7 +375,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 {
 
@@ -385,8 +385,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;
     }
@@ -396,13 +396,13 @@ 
      * @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();
-        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;
                 }
             }
@@ -416,34 +416,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 +78 added lines, -46 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);
@@ -165,7 +169,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;
     }
 
@@ -175,7 +179,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);
     }
 
@@ -185,7 +189,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('*');
     }
 
@@ -231,7 +235,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;
@@ -253,7 +257,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);
@@ -282,7 +286,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;
         }
@@ -294,8 +300,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)).'$#';
     }
@@ -307,12 +313,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;
     }
@@ -322,7 +332,7 @@ 
      * @param  string  $pattern The URL schema.
      * @return boolean
      */
-    protected static function isDynamic($pattern){
+    protected static function isDynamic($pattern) {
       return strlen($pattern) != strcspn($pattern,':(?[*+');
     }
 
@@ -331,8 +341,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;
     }
 
@@ -341,41 +353,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 {
 
@@ -385,7 +411,7 @@ 
 
     }
 
-    public static function exitWithError($code,$message="Application Error"){
+    public static function exitWithError($code,$message="Application Error") {
     	Response::error($code,$message);
     	Response::send();
     	exit;
@@ -396,12 +422,16 @@ 
      * @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();
-        foreach ((array)static::$routes as $group => $routes){
+    public static function dispatch($URL=null,$method=null) {
+        if (!$URL) {
+          $URL     = Request::URI();
+        }
+        if (!$method) {
+          $method  = Request::method();
+        }
+        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;
                 }
@@ -416,34 +446,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/View.php

Doc Comments +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
     /**
      * Load a Template Handler
-     * @param  class $handler The template handler class instance
+     * @param  View\Adapter $handler The template handler class instance
      */
     public static function using(View\Adapter &$handler){
       static::$handler = $handler;
@@ -69,7 +69,7 @@ 
 
     /**
      * Returns the handler instance
-     * @return mixed
+     * @return View\Adapter
      */
     public static function & handler(){
       return static::$handler;

Spacing +13 added lines, -13 removed lines

@@ -23,7 +23,7 @@ 
      * Construct a new view based on the passed template
      * @param  string $template The template path
      */
-    public function __construct($template){
+    public function __construct($template) {
       $this->options['template'] = $template;
     }
 
@@ -31,7 +31,7 @@ 
      * Load a Template Handler
      * @param  class $handler The template handler class instance
      */
-    public static function using(View\Adapter &$handler){
+    public static function using(View\Adapter & $handler) {
       static::$handler = $handler;
     }
 
@@ -41,7 +41,7 @@ 
      * @param  array $data     The key-value map of data to pass to the view
      * @return View
      */
-    public static function from($template,$data=null){
+    public static function from($template, $data = null) {
       $view = new self($template);
       return $data ? $view->with($data) : $view;
     }
@@ -51,9 +51,9 @@ 
      * @param  array $data     The key-value map of data to pass to the view
      * @return View
      */
-    public function with($data){
-      if ($data){
-        $tmp = array_merge($data, (isset($this->options['data'])?$this->options['data']:[]));
+    public function with($data) {
+      if ($data) {
+        $tmp = array_merge($data, (isset($this->options['data']) ? $this->options['data'] : []));
         $this->options['data'] = $tmp;
       }
       return $this;
@@ -63,8 +63,8 @@ 
      * Render view when casted to a string
      * @return string The rendered view
      */
-    public function __toString(){
-      return Filter::with('core.view',static::$handler->render($this->options['template'],$this->options['data']));
+    public function __toString() {
+      return Filter::with('core.view', static::$handler->render($this->options['template'], $this->options['data']));
     }
 
     /**
@@ -79,7 +79,7 @@ 
      * Check if a template exists
      * @return bool
      */
-    public static function exists($templatePath){
+    public static function exists($templatePath) {
       return static::$handler->exists($templatePath);
     }
 
@@ -87,15 +87,15 @@ 
     /**
      * Propagate the call to the handler
      */
-    public function __call($n,$p){
-      return call_user_func_array([static::$handler,$n],$p);
+    public function __call($n, $p) {
+      return call_user_func_array([static::$handler, $n], $p);
     }
 
     /**
      * Propagate the static call to the handler
      */
-    public static function __callStatic($n,$p){
-      return forward_static_call_array([static::$handler,$n],$p);
+    public static function __callStatic($n, $p) {
+      return forward_static_call_array([static::$handler, $n], $p);
     }
 
 

Braces +9 added lines, -9 removed lines

@@ -23,7 +23,7 @@ 
      * Construct a new view based on the passed template
      * @param  string $template The template path
      */
-    public function __construct($template){
+    public function __construct($template) {
       $this->options['template'] = $template;
     }
 
@@ -31,7 +31,7 @@ 
      * Load a Template Handler
      * @param  class $handler The template handler class instance
      */
-    public static function using(View\Adapter &$handler){
+    public static function using(View\Adapter &$handler) {
       static::$handler = $handler;
     }
 
@@ -41,7 +41,7 @@ 
      * @param  array $data     The key-value map of data to pass to the view
      * @return View
      */
-    public static function from($template,$data=null){
+    public static function from($template,$data=null) {
       $view = new self($template);
       return $data ? $view->with($data) : $view;
     }
@@ -51,8 +51,8 @@ 
      * @param  array $data     The key-value map of data to pass to the view
      * @return View
      */
-    public function with($data){
-      if ($data){
+    public function with($data) {
+      if ($data) {
         $tmp = array_merge($data, (isset($this->options['data'])?$this->options['data']:[]));
         $this->options['data'] = $tmp;
       }
@@ -63,7 +63,7 @@ 
      * Render view when casted to a string
      * @return string The rendered view
      */
-    public function __toString(){
+    public function __toString() {
       return Filter::with('core.view',static::$handler->render($this->options['template'],$this->options['data']));
     }
 
@@ -79,7 +79,7 @@ 
      * Check if a template exists
      * @return bool
      */
-    public static function exists($templatePath){
+    public static function exists($templatePath) {
       return static::$handler->exists($templatePath);
     }
 
@@ -87,14 +87,14 @@ 
     /**
      * Propagate the call to the handler
      */
-    public function __call($n,$p){
+    public function __call($n,$p) {
       return call_user_func_array([static::$handler,$n],$p);
     }
 
     /**
      * Propagate the static call to the handler
      */
-    public static function __callStatic($n,$p){
+    public static function __callStatic($n,$p) {
       return forward_static_call_array([static::$handler,$n],$p);
     }
 

classes/View/PHP.php

Doc Comments +3 added lines

@@ -57,6 +57,9 @@
 class PHPContext {
     protected $data      = [];
 
+    /**
+     * @param string $path
+     */
     public function __construct($data=[], $path=null){
         $this->data = $data;
     }

Indentation +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     const EXTENSION       = 'php';
 
     protected static $templatePath,
-                     $globals = [];
+                      $globals = [];
 
     public function __construct($path=null,$options=[]){
         self::$templatePath = ($path ? rtrim($path,'/') : __DIR__) . '/';

Spacing +17 added lines, -17 removed lines

@@ -14,32 +14,32 @@ 
 
 class PHP implements Adapter {
 
-    const EXTENSION       = 'php';
+    const EXTENSION = 'php';
 
     protected static $templatePath,
                      $globals = [];
 
-    public function __construct($path=null,$options=[]){
-        self::$templatePath = ($path ? rtrim($path,'/') : __DIR__) . '/';
+    public function __construct($path = null, $options = []) {
+        self::$templatePath = ($path ? rtrim($path, '/') : __DIR__).'/';
     }
 
-    public static function exists($path){
+    public static function exists($path) {
         return is_file(self::$templatePath.$path.'.php');
     }
 
-    public static function addGlobal($key,$val){
+    public static function addGlobal($key, $val) {
       self::$globals[$key] = $val;
     }
 
-    public static function addGlobals(array $defs){
-      foreach ((array)$defs as $key=>$val) {
+    public static function addGlobals(array $defs) {
+      foreach ((array) $defs as $key=>$val) {
           self::$globals[$key] = $val;
       }
     }
 
-    public function render($template,$data=[]){
-        $template_path = self::$templatePath . trim($template,'/') . '.php';
-        $sandbox     = function() use ($template_path){
+    public function render($template, $data = []) {
+        $template_path = self::$templatePath.trim($template, '/').'.php';
+        $sandbox = function() use ($template_path){
             ob_start();
             include($template_path);
             $__buffer__ = ob_get_contents();
@@ -55,21 +55,21 @@ 
 }
 
 class PHPContext {
-    protected $data      = [];
+    protected $data = [];
 
-    public function __construct($data=[], $path=null){
+    public function __construct($data = [], $path = null) {
         $this->data = $data;
     }
 
-    public function partial($template, $vars=[]){
-        return \View::from($template,array_merge($this->data,$vars));
+    public function partial($template, $vars = []) {
+        return \View::from($template, array_merge($this->data, $vars));
     }
 
-    public function __isset($n){ return true; }
+    public function __isset($n) { return true;}
 
-    public function __unset($n){}
+    public function __unset($n) {}
 
-    public function __get($n){
+    public function __get($n) {
       return empty($this->data[$n]) ? '' : $this->data[$n];
     }
 

Braces +9 added lines, -9 removed lines

@@ -19,27 +19,27 @@ 
     protected static $templatePath,
                      $globals = [];
 
-    public function __construct($path=null,$options=[]){
+    public function __construct($path=null,$options=[]) {
         self::$templatePath = ($path ? rtrim($path,'/') : __DIR__) . '/';
     }
 
-    public static function exists($path){
+    public static function exists($path) {
         return is_file(self::$templatePath.$path.'.php');
     }
 
-    public static function addGlobal($key,$val){
+    public static function addGlobal($key,$val) {
       self::$globals[$key] = $val;
     }
 
-    public static function addGlobals(array $defs){
+    public static function addGlobals(array $defs) {
       foreach ((array)$defs as $key=>$val) {
           self::$globals[$key] = $val;
       }
     }
 
-    public function render($template,$data=[]){
+    public function render($template,$data=[]) {
         $template_path = self::$templatePath . trim($template,'/') . '.php';
-        $sandbox     = function() use ($template_path){
+        $sandbox     = function() use ($template_path) {
             ob_start();
             include($template_path);
             $__buffer__ = ob_get_contents();
@@ -57,11 +57,11 @@ 
 class PHPContext {
     protected $data      = [];
 
-    public function __construct($data=[], $path=null){
+    public function __construct($data=[], $path=null) {
         $this->data = $data;
     }
 
-    public function partial($template, $vars=[]){
+    public function partial($template, $vars=[]) {
         return \View::from($template,array_merge($this->data,$vars));
     }
 
@@ -69,7 +69,7 @@ 
 
     public function __unset($n){}
 
-    public function __get($n){
+    public function __get($n) {
       return empty($this->data[$n]) ? '' : $this->data[$n];
     }
 

classes/SQL.php

Doc Comments +5 added lines

@@ -101,6 +101,11 @@
             $queries           = [],
             $last_exec_success = true;
 
+  /**
+   * @param string $dsn
+   * @param string $username
+   * @param string $password
+   */
   public function __construct($dsn, $username=null, $password=null, $options=[]){
     $this->connection = [
       'dsn'        => $dsn,

Indentation +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 class SQL {
   use Module;
   protected static $connections = [],
-                   $current     = 'default';
+                    $current     = 'default';
 
   /**
    * Register a new datasource
@@ -208,7 +208,7 @@ 
     }
   }
 
- public function run($script){
+  public function run($script){
     if(!$this->connection()) return false;
 
     $sql_path = Options::get('database.sql.path',APP_DIR.'/sql');
@@ -223,8 +223,8 @@ 
   }
 
   public function all($query, $params=[], callable $looper = null){
-   if(!$this->connection()) return false;
-   return $this->each($query,$params,$looper);
+    if(!$this->connection()) return false;
+    return $this->each($query,$params,$looper);
   }
 
   public function delete($table, $pks=null, $pk='id', $inclusive=true){

Spacing +70 added lines, -70 removed lines

@@ -25,7 +25,7 @@ 
    * @param  array  $options  Options to pass to the PDO constructor
    * @return SQLConnection    The datasource resource
    */
-  public static function register($name, $dsn, $username=null, $password=null, $options=[]){
+  public static function register($name, $dsn, $username = null, $password = null, $options = []) {
     return self::$connections[$name] = new SQLConnection($dsn, $username, $password, $options);
   }
 
@@ -37,7 +37,7 @@ 
    * @param  array  $options  Options to pass to the PDO constructor
    * @return SQLConnection    The datasource resource
    */
-  public static function connect($dsn, $username=null, $password=null, $options=[]){
+  public static function connect($dsn, $username = null, $password = null, $options = []) {
     return self::register('default', $dsn, $username, $password, $options);
   }
 
@@ -46,8 +46,8 @@ 
    * @param  string $name The datasource name
    * @return bool       `true` if correctly changed
    */
-  public static function defaultTo($name){
-    if (isset(self::$connections[$name])){
+  public static function defaultTo($name) {
+    if (isset(self::$connections[$name])) {
       self::$current = $name;
       return true;
     } else return false;
@@ -58,11 +58,11 @@ 
    * @param  string $name The datasource name, omit for close all of them
    * @return bool       `true` if one or more datasource where closed
    */
-  public static function close($name=null){
+  public static function close($name = null) {
     if ($name === null) {
       foreach (self::$connections as $conn) $conn->close();
       return true;
-    } else if (isset(self::$connections[$name])){
+    } else if (isset(self::$connections[$name])) {
       self::$connections[$name]->close();
       return true;
     } else return false;
@@ -73,7 +73,7 @@ 
    * @param  strinf $name The datasource name
    * @return SQLConnect   The datasource connection
    */
-  public static function using($name){
+  public static function using($name) {
     if (empty(self::$connections[$name])) throw new \Exception("[SQL] Unknown connection named '$name'.");
     return self::$connections[$name];
   }
@@ -84,9 +84,9 @@ 
    * @param  array $args    The method arguments
    * @return mixed          The method return value
    */
-  public static function __callStatic($method, $args){
+  public static function __callStatic($method, $args) {
     if (empty(self::$connections[self::$current])) throw new \Exception("[SQL] No default connection defined.");
-    return call_user_func_array([self::$connections[self::$current],$method],$args);
+    return call_user_func_array([self::$connections[self::$current], $method], $args);
   }
 
 }
@@ -101,7 +101,7 @@ 
             $queries           = [],
             $last_exec_success = true;
 
-  public function __construct($dsn, $username=null, $password=null, $options=[]){
+  public function __construct($dsn, $username = null, $password = null, $options = []) {
     $this->connection = [
       'dsn'        => $dsn,
       'pdo'        => null,
@@ -111,20 +111,20 @@ 
         PDO::ATTR_ERRMODE              => PDO::ERRMODE_EXCEPTION,
         PDO::ATTR_DEFAULT_FETCH_MODE   => PDO::FETCH_ASSOC,
         PDO::ATTR_EMULATE_PREPARES     => true,
-      ],$options),
+      ], $options),
     ];
     // "The auto-commit mode cannot be changed for this driver" SQLite workaround
-    if (strpos($dsn,'sqlite:') === 0) {
+    if (strpos($dsn, 'sqlite:') === 0) {
       $this->connection['options'] = $options;
     }
   }
 
-  public function close(){
+  public function close() {
     $this->connection['pdo'] = null;
   }
 
-  public function connection(){
-    if(empty($this->connection['pdo'])) {
+  public function connection() {
+    if (empty($this->connection['pdo'])) {
       try {
         $this->connection['pdo'] = new PDO(
             $this->connection['dsn'],
@@ -133,26 +133,26 @@ 
             $this->connection['options']
 
         );
-        Event::trigger('core.sql.connect',$this);
-      } catch(Exception $e) {
+        Event::trigger('core.sql.connect', $this);
+      } catch (Exception $e) {
         $this->connection['pdo'] = null;
       }
     }
     return $this->connection['pdo'];
   }
 
-  public function prepare($query){
-    if(!$this->connection()) return false;
+  public function prepare($query) {
+    if (!$this->connection()) return false;
     return isset($this->queries[$query]) ? $this->queries[$query] : ($this->queries[$query] = $this->connection()->prepare($query));
   }
 
-  public function exec($query, $params=[]){
-    if(!$this->connection()) return false;
+  public function exec($query, $params = []) {
+    if (!$this->connection()) return false;
 
-    if (false==is_array($params)) $params = (array)$params;
-    $query = Filter::with('core.sql.query',$query);
-    if($statement = $this->prepare($query)){
-      Event::trigger('core.sql.query',$query,$params,(bool)$statement);
+    if (false == is_array($params)) $params = (array) $params;
+    $query = Filter::with('core.sql.query', $query);
+    if ($statement = $this->prepare($query)) {
+      Event::trigger('core.sql.query', $query, $params, (bool) $statement);
 
       foreach ($params as $key => $val) {
         $type = PDO::PARAM_STR;
@@ -164,10 +164,10 @@ 
           $type = PDO::PARAM_INT;
         }
         // bindValue need a 1-based numeric parameter
-        $statement->bindValue(is_numeric($key)?$key+1:':'.$key, $val, $type);
+        $statement->bindValue(is_numeric($key) ? $key + 1 : ':'.$key, $val, $type);
       }
     } else {
-      Event::trigger('core.sql.error',$query,$params);
+      Event::trigger('core.sql.error', $query, $params);
       return false;
     }
 
@@ -175,32 +175,32 @@ 
     return $statement;
   }
 
-  public function value($query, $params=[], $column=0){
-    if(!$this->connection()) return false;
+  public function value($query, $params = [], $column = 0) {
+    if (!$this->connection()) return false;
 
-    $res = $this->exec($query,$params);
+    $res = $this->exec($query, $params);
     return $res ? $res->fetchColumn($column) : null;
   }
 
-  public function each($query, $params=[], callable $looper = null){
-    if(!$this->connection()) return false;
+  public function each($query, $params = [], callable $looper = null) {
+    if (!$this->connection()) return false;
 
     // ($query,$looper) shorthand
-    if ($looper===null && is_callable($params)) {$looper = $params; $params = [];}
-    if( $res = $this->exec($query,$params) ){
-      if(is_callable($looper))
+    if ($looper === null && is_callable($params)) {$looper = $params;$params = [];}
+    if ($res = $this->exec($query, $params)) {
+      if (is_callable($looper))
         while ($row = $res->fetchObject()) $looper($row);
       else
         return $res->fetchAll(PDO::FETCH_CLASS);
     }
   }
 
-  public function single($query, $params=[], callable $handler = null){
-    if(!$this->connection()) return false;
+  public function single($query, $params = [], callable $handler = null) {
+    if (!$this->connection()) return false;
 
     // ($query,$handler) shorthand
-    if ($handler===null && is_callable($params)) {$handler = $params; $params = [];}
-    if( $res = $this->exec($query,$params) ){
+    if ($handler === null && is_callable($params)) {$handler = $params;$params = [];}
+    if ($res = $this->exec($query, $params)) {
         if (is_callable($handler))
           $handler($res->fetchObject());
         else
@@ -208,76 +208,76 @@ 
     }
   }
 
- public function run($script){
-    if(!$this->connection()) return false;
+ public function run($script) {
+    if (!$this->connection()) return false;
 
-    $sql_path = Options::get('database.sql.path',APP_DIR.'/sql');
-    $sql_sep  = Options::get('database.sql.separator',';');
-    if (is_file($f = "$sql_path/$script.sql")){
+    $sql_path = Options::get('database.sql.path', APP_DIR.'/sql');
+    $sql_sep  = Options::get('database.sql.separator', ';');
+    if (is_file($f = "$sql_path/$script.sql")) {
         $result = true;
-        foreach(explode($sql_sep,file_get_contents($f)) as $statement) {
+        foreach (explode($sql_sep, file_get_contents($f)) as $statement) {
             $result = $this->exec($statement);
         }
         return $result;
     } else return false;
   }
 
-  public function all($query, $params=[], callable $looper = null){
-   if(!$this->connection()) return false;
-   return $this->each($query,$params,$looper);
+  public function all($query, $params = [], callable $looper = null) {
+   if (!$this->connection()) return false;
+   return $this->each($query, $params, $looper);
   }
 
-  public function delete($table, $pks=null, $pk='id', $inclusive=true){
-    if(!$this->connection()) return false;
+  public function delete($table, $pks = null, $pk = 'id', $inclusive = true) {
+    if (!$this->connection()) return false;
 
-    if (null===$pks) {
+    if (null === $pks) {
       return $this->exec("DELETE FROM `$table`");
     } else {
-      return $this->exec("DELETE FROM `$table` WHERE `$pk` ".($inclusive ? "" : "NOT " )."IN (" . implode( ',', array_fill_keys( (array)$pks, '?' ) ) . ")",(array)$pks);
+      return $this->exec("DELETE FROM `$table` WHERE `$pk` ".($inclusive ? "" : "NOT ")."IN (".implode(',', array_fill_keys((array) $pks, '?')).")", (array) $pks);
     }
   }
 
-  public function insert($table, $data=[]){
-    if(!$this->connection()) return false;
+  public function insert($table, $data = []) {
+    if (!$this->connection()) return false;
 
-    if (false==is_array($data)) $data = (array)$data;
+    if (false == is_array($data)) $data = (array) $data;
     $k = array_keys($data);
     asort($k);
     $pk = $k;
-    array_walk($pk,function(&$e){ $e = ':'.$e;});
-    $q = "INSERT INTO `$table` (`".implode('`,`',$k)."`) VALUES (".implode(',',$pk).")";
-    $this->exec($q,$data);
+    array_walk($pk, function(&$e) { $e = ':'.$e;});
+    $q = "INSERT INTO `$table` (`".implode('`,`', $k)."`) VALUES (".implode(',', $pk).")";
+    $this->exec($q, $data);
     return $this->last_exec_success ? $this->connection()->lastInsertId() : false;
   }
 
-  public function updateWhere($table, $data=[], $where, $pk='id'){
-    if(!$this->connection()) return false;
+  public function updateWhere($table, $data = [], $where, $pk = 'id') {
+    if (!$this->connection()) return false;
 
-    if (false==is_array($data)) $data = (array)$data;
+    if (false == is_array($data)) $data = (array) $data;
     if (empty($data)) return false;
     $k = array_keys($data);
     asort($k);
 
     // Remove primary key from SET
-    array_walk($k,function(&$e) use ($pk) {
-      $e = ($e==$pk) ? null : "`$e`=:$e";
+    array_walk($k, function(&$e) use ($pk) {
+      $e = ($e == $pk) ? null : "`$e`=:$e";
     });
 
-    $q = "UPDATE `$table` SET ".implode(', ',array_filter($k))." WHERE $where";
-    $this->exec($q,$data);
+    $q = "UPDATE `$table` SET ".implode(', ', array_filter($k))." WHERE $where";
+    $this->exec($q, $data);
     return $this->last_exec_success;
   }
 
-  public function update($table, $data=[], $pk='id', $extra_where=''){
+  public function update($table, $data = [], $pk = 'id', $extra_where = '') {
     return $this->updateWhere($table, $data, "`$pk`=:$pk $extra_where", $pk);
   }
 
-  public function insertOrUpdate($table, $data=[], $pk='id', $extra_where=''){
-    if(!$this->connection()) return false;
+  public function insertOrUpdate($table, $data = [], $pk = 'id', $extra_where = '') {
+    if (!$this->connection()) return false;
 
-    if (false==is_array($data)) $data = (array)$data;
+    if (false == is_array($data)) $data = (array) $data;
     if (empty($data[$pk])) return $this->insert($table, $data);
-    if( (string) $this->value("SELECT `$pk` FROM `$table` WHERE `$pk`=? LIMIT 1", [$data[$pk]]) === (string) $data[$pk] ){
+    if ((string) $this->value("SELECT `$pk` FROM `$table` WHERE `$pk`=? LIMIT 1", [$data[$pk]]) === (string) $data[$pk]) {
         return $this->update($table, $data, $pk, $extra_where);
     } else {
         return $this->insert($table, $data);

Braces +104 added lines, -58 removed lines

@@ -25,7 +25,7 @@ 
    * @param  array  $options  Options to pass to the PDO constructor
    * @return SQLConnection    The datasource resource
    */
-  public static function register($name, $dsn, $username=null, $password=null, $options=[]){
+  public static function register($name, $dsn, $username=null, $password=null, $options=[]) {
     return self::$connections[$name] = new SQLConnection($dsn, $username, $password, $options);
   }
 
@@ -37,7 +37,7 @@ 
    * @param  array  $options  Options to pass to the PDO constructor
    * @return SQLConnection    The datasource resource
    */
-  public static function connect($dsn, $username=null, $password=null, $options=[]){
+  public static function connect($dsn, $username=null, $password=null, $options=[]) {
     return self::register('default', $dsn, $username, $password, $options);
   }
 
@@ -46,11 +46,13 @@ 
    * @param  string $name The datasource name
    * @return bool       `true` if correctly changed
    */
-  public static function defaultTo($name){
-    if (isset(self::$connections[$name])){
+  public static function defaultTo($name) {
+    if (isset(self::$connections[$name])) {
       self::$current = $name;
       return true;
-    } else return false;
+    } else {
+      return false;
+    }
   }
 
   /**
@@ -58,14 +60,16 @@ 
    * @param  string $name The datasource name, omit for close all of them
    * @return bool       `true` if one or more datasource where closed
    */
-  public static function close($name=null){
+  public static function close($name=null) {
     if ($name === null) {
       foreach (self::$connections as $conn) $conn->close();
       return true;
-    } else if (isset(self::$connections[$name])){
+    } else if (isset(self::$connections[$name])) {
       self::$connections[$name]->close();
       return true;
-    } else return false;
+    } else {
+      return false;
+    }
   }
 
   /**
@@ -73,8 +77,10 @@ 
    * @param  strinf $name The datasource name
    * @return SQLConnect   The datasource connection
    */
-  public static function using($name){
-    if (empty(self::$connections[$name])) throw new \Exception("[SQL] Unknown connection named '$name'.");
+  public static function using($name) {
+    if (empty(self::$connections[$name])) {
+      throw new \Exception("[SQL] Unknown connection named '$name'.");
+    }
     return self::$connections[$name];
   }
 
@@ -84,8 +90,10 @@ 
    * @param  array $args    The method arguments
    * @return mixed          The method return value
    */
-  public static function __callStatic($method, $args){
-    if (empty(self::$connections[self::$current])) throw new \Exception("[SQL] No default connection defined.");
+  public static function __callStatic($method, $args) {
+    if (empty(self::$connections[self::$current])) {
+      throw new \Exception("[SQL] No default connection defined.");
+    }
     return call_user_func_array([self::$connections[self::$current],$method],$args);
   }
 
@@ -101,7 +109,7 @@ 
             $queries           = [],
             $last_exec_success = true;
 
-  public function __construct($dsn, $username=null, $password=null, $options=[]){
+  public function __construct($dsn, $username=null, $password=null, $options=[]) {
     $this->connection = [
       'dsn'        => $dsn,
       'pdo'        => null,
@@ -119,11 +127,11 @@ 
     }
   }
 
-  public function close(){
+  public function close() {
     $this->connection['pdo'] = null;
   }
 
-  public function connection(){
+  public function connection() {
     if(empty($this->connection['pdo'])) {
       try {
         $this->connection['pdo'] = new PDO(
@@ -141,17 +149,23 @@ 
     return $this->connection['pdo'];
   }
 
-  public function prepare($query){
-    if(!$this->connection()) return false;
+  public function prepare($query) {
+    if(!$this->connection()) {
+      return false;
+    }
     return isset($this->queries[$query]) ? $this->queries[$query] : ($this->queries[$query] = $this->connection()->prepare($query));
   }
 
-  public function exec($query, $params=[]){
-    if(!$this->connection()) return false;
+  public function exec($query, $params=[]) {
+    if(!$this->connection()) {
+      return false;
+    }
 
-    if (false==is_array($params)) $params = (array)$params;
+    if (false==is_array($params)) {
+      $params = (array)$params;
+    }
     $query = Filter::with('core.sql.query',$query);
-    if($statement = $this->prepare($query)){
+    if($statement = $this->prepare($query)) {
       Event::trigger('core.sql.query',$query,$params,(bool)$statement);
 
       foreach ($params as $key => $val) {
@@ -175,60 +189,76 @@ 
     return $statement;
   }
 
-  public function value($query, $params=[], $column=0){
-    if(!$this->connection()) return false;
+  public function value($query, $params=[], $column=0) {
+    if(!$this->connection()) {
+      return false;
+    }
 
     $res = $this->exec($query,$params);
     return $res ? $res->fetchColumn($column) : null;
   }
 
-  public function each($query, $params=[], callable $looper = null){
-    if(!$this->connection()) return false;
+  public function each($query, $params=[], callable $looper = null) {
+    if(!$this->connection()) {
+      return false;
+    }
 
     // ($query,$looper) shorthand
     if ($looper===null && is_callable($params)) {$looper = $params; $params = [];}
-    if( $res = $this->exec($query,$params) ){
-      if(is_callable($looper))
-        while ($row = $res->fetchObject()) $looper($row);
-      else
-        return $res->fetchAll(PDO::FETCH_CLASS);
+    if( $res = $this->exec($query,$params) ) {
+      if(is_callable($looper)) {
+              while ($row = $res->fetchObject()) $looper($row);
+      } else {
+              return $res->fetchAll(PDO::FETCH_CLASS);
+      }
     }
   }
 
-  public function single($query, $params=[], callable $handler = null){
-    if(!$this->connection()) return false;
+  public function single($query, $params=[], callable $handler = null) {
+    if(!$this->connection()) {
+      return false;
+    }
 
     // ($query,$handler) shorthand
     if ($handler===null && is_callable($params)) {$handler = $params; $params = [];}
-    if( $res = $this->exec($query,$params) ){
-        if (is_callable($handler))
-          $handler($res->fetchObject());
-        else
-          return $res->fetchObject();
+    if( $res = $this->exec($query,$params) ) {
+        if (is_callable($handler)) {
+                  $handler($res->fetchObject());
+        } else {
+                  return $res->fetchObject();
+        }
     }
   }
 
- public function run($script){
-    if(!$this->connection()) return false;
+ public function run($script) {
+    if(!$this->connection()) {
+      return false;
+    }
 
     $sql_path = Options::get('database.sql.path',APP_DIR.'/sql');
     $sql_sep  = Options::get('database.sql.separator',';');
-    if (is_file($f = "$sql_path/$script.sql")){
+    if (is_file($f = "$sql_path/$script.sql")) {
         $result = true;
         foreach(explode($sql_sep,file_get_contents($f)) as $statement) {
             $result = $this->exec($statement);
         }
         return $result;
-    } else return false;
+    } else {
+      return false;
+    }
   }
 
-  public function all($query, $params=[], callable $looper = null){
-   if(!$this->connection()) return false;
+  public function all($query, $params=[], callable $looper = null) {
+   if(!$this->connection()) {
+     return false;
+   }
    return $this->each($query,$params,$looper);
   }
 
-  public function delete($table, $pks=null, $pk='id', $inclusive=true){
-    if(!$this->connection()) return false;
+  public function delete($table, $pks=null, $pk='id', $inclusive=true) {
+    if(!$this->connection()) {
+      return false;
+    }
 
     if (null===$pks) {
       return $this->exec("DELETE FROM `$table`");
@@ -237,10 +267,14 @@ 
     }
   }
 
-  public function insert($table, $data=[]){
-    if(!$this->connection()) return false;
+  public function insert($table, $data=[]) {
+    if(!$this->connection()) {
+      return false;
+    }
 
-    if (false==is_array($data)) $data = (array)$data;
+    if (false==is_array($data)) {
+      $data = (array)$data;
+    }
     $k = array_keys($data);
     asort($k);
     $pk = $k;
@@ -250,11 +284,17 @@ 
     return $this->last_exec_success ? $this->connection()->lastInsertId() : false;
   }
 
-  public function updateWhere($table, $data=[], $where, $pk='id'){
-    if(!$this->connection()) return false;
+  public function updateWhere($table, $data=[], $where, $pk='id') {
+    if(!$this->connection()) {
+      return false;
+    }
 
-    if (false==is_array($data)) $data = (array)$data;
-    if (empty($data)) return false;
+    if (false==is_array($data)) {
+      $data = (array)$data;
+    }
+    if (empty($data)) {
+      return false;
+    }
     $k = array_keys($data);
     asort($k);
 
@@ -268,16 +308,22 @@ 
     return $this->last_exec_success;
   }
 
-  public function update($table, $data=[], $pk='id', $extra_where=''){
+  public function update($table, $data=[], $pk='id', $extra_where='') {
     return $this->updateWhere($table, $data, "`$pk`=:$pk $extra_where", $pk);
   }
 
-  public function insertOrUpdate($table, $data=[], $pk='id', $extra_where=''){
-    if(!$this->connection()) return false;
+  public function insertOrUpdate($table, $data=[], $pk='id', $extra_where='') {
+    if(!$this->connection()) {
+      return false;
+    }
 
-    if (false==is_array($data)) $data = (array)$data;
-    if (empty($data[$pk])) return $this->insert($table, $data);
-    if( (string) $this->value("SELECT `$pk` FROM `$table` WHERE `$pk`=? LIMIT 1", [$data[$pk]]) === (string) $data[$pk] ){
+    if (false==is_array($data)) {
+      $data = (array)$data;
+    }
+    if (empty($data[$pk])) {
+      return $this->insert($table, $data);
+    }
+    if( (string) $this->value("SELECT `$pk` FROM `$table` WHERE `$pk`=? LIMIT 1", [$data[$pk]]) === (string) $data[$pk] ) {
         return $this->update($table, $data, $pk, $extra_where);
     } else {
         return $this->insert($table, $data);

classes/Response.php

Doc Comments +7 added lines, -6 removed lines

@@ -89,7 +89,7 @@ 
 
     /**
      * Check if an response output buffering is active.
-     * @return boolean
+     * @return boolean|null
      */
     public static function isBuffering(){
         return static::$buffer;
@@ -113,7 +113,6 @@ 
 
     /**
      * Append a text to the buffer.
-     * @param  mixed $payload Text to append to the response buffer
      */
     public static function text(){
         static::type(static::TYPE_TEXT);
@@ -122,7 +121,6 @@ 
 
     /**
      * Append an XML string to the buffer.
-     * @param  mixed $payload Data to append to the response buffer
      */
     public static function xml(){
         static::type(static::TYPE_XML);
@@ -131,7 +129,6 @@ 
 
     /**
      * Append a SVG string to the buffer.
-     * @param  mixed $payload Data to append to the response buffer
      */
     public static function svg(){
         static::type(static::TYPE_SVG);
@@ -140,7 +137,6 @@ 
 
     /**
      * Append an HTML string to the buffer.
-     * @param  mixed $payload Data to append to the response buffer
      */
     public static function html(){
         static::type(static::TYPE_HTML);
@@ -149,16 +145,21 @@ 
 
     /**
      * 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());
     }
 
+    /**
+     * @param integer $code
+     */
     public static function status($code,$message=''){
         static::header('Status',$message?:$code,$code);
     }
 
+    /**
+     * @param string $name
+     */
     public static function header($name,$value,$code=null){
         static::$headers[$name] = [$value,$code];
     }

Indentation +8 added lines, -8 removed lines

@@ -23,12 +23,12 @@ 
           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;
+                      $status      = 200,
+                      $charset     = "utf-8",
+                      $headers     = ['Content-Type' => ['text/html; charset=utf-8']],
+                      $buffer      = null,
+                      $force_dl    = false,
+                      $link        = null;
 
 
     public static function charset($charset){
@@ -184,8 +184,8 @@ 
     }
 
     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

@@ -31,12 +31,12 @@ 
                      $link        = null;
 
 
-    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 : ''));
     }
 
     /**
@@ -44,39 +44,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();
@@ -88,8 +88,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;
@@ -101,14 +101,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 = [];
     }
 
@@ -116,74 +116,74 @@ 
      * 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);
+      return Filter::with('core.response.body', 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;
     }
@@ -195,7 +195,7 @@ 
      *
      * @return array Headers and body of the response
      */
-    public static function save(){
+    public static function save() {
         return [
           'head' => static::$headers,
           'body' => static::body(),
@@ -209,13 +209,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(){
+    public static function send() {
         Event::trigger('core.response.send');
         if (false === headers_sent()) foreach (static::$headers as $name => $value_code) {
             if (is_array($value_code)) {
@@ -225,8 +225,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);

Braces +43 added lines, -31 removed lines

@@ -31,11 +31,11 @@ 
                      $link        = null;
 
 
-    public static function charset($charset){
+    public static function charset($charset) {
         static::$charset = $charset;
     }
 
-    public static function type($mime){
+    public static function type($mime) {
         static::header('Content-Type',$mime . (static::$charset ? '; charset='.static::$charset : ''));
     }
 
@@ -44,21 +44,21 @@ 
      * @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')) {
@@ -88,8 +88,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;
@@ -101,14 +101,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 = [];
     }
 
@@ -116,7 +116,7 @@ 
      * 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));
     }
@@ -125,7 +125,7 @@ 
      * 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());
     }
@@ -134,7 +134,7 @@ 
      * 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());
     }
@@ -143,7 +143,7 @@ 
      * 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());
     }
@@ -152,7 +152,7 @@ 
      * 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());
     }
@@ -161,30 +161,34 @@ 
      * Append a raw string to the buffer.
      * @param  mixed $payload Data to append to the response buffer
      */
-    public static function add(){
+    public static function add() {
         static::$payload[] = implode('',func_get_args());
     }
 
-    public static function status($code,$message=''){
+    public static function status($code,$message='') {
         static::header('Status',$message?:$code,$code);
     }
 
-    public static function header($name,$value,$code=null){
+    public static function header($name,$value,$code=null) {
         static::$headers[$name] = [$value,$code];
     }
 
-    public static function error($code=500,$message='Application Error'){
+    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){
-      if ($setBody) static::$payload = [$setBody];
+    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);
     }
 
-    public static function headers($setHeaders=null){
-       if ($setHeaders) static::$headers = $setHeaders;
+    public static function headers($setHeaders=null) {
+       if ($setHeaders) {
+         static::$headers = $setHeaders;
+       }
        return static::$headers;
     }
 
@@ -195,7 +199,7 @@ 
      *
      * @return array Headers and body of the response
      */
-    public static function save(){
+    public static function save() {
         return [
           'head' => static::$headers,
           'body' => static::body(),
@@ -209,24 +213,30 @@ 
      *
      * @param  array $data head/body saved state
      */
-    public static function load($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);
+      if (isset($data->head)) {
+        static::headers($data->head);
+      }
+      if (isset($data->body)) {
+        static::body($data->body);
+      }
     }
 
-    public static function send(){
+    public static function send() {
         Event::trigger('core.response.send');
-        if (false === headers_sent()) foreach (static::$headers as $name => $value_code) {
+        if (false === headers_sent()) {
+          foreach (static::$headers as $name => $value_code) {
             if (is_array($value_code)) {
                 list($value, $code) = (count($value_code) > 1) ? $value_code : [current($value_code), 200];
+        }
             } else {
                 $value = $value_code;
                 $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);
@@ -238,7 +248,9 @@ 
                 : header("$name: $value", true);
             }
         }
-        if (static::$force_dl) header('Content-Disposition: attachment; filename="'.static::$force_dl.'"');
+        if (static::$force_dl) {
+          header('Content-Disposition: attachment; filename="'.static::$force_dl.'"');
+        }
         echo static::body();
     }
 

classes/Session.php

Doc Comments +3 added lines, -4 removed lines

@@ -32,7 +32,6 @@ 
 	 *
 	 * @access public
 	 * @static
-	 * @param string $key The session name
 	 * @return string The session value
 	 */
 	static public function name($name=null){
@@ -62,7 +61,7 @@ 
 	 *
 	 * @access public
 	 * @static
-	 * @param mixed $key The variable name
+	 * @param string $key The variable name
 	 * @param mixed $value The variable value
 	 * @return void
 	 */
@@ -80,7 +79,7 @@ 
 	 *
 	 * @access public
 	 * @static
-	 * @param mixed $key The variable name
+	 * @param string $key The variable name
 	 * @return void
 	 */
 	static public function delete($key){
@@ -107,7 +106,7 @@ 
 	 *
 	 * @access public
 	 * @static
-	 * @return void
+	 * @return boolean
 	 */
 	static public function active(){
 		return session_status() == PHP_SESSION_ACTIVE;

Indentation +147 added lines, -147 removed lines

@@ -13,126 +13,126 @@ 
 class Session {
     use Module;
 
-	/**
-	 * Start session handler
-	 *
-	 * @access public
-	 * @static
-	 * @return void
-	 */
-	static public function start($name=null){
-		if (isset($_SESSION)) return;
-		static::name($name);
-		session_cache_limiter('must-revalidate');
-		@session_start();
-	}
-
-	/**
-	 * Get/Set Session name
-	 *
-	 * @access public
-	 * @static
-	 * @param string $key The session name
-	 * @return string The session value
-	 */
-	static public function name($name=null){
-		return $name ? session_name($name) : session_name();
-	}
-
-	/**
-	 * Get a session variable reference
-	 *
-	 * @access public
-	 * @static
-	 * @param mixed $key The variable name
-	 * @return mixed The variable value
-	 */
-	static public function get($key,$default=null){
+  /**
+   * Start session handler
+   *
+   * @access public
+   * @static
+   * @return void
+   */
+  static public function start($name=null){
+    if (isset($_SESSION)) return;
+    static::name($name);
+    session_cache_limiter('must-revalidate');
+    @session_start();
+  }
+
+  /**
+   * Get/Set Session name
+   *
+   * @access public
+   * @static
+   * @param string $key The session name
+   * @return string The session value
+   */
+  static public function name($name=null){
+    return $name ? session_name($name) : session_name();
+  }
+
+  /**
+   * Get a session variable reference
+   *
+   * @access public
+   * @static
+   * @param mixed $key The variable name
+   * @return mixed The variable value
+   */
+  static public function get($key,$default=null){
                 if (($active = static::active()) && isset($_SESSION[$key])) {
-			return $_SESSION[$key];
+      return $_SESSION[$key];
                 } else if ($active) {
-                	return $_SESSION[$key] = (is_callable($default)?call_user_func($default):$default);
+                  return $_SESSION[$key] = (is_callable($default)?call_user_func($default):$default);
                 } else {
-                 	return (is_callable($default)?call_user_func($default):$default);
+                    return (is_callable($default)?call_user_func($default):$default);
                 }
-	}
-
-	/**
-	 * Set a session variable
-	 *
-	 * @access public
-	 * @static
-	 * @param mixed $key The variable name
-	 * @param mixed $value The variable value
-	 * @return void
-	 */
-	static public function set($key,$value=null){
-		static::start();
-		if($value==null && is_array($key)){
-			foreach($key as $k=>$v) $_SESSION[$k]=$v;
-		} else {
-			$_SESSION[$key] = $value;
-		}
-	}
-
-	/**
-	 * Delete a session variable
-	 *
-	 * @access public
-	 * @static
-	 * @param mixed $key The variable name
-	 * @return void
-	 */
-	static public function delete($key){
-		static::start();
-		unset($_SESSION[$key]);
-	}
-
-
-	/**
-	 * Delete all session variables
-	 *
-	 * @access public
-	 * @static
-	 * @return void
-	 */
-	static public function clear(){
-		static::start();
-		session_unset();
-		session_destroy();
-	}
-
-	/**
-	 * Check if session is active
-	 *
-	 * @access public
-	 * @static
-	 * @return void
-	 */
-	static public function active(){
-		return session_status() == PHP_SESSION_ACTIVE;
-	}
-
-	/**
-	 * Check if a session variable exists
-	 *
-	 * @access public
-	 * @static
-	 * @param mixed $key The variable name
-	 * @return bool
-	 */
-	static public function exists($key){
-		static::start();
-		return isset($_SESSION[$key]);
-	}
-
-	/**
-	 * Return a read-only accessor to session variables for in-view use.
-	 * @return SessionReadOnly
-	 */
-	static public function readOnly(){
-		return new SessionReadOnly;
-	}
+  }
+
+  /**
+   * Set a session variable
+   *
+   * @access public
+   * @static
+   * @param mixed $key The variable name
+   * @param mixed $value The variable value
+   * @return void
+   */
+  static public function set($key,$value=null){
+    static::start();
+    if($value==null && is_array($key)){
+      foreach($key as $k=>$v) $_SESSION[$k]=$v;
+    } else {
+      $_SESSION[$key] = $value;
+    }
+  }
+
+  /**
+   * Delete a session variable
+   *
+   * @access public
+   * @static
+   * @param mixed $key The variable name
+   * @return void
+   */
+  static public function delete($key){
+    static::start();
+    unset($_SESSION[$key]);
+  }
+
+
+  /**
+   * Delete all session variables
+   *
+   * @access public
+   * @static
+   * @return void
+   */
+  static public function clear(){
+    static::start();
+    session_unset();
+    session_destroy();
+  }
+
+  /**
+   * Check if session is active
+   *
+   * @access public
+   * @static
+   * @return void
+   */
+  static public function active(){
+    return session_status() == PHP_SESSION_ACTIVE;
+  }
+
+  /**
+   * Check if a session variable exists
+   *
+   * @access public
+   * @static
+   * @param mixed $key The variable name
+   * @return bool
+   */
+  static public function exists($key){
+    static::start();
+    return isset($_SESSION[$key]);
+  }
+
+  /**
+   * Return a read-only accessor to session variables for in-view use.
+   * @return SessionReadOnly
+   */
+  static public function readOnly(){
+    return new SessionReadOnly;
+  }
 
 }  /* End of class */
 
@@ -144,36 +144,36 @@ 
 
 class SessionReadOnly {
 
-	/**
-	 * Get a session variable reference
-	 *
-	 * @access public
-	 * @param mixed $key The variable name
-	 * @return mixed The variable value
-	 */
-	public function get($key){
-		return Session::get($key);
-	}
-	public function __get($key){
-		return Session::get($key);
-	}
-
-	public function name(){
-		return Session::name();
-	}
-
-	/**
-	 * Check if a session variable exists
-	 *
-	 * @access public
-	 * @param mixed $key The variable name
-	 * @return bool
-	 */
-	public function exists($key){
-		return Session::exists($key);
-	}
-	public function __isset($key){
-		return Session::exists($key);
-	}
+  /**
+   * Get a session variable reference
+   *
+   * @access public
+   * @param mixed $key The variable name
+   * @return mixed The variable value
+   */
+  public function get($key){
+    return Session::get($key);
+  }
+  public function __get($key){
+    return Session::get($key);
+  }
+
+  public function name(){
+    return Session::name();
+  }
+
+  /**
+   * Check if a session variable exists
+   *
+   * @access public
+   * @param mixed $key The variable name
+   * @return bool
+   */
+  public function exists($key){
+    return Session::exists($key);
+  }
+  public function __isset($key){
+    return Session::exists($key);
+  }
 
 }  /* End of class */

Spacing +18 added lines, -18 removed lines

@@ -20,7 +20,7 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function start($name=null){
+	static public function start($name = null) {
 		if (isset($_SESSION)) return;
 		static::name($name);
 		session_cache_limiter('must-revalidate');
@@ -35,7 +35,7 @@ 
 	 * @param string $key The session name
 	 * @return string The session value
 	 */
-	static public function name($name=null){
+	static public function name($name = null) {
 		return $name ? session_name($name) : session_name();
 	}
 
@@ -47,13 +47,13 @@ 
 	 * @param mixed $key The variable name
 	 * @return mixed The variable value
 	 */
-	static public function get($key,$default=null){
+	static public function get($key, $default = null) {
                 if (($active = static::active()) && isset($_SESSION[$key])) {
 			return $_SESSION[$key];
                 } else if ($active) {
-                	return $_SESSION[$key] = (is_callable($default)?call_user_func($default):$default);
+                	return $_SESSION[$key] = (is_callable($default) ? call_user_func($default) : $default);
                 } else {
-                 	return (is_callable($default)?call_user_func($default):$default);
+                 	return (is_callable($default) ? call_user_func($default) : $default);
                 }
 	}
 
@@ -66,10 +66,10 @@ 
 	 * @param mixed $value The variable value
 	 * @return void
 	 */
-	static public function set($key,$value=null){
+	static public function set($key, $value = null) {
 		static::start();
-		if($value==null && is_array($key)){
-			foreach($key as $k=>$v) $_SESSION[$k]=$v;
+		if ($value == null && is_array($key)) {
+			foreach ($key as $k=>$v) $_SESSION[$k] = $v;
 		} else {
 			$_SESSION[$key] = $value;
 		}
@@ -83,7 +83,7 @@ 
 	 * @param mixed $key The variable name
 	 * @return void
 	 */
-	static public function delete($key){
+	static public function delete($key) {
 		static::start();
 		unset($_SESSION[$key]);
 	}
@@ -96,7 +96,7 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function clear(){
+	static public function clear() {
 		static::start();
 		session_unset();
 		session_destroy();
@@ -109,7 +109,7 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function active(){
+	static public function active() {
 		return session_status() == PHP_SESSION_ACTIVE;
 	}
 
@@ -121,7 +121,7 @@ 
 	 * @param mixed $key The variable name
 	 * @return bool
 	 */
-	static public function exists($key){
+	static public function exists($key) {
 		static::start();
 		return isset($_SESSION[$key]);
 	}
@@ -130,7 +130,7 @@ 
 	 * Return a read-only accessor to session variables for in-view use.
 	 * @return SessionReadOnly
 	 */
-	static public function readOnly(){
+	static public function readOnly() {
 		return new SessionReadOnly;
 	}
 
@@ -151,14 +151,14 @@ 
 	 * @param mixed $key The variable name
 	 * @return mixed The variable value
 	 */
-	public function get($key){
+	public function get($key) {
 		return Session::get($key);
 	}
-	public function __get($key){
+	public function __get($key) {
 		return Session::get($key);
 	}
 
-	public function name(){
+	public function name() {
 		return Session::name();
 	}
 
@@ -169,10 +169,10 @@ 
 	 * @param mixed $key The variable name
 	 * @return bool
 	 */
-	public function exists($key){
+	public function exists($key) {
 		return Session::exists($key);
 	}
-	public function __isset($key){
+	public function __isset($key) {
 		return Session::exists($key);
 	}
 

Braces +18 added lines, -16 removed lines

@@ -20,8 +20,10 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function start($name=null){
-		if (isset($_SESSION)) return;
+	static public function start($name=null) {
+		if (isset($_SESSION)) {
+		  return;
+		}
 		static::name($name);
 		session_cache_limiter('must-revalidate');
 		@session_start();
@@ -35,7 +37,7 @@ 
 	 * @param string $key The session name
 	 * @return string The session value
 	 */
-	static public function name($name=null){
+	static public function name($name=null) {
 		return $name ? session_name($name) : session_name();
 	}
 
@@ -47,7 +49,7 @@ 
 	 * @param mixed $key The variable name
 	 * @return mixed The variable value
 	 */
-	static public function get($key,$default=null){
+	static public function get($key,$default=null) {
                 if (($active = static::active()) && isset($_SESSION[$key])) {
 			return $_SESSION[$key];
                 } else if ($active) {
@@ -66,9 +68,9 @@ 
 	 * @param mixed $value The variable value
 	 * @return void
 	 */
-	static public function set($key,$value=null){
+	static public function set($key,$value=null) {
 		static::start();
-		if($value==null && is_array($key)){
+		if($value==null && is_array($key)) {
 			foreach($key as $k=>$v) $_SESSION[$k]=$v;
 		} else {
 			$_SESSION[$key] = $value;
@@ -83,7 +85,7 @@ 
 	 * @param mixed $key The variable name
 	 * @return void
 	 */
-	static public function delete($key){
+	static public function delete($key) {
 		static::start();
 		unset($_SESSION[$key]);
 	}
@@ -96,7 +98,7 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function clear(){
+	static public function clear() {
 		static::start();
 		session_unset();
 		session_destroy();
@@ -109,7 +111,7 @@ 
 	 * @static
 	 * @return void
 	 */
-	static public function active(){
+	static public function active() {
 		return session_status() == PHP_SESSION_ACTIVE;
 	}
 
@@ -121,7 +123,7 @@ 
 	 * @param mixed $key The variable name
 	 * @return bool
 	 */
-	static public function exists($key){
+	static public function exists($key) {
 		static::start();
 		return isset($_SESSION[$key]);
 	}
@@ -130,7 +132,7 @@ 
 	 * Return a read-only accessor to session variables for in-view use.
 	 * @return SessionReadOnly
 	 */
-	static public function readOnly(){
+	static public function readOnly() {
 		return new SessionReadOnly;
 	}
 
@@ -151,14 +153,14 @@ 
 	 * @param mixed $key The variable name
 	 * @return mixed The variable value
 	 */
-	public function get($key){
+	public function get($key) {
 		return Session::get($key);
 	}
-	public function __get($key){
+	public function __get($key) {
 		return Session::get($key);
 	}
 
-	public function name(){
+	public function name() {
 		return Session::name();
 	}
 
@@ -169,10 +171,10 @@ 
 	 * @param mixed $key The variable name
 	 * @return bool
 	 */
-	public function exists($key){
+	public function exists($key) {
 		return Session::exists($key);
 	}
-	public function __isset($key){
+	public function __isset($key) {
 		return Session::exists($key);
 	}