Completed
Push — 4.0 ( b44693...cafdfc )
by Marco
02:39
created
src/Comodojo/Dispatcher/Request/Headers.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,9 @@
 block discarded – undo
41 41
      */
42 42
     private static function getHeaders() {
43 43
 
44
-        if ( function_exists('getallheaders') ) return getallheaders();
44
+        if ( function_exists('getallheaders') ) {
45
+            return getallheaders();
46
+        }
45 47
 
46 48
         $headers = array();
47 49
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Response/Status.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,9 @@
 block discarded – undo
62 62
 
63 63
     public function description($code=null) {
64 64
 
65
-        if ( is_null($code) ) return $this->codes->getMessage($this->status_code);
65
+        if ( is_null($code) ) {
66
+            return $this->codes->getMessage($this->status_code);
67
+        }
66 68
         
67 69
         return $this->codes->getMessage($code);
68 70
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Components/HttpStatusCodes.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,9 @@
 block discarded – undo
102 102
 
103 103
     public function getMessage($code) {
104 104
 
105
-        if ( $this->exists($code) ) return $this->codes[$code];
105
+        if ( $this->exists($code) ) {
106
+            return $this->codes[$code];
107
+        }
106 108
 
107 109
         throw new Exception("Invalid HTTP status code $code");
108 110
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Request/File.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -84,7 +84,9 @@
 block discarded – undo
84 84
 
85 85
         $file = $this->getTemporaryName();
86 86
 
87
-        if (file_exists($file)) return file_get_contents($file);
87
+        if (file_exists($file)) {
88
+            return file_get_contents($file);
89
+        }
88 90
 
89 91
         throw new Exception("File does not exists");
90 92
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Traits/HeadersTrait.php 1 patch
Braces   +10 added lines, -6 removed lines patch added patch discarded remove patch
@@ -28,11 +28,13 @@  discard block
 block discarded – undo
28 28
 
29 29
     public function get($header = null) {
30 30
 
31
-        if ( is_null($header) ) return $this->headers;
32
-
33
-        else if ( array_key_exists($header, $this->headers) ) return $this->headers[$header];
34
-
35
-        else return null;
31
+        if ( is_null($header) ) {
32
+            return $this->headers;
33
+        } else if ( array_key_exists($header, $this->headers) ) {
34
+            return $this->headers[$header];
35
+        } else {
36
+            return null;
37
+        }
36 38
 
37 39
     }
38 40
 
@@ -49,7 +51,9 @@  discard block
 block discarded – undo
49 51
 
50 52
             return self::headerToString($header, $this->headers[$header]);
51 53
 
52
-        } else return null;
54
+        } else {
55
+            return null;
56
+        }
53 57
 
54 58
     }
55 59
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Router/Route.php 1 patch
Braces   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -207,7 +207,10 @@
 block discarded – undo
207 207
                  */
208 208
                 if ( preg_match('/^' . $value['regex'] . '$/', $path[$key], $matches) ) {
209 209
 
210
-                    if ( count($matches) == 1 ) $matches = $matches[0]; // This is the case where no backreferences are present or available.
210
+                    if ( count($matches) == 1 ) {
211
+                        $matches = $matches[0];
212
+                    }
213
+                    // This is the case where no backreferences are present or available.
211 214
 
212 215
                     // The extracted value (with any backreference available) is added to the query parameters.
213 216
                     $this->setRequestParameter($key, $matches);
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Router/Table.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -156,7 +156,9 @@  discard block
 block discarded – undo
156 156
 
157 157
     private function readCache() {
158 158
 
159
-        if ( $this->configuration->get('routing-table-cache') !== true ) return;
159
+        if ( $this->configuration->get('routing-table-cache') !== true ) {
160
+            return;
161
+        }
160 162
 
161 163
         $data = $this->cache->read();
162 164
 
@@ -176,7 +178,9 @@  discard block
 block discarded – undo
176 178
 
177 179
     private function dumpCache() {
178 180
 
179
-        if ( $this->configuration->get('routing-table-cache') !== true ) return;
181
+        if ( $this->configuration->get('routing-table-cache') !== true ) {
182
+            return;
183
+        }
180 184
 
181 185
         $ttl = $this->configuration->get('routing-table-ttl');
182 186
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Router/Parser.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,9 @@
 block discarded – undo
96 96
                      */
97 97
                     $param_regex .= $this->param($key, $string, $value);
98 98
 
99
-                    if ($value->isQueryRequired($key)) $param_required = true;
99
+                    if ($value->isQueryRequired($key)) {
100
+                        $param_required = true;
101
+                    }
100 102
 
101 103
                     $this->logger->debug("Route parser - parameter regex: $param_regex");
102 104
 
Please login to merge, or discard this patch.
src/Comodojo/Dispatcher/Router/Model.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -122,7 +122,9 @@
 block discarded – undo
122 122
 
123 123
         if ( $this->bypass_routing === false ) {
124 124
 
125
-            if ( !$this->parse() ) throw new DispatcherException("Unable to find a valid route for the specified uri", 0, null, 404);
125
+            if ( !$this->parse() ) {
126
+                throw new DispatcherException("Unable to find a valid route for the specified uri", 0, null, 404);
127
+            }
126 128
 
127 129
         }
128 130
 
Please login to merge, or discard this patch.