HexMakina / LocalFS

File.class.php

Braces +21 added lines, -14 removed lines

@@ -15,8 +15,9 @@ 
 
   public function __construct($path_to_file, $mode='r')
   {
-    if(!FileSystem::exists($path_to_file) && self::requires_existing_file($mode))
-      throw new \Exception('FILE_MUST_ALREADY_EXIST ('.$this->filepath().', '.$this->mode.')');
+    if(!FileSystem::exists($path_to_file) && self::requires_existing_file($mode)) {
+          throw new \Exception('FILE_MUST_ALREADY_EXIST ('.$this->filepath().', '.$this->mode.')');
+    }
 
     $this->filepath = new FilePath($path_to_file);
     $this->mode = $mode;
@@ -25,18 +26,21 @@ 
   public function open()
   {
     $this->pointer = fopen($this->filepath, $this->mode);
-    if($this->pointer === false)
-      throw new \Exception('FILE_OPEN_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    if($this->pointer === false) {
+          throw new \Exception('FILE_OPEN_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    }
     return $this->pointer;
   }
 
   public function close()
   {
-    if(!is_resource($this->pointer))
-      return true;
+    if(!is_resource($this->pointer)) {
+          return true;
+    }
 
-    if(fclose($this->pointer) === false)
-      throw new \Exception('FILE_CLOSE_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    if(fclose($this->pointer) === false) {
+          throw new \Exception('FILE_CLOSE_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    }
 
     return true;
   }
@@ -48,12 +52,14 @@ 
 
   public function set_content($content)
   {
-    if(is_writable($this->filepath) !== true && self::requires_existing_file($this->mode))
-      throw new \Exception('FILE_IS_NOT_WRITABLE ('.$this->filepath().', '.$this->mode.')');
+    if(is_writable($this->filepath) !== true && self::requires_existing_file($this->mode)) {
+          throw new \Exception('FILE_IS_NOT_WRITABLE ('.$this->filepath().', '.$this->mode.')');
+    }
 
     $this->open();
-    if(fwrite($this->pointer, $content) === false)
-      throw new \Exception('FILE_WRITE_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    if(fwrite($this->pointer, $content) === false) {
+          throw new \Exception('FILE_WRITE_FAILURE ('.$this->filepath().', '.$this->mode.')');
+    }
     $this->close();
   }
 
@@ -71,8 +77,9 @@ 
   {
     $res = $this->size_in_bytes;
 
-    if(is_null($res) && ($res = filesize("$this->filepath")) !== false)
-      $this->size_in_bytes = $res;
+    if(is_null($res) && ($res = filesize("$this->filepath")) !== false) {
+          $this->size_in_bytes = $res;
+    }
 
     return $res;
   }

FileSystem.class.php

Braces +32 added lines, -20 removed lines

@@ -19,12 +19,14 @@ 
 
   public function filenames($regex=null) : array
   {
-		if(!file_exists($this->root_path) && mkdir($this->root_path) === false)
-			return [];
+		if(!file_exists($this->root_path) && mkdir($this->root_path) === false) {
+					return [];
+		}
 
     $filenames = self::preg_scandir($this->root_path, $regex);// ID_SEQUENCENUMBER.ext
-    if(!is_null($filenames))
-      sort($filenames);
+    if(!is_null($filenames)) {
+          sort($filenames);
+    }
 
     return $filenames;
   }
@@ -33,8 +35,9 @@ 
   {
     $filenames = $this->filenames($regex);
     $filepathes = [];
-    foreach($filenames as $filename)
-      $filepathes[] = $this->path_to($filename);
+    foreach($filenames as $filename) {
+          $filepathes[] = $this->path_to($filename);
+    }
 
     return $filepathes;
   }
@@ -52,8 +55,9 @@ 
     if(is_link($path))
     {
       $res = readlink($path);
-      if($res === false)
-        throw new \Exception('readlink failed on '.$path);
+      if($res === false) {
+              throw new \Exception('readlink failed on '.$path);
+      }
 
       $path = $res;
     }
@@ -65,8 +69,10 @@ 
     if(self::exists($src_path) && self::is_file($src_path))
     {
       $dst = new FilePath($dst_path);
-      if(self::exists($dst->dir()) && self::is_dir($dst->dir()))
-        return copy($src_path, $dst_path); // Returns TRUE on success or FALSE on failure.
+      if(self::exists($dst->dir()) && self::is_dir($dst->dir())) {
+              return copy($src_path, $dst_path);
+      }
+      // Returns TRUE on success or FALSE on failure.
 
       // vd(__FUNCTION__.'ERR: self::exists('.$dst->dir().') && self::is_dir('.$dst->dir().')');
     }
@@ -102,23 +108,29 @@ 
     $ret = false;
     if(self::exists($src_path))
     {
-      if(self::is_link($src_path) && $follow_link === true)
-        $ret = self::remove(readlink($src_path));
-      elseif(self::is_file($src_path))
-        $ret = unlink($src_path); // Returns TRUE on success or FALSE on failure.
-      else if(self::is_dir($src_path))
-        $ret = rmdir($src_path); // Returns TRUE on success or FALSE on failure.
+      if(self::is_link($src_path) && $follow_link === true) {
+              $ret = self::remove(readlink($src_path));
+      } elseif(self::is_file($src_path)) {
+              $ret = unlink($src_path);
+      }
+      // Returns TRUE on success or FALSE on failure.
+      else if(self::is_dir($src_path)) {
+              $ret = rmdir($src_path);
+      }
+      // Returns TRUE on success or FALSE on failure.
     }
     return $ret;
   }
 
   public static function preg_scandir($dir_path, $regex=null)
   {
-    if(!self::exists($dir_path) || !self::is_dir($dir_path))
-      return null;
+    if(!self::exists($dir_path) || !self::is_dir($dir_path)) {
+          return null;
+    }
 
-    if(($filenames = scandir($dir_path, SCANDIR_SORT_ASCENDING)) !== false)
-      return is_null($regex)? $filenames : preg_grep($regex, $filenames);
+    if(($filenames = scandir($dir_path, SCANDIR_SORT_ASCENDING)) !== false) {
+          return is_null($regex)? $filenames : preg_grep($regex, $filenames);
+    }
 
     throw new \Exception("directory path '$dir_path' cannot be scanned");
   }

Text/TextFile.class.php

Braces +11 added lines, -10 removed lines

@@ -8,16 +8,18 @@ 
   public static function identical($filepath_1, $filepath_2, $read_length = 8192)
   {
     //** TEST FOR EXISTENCE
-    if(!file_exists($filepath_1) || !file_exists($filepath_2))
-      throw new \Exception('file_exists false');
+    if(!file_exists($filepath_1) || !file_exists($filepath_2)) {
+          throw new \Exception('file_exists false');
+    }
 
     //** TEST FOR SYMLINK
     $filepath_1 = self::resolve_symlink($filepath_1);
     $filepath_2 = self::resolve_symlink($filepath_2);
 
     //** TEST FOR IDENTICAL TYPE AND SIZE
-    if(filetype($filepath_1) !== filetype($filepath_2) || filesize($filepath_1) !== filesize($filepath_2))
-      return false;
+    if(filetype($filepath_1) !== filetype($filepath_2) || filesize($filepath_1) !== filesize($filepath_2)) {
+          return false;
+    }
 
     //** TEST FOR IDENTICAL CONTENT
     return self::compare_content($filepath_1, $filepath_2, $read_length);
@@ -43,15 +45,14 @@ 
         $identical = (0 === strcmp($buffer_1, $buffer_2));
       }
 
-      if($identical === true && feof($filepointer_1) !== feof($filepointer_2))
-        $identical = false;
+      if($identical === true && feof($filepointer_1) !== feof($filepointer_2)) {
+              $identical = false;
+      }
 
-    }
-    catch (\Exception $e)
+    } catch (\Exception $e)
     {
       return false;
-    }
-    finally
+    } finally
     {
       $file_1->close();
       $file_2->close();

Text/JSON.class.php

Braces +9 added lines, -6 removed lines

@@ -10,8 +10,9 @@ 
     $ret = json_decode($string, $assoc, $depth, $options);
 
     $error = self::has_errors();
-    if($error !== false)
-      throw new \Exception("ParsingException: $error");
+    if($error !== false) {
+          throw new \Exception("ParsingException: $error");
+    }
 
     return $ret;
   }
@@ -21,16 +22,18 @@ 
     $ret = json_encode($var, $options, $depth);
 
     $error = self::has_errors();
-    if($error !== false)
-      throw new \Exception("ParsingException: $error");
+    if($error !== false) {
+          throw new \Exception("ParsingException: $error");
+    }
 
     return $ret;
   }
 
   private static function has_errors()
   {
-    if(json_last_error() === JSON_ERROR_NONE)
-      return false;
+    if(json_last_error() === JSON_ERROR_NONE) {
+          return false;
+    }
 
     return json_last_error_msg();
   }

Text/INI.class.php

Braces +9 added lines, -5 removed lines

@@ -9,18 +9,22 @@
   {
     $ret = '# INI Dump';
 
-    if(is_array(current($array))) // with sections
+    if(is_array(current($array))) {
+      // with sections
     {
       foreach($array as $section => $data)
       {
         $ret.= PHP_EOL.PHP_EOL.self::section($section);
-        foreach($data as $key => $value)
-          $ret.= PHP_EOL.self::line($key, $value);
-      }
     }
-    else // no section
+        foreach($data as $key => $value) {
+                  $ret.= PHP_EOL.self::line($key, $value);
+        }
+      }
+    } else {
+      // no section
       foreach($array as $key => $value)
         $ret.= self::line($key, $value);
+    }
 
     return $ret;
   }