Pleets / dronephp

src/Db/Driver/AbstractDriver.php

Spacing +2 added lines, -2 removed lines

@@ -298,8 +298,8 @@
     {
         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);
         }
     }
 

Braces +32 added lines, -22 removed lines

@@ -216,8 +216,9 @@ 
      */
     public function getArrayResult()
     {
-        if (!empty($this->arrayResult))
-            return $this->arrayResult;
+        if (!empty($this->arrayResult)) {
+                    return $this->arrayResult;
+        }
 
         return $this->toArray();
     }
@@ -313,8 +314,9 @@ 
      */
     public function autocommit($value)
     {
-        if ($this->transac_mode)
-            throw new \LogicException("You cannot change autocommit behavior during a transaction");
+        if ($this->transac_mode) {
+                    throw new \LogicException("You cannot change autocommit behavior during a transaction");
+        }
 
         $this->autocommit = $value;
     }
@@ -330,8 +332,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);
+            }
         }
     }
 
@@ -375,8 +378,9 @@ 
      */
     public function reconnect()
     {
-        if (!$this->isConnected())
-            throw new \LogicException("Connection was not established");
+        if (!$this->isConnected()) {
+                    throw new \LogicException("Connection was not established");
+        }
 
         $this->disconnect();
         return $this->connect();
@@ -405,8 +409,9 @@ 
      */
     public function disconnect()
     {
-        if (!$this->isConnected())
-            throw new \LogicException("Connection was not established");
+        if (!$this->isConnected()) {
+                    throw new \LogicException("Connection was not established");
+        }
     }
 
     /**
@@ -418,11 +423,13 @@ 
      */
     public function beginTransaction()
     {
-        if (!$this->isConnected())
-            $this->connect();
+        if (!$this->isConnected()) {
+                    $this->connect();
+        }
 
-        if ($this->transac_mode)
-            throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_STARTED]);
+        if ($this->transac_mode) {
+                    throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_STARTED]);
+        }
 
         $this->transac_mode = true;
     }
@@ -436,16 +443,19 @@ 
      */
     public function endTransaction()
     {
-        if (!$this->transac_mode)
-            throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_NOT_STARTED]);
+        if (!$this->transac_mode) {
+                    throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_NOT_STARTED]);
+        }
 
-        if (is_null($this->transac_result))
-            throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_EMPTY]);
+        if (is_null($this->transac_result)) {
+                    throw new \LogicException($this->standardErrors[Errno::DB_TRANSACTION_EMPTY]);
+        }
 
-        if ($this->transac_result)
-            $this->commit();
-        else
-            $this->rollback();
+        if ($this->transac_result) {
+                    $this->commit();
+        } else {
+                    $this->rollback();
+        }
 
         $this->result = $this->transac_result;
 

src/Db/Driver/DriverAdapter.php

Braces +8 added lines, -7 removed lines

@@ -85,13 +85,14 @@ 
             "Sqlsrv" => "Drone\Db\Driver\SQLServer",
         ];
 
-        if (gettype($connection_identifier) == 'array')
-            $connection_array = $connection_identifier;
-        else
+        if (gettype($connection_identifier) == 'array') {
+                    $connection_array = $connection_identifier;
+        } else
         {
             # Take connection parameters from configuration file
-            if (!file_exists("config/database.config.php"))
-                throw new \RuntimeException("config/data.base.config.php is missing!");
+            if (!file_exists("config/database.config.php")) {
+                            throw new \RuntimeException("config/data.base.config.php is missing!");
+            }
 
             $dbsettings = include("config/database.config.php");
             $connection_array = $dbsettings[$connection_identifier];
@@ -106,8 +107,8 @@ 
 
             $this->driverName = $drv;
             $this->db = new $driver[$drv]($connection_array);
+        } else {
+                    throw new \RuntimeException("The Database driver does not exists");
         }
-        else
-            throw new \RuntimeException("The Database driver does not exists");
     }
 }
\ 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/TableGatewayInterface.php

Indentation +4 added lines, -4 removed lines

@@ -22,14 +22,14 @@ 
      *
      * @param array $where
      */
-	public function select(Array $where);
+    public function select(Array $where);
 
     /**
      * Insert statement
      *
      * @param array $data
      */
-	public function insert(Array $data);
+    public function insert(Array $data);
 
     /**
      * Update statement
@@ -37,12 +37,12 @@ 
      * @param array $set
      * @param array $where
      */
-	public function update(Array $set, Array $where);
+    public function update(Array $set, Array $where);
 
     /**
      * Delete statement
      *
      * @param array $where
      */
-	public function delete(Array $where);
+    public function delete(Array $where);
 }
\ No newline at end of file

src/Db/TableGateway/AbstractTableGateway.php

Braces +3 added lines, -2 removed lines

@@ -73,7 +73,8 @@
     {
         $this->currentDriverIdentifier = $connection_identifier;
 
-        if (!isset(self::$drivers[$connection_identifier]))
-            self::$drivers[$connection_identifier] = new DriverAdapter($connection_identifier, $auto_connect);
+        if (!isset(self::$drivers[$connection_identifier])) {
+                    self::$drivers[$connection_identifier] = new DriverAdapter($connection_identifier, $auto_connect);
+        }
     }
 }
\ No newline at end of file

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

Spacing +15 added lines, -15 removed lines

@@ -93,24 +93,24 @@ 
             {
                 foreach ($contents as $i)
                 {
-                    if (is_file($handler.'/'.$i))
+                    if (is_file($handler . '/' . $i))
                     {
-                        $allContents[] = $handler.'/'.$i;
+                        $allContents[] = $handler . '/' . $i;
 
-                        $this->buffer = $handler.'/'.$i;
+                        $this->buffer = $handler . '/' . $i;
                         call_user_func($fileCallback, $this);
                     }
-                    elseif (is_dir($handler.'/'.$i))
+                    elseif (is_dir($handler . '/' . $i))
                     {
-                        $allContents[] = $handler.'/'.$i;
+                        $allContents[] = $handler . '/' . $i;
 
-                        $this->buffer = $handler.'/'.$i;
-                        $this->getContents($handler.'/'.$i, $fileCallback, $dirCallback);
+                        $this->buffer = $handler . '/' . $i;
+                        $this->getContents($handler . '/' . $i, $fileCallback, $dirCallback);
 
                         $directory = scandir($handler);
 
                         if (!count($directory) < 3)
-                            $this->buffer = $handler.'/'.$i;
+                            $this->buffer = $handler . '/' . $i;
 
                         call_user_func($dirCallback, $this);
                     }
@@ -196,7 +196,7 @@ 
                 if (!empty($path))
                     if (!strlen(stristr($item, $path)) > 0)
                         continue;
-                if (strstr($item,'~') === false && $item != '.' && $item != '..')
+                if (strstr($item, '~') === false && $item != '.' && $item != '..')
                     $filesToReturn[] = $item;
             }
         }
@@ -299,7 +299,7 @@ 
         if (is_dir($dest))
         {
             if (!$recursive)
-                copy($file, $dest.'/'.$file);
+                copy($file, $dest . '/' . $file);
             else {
 
                 $files = array();
@@ -318,14 +318,14 @@ 
 
                     foreach ($files[1] as $item)
                     {
-                        if (!file_exists($dest.'/'.$item))
+                        if (!file_exists($dest . '/' . $item))
                             mkdir("$dest/$item", 0777, true);
                     }
 
                     foreach ($files[0] as $item)
                     {
                         if (!file_exists("$dest/$files"))
-                            copy($item, $dest.'/'.$item);
+                            copy($item, $dest . '/' . $item);
                     }
                 });
             }
@@ -350,12 +350,12 @@ 
             throw new Exception("Missing parameter for mv!");
 
         if (is_dir($newfile))
-                $newfile .= '/'.basename($oldfile);
+                $newfile .= '/' . basename($oldfile);
 
         if ($oldfile == $newfile)
             throw new Exception("'$oldfile' and '$newfile' are the same file");
 
-        if(!rename($oldfile, $newfile))
+        if (!rename($oldfile, $newfile))
             return false;
 
         return true;
@@ -385,7 +385,7 @@ 
         else {
             if (!is_dir($dir))
             {
-                if(!mkdir("$dir", 0777))
+                if (!mkdir("$dir", 0777))
                     return false;
             }
         }

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,25 +182,25 @@ 
                 foreach ($files as $item) {
                     $filesToReturn[] = $item;
                 }
-            }
-            else {
+            } else {
                 while (false !== ($item = $pathIns->read())) {
                     $filesToReturn[] = $item;
                 }
                 $pathIns->close();
             }
-        }
-        else {
+        } else {
 
             $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;
+                }
             }
         }
 
@@ -217,8 +220,9 @@ 
 
         if (is_dir($path))
         {
-            if (chdir($moveTo))
-                return true;
+            if (chdir($moveTo)) {
+                            return true;
+            }
         }
 
         return false;
@@ -235,11 +239,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);
         }
@@ -259,12 +265,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;
 
@@ -293,14 +300,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();
@@ -318,20 +326,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;
     }
@@ -346,17 +356,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)
-            throw new Exception("'$oldfile' and '$newfile' are the same file");
+        if ($oldfile == $newfile) {
+                    throw new Exception("'$oldfile' and '$newfile' are the same file");
+        }
 
-        if(!rename($oldfile, $newfile))
-            return false;
+        if(!rename($oldfile, $newfile)) {
+                    return false;
+        }
 
         return true;
     }
@@ -372,21 +386,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;
@@ -401,12 +418,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/Network/Socket/AbstractSocket.php

Indentation +4 added lines, -4 removed lines

@@ -121,7 +121,7 @@ 
         }
 
         $this->socket = $socket;
-	}
+    }
 
     /**
      * Binds the socket
@@ -145,8 +145,8 @@ 
      *
      * @return null
      */
-	public function close()
-	{
+    public function close()
+    {
         return socket_close($this->socket);
-	}
+    }
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

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

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 (!($socket = @socket_create(AF_INET, SOCK_STREAM, SOL_TCP)))

src/Db/Entity.php

Braces +5 added lines, -4 removed lines

@@ -107,11 +107,12 @@
             {
                 $this->$prop = $value;
 
-                if (!in_array($prop, $this->changedFields))
-                    $this->changedFields[] = $prop;
+                if (!in_array($prop, $this->changedFields)) {
+                                    $this->changedFields[] = $prop;
+                }
+            } else {
+                            throw new \LogicException("The property '$prop' does not exists in the class ' " . get_class($this) . " '");
             }
-            else
-                throw new \LogicException("The property '$prop' does not exists in the class ' " . get_class($this) . " '");
         }
     }