Passed
Push — master ( c6e4cd...5de612 )
by Darío
01:48
created
test/FileSystem/ShellTest.php 2 patches
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -141,13 +141,11 @@
 block discarded – undo
141 141
         try
142 142
         {
143 143
             $shell->cp('foo2', 'foo3');
144
-        }
145
-        catch (\Exception $e)
144
+        } catch (\Exception $e)
146 145
         {
147 146
             # omitting directory
148 147
             $errorObject = ($e instanceof \RuntimeException);
149
-        }
150
-        finally
148
+        } finally
151 149
         {
152 150
             $this->assertTrue($errorObject, $e->getMessage());
153 151
         }
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -194,7 +194,7 @@
 block discarded – undo
194 194
         $this->assertTrue(is_dir('foo4/foo2'));
195 195
         $this->assertTrue(file_exists('foo4/foo2/new3.txt'));
196 196
         $this->assertTrue(file_exists('foo4/foo2/new4.txt'));
197
-   }
197
+    }
198 198
 
199 199
     /**
200 200
      * Tests removing files
Please login to merge, or discard this patch.
src/FileSystem/Shell.php 2 patches
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -92,28 +92,28 @@  discard block
 block discarded – undo
92 92
             {
93 93
                 foreach ($contents as $i)
94 94
                 {
95
-                    if (is_file($directory.'/'.$i))
95
+                    if (is_file($directory . '/' . $i))
96 96
                     {
97 97
                         $allContents[] = $i;
98 98
 
99
-                        $this->buffer = $directory.'/'.$i;
99
+                        $this->buffer = $directory . '/' . $i;
100 100
                         call_user_func($fileCallback, $this);
101 101
                     }
102
-                    else if (is_dir($directory.'/'.$i))
102
+                    else if (is_dir($directory . '/' . $i))
103 103
                     {
104
-                        $directory_name = basename($directory).'/'.$i;
104
+                        $directory_name = basename($directory) . '/' . $i;
105 105
 
106 106
                         if (strpos($directory_name, './') === 0)
107 107
                         {
108 108
                             $from = './';
109
-                            $from = '/'.preg_quote($from, '/').'/';
109
+                            $from = '/' . preg_quote($from, '/') . '/';
110 110
                             $directory_name = preg_replace($from, "", $directory_name, 1);
111 111
                         }
112 112
 
113 113
                         $allContents[$directory_name] =
114
-                            $this->getContents($directory.'/'.$i, $fileCallback, $dirCallback);
114
+                            $this->getContents($directory . '/' . $i, $fileCallback, $dirCallback);
115 115
 
116
-                        $this->buffer = $directory.'/'.$i;
116
+                        $this->buffer = $directory . '/' . $i;
117 117
                         call_user_func($dirCallback, $this);
118 118
                     }
119 119
                 }
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
                 if (!empty($path))
188 188
                     if (!strlen(stristr($item, $path)) > 0)
189 189
                         continue;
190
-                if (strstr($item,'~') === false && $item != '.' && $item != '..')
190
+                if (strstr($item, '~') === false && $item != '.' && $item != '..')
191 191
                     $filesToReturn[] = $item;
192 192
             }
193 193
         }
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
                     {
332 332
                         foreach ($files["folders"] as $folder)
333 333
                         {
334
-                            if (!file_exists($dest.'/'.$folder))
334
+                            if (!file_exists($dest . '/' . $folder))
335 335
                                 mkdir("$dest/$folder", 0777, true);
336 336
                         }
337 337
 
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
                             foreach ($files["files"] as $item)
341 341
                             {
342 342
                                 if (!file_exists("$dest/$item"))
343
-                                    copy($item, $dest.'/'.$item);
343
+                                    copy($item, $dest . '/' . $item);
344 344
                             }
345 345
                         }
346 346
                     }
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
                         array_shift($files["folders"]);
351 351
 
352 352
                         $from = "$file/";
353
-                        $from = '/'.preg_quote($from, '/').'/';
353
+                        $from = '/' . preg_quote($from, '/') . '/';
354 354
 
355 355
                         if (count($files["folders"]))
356 356
                         {
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
                             {
359 359
                                 $_folder = preg_replace($from, "", $folder, 1);
360 360
 
361
-                                if (!file_exists($dest.'/'.$_folder))
361
+                                if (!file_exists($dest . '/' . $_folder))
362 362
                                     mkdir("$dest/$_folder", 0777, true);
363 363
                             }
364 364
                         }
@@ -370,7 +370,7 @@  discard block
 block discarded – undo
370 370
                                 $_item = preg_replace($from, "", $item, 1);
371 371
 
372 372
                                 if (!file_exists("$dest/$_item"))
373
-                                    copy($item, $dest.'/'.$_item);
373
+                                    copy($item, $dest . '/' . $_item);
374 374
                             }
375 375
                         }
376 376
                     }
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
 
381 381
         }
382 382
         else if (is_dir($dest))
383
-            copy($file, $dest.'/'.$file);
383
+            copy($file, $dest . '/' . $file);
384 384
         else
385 385
             copy($file, $dest);
386 386
 
@@ -401,12 +401,12 @@  discard block
 block discarded – undo
401 401
             throw new \InvalidArgumentException("Missing parameters");
402 402
 
403 403
         if (is_dir($newfile))
404
-            $newfile .= '/'.basename($oldfile);
404
+            $newfile .= '/' . basename($oldfile);
405 405
 
406 406
         if ($oldfile == $newfile)
407 407
             throw new \Exception("'$oldfile' and '$newfile' are the same file");
408 408
 
409
-        if(!rename($oldfile, $newfile))
409
+        if (!rename($oldfile, $newfile))
410 410
             return false;
411 411
 
412 412
         return true;
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
         else {
437 437
             if (!is_dir($dir))
438 438
             {
439
-                if(!mkdir("$dir", 0777))
439
+                if (!mkdir("$dir", 0777))
440 440
                     return false;
441 441
             }
442 442
         }
Please login to merge, or discard this patch.
Braces   +99 added lines, -78 removed lines patch added patch discarded remove patch
@@ -84,8 +84,9 @@  discard block
 block discarded – undo
84 84
         {
85 85
             foreach ($this->ls($directory) as $item)
86 86
             {
87
-                if ($item != '.' && $item != '..')
88
-                    $contents[] = $item;
87
+                if ($item != '.' && $item != '..') {
88
+                                    $contents[] = $item;
89
+                }
89 90
             }
90 91
 
91 92
             if (count($contents) > 0)
@@ -98,8 +99,7 @@  discard block
 block discarded – undo
98 99
 
99 100
                         $this->buffer = $directory.'/'.$i;
100 101
                         call_user_func($fileCallback, $this);
101
-                    }
102
-                    else if (is_dir($directory.'/'.$i))
102
+                    } else if (is_dir($directory.'/'.$i))
103 103
                     {
104 104
                         $directory_name = basename($directory).'/'.$i;
105 105
 
@@ -118,14 +118,15 @@  discard block
 block discarded – undo
118 118
                     }
119 119
                 }
120 120
             }
121
+        } else if (is_file($directory)) {
122
+                    throw new \InvalidArgumentException("'$directory' is actually a file");
123
+        } else {
124
+                    throw new \InvalidArgumentException("The directory '$directory' does not exists");
121 125
         }
122
-        else if (is_file($directory))
123
-            throw new \InvalidArgumentException("'$directory' is actually a file");
124
-        else
125
-            throw new \InvalidArgumentException("The directory '$directory' does not exists");
126 126
 
127
-        if (!is_null($callback))
128
-            call_user_func($callback, $this);
127
+        if (!is_null($callback)) {
128
+                    call_user_func($callback, $this);
129
+        }
129 130
 
130 131
         return $allContents;
131 132
     }
@@ -137,10 +138,11 @@  discard block
 block discarded – undo
137 138
      */
138 139
     public function pwd()
139 140
     {
140
-        if (getcwd())
141
-            $this->buffer = getcwd();
142
-        else
143
-            return false;
141
+        if (getcwd()) {
142
+                    $this->buffer = getcwd();
143
+        } else {
144
+                    return false;
145
+        }
144 146
 
145 147
         return $this->buffer;
146 148
     }
@@ -159,36 +161,38 @@  discard block
 block discarded – undo
159 161
 
160 162
         $path = (is_null($path) || empty($path)) ? '.' : $path;
161 163
 
162
-        if (is_file($path))
163
-            $filesToReturn = array($path);
164
-        else if (is_dir($path))
164
+        if (is_file($path)) {
165
+                    $filesToReturn = array($path);
166
+        } else if (is_dir($path))
165 167
         {
166 168
             $pathIns = dir($path);
167 169
 
168
-            if ($recursive)
169
-                $filesToReturn = $this->getContents($path);
170
-            else
170
+            if ($recursive) {
171
+                            $filesToReturn = $this->getContents($path);
172
+            } else
171 173
             {
172 174
                 while (($item = $pathIns->read()) !== false)
173 175
                 {
174
-                    if ($item != '.' && $item != '..')
175
-                        $filesToReturn[] = $item;
176
+                    if ($item != '.' && $item != '..') {
177
+                                            $filesToReturn[] = $item;
178
+                    }
176 179
                 }
177 180
 
178 181
                 $pathIns->close();
179 182
             }
180
-        }
181
-        else {
183
+        } else {
182 184
 
183 185
             $contents = $this->ls();
184 186
 
185 187
             foreach ($contents as $item)
186 188
             {
187
-                if (!empty($path))
188
-                    if (!strlen(stristr($item, $path)) > 0)
189
+                if (!empty($path)) {
190
+                                    if (!strlen(stristr($item, $path)) > 0)
189 191
                         continue;
190
-                if (strstr($item,'~') === false && $item != '.' && $item != '..')
191
-                    $filesToReturn[] = $item;
192
+                }
193
+                if (strstr($item,'~') === false && $item != '.' && $item != '..') {
194
+                                    $filesToReturn[] = $item;
195
+                }
192 196
             }
193 197
         }
194 198
 
@@ -208,8 +212,9 @@  discard block
 block discarded – undo
208 212
 
209 213
         if (is_dir($path))
210 214
         {
211
-            if (chdir($path))
212
-                return true;
215
+            if (chdir($path)) {
216
+                            return true;
217
+            }
213 218
         }
214 219
 
215 220
         return false;
@@ -249,12 +254,13 @@  discard block
 block discarded – undo
249 254
      */
250 255
     public function rm($file, $recursive = false)
251 256
     {
252
-        if (is_null($file))
253
-            throw new \InvalidArgumentException("Missing first parameter");
257
+        if (is_null($file)) {
258
+                    throw new \InvalidArgumentException("Missing first parameter");
259
+        }
254 260
 
255
-        if (file_exists($file) && !$recursive)
256
-            unlink($file);
257
-        elseif (is_dir($file) && $recursive)
261
+        if (file_exists($file) && !$recursive) {
262
+                    unlink($file);
263
+        } elseif (is_dir($file) && $recursive)
258 264
         {
259 265
             $that = $this;
260 266
 
@@ -291,11 +297,13 @@  discard block
 block discarded – undo
291 297
      */
292 298
     public function cp($file, $dest, $recursive = false)
293 299
     {
294
-        if (empty($file) || empty($dest))
295
-            throw new \InvalidArgumentException("Missing parameters");
300
+        if (empty($file) || empty($dest)) {
301
+                    throw new \InvalidArgumentException("Missing parameters");
302
+        }
296 303
 
297
-        if (is_dir($file) && !$recursive)
298
-            throw new \RuntimeException("Ommiting directory <<$foo>>");
304
+        if (is_dir($file) && !$recursive) {
305
+                    throw new \RuntimeException("Ommiting directory <<$foo>>");
306
+        }
299 307
 
300 308
         $dest_exists = file_exists($dest);
301 309
 
@@ -308,8 +316,9 @@  discard block
 block discarded – undo
308 316
 
309 317
             $files["folders"][] = is_dir($dest) ? basename($file) : $dest;
310 318
 
311
-            if (!$dest_exists)
312
-                mkdir($dest);
319
+            if (!$dest_exists) {
320
+                            mkdir($dest);
321
+            }
313 322
 
314 323
             $that = $this;
315 324
 
@@ -331,20 +340,21 @@  discard block
 block discarded – undo
331 340
                     {
332 341
                         foreach ($files["folders"] as $folder)
333 342
                         {
334
-                            if (!file_exists($dest.'/'.$folder))
335
-                                mkdir("$dest/$folder", 0777, true);
343
+                            if (!file_exists($dest.'/'.$folder)) {
344
+                                                            mkdir("$dest/$folder", 0777, true);
345
+                            }
336 346
                         }
337 347
 
338 348
                         if (count($files["files"]))
339 349
                         {
340 350
                             foreach ($files["files"] as $item)
341 351
                             {
342
-                                if (!file_exists("$dest/$item"))
343
-                                    copy($item, $dest.'/'.$item);
352
+                                if (!file_exists("$dest/$item")) {
353
+                                                                    copy($item, $dest.'/'.$item);
354
+                                }
344 355
                             }
345 356
                         }
346
-                    }
347
-                    else
357
+                    } else
348 358
                     {
349 359
                         # directory previoulsy created
350 360
                         array_shift($files["folders"]);
@@ -358,8 +368,9 @@  discard block
 block discarded – undo
358 368
                             {
359 369
                                 $_folder = preg_replace($from, "", $folder, 1);
360 370
 
361
-                                if (!file_exists($dest.'/'.$_folder))
362
-                                    mkdir("$dest/$_folder", 0777, true);
371
+                                if (!file_exists($dest.'/'.$_folder)) {
372
+                                                                    mkdir("$dest/$_folder", 0777, true);
373
+                                }
363 374
                             }
364 375
                         }
365 376
 
@@ -369,8 +380,9 @@  discard block
 block discarded – undo
369 380
                             {
370 381
                                 $_item = preg_replace($from, "", $item, 1);
371 382
 
372
-                                if (!file_exists("$dest/$_item"))
373
-                                    copy($item, $dest.'/'.$_item);
383
+                                if (!file_exists("$dest/$_item")) {
384
+                                                                    copy($item, $dest.'/'.$_item);
385
+                                }
374 386
                             }
375 387
                         }
376 388
                     }
@@ -378,11 +390,11 @@  discard block
 block discarded – undo
378 390
             );
379 391
 
380 392
 
393
+        } else if (is_dir($dest)) {
394
+                    copy($file, $dest.'/'.$file);
395
+        } else {
396
+                    copy($file, $dest);
381 397
         }
382
-        else if (is_dir($dest))
383
-            copy($file, $dest.'/'.$file);
384
-        else
385
-            copy($file, $dest);
386 398
 
387 399
         return true;
388 400
     }
@@ -397,17 +409,21 @@  discard block
 block discarded – undo
397 409
      */
398 410
     public function mv($oldfile, $newfile)
399 411
     {
400
-        if (empty($oldfile) || empty($newfile))
401
-            throw new \InvalidArgumentException("Missing parameters");
412
+        if (empty($oldfile) || empty($newfile)) {
413
+                    throw new \InvalidArgumentException("Missing parameters");
414
+        }
402 415
 
403
-        if (is_dir($newfile))
404
-            $newfile .= '/'.basename($oldfile);
416
+        if (is_dir($newfile)) {
417
+                    $newfile .= '/'.basename($oldfile);
418
+        }
405 419
 
406
-        if ($oldfile == $newfile)
407
-            throw new \Exception("'$oldfile' and '$newfile' are the same file");
420
+        if ($oldfile == $newfile) {
421
+                    throw new \Exception("'$oldfile' and '$newfile' are the same file");
422
+        }
408 423
 
409
-        if(!rename($oldfile, $newfile))
410
-            return false;
424
+        if(!rename($oldfile, $newfile)) {
425
+                    return false;
426
+        }
411 427
 
412 428
         return true;
413 429
     }
@@ -423,21 +439,24 @@  discard block
 block discarded – undo
423 439
      */
424 440
     public function mkdir($dir, $dest = null, $recursive = null)
425 441
     {
426
-        if (empty($dir))
427
-            throw new \InvalidArgumentException("Missing first parameter");
442
+        if (empty($dir)) {
443
+                    throw new \InvalidArgumentException("Missing first parameter");
444
+        }
428 445
 
429
-        if (empty($dest))
430
-            $dest = '.';
446
+        if (empty($dest)) {
447
+                    $dest = '.';
448
+        }
431 449
 
432 450
         $recursive = (is_null($recursive)) ? false : $recursive;
433 451
 
434
-        if ($recursive)
435
-            mkdir("$dest/$dir", 0777, true);
436
-        else {
452
+        if ($recursive) {
453
+                    mkdir("$dest/$dir", 0777, true);
454
+        } else {
437 455
             if (!is_dir($dir))
438 456
             {
439
-                if(!mkdir("$dir", 0777))
440
-                    return false;
457
+                if(!mkdir("$dir", 0777)) {
458
+                                    return false;
459
+                }
441 460
             }
442 461
         }
443 462
         return true;
@@ -452,12 +471,14 @@  discard block
 block discarded – undo
452 471
      */
453 472
     public function rmdir($dir)
454 473
     {
455
-        if (is_null($dir) || empty($dir))
456
-            throw new \RuntimeException("Missing first parameter");
474
+        if (is_null($dir) || empty($dir)) {
475
+                    throw new \RuntimeException("Missing first parameter");
476
+        }
457 477
 
458
-        if (rmdir($dir))
459
-            return true;
460
-        else
461
-            return false;
478
+        if (rmdir($dir)) {
479
+                    return true;
480
+        } else {
481
+                    return false;
482
+        }
462 483
     }
463 484
 }
464 485
\ No newline at end of file
Please login to merge, or discard this patch.
src/Network/Socket/Server.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
     public function listen(Array $eventHandlers = array())
75 75
     {
76 76
         $event = $eventHandlers;
77
-        $clousure = function(){};
77
+        $clousure = function() {};
78 78
 
79 79
         if (!array_key_exists('success', $event))
80 80
             $event["success"] = $clousure;
Please login to merge, or discard this patch.
Braces   +10 added lines, -8 removed lines patch added patch discarded remove patch
@@ -76,11 +76,13 @@  discard block
 block discarded – undo
76 76
         $event = $eventHandlers;
77 77
         $clousure = function(){};
78 78
 
79
-        if (!array_key_exists('success', $event))
80
-            $event["success"] = $clousure;
79
+        if (!array_key_exists('success', $event)) {
80
+                    $event["success"] = $clousure;
81
+        }
81 82
 
82
-        if (!array_key_exists('error', $event))
83
-            $event["error"] = $clousure;
83
+        if (!array_key_exists('error', $event)) {
84
+                    $event["error"] = $clousure;
85
+        }
84 86
 
85 87
         $listener = false;
86 88
 
@@ -90,8 +92,7 @@  discard block
 block discarded – undo
90 92
             $this->error($errno, socket_strerror($errno));
91 93
 
92 94
             throw new \RuntimeException("Could not set socket to listen");
93
-        }
94
-        else {
95
+        } else {
95 96
 
96 97
             echo "\n";
97 98
             echo "Server Started : " . date('Y-m-d H:i:s') . "\n";
@@ -114,8 +115,9 @@  discard block
 block discarded – undo
114 115
             socket_close($spawn);
115 116
         }
116 117
 
117
-        if (!$listener)
118
-            call_user_func($event["error"], $this->getLastError());
118
+        if (!$listener) {
119
+                    call_user_func($event["error"], $this->getLastError());
120
+        }
119 121
 
120 122
         return $listener;
121 123
     }
Please login to merge, or discard this patch.
test/Db/Driver/MySQLTest.php 2 patches
Braces   +9 added lines, -18 removed lines patch added patch discarded remove patch
@@ -84,12 +84,10 @@  discard block
 block discarded – undo
84 84
 
85 85
         try {
86 86
             $conn->disconnect();
87
-        }
88
-        catch (\Exception $e)
87
+        } catch (\Exception $e)
89 88
         {
90 89
             $errorObject = ($e instanceof \LogicException);
91
-        }
92
-        finally
90
+        } finally
93 91
         {
94 92
             $this->assertNotTrue($conn->isConnected());
95 93
             $this->assertTrue($errorObject, $e->getMessage());
@@ -125,12 +123,10 @@  discard block
 block discarded – undo
125 123
 
126 124
         try {
127 125
             $conn->reconnect();
128
-        }
129
-        catch (\Exception $e)
126
+        } catch (\Exception $e)
130 127
         {
131 128
             $errorObject = ($e instanceof \LogicException);
132
-        }
133
-        finally
129
+        } finally
134 130
         {
135 131
             $this->assertTrue($errorObject, $e->getMessage());
136 132
             $this->assertNotTrue($conn->isConnected());
@@ -156,13 +152,11 @@  discard block
 block discarded – undo
156 152
         try
157 153
         {
158 154
             $conn->connect();
159
-        }
160
-        catch (\Exception $e)
155
+        } catch (\Exception $e)
161 156
         {
162 157
             $errorObject = ($e instanceof ConnectionException);
163 158
             $message = $e->getMessage();
164
-        }
165
-        finally
159
+        } finally
166 160
         {
167 161
             $this->assertTrue($errorObject, $message);
168 162
             $this->assertNotTrue($conn->isConnected());
@@ -242,13 +236,11 @@  discard block
 block discarded – undo
242 236
         {
243 237
             $sql = "INSERT INTO MYTABLE (DESCRIPTION, WRONG) VALUES ('Hello world!')";
244 238
             $conn->execute($sql);
245
-        }
246
-        catch (\Exception $e)
239
+        } catch (\Exception $e)
247 240
         {
248 241
             $errorObject = ($e instanceof InvalidQueryException);
249 242
             $message = $e->getMessage();
250
-        }
251
-        finally
243
+        } finally
252 244
         {
253 245
             $this->assertTrue($errorObject, $message);
254 246
         }
@@ -517,8 +509,7 @@  discard block
 block discarded – undo
517 509
 
518 510
             $sql = "INSERT INTO MYTABLE (DESCRIPTION, WRONG) VALUES ('TRANS_SHORTCUT_2')";
519 511
             $conn->execute($sql);
520
-        }
521
-        catch (InvalidQueryException $e)
512
+        } catch (InvalidQueryException $e)
522 513
         {
523 514
             $message = $e->getMessage();
524 515
             #·not necessary!
Please login to merge, or discard this patch.
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
     public function testCannotStablishConnection()
146 146
     {
147 147
         $options = $this->options;
148
-        $options["dbhost"] = 'myserver';   // this server does not exists
148
+        $options["dbhost"] = 'myserver'; // this server does not exists
149 149
 
150 150
         $conn = new MySQL($options);
151 151
 
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
         $this->assertEquals(2, $conn->getNumFields());
275 275
         $this->assertEquals(0, $conn->getRowsAffected());
276 276
 
277
-        $rowset = $conn->getArrayResult();    # array with results
277
+        $rowset = $conn->getArrayResult(); # array with results
278 278
         $row = array_shift($rowset);
279 279
 
280 280
         $this->assertArrayHasKey("ID", $row);
@@ -301,12 +301,12 @@  discard block
 block discarded – undo
301 301
         $conn->execute($sql);
302 302
         $rowcount = count($conn->getArrayResult());
303 303
 
304
-        $this->assertTrue(($rowcount === 1));    # the row is available for now
304
+        $this->assertTrue(($rowcount === 1)); # the row is available for now
305 305
 
306 306
         # properties modified by execute() method
307 307
         $this->assertEquals(1, $conn->getNumRows());
308 308
         $this->assertEquals(2, $conn->getNumFields());
309
-        $this->assertEquals(0, $conn->getRowsAffected());    # nothing affected (autocommit = false)
309
+        $this->assertEquals(0, $conn->getRowsAffected()); # nothing affected (autocommit = false)
310 310
 
311 311
         $this->assertTrue($conn->commit());
312 312
 
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
         $conn->execute($sql);
316 316
         $rowcount = count($conn->getArrayResult());
317 317
 
318
-        $this->assertTrue(($rowcount === 1));    # the row is available
318
+        $this->assertTrue(($rowcount === 1)); # the row is available
319 319
     }
320 320
 
321 321
     /**
@@ -338,12 +338,12 @@  discard block
 block discarded – undo
338 338
         $conn->execute($sql);
339 339
         $rowcount = count($conn->getArrayResult());
340 340
 
341
-        $this->assertTrue(($rowcount === 1));    # the row is available for now
341
+        $this->assertTrue(($rowcount === 1)); # the row is available for now
342 342
 
343 343
         # properties modified by execute() method
344 344
         $this->assertEquals(1, $conn->getNumRows());
345 345
         $this->assertEquals(2, $conn->getNumFields());
346
-        $this->assertEquals(0, $conn->getRowsAffected());    # nothing affected (autocommit = false)
346
+        $this->assertEquals(0, $conn->getRowsAffected()); # nothing affected (autocommit = false)
347 347
 
348 348
         $this->assertTrue($conn->rollback());
349 349
 
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
         $conn->execute($sql);
353 353
         $rowcount = count($conn->getArrayResult());
354 354
 
355
-        $this->assertNotTrue(($rowcount === 1));    # the row is not available
355
+        $this->assertNotTrue(($rowcount === 1)); # the row is not available
356 356
     }
357 357
 
358 358
     /**
@@ -384,12 +384,12 @@  discard block
 block discarded – undo
384 384
         $conn->execute($sql);
385 385
         $rowcount = count($conn->getArrayResult());
386 386
 
387
-        $this->assertTrue(($rowcount === 4));    # the rows are available for now
387
+        $this->assertTrue(($rowcount === 4)); # the rows are available for now
388 388
 
389 389
         # properties modified by execute() method
390 390
         $this->assertEquals(4, $conn->getNumRows());
391 391
         $this->assertEquals(2, $conn->getNumFields());
392
-        $this->assertEquals(0, $conn->getRowsAffected());    # nothing affected (autocommit = false)
392
+        $this->assertEquals(0, $conn->getRowsAffected()); # nothing affected (autocommit = false)
393 393
 
394 394
         $this->assertTrue($conn->commit());
395 395
 
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
         $conn->execute($sql);
399 399
         $rowcount = count($conn->getArrayResult());
400 400
 
401
-        $this->assertTrue(($rowcount === 4));    # the rows are available
401
+        $this->assertTrue(($rowcount === 4)); # the rows are available
402 402
     }
403 403
 
404 404
     /**
@@ -430,12 +430,12 @@  discard block
 block discarded – undo
430 430
         $conn->execute($sql);
431 431
         $rowcount = count($conn->getArrayResult());
432 432
 
433
-        $this->assertTrue(($rowcount === 4));    # the rows are available for now
433
+        $this->assertTrue(($rowcount === 4)); # the rows are available for now
434 434
 
435 435
         # properties modified by execute() method
436 436
         $this->assertEquals(4, $conn->getNumRows());
437 437
         $this->assertEquals(2, $conn->getNumFields());
438
-        $this->assertEquals(0, $conn->getRowsAffected());    # nothing affected (autocommit = false)
438
+        $this->assertEquals(0, $conn->getRowsAffected()); # nothing affected (autocommit = false)
439 439
 
440 440
         $this->assertTrue($conn->rollback());
441 441
 
@@ -444,7 +444,7 @@  discard block
 block discarded – undo
444 444
         $conn->execute($sql);
445 445
         $rowcount = count($conn->getArrayResult());
446 446
 
447
-        $this->assertNotTrue(($rowcount === 4));    # the rows are not available
447
+        $this->assertNotTrue(($rowcount === 4)); # the rows are not available
448 448
     }
449 449
 
450 450
     /**
@@ -475,12 +475,12 @@  discard block
 block discarded – undo
475 475
         $conn->execute($sql);
476 476
         $rowcount = count($conn->getArrayResult());
477 477
 
478
-        $this->assertTrue(($rowcount === 2));    # the rows are available for now
478
+        $this->assertTrue(($rowcount === 2)); # the rows are available for now
479 479
 
480 480
         # properties modified by execute() method
481 481
         $this->assertEquals(2, $conn->getNumRows());
482 482
         $this->assertEquals(2, $conn->getNumFields());
483
-        $this->assertEquals(0, $conn->getRowsAffected());    # nothing affected (autocommit = false)
483
+        $this->assertEquals(0, $conn->getRowsAffected()); # nothing affected (autocommit = false)
484 484
 
485 485
         # ends the transaction
486 486
         $conn->endTransaction();
@@ -490,7 +490,7 @@  discard block
 block discarded – undo
490 490
         $conn->execute($sql);
491 491
         $rowcount = count($conn->getArrayResult());
492 492
 
493
-        $this->assertTrue(($rowcount === 2));    # the row is available
493
+        $this->assertTrue(($rowcount === 2)); # the row is available
494 494
     }
495 495
 
496 496
     /**
@@ -534,7 +534,7 @@  discard block
 block discarded – undo
534 534
         $conn->execute($sql);
535 535
         $rowcount = count($conn->getArrayResult());
536 536
 
537
-        $this->assertNotTrue(($rowcount === 0));    # the rows are not available
537
+        $this->assertNotTrue(($rowcount === 0)); # the rows are not available
538 538
 
539 539
         # remove all work
540 540
         $conn->execute("DROP TABLE MYTABLE");
Please login to merge, or discard this patch.
src/Network/Http.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -127,8 +127,9 @@
 block discarded – undo
127 127
     {
128 128
         $codes = $this->httpStatusCodes;
129 129
 
130
-        if (!in_array($code, array_keys($codes)))
131
-            throw new \RuntimeException("Status code not supported");
130
+        if (!in_array($code, array_keys($codes))) {
131
+                    throw new \RuntimeException("Status code not supported");
132
+        }
132 133
 
133 134
         return $this->httpStatusCodes[$code];
134 135
     }
Please login to merge, or discard this patch.
test/Network/HttpTest.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,13 +56,11 @@
 block discarded – undo
56 56
         try
57 57
         {
58 58
             $text = $http->getStatusText(65431);
59
-        }
60
-        catch (\Exception $e)
59
+        } catch (\Exception $e)
61 60
         {
62 61
             $errorObject = ($e instanceof \RuntimeException);
63 62
             $message = $e->getMessage();
64
-        }
65
-        finally
63
+        } finally
66 64
         {
67 65
             $this->assertTrue($errorObject, $message);
68 66
         }
Please login to merge, or discard this patch.
src/Util/DateTime.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
                 break;
95 95
 
96 96
             case self::LAST_YEAR:
97
-                $year = ((int) date('Y'))- 1;
97
+                $year = ((int) date('Y')) - 1;
98 98
                 break;
99 99
 
100 100
             default:
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
                 $d->setDate($year, (int) $d->format('m'), (int) $d->format('d'));
143 143
             }
144 144
 
145
-            $d->add(new \DateInterval('P'.($day - 1).'D'));
145
+            $d->add(new \DateInterval('P' . ($day - 1) . 'D'));
146 146
         }
147 147
 
148 148
         return $d;
Please login to merge, or discard this patch.
Braces   +18 added lines, -14 removed lines patch added patch discarded remove patch
@@ -63,8 +63,9 @@  discard block
 block discarded – undo
63 63
      */
64 64
     public static function getMonth($month_number)
65 65
     {
66
-        if (!array_key_exists($month_number, self::MONTHS))
67
-            throw new Exception\MonthOutOfRange("Invalid Month");
66
+        if (!array_key_exists($month_number, self::MONTHS)) {
67
+                    throw new Exception\MonthOutOfRange("Invalid Month");
68
+        }
68 69
 
69 70
         return self::MONTHS[$month_number];
70 71
     }
@@ -78,14 +79,17 @@  discard block
 block discarded – undo
78 79
      */
79 80
     public static function create(array $date = [])
80 81
     {
81
-        if (!array_key_exists('day', $date))
82
-            $date["day"] = self::TODAY;
82
+        if (!array_key_exists('day', $date)) {
83
+                    $date["day"] = self::TODAY;
84
+        }
83 85
 
84
-        if (!array_key_exists('month', $date))
85
-            $date["month"] = self::THIS_MONTH;
86
+        if (!array_key_exists('month', $date)) {
87
+                    $date["month"] = self::THIS_MONTH;
88
+        }
86 89
 
87
-        if (!array_key_exists('year', $date))
88
-            $date["year"] = self::THIS_YEAR;
90
+        if (!array_key_exists('year', $date)) {
91
+                    $date["year"] = self::THIS_YEAR;
92
+        }
89 93
 
90 94
         switch ($date['year'])
91 95
         {
@@ -130,13 +134,13 @@  discard block
 block discarded – undo
130 134
                 break;
131 135
         }
132 136
 
133
-        if (is_string($day))
134
-            $d = new \DateTime("$day $month $year");
135
-        else
137
+        if (is_string($day)) {
138
+                    $d = new \DateTime("$day $month $year");
139
+        } else
136 140
         {
137
-            if (in_array($month, self::MONTHS))
138
-                $d = new \DateTime("first day of $month $year");
139
-            else
141
+            if (in_array($month, self::MONTHS)) {
142
+                            $d = new \DateTime("first day of $month $year");
143
+            } else
140 144
             {
141 145
                 $d = new \DateTime("first day of $month");
142 146
                 $d->setDate($year, (int) $d->format('m'), (int) $d->format('d'));
Please login to merge, or discard this patch.
src/Util/ArrayDimension.php 1 patch
Braces   +10 added lines, -8 removed lines patch added patch discarded remove patch
@@ -39,9 +39,9 @@  discard block
 block discarded – undo
39 39
                     $again = true;
40 40
                     $new_config[$param . $glue . $key] = $value;
41 41
                 }
42
+            } else {
43
+                            $new_config[$param] = $configure;
42 44
             }
43
-            else
44
-                $new_config[$param] = $configure;
45 45
         }
46 46
 
47 47
         return (!$again) ? $new_config : self::toUnidimensional($new_config, $glue);
@@ -64,15 +64,17 @@  discard block
 block discarded – undo
64 64
 
65 65
         do
66 66
         {
67
-            if (array_key_exists($key, $haystack))
68
-                $haystack = $haystack[$key];
69
-            else
70
-                return $value;
67
+            if (array_key_exists($key, $haystack)) {
68
+                            $haystack = $haystack[$key];
69
+            } else {
70
+                            return $value;
71
+            }
71 72
 
72 73
             $key = count($needle) ? array_shift($needle) : NULL;
73 74
 
74
-            if (is_null($key))
75
-                return $haystack;
75
+            if (is_null($key)) {
76
+                            return $haystack;
77
+            }
76 78
 
77 79
         } while (!is_null($key));
78 80
     }
Please login to merge, or discard this patch.
test/Error/ErrorTraitTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -126,7 +126,7 @@
 block discarded – undo
126 126
     public function getContentsFromUrl($filename)
127 127
     {
128 128
         $headers = get_headers($url);
129
-        $exists = stripos($headers[0],"200 OK") ? true : false;
129
+        $exists = stripos($headers[0], "200 OK") ? true : false;
130 130
 
131 131
         if ($exists)
132 132
             return file_get_contents($filename);
Please login to merge, or discard this patch.
Braces   +17 added lines, -16 removed lines patch added patch discarded remove patch
@@ -90,13 +90,11 @@  discard block
 block discarded – undo
90 90
         try
91 91
         {
92 92
             $ctrl->addNonExistingError();
93
-        }
94
-        catch (\Exception $e)
93
+        } catch (\Exception $e)
95 94
         {
96 95
             $errorObject = ($e instanceof \LogicException);
97 96
             $message = $e->getMessage();
98
-        }
99
-        finally
97
+        } finally
100 98
         {
101 99
             $this->assertTrue($errorObject, $message);
102 100
         }
@@ -109,18 +107,20 @@  discard block
 block discarded – undo
109 107
 
110 108
     public function getContentsFromFile($filename)
111 109
     {
112
-        if (file_exists($filename))
113
-            return file_get_contents($filename);
114
-        else
115
-            $this->error(Errno::FILE_NOT_FOUND, $filename);
110
+        if (file_exists($filename)) {
111
+                    return file_get_contents($filename);
112
+        } else {
113
+                    $this->error(Errno::FILE_NOT_FOUND, $filename);
114
+        }
116 115
     }
117 116
 
118 117
     public function getContentsFromFileWithoutFilenameOnError($filename)
119 118
     {
120
-        if (file_exists($filename))
121
-            return file_get_contents($filename);
122
-        else
123
-            $this->error(Errno::FILE_NOT_FOUND);
119
+        if (file_exists($filename)) {
120
+                    return file_get_contents($filename);
121
+        } else {
122
+                    $this->error(Errno::FILE_NOT_FOUND);
123
+        }
124 124
     }
125 125
 
126 126
     public function getContentsFromUrl($filename)
@@ -128,10 +128,11 @@  discard block
 block discarded – undo
128 128
         $headers = get_headers($url);
129 129
         $exists = stripos($headers[0],"200 OK") ? true : false;
130 130
 
131
-        if ($exists)
132
-            return file_get_contents($filename);
133
-        else
134
-            $this->error("URL not found");
131
+        if ($exists) {
132
+                    return file_get_contents($filename);
133
+        } else {
134
+                    $this->error("URL not found");
135
+        }
135 136
     }
136 137
 
137 138
     public function addNonExistingError()
Please login to merge, or discard this patch.