comodojo / dispatcher.framework

src/ObjectRoute/ObjectRoute.php

Braces +9 added lines, -6 removed lines

@@ -186,8 +186,7 @@ 
 
             $this->cache = false;
 
-        }
-        else {
+        } else {
 
             $cache = strtoupper($cache);
 
@@ -279,7 +278,9 @@ 
      */
     public function getHeader($header) {
 
-        if ( isset($this->headers[$header]) ) return $this->headers[$header];
+        if ( isset($this->headers[$header]) ) {
+            return $this->headers[$header];
+        }
 
         return null;
 
@@ -344,9 +345,11 @@ 
      */
     public function getParameter($parameter) {
 
-        if ( isset($this->parameters[$parameter]) ) return $this->parameters[$parameter];
-
-        else return null;
+        if ( isset($this->parameters[$parameter]) ) {
+            return $this->parameters[$parameter];
+        } else {
+            return null;
+        }
 
     }
 

src/Serialization.php

Braces +20 added lines, -8 removed lines

@@ -39,7 +39,9 @@ 
      */
     final public function toJson($data, $flags=null) {
 
-        if ( !( is_array($data) OR is_object($data) ) ) throw new Exception("Invalid data for JSON serialization");
+        if ( !( is_array($data) OR is_object($data) ) ) {
+            throw new Exception("Invalid data for JSON serialization");
+        }
 
         return json_encode($data, $flags);
 
@@ -57,9 +59,13 @@ 
      */
     final public function toXml($data, $prettify=false) {
 
-        if ( !( is_array($data) OR is_object($data) ) ) throw new Exception("Invalid data for XML serialization");
+        if ( !( is_array($data) OR is_object($data) ) ) {
+            throw new Exception("Invalid data for XML serialization");
+        }
 
-        if ( is_object($data) ) $data = $this->objectToArray($data);
+        if ( is_object($data) ) {
+            $data = $this->objectToArray($data);
+        }
 
         $xmlEngine = new XML();
         
@@ -103,9 +109,13 @@ 
      */
     final public function toYaml($data) {
 
-        if ( !( is_array($data) OR is_object($data) ) ) throw new Exception("Invalid data for XML serialization");
+        if ( !( is_array($data) OR is_object($data) ) ) {
+            throw new Exception("Invalid data for XML serialization");
+        }
 
-        if ( is_object($data) ) $data = $this->objectToArray($data);
+        if ( is_object($data) ) {
+            $data = $this->objectToArray($data);
+        }
 
         return \Spyc::YAMLDump($data);
 
@@ -158,7 +168,9 @@ 
         
         foreach($xmlStringArray as $k=>$tag){
 
-            if ($tag == "") continue;
+            if ($tag == "") {
+                continue;
+            }
             
             if ($tag[0]=="<" AND $tag[1] != "/") {
 
@@ -203,9 +215,9 @@ 
 
             return $array;
 
+        } else {
+            return $stdObj;
         }
-        
-        else return $stdObj;
 
     }
 

src/Cache.php

Braces +15 added lines, -14 removed lines

@@ -95,8 +95,7 @@ 
 
                 return false;
 
-            }
-            else {
+            } else {
 
                 $this->logger->error('Empty data, nothign to cache');
 
@@ -120,8 +119,7 @@ 
             
             if ($this->fail_silently) {
                 return false;
-            }
-            else {
+            } else {
                 throw new IOException("Error writing to cache");
             }
         }
@@ -176,8 +174,7 @@ 
 
                     return false;
 
-                }
-                else {
+                } else {
 
                     throw new IOException("Error reading from cache");
 
@@ -191,10 +188,10 @@ 
                 "object"    =>  $u_data
              );
 
+        } else {
+            return false;
         }
         
-        else return false;
-        
     }
 
     /**
@@ -226,17 +223,20 @@ 
             while( false !== ( $cache_file = readdir($cache_path) ) ) {
 
                 if ( pathinfo($cache_file, PATHINFO_EXTENSION) == "cache" ) {
-                    if ( unlink( $this->cache_path . ( $this->cache_path[strlen($this->cache_path)-1] == "/" ? "" : "/" ) . $cache_file ) == false ) return false;
-                    else $cache_files_number++;
+                    if ( unlink( $this->cache_path . ( $this->cache_path[strlen($this->cache_path)-1] == "/" ? "" : "/" ) . $cache_file ) == false ) {
+                        return false;
+                    } else {
+                        $cache_files_number++;
+                    }
+                } else {
+                    continue;
                 }
-                else continue;
                 
             }
           
             closedir($cache_path);
 
-        }
-        else {
+        } else {
 
             $this->logger->info('Purging request cache', array(
                 'REQUEST' => $request
@@ -251,8 +251,9 @@ 
                 $unlink = unlink($cacheFile);
                 $cache_files_number = $unlink ? 1 : false;
 
+            } else {
+                $cache_files_number = 0;
             }
-            else $cache_files_number = 0;
 
         }
 

src/ObjectRoutingTable/ObjectRoutingTable.php

Braces +22 added lines, -14 removed lines

@@ -121,7 +121,9 @@ 
 
         // If service is null, route to landing or (if not specified) to default route
 
-        if ( empty($service) ) return $this->routeToLanding();
+        if ( empty($service) ) {
+            return $this->routeToLanding();
+        }
 
         // If service is in table, check if it is routable (in case of "ROUTE" route).
         // If true, route request
@@ -131,14 +133,16 @@ 
 
             if ( $this->table[$service]["type"] == "ROUTE" ) {
 
-                if ( $this->routable($this->table[$service]["target"]) ) return $this->table[$service];
-
-                else return $this->routeToNowhere();
+                if ( $this->routable($this->table[$service]["target"]) ) {
+                    return $this->table[$service];
+                } else {
+                    return $this->routeToNowhere();
+                }
 
+            } else {
+                return $this->table[$service];
             }
 
-            else return $this->table[$service];
-
         }
 
         // If autoroute is enabled, try to match service name with a service file.
@@ -156,7 +160,9 @@ 
 
         // If a route cannot be traced, emit a 404 - not found - response
 
-        else return $this->routeToNotfound();
+        else {
+            return $this->routeToNotfound();
+        }
 
     }
 
@@ -180,9 +186,11 @@ 
      */
     private function routable($target, $relative=false) {
 
-        if ( $relative ) return is_readable(DISPATCHER_SERVICES_FOLDER.$target) ? true : false;
-
-        else return is_readable($target) ? true : false;
+        if ( $relative ) {
+            return is_readable(DISPATCHER_SERVICES_FOLDER.$target) ? true : false;
+        } else {
+            return is_readable($target) ? true : false;
+        }
 
     }
 
@@ -197,10 +205,10 @@ 
 
             return $this->table[""];
 
+        } else {
+            return $this->routeToDefault();
         }
 
-        else return $this->routeToDefault();
-
     }
 
     /**
@@ -214,10 +222,10 @@ 
 
             return $this->table["default"];
 
+        } else {
+            return $this->routeToNotfound();
         }
 
-        else return $this->routeToNotfound();
-
     }
 
     /**

src/ObjectRequest/ObjectRequest.php

Braces +8 added lines, -4 removed lines

@@ -165,9 +165,11 @@ 
      */
     public function setAttribute($name=null, $value) {
 
-        if ( is_null($name) ) array_push($this->attributes, $value);
-
-        else $this->attributes[$name] = $value;
+        if ( is_null($name) ) {
+            array_push($this->attributes, $value);
+        } else {
+            $this->attributes[$name] = $value;
+        }
 
         return $this;
 
@@ -417,7 +419,9 @@ 
      */
     public function getHeader($header) {
 
-        if ( isset($this->headers[$header]) ) return $this->headers[$header];
+        if ( isset($this->headers[$header]) ) {
+            return $this->headers[$header];
+        }
 
         return null;
 

src/Header.php

Braces +31 added lines, -16 removed lines

@@ -77,9 +77,11 @@ 
      */
     final public function get($header) {
 
-        if ( array_key_exists($header, $this->headers) ) return $this->headers[$header];
-
-        else return null;
+        if ( array_key_exists($header, $this->headers) ) {
+            return $this->headers[$header];
+        } else {
+            return null;
+        }
 
     }
 
@@ -163,10 +165,16 @@ 
 
             case 304: //Not Modified
 
-            if ( $value === false ) header($_SERVER["SERVER_PROTOCOL"].' 304 Not Modified');
-            else header('Last-Modified: '.gmdate('D, d M Y H:i:s', $value).' GMT', true, 304);
+            if ( $value === false ) {
+                header($_SERVER["SERVER_PROTOCOL"].' 304 Not Modified');
+            } else {
+                header('Last-Modified: '.gmdate('D, d M Y H:i:s', $value).' GMT', true, 304);
+            }
 
-                if ($contentLength !== 0) header('Content-Length: '.$contentLength); //is it needed?
+                if ($contentLength !== 0) {
+                    header('Content-Length: '.$contentLength);
+                }
+                //is it needed?
 
                 $this->processExtraHeaders($this->headers);
 
@@ -240,8 +248,7 @@ 
         if ( $ttl > 0 ) {
             $this->set("Cache-Control","max-age=".$ttl.", must-revalidate");
             $this->set("Expires",gmdate("D, d M Y H:i:s", (int)$this->current_time + $ttl)." GMT");
-        }
-        else {
+        } else {
             $this->set("Cache-Control","no-cache, must-revalidate");
             $this->set("Expires","Mon, 26 Jul 1997 05:00:00 GMT");
         }
@@ -260,8 +267,11 @@ 
     public function setContentType($type, $charset=null) {
 
 
-        if ( is_null($charset) ) $this->set("Content-type",strtolower($type));
-        else $this->set("Content-type",strtolower($type)."; charset=".$charset);
+        if ( is_null($charset) ) {
+            $this->set("Content-type",strtolower($type));
+        } else {
+            $this->set("Content-type",strtolower($type)."; charset=".$charset);
+        }
 
         return $this;
 
@@ -276,13 +286,15 @@ 
 
         $headers = '';
 
-        if (function_exists('getallheaders')) $headers = getallheaders();
-
-        else {
+        if (function_exists('getallheaders')) {
+            $headers = getallheaders();
+        } else {
 
             foreach ($_SERVER as $name => $value) {
 
-                if (substr($name, 0, 5) == 'HTTP_') $headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
+                if (substr($name, 0, 5) == 'HTTP_') {
+                    $headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
+                }
             }
 
         }
@@ -300,8 +312,11 @@ 
 
         foreach ( $headers as $header => $value ) {
 
-            if ( is_null($value) ) header($header, true);
-            else  header($header.": ".$value, true);
+            if ( is_null($value) ) {
+                header($header, true);
+            } else {
+                header($header.": ".$value, true);
+            }
 
         }
 

src/ObjectResult/ObjectError.php

Braces +3 added lines, -1 removed lines

@@ -175,7 +175,9 @@
      */
     public function getHeader($header) {
 
-        if ( isset($this->headers[$header]) ) return $this->headers[$header];
+        if ( isset($this->headers[$header]) ) {
+            return $this->headers[$header];
+        }
 
         return null;
 

src/Debug.php

Braces +1 added lines, -2 removed lines

@@ -48,8 +48,7 @@
 
             $handler = is_null($target) ? new ErrorLogHandler() : new StreamHandler( defined('DISPATCHER_LOG_FOLDER') ? DISPATCHER_LOG_FOLDER.$target : $target, $level);
 
-        }
-        else {
+        } else {
 
             $handler = new NullHandler($level);
 

src/XML.php

Braces +28 added lines, -15 removed lines

@@ -210,7 +210,9 @@ 
         
         $xml = $this->ObjectXML->asXML();
 
-        if ( $xml === false ) throw new XMLException("Error formatting object");
+        if ( $xml === false ) {
+            throw new XMLException("Error formatting object");
+        }
         
         return $xml;
         
@@ -231,7 +233,9 @@ 
         
         $parser_structs = xml_parse_into_struct($this->parser, $xml, $this->struct, $index);
         
-        if ( $parser_structs === 0 ) throw new XMLException("Failed to parse xml data into structs");
+        if ( $parser_structs === 0 ) {
+            throw new XMLException("Failed to parse xml data into structs");
+        }
         
         $this->counter = count($this->struct);
 
@@ -350,11 +354,15 @@ 
         
         if ( !isset($target[$key]) ) {
             
-            if ($child != "") $target[$key] = $child;
-
-            if ($attributes != "") foreach($attributes as $akey => $avalue) $target[$key][$akey] = $avalue;
+            if ($child != "") {
+                $target[$key] = $child;
+            }
 
-            else if ( $value != "" ) $target[$key] = $value;
+            if ($attributes != "") {
+                foreach($attributes as $akey => $avalue) $target[$key][$akey] = $avalue;
+            } else if ( $value != "" ) {
+                $target[$key] = $value;
+            }
             
         } else {
 
@@ -365,18 +373,21 @@ 
                 $target[$key][0] = $oldval;
                 $index=1;
 
-            }
-            else {
+            } else {
 
                 $index = count($target[$key]);
 
             }
 
-            if ( $child != "" ) $target[$key][$index] = $child;
-
-            if ( $attributes != "" ) foreach($attributes as $akey => $avalue) $target[$key][$index][$akey] = $avalue;
+            if ( $child != "" ) {
+                $target[$key][$index] = $child;
+            }
 
-            elseif ( $value != "" ) $target[$key][$index] = $value;
+            if ( $attributes != "" ) {
+                foreach($attributes as $akey => $avalue) $target[$key][$index][$akey] = $avalue;
+            } elseif ( $value != "" ) {
+                $target[$key][$index] = $value;
+            }
         }
 
         return $target;
@@ -393,9 +404,11 @@ 
 
         foreach ($data as $key => $value) {
 
-            if ( is_array($value) ) $this->pushElement($ObjectXML->addChild(is_numeric($key) ? 'KEY_'.$key : $key), $value);
-
-            else $ObjectXML->addChild(is_numeric($key) ? 'KEY_'.$key : $key, mb_convert_encoding($value, $this->encoding));
+            if ( is_array($value) ) {
+                $this->pushElement($ObjectXML->addChild(is_numeric($key) ? 'KEY_'.$key : $key), $value);
+            } else {
+                $ObjectXML->addChild(is_numeric($key) ? 'KEY_'.$key : $key, mb_convert_encoding($value, $this->encoding));
+            }
 
         }