Pleets / dronephp

src/Network/Http.php

Braces +3 added lines, -2 removed lines

@@ -127,8 +127,9 @@
     {
         $codes = $this->httpStatusCodes;
 
-        if (!in_array($code, array_keys($codes)))
-            throw new \RuntimeException("Status code not supported");
+        if (!in_array($code, array_keys($codes))) {
+                    throw new \RuntimeException("Status code not supported");
+        }
 
         return $this->httpStatusCodes[$code];
     }

test/Network/HttpTest.php

Braces +2 added lines, -4 removed lines

@@ -56,13 +56,11 @@
         try
         {
             $text = $http->getStatusText(65431);
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             $errorObject = ($e instanceof \RuntimeException);
             $message = $e->getMessage();
-        }
-        finally
+        } finally
         {
             $this->assertTrue($errorObject, $message);
         }

src/Util/DateTime.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
                 break;
 
             case self::LAST_YEAR:
-                $year = ((int) date('Y'))- 1;
+                $year = ((int) date('Y')) - 1;
                 break;
 
             default:
@@ -142,7 +142,7 @@ 
                 $d->setDate($year, (int) $d->format('m'), (int) $d->format('d'));
             }
 
-            $d->add(new \DateInterval('P'.($day - 1).'D'));
+            $d->add(new \DateInterval('P' . ($day - 1) . 'D'));
         }
 
         return $d;

Braces +18 added lines, -14 removed lines

@@ -63,8 +63,9 @@ 
      */
     public static function getMonth($month_number)
     {
-        if (!array_key_exists($month_number, self::MONTHS))
-            throw new Exception\MonthOutOfRange("Invalid Month");
+        if (!array_key_exists($month_number, self::MONTHS)) {
+                    throw new Exception\MonthOutOfRange("Invalid Month");
+        }
 
         return self::MONTHS[$month_number];
     }
@@ -78,14 +79,17 @@ 
      */
     public static function create(array $date = [])
     {
-        if (!array_key_exists('day', $date))
-            $date["day"] = self::TODAY;
+        if (!array_key_exists('day', $date)) {
+                    $date["day"] = self::TODAY;
+        }
 
-        if (!array_key_exists('month', $date))
-            $date["month"] = self::THIS_MONTH;
+        if (!array_key_exists('month', $date)) {
+                    $date["month"] = self::THIS_MONTH;
+        }
 
-        if (!array_key_exists('year', $date))
-            $date["year"] = self::THIS_YEAR;
+        if (!array_key_exists('year', $date)) {
+                    $date["year"] = self::THIS_YEAR;
+        }
 
         switch ($date['year'])
         {
@@ -130,13 +134,13 @@ 
                 break;
         }
 
-        if (is_string($day))
-            $d = new \DateTime("$day $month $year");
-        else
+        if (is_string($day)) {
+                    $d = new \DateTime("$day $month $year");
+        } else
         {
-            if (in_array($month, self::MONTHS))
-                $d = new \DateTime("first day of $month $year");
-            else
+            if (in_array($month, self::MONTHS)) {
+                            $d = new \DateTime("first day of $month $year");
+            } else
             {
                 $d = new \DateTime("first day of $month");
                 $d->setDate($year, (int) $d->format('m'), (int) $d->format('d'));

src/Util/ArrayDimension.php

Braces +10 added lines, -8 removed lines

@@ -39,9 +39,9 @@ 
                     $again = true;
                     $new_config[$param . $glue . $key] = $value;
                 }
+            } else {
+                            $new_config[$param] = $configure;
             }
-            else
-                $new_config[$param] = $configure;
         }
 
         return (!$again) ? $new_config : self::toUnidimensional($new_config, $glue);
@@ -64,15 +64,17 @@ 
 
         do
         {
-            if (array_key_exists($key, $haystack))
-                $haystack = $haystack[$key];
-            else
-                return $value;
+            if (array_key_exists($key, $haystack)) {
+                            $haystack = $haystack[$key];
+            } else {
+                            return $value;
+            }
 
             $key = count($needle) ? array_shift($needle) : NULL;
 
-            if (is_null($key))
-                return $haystack;
+            if (is_null($key)) {
+                            return $haystack;
+            }
 
         } while (!is_null($key));
     }

test/Error/ErrorTraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
     public function getContentsFromUrl($filename)
     {
         $headers = get_headers($url);
-        $exists = stripos($headers[0],"200 OK") ? true : false;
+        $exists = stripos($headers[0], "200 OK") ? true : false;
 
         if ($exists)
             return file_get_contents($filename);

Braces +17 added lines, -16 removed lines

@@ -90,13 +90,11 @@ 
         try
         {
             $ctrl->addNonExistingError();
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             $errorObject = ($e instanceof \LogicException);
             $message = $e->getMessage();
-        }
-        finally
+        } finally
         {
             $this->assertTrue($errorObject, $message);
         }
@@ -109,18 +107,20 @@ 
 
     public function getContentsFromFile($filename)
     {
-        if (file_exists($filename))
-            return file_get_contents($filename);
-        else
-            $this->error(Errno::FILE_NOT_FOUND, $filename);
+        if (file_exists($filename)) {
+                    return file_get_contents($filename);
+        } else {
+                    $this->error(Errno::FILE_NOT_FOUND, $filename);
+        }
     }
 
     public function getContentsFromFileWithoutFilenameOnError($filename)
     {
-        if (file_exists($filename))
-            return file_get_contents($filename);
-        else
-            $this->error(Errno::FILE_NOT_FOUND);
+        if (file_exists($filename)) {
+                    return file_get_contents($filename);
+        } else {
+                    $this->error(Errno::FILE_NOT_FOUND);
+        }
     }
 
     public function getContentsFromUrl($filename)
@@ -128,10 +128,11 @@ 
         $headers = get_headers($url);
         $exists = stripos($headers[0],"200 OK") ? true : false;
 
-        if ($exists)
-            return file_get_contents($filename);
-        else
-            $this->error("URL not found");
+        if ($exists) {
+                    return file_get_contents($filename);
+        } else {
+                    $this->error("URL not found");
+        }
     }
 
     public function addNonExistingError()

src/Error/ErrorTrait.php

Braces +17 added lines, -13 removed lines

@@ -92,15 +92,16 @@ 
      */
     protected function _error($code, $message = null)
     {
-        if (!is_null($code) && !is_integer($code))
-            throw new \InvalidArgumentException("Invalid type given. Integer expected");
+        if (!is_null($code) && !is_integer($code)) {
+                    throw new \InvalidArgumentException("Invalid type given. Integer expected");
+        }
 
-        if (is_null($code))
-            $code = preg_replace('/=|\/|\+/', "", base64_encode($message));
-        else
+        if (is_null($code)) {
+                    $code = preg_replace('/=|\/|\+/', "", base64_encode($message));
+        } else
         {
-            if (!array_key_exists($code, $this->standardErrors) && empty($message))
-                /*
+            if (!array_key_exists($code, $this->standardErrors) && empty($message)) {
+                            /*
                  * "This kind of exception should lead directly to a fix in your code"
                  * Non-standard errors must have a message to describe the error, make sure
                  * you execute the error() method with a message as the second parameter.
@@ -108,16 +109,18 @@ 
                  * Ref: http://php.net/manual/en/class.logicexception.php
                  */
                 throw new \LogicException('The message does not be empty in non-standard errors!');
+            }
         }
 
-        if (!array_key_exists($code, $this->errors))
-            $this->errors[$code] = (array_key_exists($code, $this->standardErrors))
+        if (!array_key_exists($code, $this->errors)) {
+                    $this->errors[$code] = (array_key_exists($code, $this->standardErrors))
                 ?
                     is_null($message)
                         ? preg_replace('/\s\'%[a-zA-Z]*%\'/', $message, $this->standardErrors[$code])
                         # if $message is not null it will replace the %file% wildcard
                         : preg_replace('/%[a-zA-Z]*%/', $message, $this->standardErrors[$code])
                 : $message;
+        }
     }
 
     function __call($method, $arguments)
@@ -127,10 +130,11 @@ 
             switch (count($arguments))
             {
                 case 1:
-                    if (is_integer($arguments[0]))
-                        return call_user_func([$this, '_error'], array_shift($arguments));
-                    else
-                        return call_user_func([$this, '_error'], null, array_shift($arguments));
+                    if (is_integer($arguments[0])) {
+                                            return call_user_func([$this, '_error'], array_shift($arguments));
+                    } else {
+                                            return call_user_func([$this, '_error'], null, array_shift($arguments));
+                    }
                     break;
                 case 2:
                     return call_user_func([$this, '_error'], $arguments[0], $arguments[1]);

test/Util/DateTimeTest.php

Braces +2 added lines, -4 removed lines

@@ -39,13 +39,11 @@
         try
         {
             $this->assertEquals('June', Dt::getMonth(13));
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             $errorObject = ($e instanceof MonthOutOfRange);
             $message = $e->getMessage();
-        }
-        finally
+        } finally
         {
             $this->assertTrue($errorObject, $message);
         }