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/FileSystem/Shell.php

Braces +89 added lines, -70 removed lines

@@ -83,8 +83,9 @@ 
 
             foreach ($filesForHandler as $item)
             {
-                if ($item != '.' && $item != '..')
-                    $contents[] = $item;
+                if ($item != '.' && $item != '..') {
+                                    $contents[] = $item;
+                }
             }
 
             $allContents = $contents;
@@ -99,8 +100,7 @@ 
 
                         $this->buffer = $handler.'/'.$i;
                         call_user_func($fileCallback, $this);
-                    }
-                    elseif (is_dir($handler.'/'.$i))
+                    } elseif (is_dir($handler.'/'.$i))
                     {
                         $allContents[] = $handler.'/'.$i;
 
@@ -109,19 +109,21 @@ 
 
                         $directory = scandir($handler);
 
-                        if (!count($directory) < 3)
-                            $this->buffer = $handler.'/'.$i;
+                        if (!count($directory) < 3) {
+                                                    $this->buffer = $handler.'/'.$i;
+                        }
 
                         call_user_func($dirCallback, $this);
                     }
                 }
             }
+        } else {
+                    throw new Exception("The directory '$handler' does not exists");
         }
-        else
-            throw new Exception("The directory '$handler' does not exists");
 
-        if (!is_null($callback))
-            call_user_func($callback, $this);
+        if (!is_null($callback)) {
+                    call_user_func($callback, $this);
+        }
 
         return $allContents;
     }
@@ -133,10 +135,11 @@ 
      */
     public function pwd()
     {
-        if (getcwd())
-            $this->buffer = getcwd();
-        else
-            return false;
+        if (getcwd()) {
+                    $this->buffer = getcwd();
+        } else {
+                    return false;
+        }
 
         return $this->buffer;
     }
@@ -155,9 +158,9 @@ 
 
         $path = (is_null($path) || empty($path)) ? '.' : $path;
 
-        if (is_file($path))
-            $filesToReturn = array($path);
-        elseif (is_dir($path))
+        if (is_file($path)) {
+                    $filesToReturn = array($path);
+        } elseif (is_dir($path))
         {
             $pathIns = dir($path);
 
@@ -179,26 +182,26 @@ 
                 foreach ($files as $item) {
                     $filesToReturn[] = $item;
                 }
-            }
-            else {
+            } else {
                 while (false !== ($item = $pathIns->read())) {
                     $filesToReturn[] = $item;
                 }
                 $pathIns->close();
             }
-        }
-        else {
+        } else {
 
             $pathIns = dir('.');
             $contents = $this->ls();
 
             foreach ($contents as $item)
             {
-                if (!empty($path))
-                    if (!strlen(stristr($item, $path)) > 0)
+                if (!empty($path)) {
+                                    if (!strlen(stristr($item, $path)) > 0)
                         continue;
-                if (strstr($item,'~') === false && $item != '.' && $item != '..')
-                    $filesToReturn[] = $item;
+                }
+                if (strstr($item,'~') === false && $item != '.' && $item != '..') {
+                                    $filesToReturn[] = $item;
+                }
             }
         }
 
@@ -218,8 +221,9 @@ 
 
         if (is_dir($path))
         {
-            if (chdir($moveTo))
-                return true;
+            if (chdir($moveTo)) {
+                            return true;
+            }
         }
 
         return false;
@@ -236,11 +240,13 @@ 
     {
         if (file_exists($file))
         {
-            if (!$openFile = fopen($file, 'w+'))
-                return false;
+            if (!$openFile = fopen($file, 'w+')) {
+                            return false;
+            }
 
-            if (fwrite($openFile, ""))
-                return true;
+            if (fwrite($openFile, "")) {
+                            return true;
+            }
 
             fclose($openFile);
         }
@@ -260,12 +266,13 @@ 
     {
         $recursive = is_null($recursive) ? false : $recursive;
 
-        if (is_null($file))
-            throw new Exception("Missing parameter for rm!");
+        if (is_null($file)) {
+                    throw new Exception("Missing parameter for rm!");
+        }
 
-        if (file_exists($file) && !$recursive)
-            unlink($file);
-        elseif (is_dir($file) && $recursive)
+        if (file_exists($file) && !$recursive) {
+                    unlink($file);
+        } elseif (is_dir($file) && $recursive)
         {
             $that = $this;
 
@@ -294,14 +301,15 @@ 
     {
         $recursive = (is_null($recursive)) ? false : $recursive;
 
-        if (empty($file) || empty($dest))
-            throw new Exception("Missing parameters!");
+        if (empty($file) || empty($dest)) {
+                    throw new Exception("Missing parameters!");
+        }
 
         if (is_dir($dest))
         {
-            if (!$recursive)
-                copy($file, $dest.'/'.$file);
-            else {
+            if (!$recursive) {
+                            copy($file, $dest.'/'.$file);
+            } else {
 
                 $files = array();
                 $files[0] = array();
@@ -319,20 +327,22 @@ 
 
                     foreach ($files[1] as $item)
                     {
-                        if (!file_exists($dest.'/'.$item))
-                            mkdir("$dest/$item", 0777, true);
+                        if (!file_exists($dest.'/'.$item)) {
+                                                    mkdir("$dest/$item", 0777, true);
+                        }
                     }
 
                     foreach ($files[0] as $item)
                     {
-                        if (!file_exists("$dest/$files"))
-                            copy($item, $dest.'/'.$item);
+                        if (!file_exists("$dest/$files")) {
+                                                    copy($item, $dest.'/'.$item);
+                        }
                     }
                 });
             }
+        } else {
+                    copy($file, $dest);
         }
-        else
-            copy($file, $dest);
 
         return true;
     }
@@ -347,17 +357,21 @@ 
      */
     public function mv($oldfile, $newfile)
     {
-        if (empty($oldfile))
-            throw new Exception("Missing parameter for mv!");
+        if (empty($oldfile)) {
+                    throw new Exception("Missing parameter for mv!");
+        }
 
-        if (is_dir($newfile))
-                $newfile .= '/'.basename($oldfile);
+        if (is_dir($newfile)) {
+                        $newfile .= '/'.basename($oldfile);
+        }
 
-        if ($oldfile == $newfile)
-            return $this;
+        if ($oldfile == $newfile) {
+                    return $this;
+        }
 
-        if(!rename($oldfile, $newfile))
-            return false;
+        if(!rename($oldfile, $newfile)) {
+                    return false;
+        }
 
         return true;
     }
@@ -373,21 +387,24 @@ 
      */
     public function mkdir($dir, $dest = null, $recursive = null)
     {
-        if (empty($dir))
-            throw new Exception("Missing parameter for mkdir!");
+        if (empty($dir)) {
+                    throw new Exception("Missing parameter for mkdir!");
+        }
 
-        if (empty($dest))
-            $dest = '.';
+        if (empty($dest)) {
+                    $dest = '.';
+        }
 
         $recursive = (is_null($recursive)) ? false : $recursive;
 
-        if ($recursive)
-            mkdir("$dest/$dir", 0777, true);
-        else {
+        if ($recursive) {
+                    mkdir("$dest/$dir", 0777, true);
+        } else {
             if (!is_dir($dir))
             {
-                if(!mkdir("$dir", 0777))
-                    return false;
+                if(!mkdir("$dir", 0777)) {
+                                    return false;
+                }
             }
         }
         return true;
@@ -402,12 +419,14 @@ 
      */
     public function rmdir($dir)
     {
-        if (is_null($dir) || empty($dir))
-            throw new Exception("Missing parameter for rmdir!");
+        if (is_null($dir) || empty($dir)) {
+                    throw new Exception("Missing parameter for rmdir!");
+        }
 
-        if (rmdir($dir))
-            return true;
-        else
-            return false;
+        if (rmdir($dir)) {
+                    return true;
+        } else {
+                    return false;
+        }
     }
 }
\ No newline at end of file

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/Socket/AbstractSocket.php

Braces +3 added lines, -2 removed lines

@@ -108,8 +108,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);
+            }
         }
 
         if (!($this->socket = @socket_create(AF_INET, SOCK_STREAM, SOL_TCP)))

src/Network/Socket/Server.php

Braces +10 added lines, -8 removed lines

@@ -76,11 +76,13 @@ 
 		$event = $eventHandlers;
 		$clousure = function(){};
 
-		if (!array_key_exists('success', $event))
-			$event["success"] = $clousure;
+		if (!array_key_exists('success', $event)) {
+					$event["success"] = $clousure;
+		}
 
-		if (!array_key_exists('error', $event))
-			$event["error"] = $clousure;
+		if (!array_key_exists('error', $event)) {
+					$event["error"] = $clousure;
+		}
 
 		$listener = false;
 
@@ -90,8 +92,7 @@ 
 			$this->error($errno, socket_strerror($errno));
 
 			throw new \RuntimeException("Could not set socket to listen");
-		}
-		else {
+		} else {
 
 			echo "\n";
 			echo "Server Started : " . date('Y-m-d H:i:s') . "\n";
@@ -116,8 +117,9 @@ 
 			socket_close($spawn);
 		}
 
-		if (!$listener)
-			call_user_func($event["error"], $this->getLastError());
+		if (!$listener) {
+					call_user_func($event["error"], $this->getLastError());
+		}
 
 		return $listener;
 	}

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 \LogicException("Status code not supported");
+        if (!in_array($code, array_keys($codes))) {
+                    throw new \LogicException("Status code not supported");
+        }
 
         return $this->httpStatusCodes[$code];
     }

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/Pear/Mail.php

Braces +12 added lines, -8 removed lines

@@ -74,18 +74,21 @@ 
     {
         @include_once 'System.php';
 
-        if (!class_exists('System', false))
-            throw new \RuntimeException("PEAR is not installed in your system!");
+        if (!class_exists('System', false)) {
+                    throw new \RuntimeException("PEAR is not installed in your system!");
+        }
 
         include_once 'Net/SMTP.php';
 
-        if (!class_exists('\Net_SMTP', false))
-            throw new \RuntimeException("PEAR::Net_SMTP is not installed!");
+        if (!class_exists('\Net_SMTP', false)) {
+                    throw new \RuntimeException("PEAR::Net_SMTP is not installed!");
+        }
 
         @include_once 'Mail.php';
 
-        if (!class_exists('\Mail', false))
-            throw new \RuntimeException("PEAR::Mail is not installed!");
+        if (!class_exists('\Mail', false)) {
+                    throw new \RuntimeException("PEAR::Mail is not installed!");
+        }
     }
 
     /**
@@ -98,8 +101,9 @@ 
      */
     protected function error($code, $message = null)
     {
-        if (!array_key_exists($code, $this->errors))
-            $this->errors[$message] = (is_null($message) && array_key_exists($code, $this->messagesTemplates)) ? $this->messagesTemplates[$code] : $message;
+        if (!array_key_exists($code, $this->errors)) {
+                    $this->errors[$message] = (is_null($message) && array_key_exists($code, $this->messagesTemplates)) ? $this->messagesTemplates[$code] : $message;
+        }
     }
 
     /**