Merge pull request #547 from svrnm/master

Fix for 'The first argument to copy() function cannot be a directory'
fzaninotto · Apr 27, 2015 · e1e2782 · e1e2782
2 parents 082c744 + 1ca84ea
commit e1e2782
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/Faker/Provider/File.php b/src/Faker/Provider/File.php
@@ -584,7 +584,9 @@ public static function file($sourceDirectory = '/tmp', $targetDirectory = '/tmp'
         }
 
         // Drop . and .. and reset array keys
-        $files = array_values(array_diff(scandir($sourceDirectory), array('.', '..')));
+        $files = array_filter(array_values(array_diff(scandir($sourceDirectory), array('.', '..'))), function ($file) use ($sourceDirectory) {
+                return !is_dir($sourceDirectory . DIRECTORY_SEPARATOR . $file);
+        });
 
         $sourceFullPath = $sourceDirectory . DIRECTORY_SEPARATOR . static::randomElement($files);