Pleets / dronephp

src/Network/Rest/AbstractRest.php

Braces +6 added lines, -4 removed lines

@@ -127,8 +127,9 @@ 
      */
     public function setWhiteList(array $whiteList)
     {
-        if (empty($whiteList))
-            throw new \RuntimeException("Empty whitelist!");
+        if (empty($whiteList)) {
+                    throw new \RuntimeException("Empty whitelist!");
+        }
 
         $this->whiteList = $whiteList;
     }
@@ -156,8 +157,9 @@ 
     {
         foreach ($options as $option => $value)
         {
-            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set'.$option))
-            $this->{'set'.$option}($value);
+            if (property_exists(__CLASS__, strtolower($option)) && method_exists($this, 'set'.$option)) {
+                        $this->{'set'.$option}($value);
+            }
         }
 
         # HTTP instance

src/Network/Server.php

Braces +6 added lines, -4 removed lines

@@ -64,11 +64,13 @@
      */
     public static function getClientIP()
     {
-        if (!empty($_SERVER['HTTP_CLIENT_IP']))
-            return $_SERVER['HTTP_CLIENT_IP'];
+        if (!empty($_SERVER['HTTP_CLIENT_IP'])) {
+                    return $_SERVER['HTTP_CLIENT_IP'];
+        }
 
-        if (!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
-            return $_SERVER['HTTP_X_FORWARDED_FOR'];
+        if (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
+                    return $_SERVER['HTTP_X_FORWARDED_FOR'];
+        }
 
         return $_SERVER['REMOTE_ADDR'];
     }

src/Util/ParamTrait.php

Braces +3 added lines, -2 removed lines

@@ -70,8 +70,9 @@
     {
         $parameters = $this->getParams();
 
-        if (array_key_exists($param, $parameters))
-            return true;
+        if (array_key_exists($param, $parameters)) {
+                    return true;
+        }
 
         return false;
     }

src/Exception/Storage.php

Braces +9 added lines, -6 removed lines

@@ -76,17 +76,20 @@
             "object"  => serialize($exception)
         ];
 
-        if (!function_exists('mb_detect_encoding'))
-            throw new \RuntimeException("mbstring library is not installed!");
+        if (!function_exists('mb_detect_encoding')) {
+                    throw new \RuntimeException("mbstring library is not installed!");
+        }
 
         /*
          * Encodes to UTF8 all messages. It ensures JSON encoding.
          */
-        if (!mb_detect_encoding($data[$id]["message"], 'UTF-8', true))
-            $data[$id]["message"] = utf8_encode($data[$id]["message"]);
+        if (!mb_detect_encoding($data[$id]["message"], 'UTF-8', true)) {
+                    $data[$id]["message"] = utf8_encode($data[$id]["message"]);
+        }
 
-        if (!mb_detect_encoding($data[$id]["object"], 'UTF-8', true))
-            $data[$id]["object"] = utf8_decode($data[$id]["object"]);
+        if (!mb_detect_encoding($data[$id]["object"], 'UTF-8', true)) {
+                    $data[$id]["object"] = utf8_decode($data[$id]["object"]);
+        }
 
         if (($encoded_data = json_encode($data)) === false)
         {

src/Mvc/Router.php

Braces +11 added lines, -8 removed lines

@@ -145,8 +145,9 @@ 
         $module = (is_null($this->identifiers["module"]) || empty($this->identifiers["module"]))
                     ? $this->routes["defaults"]["module"] : $this->identifiers["module"];
 
-        if (!array_key_exists($module, $this->routes))
-            throw new Exception\ModuleNotFoundException("The key '$module' does not exists in routes!");
+        if (!array_key_exists($module, $this->routes)) {
+                    throw new Exception\ModuleNotFoundException("The key '$module' does not exists in routes!");
+        }
 
         $controller = (is_null($this->identifiers["controller"]) || empty($this->identifiers["controller"]))
                     ? $this->routes[$module]["controller"] : $this->identifiers["controller"];
@@ -156,10 +157,11 @@ 
 
         $fqn_controller = '\\' . $module . "\Controller\\" . $controller;
 
-        if (class_exists($fqn_controller))
-            $this->controller = new $fqn_controller($module, $view, $this->basePath);
-        else
-            throw new Exception\ControllerNotFoundException("The control class '$fqn_controller' does not exists!");
+        if (class_exists($fqn_controller)) {
+                    $this->controller = new $fqn_controller($module, $view, $this->basePath);
+        } else {
+                    throw new Exception\ControllerNotFoundException("The control class '$fqn_controller' does not exists!");
+        }
     }
 
     /**
@@ -176,8 +178,9 @@ 
         $key = array_keys($route);
         $key = array_shift($key);
 
-        if (array_key_exists($key, $this->routes))
-            throw new \LogicException("The key '$key' was already defined as route");
+        if (array_key_exists($key, $this->routes)) {
+                    throw new \LogicException("The key '$key' was already defined as route");
+        }
 
         $this->routes = array_merge($this->routes, $route);
     }

src/Mvc/AbstractionModule.php

Braces +3 added lines, -2 removed lines

@@ -80,7 +80,8 @@
 
         $class = "module/" . $module . "/source/" . implode("/", $nm) . ".php";
 
-        if (file_exists($class))
-            include $class;
+        if (file_exists($class)) {
+                    include $class;
+        }
     }
 }
\ No newline at end of file

src/Db/SQLFunction.php

Braces +3 added lines, -2 removed lines

@@ -77,8 +77,9 @@
 
         foreach ($arguments as $key => $arg)
         {
-            if (is_string($arg))
-                $arguments[$key] = "'$arg'";
+            if (is_string($arg)) {
+                            $arguments[$key] = "'$arg'";
+            }
         }
 
         $arguments = implode(", ", array_values($arguments));

src/Db/TableGateway/EntityAdapter.php

Braces +21 added lines, -16 removed lines

@@ -58,8 +58,9 @@ 
     {
         $result = $this->tableGateway->select($where);
 
-        if (!count($result))
-            return $result;
+        if (!count($result)) {
+                    return $result;
+        }
 
         $array_result = [];
 
@@ -69,8 +70,9 @@ 
 
             foreach ($row as $key => $value)
             {
-                if (is_string($key))
-                    $filtered_array[$key] = $value;
+                if (is_string($key)) {
+                                    $filtered_array[$key] = $value;
+                }
             }
 
             $user_entity = get_class($this->tableGateway->getEntity());
@@ -94,10 +96,11 @@ 
      */
     public function insert($entity)
     {
-        if ($entity instanceof Entity)
-            $entity = get_object_vars($entity);
-        else if (!is_array($entity))
-            throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
+        if ($entity instanceof Entity) {
+                    $entity = get_object_vars($entity);
+        } else if (!is_array($entity)) {
+                    throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
+        }
 
         $this->parseEntity($entity);
 
@@ -126,14 +129,15 @@ 
 
             foreach ($entity as $key => $value)
             {
-                if (in_array($key, $changedFields))
-                    $fieldsToModify[$key] = $value;
+                if (in_array($key, $changedFields)) {
+                                    $fieldsToModify[$key] = $value;
+                }
             }
 
             $entity = $fieldsToModify;
+        } else if (!is_array($entity)) {
+                    throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
         }
-        else if (!is_array($entity))
-            throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
 
         $this->parseEntity($entity);
 
@@ -152,10 +156,11 @@ 
      */
     public function delete($entity)
     {
-        if ($entity instanceof Entity)
-            $entity = get_object_vars($entity);
-        else if (!is_array($entity))
-            throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
+        if ($entity instanceof Entity) {
+                    $entity = get_object_vars($entity);
+        } else if (!is_array($entity)) {
+                    throw new \InvalidArgumentException("Invalid type given. Drone\Db\Entity or Array expected");
+        }
 
         return $this->tableGateway->delete($entity);
     }

src/Mvc/AbstractionController.php

Braces +32 added lines, -24 removed lines

@@ -150,8 +150,9 @@ 
      */
     public function getPost()
     {
-        if ($_SERVER['REQUEST_METHOD'] == 'POST' && empty($_POST))
-            $_POST = json_decode(file_get_contents('php://input'), true);
+        if ($_SERVER['REQUEST_METHOD'] == 'POST' && empty($_POST)) {
+                    $_POST = json_decode(file_get_contents('php://input'), true);
+        }
 
         return (array) $_POST;
     }
@@ -165,8 +166,9 @@ 
      */
     public function getJson()
     {
-        if ($_SERVER['REQUEST_METHOD'] != 'JSON')
-            throw new \LogicException("Request method is not JSON");
+        if ($_SERVER['REQUEST_METHOD'] != 'JSON') {
+                    throw new \LogicException("Request method is not JSON");
+        }
 
         $input =  file_get_contents('php://input');
         $array = explode("&", $input);
@@ -254,8 +256,9 @@ 
         $this->module = new $fqn($module, $this);
 
         # detects method change inside Module.php
-        if (!is_null($this->getMethod()))
-            $method = $this->getMethod();
+        if (!is_null($this->getMethod())) {
+                    $method = $this->getMethod();
+        }
 
         if (!is_null($method) && $this->initExecution)
         {
@@ -265,8 +268,9 @@ 
 
                 $reflection = new \ReflectionMethod($this, $method);
 
-                if (!$reflection->isPublic())
-                    throw new Exception\PageNotFoundException("The method '$method' is not public in the control class '$class'");
+                if (!$reflection->isPublic()) {
+                                    throw new Exception\PageNotFoundException("The method '$method' is not public in the control class '$class'");
+                }
 
                 $this->method = $method;
 
@@ -282,8 +286,7 @@ 
                     $layoutManager = new Layout($layout_params);
                     $layoutManager->fromController($this);
                 }
-            }
-            else {
+            } else {
                 $class = __CLASS__;
                 throw new Exception\PageNotFoundException("The method '$method' doesn't exists in the control class '$class'");
             }
@@ -308,8 +311,9 @@ 
     public function isXmlHttpRequest()
     {
         # non standard (HTTP_X_REQUESTED_WITH is not a part of PHP)
-        if (isset($_SERVER['HTTP_X_REQUESTED_WITH']))
-            return true;
+        if (isset($_SERVER['HTTP_X_REQUESTED_WITH'])) {
+                    return true;
+        }
         return false;
     }
 
@@ -320,8 +324,9 @@ 
      */
     public function isPost()
     {
-        if ($_SERVER["REQUEST_METHOD"] == "POST")
-            return true;
+        if ($_SERVER["REQUEST_METHOD"] == "POST") {
+                    return true;
+        }
         return false;
     }
 
@@ -332,8 +337,9 @@ 
      */
     public function isGet()
     {
-        if ($_SERVER["REQUEST_METHOD"] == "GET")
-            return true;
+        if ($_SERVER["REQUEST_METHOD"] == "GET") {
+                    return true;
+        }
         return false;
     }
 
@@ -357,10 +363,11 @@ 
             $i = 1;
             foreach ($params as $item)
             {
-                if ($i % 2 != 0)
-                    $vars[] = $item;
-                else
-                    $values[] = $item;
+                if ($i % 2 != 0) {
+                                    $vars[] = $item;
+                } else {
+                                    $values[] = $item;
+                }
                 $i++;
             }
 
@@ -368,10 +375,11 @@ 
 
             for ($i = 0; $i < $vars_count; $i++)
             {
-                if (array_key_exists($i, $values))
-                    $_GET[$vars[$i]] = $values[$i];
-                else
-                    $_GET[$vars[$i]] = '';
+                if (array_key_exists($i, $values)) {
+                                    $_GET[$vars[$i]] = $values[$i];
+                } else {
+                                    $_GET[$vars[$i]] = '';
+                }
             }
 
             unset($_GET["params"]);