comodojo / dispatcher.framework

src/Cache/DispatcherCache.php

Doc Comments -2 removed lines

@@ -66,8 +66,6 @@
     /**

      * Map provided log level to level code

      *

-     * @param   string    $level

-     *

      * @return  integer

      */

     protected static function getAlgorithm($algorithm) {

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
         

         $ttl = $configuration->get('dispatcher-cache-ttl');

         

-        if ( $ttl !== null && !defined('COMODOJO_CACHE_DEFAULT_TTL') ) {

+        if ($ttl !== null && !defined('COMODOJO_CACHE_DEFAULT_TTL')) {

             

             define('COMODOJO_CACHE_DEFAULT_TTL', $ttl);

             

@@ -49,13 +49,13 @@ 
         

         $folder = $configuration->get('dispatcher-cache-folder');

         

-        $algorithm = self::getAlgorithm( $configuration->get('dispatcher-cache-algorithm') );

+        $algorithm = self::getAlgorithm($configuration->get('dispatcher-cache-algorithm'));

     

-        $manager = new CacheManager( $algorithm );

+        $manager = new CacheManager($algorithm);

         

-        if ( $enabled === true ) {

+        if ($enabled === true) {

             

-            $manager->addProvider( new FileCache($folder) );

+            $manager->addProvider(new FileCache($folder));

             

         }

         

@@ -72,7 +72,7 @@ 
      */

     protected static function getAlgorithm($algorithm) {

 

-        switch ( strtoupper($algorithm) ) {

+        switch (strtoupper($algorithm)) {

 

             case 'PICK_LAST':

                 $selected = CacheManager::PICK_LAST;

src/Log/DispatcherLogger.php

Doc Comments -2 removed lines

@@ -33,8 +33,6 @@
     /**

      * Create the logger

      *

-     * @param bool $verbose

-     *

      * @return \Monolog\Logger

      */

     public static function create(Configuration $configuration) {

Spacing +5 added lines, -5 removed lines

@@ -43,19 +43,19 @@ 
         

         $enabled = $configuration->get('dispatcher-log-enabled');

         

-        $level = self::getLevel( $configuration->get('dispatcher-log-level') );

+        $level = self::getLevel($configuration->get('dispatcher-log-level'));

         

         $target = $configuration->get('dispatcher-log-target');

     

         $logger = new Logger($name);

         

-        if ( $enabled === true ) {

+        if ($enabled === true) {

             

-            $logger->pushHandler( new StreamHandler( $target, $level) );

+            $logger->pushHandler(new StreamHandler($target, $level));

             

         } else {

             

-            $logger->pushHandler( new NullHandler($level) );

+            $logger->pushHandler(new NullHandler($level));

             

         }

         

@@ -72,7 +72,7 @@ 
      */

     protected static function getLevel($level) {

 

-        switch ( strtoupper($level) ) {

+        switch (strtoupper($level)) {

 

             case 'INFO':

                 $logger_level = Logger::INFO;

src/Router/RoutingTable.php

Doc Comments +3 added lines

@@ -62,6 +62,9 @@
         
     }
     
+    /**
+     * @param string $route
+     */
     public function get($route) {
         
         $folders = explode("/", $route);

Spacing +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
                 $this->readpath( 
                     $folders,
                     $value,
-                    $regex.'(?:\/'.$param_regex.')'. (($param_required)?'{1}':'?')
+                    $regex.'(?:\/'.$param_regex.')'.(($param_required) ? '{1}' : '?')
                 );
                 
             } else {
@@ -171,7 +171,7 @@ 
         $string = preg_replace('/(?<!\\)\((?!\?)/', '(?:', $string);
         $string = preg_replace('/\.([\*\+])(?!\?)/', '.\${1}?', $string);
         
-        return '(?P<' . $key . '>' . $string . ')' . (($field_required)?'{1}':'?');
+        return '(?P<'.$key.'>'.$string.')'.(($field_required) ? '{1}' : '?');
         
     }
     

Braces +11 added lines, -6 removed lines

@@ -66,10 +66,11 @@ 
 
         $regex = $this->readpath($folders);
 
-        if (isset($this->routes[$regex]))
-            return $this->routes[$regex];
-        else
-            return null;
+        if (isset($this->routes[$regex])) {
+                    return $this->routes[$regex];
+        } else {
+                    return null;
+        }
 
     }
 
@@ -79,7 +80,9 @@ 
 
         $regex = $this->readpath($folders);
 
-        if (isset($this->routes[$regex])) unset($this->routes[$regex]);
+        if (isset($this->routes[$regex])) {
+            unset($this->routes[$regex]);
+        }
 
     }
 
@@ -97,7 +100,9 @@ 
 
     private function readpath($folders = array(), &$value = null, $regex = '') {
 
-        if (!empty($folders) && empty($folders[0])) array_shift($folders);
+        if (!empty($folders) && empty($folders[0])) {
+            array_shift($folders);
+        }
 
         if (empty($folders)) {
 

src/Components/Configuration.php

Spacing +7 added lines, -7 removed lines

@@ -34,7 +34,7 @@ 
         'dispatcher-log-level' => 'INFO',
         'dispatcher-log-target' => '%dispatcher-log-folder%/dispatcher.log',
         'dispatcher-log-folder' => '/log',
-        'dispatcher-supported-methods' => array('GET','PUT','POST','DELETE','OPTIONS','HEAD'),
+        'dispatcher-supported-methods' => array('GET', 'PUT', 'POST', 'DELETE', 'OPTIONS', 'HEAD'),
         'dispatcher-default-encoding' => 'UTF-8',
         'dispatcher-cache-enabled' => true,
         'dispatcher-cache-ttl' => 3600,
@@ -44,7 +44,7 @@ 
         //'dispatcher-autoroute' => false
     );
 
-    public function __construct( $configuration = array() ) {
+    public function __construct($configuration = array()) {
 
         $this->attributes['dispatcher-base-url'] = self::urlGetAbsolute();
 
@@ -60,15 +60,15 @@ 
 
             $value = $this->attributes[$property];
 
-            if ( is_scalar($value) && preg_match_all('/%(.+?)%/', $value, $matches) ) {
+            if (is_scalar($value) && preg_match_all('/%(.+?)%/', $value, $matches)) {
 
                 $substitutions = array();
 
-                foreach ( $matches as $match ) {
+                foreach ($matches as $match) {
 
                     $backreference = $match[1];
 
-                    if ( $backreference != $property && !isset($substitutions['/%'.$backreference.'%/']) ) {
+                    if ($backreference != $property && !isset($substitutions['/%'.$backreference.'%/'])) {
 
                         $substitutions['/%'.$backreference.'%/'] = $this->$backreference;
 
@@ -112,11 +112,11 @@ 
 
     private static function urlGetAbsolute() {
 
-        $http = 'http' . ((isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on') ? 's' : '') . '://';
+        $http = 'http'.((isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on') ? 's' : '').'://';
 
         $uri = preg_replace("/\/index.php(.*?)$/i", "", $_SERVER['PHP_SELF']);
 
-        return ( $http . $_SERVER['HTTP_HOST'] . $uri . "/" );
+        return ($http.$_SERVER['HTTP_HOST'].$uri."/");
 
     }
 

src/Service/AbstractService.php

Spacing +5 added lines, -5 removed lines

@@ -94,7 +94,7 @@ 
 
         $supported_methods = $this->configuration()->get('dispatcher-supported-methods');
 
-        if ( method_exists($this, 'any') ) {
+        if (method_exists($this, 'any')) {
 
             return $supported_methods;
 
@@ -102,9 +102,9 @@ 
 
         $implemented_methods = array();
 
-        foreach ( $supported_methods as $method ) {
+        foreach ($supported_methods as $method) {
 
-            if ( method_exists($this, strtolower($method)) ) array_push($implemented_methods,$method);
+            if (method_exists($this, strtolower($method))) array_push($implemented_methods, $method);
 
         }
 
@@ -118,11 +118,11 @@ 
      */
     final public function getMethod($method) {
 
-        if ( method_exists($this, strtolower($method)) ) {
+        if (method_exists($this, strtolower($method))) {
 
             return strtolower($method);
 
-        } else if ( method_exists($this, strtolower('any')) ) {
+        } else if (method_exists($this, strtolower('any'))) {
 
             return 'any';
 

Braces +3 added lines, -1 removed lines

@@ -104,7 +104,9 @@
 
         foreach ( $supported_methods as $method ) {
 
-            if ( method_exists($this, strtolower($method)) ) array_push($implemented_methods,$method);
+            if ( method_exists($this, strtolower($method)) ) {
+                array_push($implemented_methods,$method);
+            }
 
         }