comodojo / dispatcher.framework

src/Components/Headers.php

Spacing +6 added lines, -6 removed lines

@@ -28,17 +28,17 @@ 
 
     final public function get($header = null) {
 
-        if ( is_null($header) ) return $this->headers;
+        if (is_null($header)) return $this->headers;
 
-        else if ( array_key_exists($header, $this->headers) ) return $this->headers[$header];
+        else if (array_key_exists($header, $this->headers)) return $this->headers[$header];
 
         else return null;
 
     }
 
-    final public function set($header, $value=null) {
+    final public function set($header, $value = null) {
 
-        if ( is_null($value) ) {
+        if (is_null($value)) {
 
             $header = explode(":", $header);
 
@@ -56,13 +56,13 @@ 
 
     final public function unset($header = null) {
 
-        if ( is_null($header) ) {
+        if (is_null($header)) {
 
             $this->headers = array();
 
             return true;
 
-        } else if ( array_key_exists($header, $this->headers) ) {
+        } else if (array_key_exists($header, $this->headers)) {
 
             unset($this->headers[$header]);
 

Braces +7 added lines, -5 removed lines

@@ -28,11 +28,13 @@
 
     final public function get($header = null) {
 
-        if ( is_null($header) ) return $this->headers;
-
-        else if ( array_key_exists($header, $this->headers) ) return $this->headers[$header];
-
-        else return null;
+        if ( is_null($header) ) {
+            return $this->headers;
+        } else if ( array_key_exists($header, $this->headers) ) {
+            return $this->headers[$header];
+        } else {
+            return null;
+        }
 
     }
 

src/Components/Parameters.php

Spacing +5 added lines, -5 removed lines

@@ -26,11 +26,11 @@ 
 
     protected $parameters = array();
 
-    final public function get($parameter=null) {
+    final public function get($parameter = null) {
 
-        if ( is_null($parameter) ) return $this->parameters;
+        if (is_null($parameter)) return $this->parameters;
 
-        else if ( array_key_exists($parameter, $this->parameters) ) {
+        else if (array_key_exists($parameter, $this->parameters)) {
 
             return $this->parameters[$parameter];
 
@@ -50,13 +50,13 @@ 
 
     final public function unset($parameter = null) {
 
-        if ( is_null($parameter) ) {
+        if (is_null($parameter)) {
 
             $this->parameters = array();
 
             return true;
 
-        } else if ( array_key_exists($parameter, $this->parameters) ) {
+        } else if (array_key_exists($parameter, $this->parameters)) {
 
             unset($this->parameters[$parameter]);
 

Braces +5 added lines, -5 removed lines

@@ -28,16 +28,16 @@
 
     final public function get($parameter=null) {
 
-        if ( is_null($parameter) ) return $this->parameters;
-
-        else if ( array_key_exists($parameter, $this->parameters) ) {
+        if ( is_null($parameter) ) {
+            return $this->parameters;
+        } else if ( array_key_exists($parameter, $this->parameters) ) {
 
             return $this->parameters[$parameter];
 
+        } else {
+            return null;
         }
 
-        else return null;
-
     }
 
     final public function set($parameter, $value) {

src/Output/Processor.php

Spacing +6 added lines, -6 removed lines

@@ -108,7 +108,7 @@ 
 
                 header($_SERVER["SERVER_PROTOCOL"].' 204 No Content');
                 header('Status: 204 No Content');
-                header('Content-Length: 0',true);
+                header('Content-Length: 0', true);
 
                 $return = null;
 
@@ -120,7 +120,7 @@ 
             case 303: //See Other
             case 307: //Temporary Redirect
 
-                header("Location: ".$location->get(),true,$status);
+                header("Location: ".$location->get(), true, $status);
 
                 break;
 
@@ -128,11 +128,11 @@ 
 
                 $last_modified = $headers->get('Last-Modified');
 
-                if ( is_null($last_modified) ) {
+                if (is_null($last_modified)) {
 
                     header($_SERVER["SERVER_PROTOCOL"].' 304 Not Modified');
 
-                } else if ( is_int($last_modified) ) {
+                } else if (is_int($last_modified)) {
 
                     header('Last-Modified: '.gmdate('D, d M Y H:i:s', $last_modified).' GMT', true, 304);
 
@@ -170,7 +170,7 @@ 
 
             case 405: //Not allowed
 
-            header('Allow: ' . $value, true, 405);
+            header('Allow: '.$value, true, 405);
 
             break;
 
@@ -183,7 +183,7 @@ 
 
             case 501: //Not implemented
 
-            header('Allow: ' . $value, true, 501);
+            header('Allow: '.$value, true, 501);
 
             break;
 

src/Request/Headers.php

Spacing +3 added lines, -3 removed lines

@@ -41,13 +41,13 @@
      */
     private static function getHeaders() {
 
-        if ( function_exists('getallheaders') ) return getallheaders();
+        if (function_exists('getallheaders')) return getallheaders();
 
         $headers = array();
 
-        foreach ( $_SERVER as $name => $value ) {
+        foreach ($_SERVER as $name => $value) {
 
-            if ( substr($name, 0, 5) == 'HTTP_' ) {
+            if (substr($name, 0, 5) == 'HTTP_') {
 
                 $headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
 

Braces +3 added lines, -1 removed lines

@@ -41,7 +41,9 @@
      */
     private static function getHeaders() {
 
-        if ( function_exists('getallheaders') ) return getallheaders();
+        if ( function_exists('getallheaders') ) {
+            return getallheaders();
+        }
 
         $headers = array();
 

src/Request/Post.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 
     private static function getParameters() {
 
-        switch( $_SERVER['REQUEST_METHOD'] ) {
+        switch ($_SERVER['REQUEST_METHOD']) {
 
             case 'POST':
 

src/Response/Content.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
 
     public function set($content) {
 
-        if ( !is_scalar($content) ) {
+        if (!is_scalar($content)) {
 
             throw new Exception("Invalid HTTP content");
 
@@ -54,7 +54,7 @@ 
 
     public function type($type = null) {
 
-        if ( is_null($type) ) {
+        if (is_null($type)) {
 
             return $this->type;
 
@@ -68,7 +68,7 @@ 
 
     public function charset($charset = null) {
 
-        if ( is_null($charset) ) {
+        if (is_null($charset)) {
 
             return $this->charset;
 

src/Response/Status.php

Spacing +3 added lines, -3 removed lines

@@ -85,7 +85,7 @@ 
 
     public function set($code) {
 
-        if ( !array_key_exists($code, $this->http_status_codes) ) {
+        if (!array_key_exists($code, $this->http_status_codes)) {
 
             throw new Exception("Invalid HTTP Status Code");
 
@@ -97,9 +97,9 @@ 
 
     }
 
-    public function description($code=null) {
+    public function description($code = null) {
 
-        if ( is_null($code) || !array_key_exists($code, $this->http_status_codes) ) {
+        if (is_null($code) || !array_key_exists($code, $this->http_status_codes)) {
 
             $message = $this->http_status_codes[$this->status_code];
 

src/Request/Query.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 
     private static function getParameters() {
 
-        switch( $_SERVER['REQUEST_METHOD'] ) {
+        switch ($_SERVER['REQUEST_METHOD']) {
 
             case 'POST':
 

src/Router/Collector.php

Braces +5 added lines, -2 removed lines

@@ -96,8 +96,9 @@ 
                 $this->response
             );
             
+        } else {
+            return null;
         }
-        else return null;
         
     }
 
@@ -183,7 +184,9 @@ 
                 
                 if (preg_match('/' . $value['regex'] . '/', $bits[$key], $matches)) {
                     
-                    if (count($matches) == 1) $matches = $matches[0];
+                    if (count($matches) == 1) {
+                        $matches = $matches[0];
+                    }
                     
                     $this->request->query()->set($key, $matches);
                     

Spacing +5 added lines, -5 removed lines

@@ -132,7 +132,7 @@ 
             
             $result = "";
 
-            switch( $_SERVER['REQUEST_METHOD'] ) {
+            switch ($_SERVER['REQUEST_METHOD']) {
     
                 case 'POST':
     
@@ -177,7 +177,7 @@ 
         
         foreach ($this->table as $regex => $value) {
             
-            if (preg_match("/" . $regex . "/", $path, $matches)) {
+            if (preg_match("/".$regex."/", $path, $matches)) {
                 
                 $this->evalUri($value['query'], $matches);
                 
@@ -190,7 +190,7 @@ 
                 $this->classname  = $value['class'];
                 $this->type       = $value['type'];
                 $this->service    = implode('.', $value['service']);
-                $this->service    = empty($this->service)?"default":$this->service;
+                $this->service    = empty($this->service) ? "default" : $this->service;
                 
                 return true;
                 
@@ -204,13 +204,13 @@ 
     
     private function evalUri($parameters, $bits) {
         
-        $count  = 0;
+        $count = 0;
         
         foreach ($parameters as $key => $value) {
             
             if (isset($bits[$key])) {
                 
-                if (preg_match('/^' . $value['regex'] . '$/', $bits[$key], $matches)) {
+                if (preg_match('/^'.$value['regex'].'$/', $bits[$key], $matches)) {
                     
                     if (count($matches) == 1) $matches = $matches[0];
                     

Doc Comments +3 added lines

@@ -209,6 +209,9 @@
 
     }
 
+    /**
+     * @param string[] $bits
+     */
     private function evalUri($parameters, $bits) {
 
         $count  = 0;