Pleets / dronephp

src/Validator/FormValidator.php

Braces +34 added lines, -26 removed lines

@@ -103,8 +103,9 @@ 
      */
     public function getOption($key, $name)
     {
-        if (!array_key_exists($key, $this->options))
-            throw new \LogicException("The option '$key' does not exists");
+        if (!array_key_exists($key, $this->options)) {
+                    throw new \LogicException("The option '$key' does not exists");
+        }
 
         return array_key_exists($name, $this->options[$key]) ? $this->options[$key][$name] : null;
     }
@@ -158,8 +159,9 @@ 
 
         foreach ($attribs as $key => $attributes)
         {
-            if (!array_key_exists($key, $attribs))
-                throw new \LogicException("The field '$key' does not exists!");
+            if (!array_key_exists($key, $attribs)) {
+                            throw new \LogicException("The field '$key' does not exists!");
+            }
 
             $label = (array_key_exists('label', array_keys($this->options))) ? $attributes["label"] : $key;
 
@@ -225,19 +227,21 @@ 
 
                     case 'min':
 
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range']))
-                            $validator = new GreaterThan(['min' => $value, 'inclusive' => true]);
-                        else
-                            throw new \LogicException("The input type must be 'range' or 'number'");
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range'])) {
+                                                    $validator = new GreaterThan(['min' => $value, 'inclusive' => true]);
+                        } else {
+                                                    throw new \LogicException("The input type must be 'range' or 'number'");
+                        }
 
                         break;
 
                     case 'max':
 
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range']))
-                            $validator = new LessThan(['max' => $value, 'inclusive' => true]);
-                        else
-                            throw new \LogicException("The input type must be 'range' or 'number'");
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['number', 'range'])) {
+                                                    $validator = new LessThan(['max' => $value, 'inclusive' => true]);
+                        } else {
+                                                    throw new \LogicException("The input type must be 'range' or 'number'");
+                        }
 
                         break;
 
@@ -245,10 +249,11 @@ 
 
                         $baseValue = (array_key_exists('min', $all_attribs)) ? $all_attribs['min'] : 0;
 
-                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['range']))
-                            $validator = new Step(['baseValue' => $baseValue, 'step' => $value]);
-                        else
-                            throw new \LogicException("The input type must be 'range'");
+                        if (array_key_exists('type', $all_attribs) && in_array($all_attribs['type'], ['range'])) {
+                                                    $validator = new Step(['baseValue' => $baseValue, 'step' => $value]);
+                        } else {
+                                                    throw new \LogicException("The input type must be 'range'");
+                        }
 
                         break;
                 }
@@ -272,8 +277,9 @@ 
                 $validator = new NotEmpty();
                 $attribute = $this->formHandler->getAttribute($key, "value");
 
-                if (is_null($attribute))
-                    throw new \LogicException("Missing attribute 'value' in '" . $key . "'.");
+                if (is_null($attribute)) {
+                                    throw new \LogicException("Missing attribute 'value' in '" . $key . "'.");
+                }
 
                 $value = $attribute->getValue();
 
@@ -287,8 +293,9 @@ 
                         {
                             $className = "\Zend\I18n\Validator\\" . $class;
 
-                            if (!class_exists($className))
-                                throw new \RuntimeException("The class '$userInputClass' or '$className' does not exists");
+                            if (!class_exists($className)) {
+                                                            throw new \RuntimeException("The class '$userInputClass' or '$className' does not exists");
+                            }
                         }
 
                         $validator = new $className($params);
@@ -326,21 +333,22 @@ 
             $v->setTranslator($this->translator);
             $notEmpty = $v->isValid($val);
 
-            if (!$required && !$notEmpty)
-                return null;
+            if (!$required && !$notEmpty) {
+                            return null;
+            }
 
             $valid = $validator->isValid($val);
             $this->setValid($valid);
 
             if (!$valid)
             {
-                if (!in_array($key, array_keys($this->messages)))
-                    $this->messages[$key] = array();
+                if (!in_array($key, array_keys($this->messages))) {
+                                    $this->messages[$key] = array();
+                }
 
                 $this->messages[$key] = array_merge($this->messages[$key], $validator->getMessages());
             }
-        }
-        else {
+        } else {
             foreach ($form_value as $val)
             {
                 $this->_validate($validator, $val, $key, $required);

src/Dom/Element/AbstractElement.php

Braces +12 added lines, -10 removed lines

@@ -108,8 +108,9 @@ 
 
         foreach ($attribs[$label] as $attrib)
         {
-            if ($attrib->getName() == $name)
-                return $attrib;
+            if ($attrib->getName() == $name) {
+                            return $attrib;
+            }
         }
     }
 
@@ -129,12 +130,12 @@ 
         if (array_key_exists($label, $attribs))
         {
 
-            if (!array_key_exists($name, $attribs))
-                $this->attributes[$label][] = new Attribute($name, $value);
-            else
-                $this->getAttribute($label, $name)->setValue($value);
-        }
-        else {
+            if (!array_key_exists($name, $attribs)) {
+                            $this->attributes[$label][] = new Attribute($name, $value);
+            } else {
+                            $this->getAttribute($label, $name)->setValue($value);
+            }
+        } else {
             $this->attributes[$label][] = new Attribute($name, $value);
         }
     }
@@ -166,8 +167,9 @@ 
      */
     public function __construct($options)
     {
-        if (!array_key_exists('endTag', $options))
-            $this->setEndTag(false);
+        if (!array_key_exists('endTag', $options)) {
+                    $this->setEndTag(false);
+        }
 
         foreach ($options as $key => $value)
         {

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/Error/ErrorTrait.php

Braces +3 added lines, -2 removed lines

@@ -67,12 +67,13 @@
      */
     protected function error($code, $message = null)
     {
-        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)
                         ? $this->standardErrors[$code]
                         : preg_replace('/%[a-zA-Z]*%/', $message, $this->standardErrors[$code])
                 : $message;
+        }
     }
 }
\ No newline at end of file

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/Util/ArrayDimension.php

Braces +2 added lines, -2 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);

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);
     }