HexMakina / LocalFS

tests/FileSystemTest.php

Spacing +23 added lines, -23 removed lines

@@ -12,7 +12,7 @@ 
 
     protected function setUp(): void
     {
-        $this->testRootPath = sys_get_temp_dir() . '/hexmakina_localfs_test';
+        $this->testRootPath = sys_get_temp_dir().'/hexmakina_localfs_test';
         if (!file_exists($this->testRootPath)) {
             mkdir($this->testRootPath);
         }
@@ -30,10 +30,10 @@ 
             $objects = scandir($dir);
             foreach ($objects as $object) {
                 if ($object != "." && $object != "..") {
-                    if (is_dir($dir . "/" . $object)) {
-                        $this->removeDirectory($dir . "/" . $object);
+                    if (is_dir($dir."/".$object)) {
+                        $this->removeDirectory($dir."/".$object);
                     } else {
-                        unlink($dir . "/" . $object);
+                        unlink($dir."/".$object);
                     }
                 }
             }
@@ -56,7 +56,7 @@ 
     $fileSystem = new FileSystem($testRootPath);
 
     $relativePath = 'subdir/file.txt';
-    $expectedPath = $testRootPath . '/' . $relativePath;
+    $expectedPath = $testRootPath.'/'.$relativePath;
 
     $result = $fileSystem->absolutePathFor($relativePath);
 
@@ -77,8 +77,8 @@ 
 
 public function testCopyNonExistentSourceFile()
 {
-    $nonExistentSourcePath = $this->testRootPath . '/non_existent_file.txt';
-    $destinationPath = $this->testRootPath . '/destination.txt';
+    $nonExistentSourcePath = $this->testRootPath.'/non_existent_file.txt';
+    $destinationPath = $this->testRootPath.'/destination.txt';
 
     $result = FileSystem::copy($nonExistentSourcePath, $destinationPath);
 
@@ -88,10 +88,10 @@ 
 
 public function testResolveSymlink()
 {
-    $tempDir = sys_get_temp_dir() . '/symlink_test';
+    $tempDir = sys_get_temp_dir().'/symlink_test';
     mkdir($tempDir);
-    $targetFile = $tempDir . '/target.txt';
-    $symlinkFile = $tempDir . '/symlink.txt';
+    $targetFile = $tempDir.'/target.txt';
+    $symlinkFile = $tempDir.'/symlink.txt';
 
     file_put_contents($targetFile, 'Test content');
     symlink($targetFile, $symlinkFile);
@@ -107,7 +107,7 @@ 
 
 public function testEnsureWritablePathCreatesDirectories()
 {
-    $nonExistentPath = $this->testRootPath . '/new/nested/directory';
+    $nonExistentPath = $this->testRootPath.'/new/nested/directory';
     $this->assertFalse(is_dir($nonExistentPath));
 
     $result = $this->fileSystem->ensureWritablePath($nonExistentPath);
@@ -129,12 +129,12 @@ 
 
 public function testDirectoriesListsOnlyDirectories()
 {
-    $testDir = $this->testRootPath . '/test_directories';
+    $testDir = $this->testRootPath.'/test_directories';
     mkdir($testDir);
-    mkdir($testDir . '/dir1');
-    mkdir($testDir . '/dir2');
-    touch($testDir . '/file1.txt');
-    touch($testDir . '/file2.txt');
+    mkdir($testDir.'/dir1');
+    mkdir($testDir.'/dir2');
+    touch($testDir.'/file1.txt');
+    touch($testDir.'/file2.txt');
 
     $result = $this->fileSystem->directories('test_directories');
 
@@ -147,13 +147,13 @@ 
 
 public function testFilesListsOnlyFilesInSpecifiedDirectory()
 {
-    $testDir = $this->testRootPath . '/test_files';
+    $testDir = $this->testRootPath.'/test_files';
     mkdir($testDir);
     
     // Create test files and directories
-    touch($testDir . '/file1.txt');
-    touch($testDir . '/file2.txt');
-    mkdir($testDir . '/subdir');
+    touch($testDir.'/file1.txt');
+    touch($testDir.'/file2.txt');
+    mkdir($testDir.'/subdir');
     
     $fileSystem = new FileSystem($this->testRootPath);
     $files = $fileSystem->files('test_files');
@@ -164,9 +164,9 @@ 
     $this->assertNotContains('subdir', $files);
     
     // Clean up
-    unlink($testDir . '/file1.txt');
-    unlink($testDir . '/file2.txt');
-    rmdir($testDir . '/subdir');
+    unlink($testDir.'/file1.txt');
+    unlink($testDir.'/file2.txt');
+    rmdir($testDir.'/subdir');
     rmdir($testDir);
 }
 }

src/FileSystem.php

Spacing +7 added lines, -7 removed lines

@@ -94,8 +94,8 @@ 
     {
         $absolutePath = $this->absolutePathFor($relativePath);
         // Filter the list of files to include only files (not directories).
-        $files = array_filter($this->list($relativePath), function ($filename) use ($absolutePath) {
-            return is_file($absolutePath . DIRECTORY_SEPARATOR . $filename);
+        $files = array_filter($this->list($relativePath), function($filename) use ($absolutePath) {
+            return is_file($absolutePath.DIRECTORY_SEPARATOR.$filename);
         });
 
         return $files;
@@ -112,8 +112,8 @@ 
         $absolutePath = $this->absolutePathFor($relativePath);
 
         // Filter the list of files to include only files (not directories).
-        $files = array_filter($this->list($relativePath), function ($filename) use ($absolutePath) {
-            return is_dir($absolutePath . DIRECTORY_SEPARATOR . $filename);
+        $files = array_filter($this->list($relativePath), function($filename) use ($absolutePath) {
+            return is_dir($absolutePath.DIRECTORY_SEPARATOR.$filename);
         });
 
         return $files;
@@ -127,19 +127,19 @@ 
      * @return bool True if the path is writable, false otherwise.
      */
 
-    public function ensureWritablePath(string $absoluteDirectoryPath, string $filename=null): bool
+    public function ensureWritablePath(string $absoluteDirectoryPath, string $filename = null): bool
     {
         if (strpos($absoluteDirectoryPath, $this->root()) !== 0) {
             throw new \InvalidArgumentException('PATH_NOT_INSIDE_ROOT_PATH');
         }
 
-        $relativeDirectoryPath = substr($absoluteDirectoryPath, strlen($this->root()) + 1);
+        $relativeDirectoryPath = substr($absoluteDirectoryPath, strlen($this->root())+1);
         $pathParts = explode('/', $relativeDirectoryPath);
         
 
         $targetDir = $this->root();
         foreach ($pathParts as $i => $part) {
-            $targetDir .= '/' . $part;
+            $targetDir .= '/'.$part;
 
             // Create the folder if it doesn't exist
             if (!file_exists($targetDir) && mkdir($targetDir, 0755, true) === false) {

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
     function __construct(string $rootPath)
     {
         $rootPath = realpath($rootPath);
-        if (!$rootPath)
-            throw new \InvalidArgumentException('INVALID_ROOT_PATH');
+        if (!$rootPath) {
+                    throw new \InvalidArgumentException('INVALID_ROOT_PATH');
+        }
 
         $this->rootPath = $rootPath;
     }

src/File.php

Spacing +2 added lines, -2 removed lines

@@ -80,9 +80,9 @@
     }
 
    
-    public function getMIMEType($fast=true): string
+    public function getMIMEType($fast = true): string
     {
-        if($fast === true || extension_loaded('fileinfo') === false){
+        if ($fast === true || extension_loaded('fileinfo') === false) {
             return mime_content_type($this->path());
         }
 

src/FilePath.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function __toString()
     {
-        return '' . $this->filepath;
+        return ''.$this->filepath;
     }
 
     public function dir(): string

src/Text/INI.php

Spacing +3 added lines, -3 removed lines

@@ -11,9 +11,9 @@ 
 
         if (is_array(current($array))) { // with sections
             foreach ($array as $section => $data) {
-                $ret .= PHP_EOL . PHP_EOL . self::section($section);
+                $ret .= PHP_EOL.PHP_EOL.self::section($section);
                 foreach ($data as $key => $value) {
-                    $ret .= PHP_EOL . self::line($key, $value);
+                    $ret .= PHP_EOL.self::line($key, $value);
                 }
             }
         } else { // no section
@@ -29,7 +29,7 @@ 
     {
         // https://secure.php.net/manual/en/function.parse-ini-file.php
         $ret = parse_ini_file($filepath, $with_sections, $mode);
-        if($ret === false)
+        if ($ret === false)
             return null;
 
         return $ret;

Braces +3 added lines, -2 removed lines

@@ -29,8 +29,9 @@
     {
         // https://secure.php.net/manual/en/function.parse-ini-file.php
         $ret = parse_ini_file($filepath, $with_sections, $mode);
-        if($ret === false)
-            return null;
+        if($ret === false) {
+                    return null;
+        }
 
         return $ret;
     }

src/Text/TextFile.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
     public function __toString()
     {
         $ret = file_get_contents($this->path());
-        if($ret === false)
+        if ($ret === false)
             $ret = $this->path();
 
         return $ret;

Braces +3 added lines, -2 removed lines

@@ -67,8 +67,9 @@
     public function __toString()
     {
         $ret = file_get_contents($this->path());
-        if($ret === false)
-            $ret = $this->path();
+        if($ret === false) {
+                    $ret = $this->path();
+        }
 
         return $ret;
     }

src/Text/JSON.php

Spacing +1 added lines, -1 removed lines

@@ -41,6 +41,6 @@
         if (json_last_error() === JSON_ERROR_NONE) {
             return false;
         }
-        throw new \Exception('ParsingException: '.json_last_error_msg());;
+        throw new \Exception('ParsingException: '.json_last_error_msg()); ;
     }
 }