Prevent copy and move to same destinations as the source.

src/Filesystem.php

@@ -119,20 +119,27 @@ private function pipeListing(string $location, bool $deep, iterable $listing): G
 
     public function move(string $source, string $destination, array $config = []): void
     {
-        $this->adapter->move(
-            $this->pathNormalizer->normalizePath($source),
-            $this->pathNormalizer->normalizePath($destination),
-            $this->config->extend($config)
-        );
+        $from = $this->pathNormalizer->normalizePath($source);
+        $to = $this->pathNormalizer->normalizePath($destination);
+
+        if ($from === $to) {
+            throw UnableToMoveFile::sourceAndDestinationAreTheSame($source, $destination);
+            throw UnableToMoveFile::because('Source and destination are the same', $source, $destination);
+        }
+
+        $this->adapter->move($from, $to, $this->config->extend($config));
     }
 
     public function copy(string $source, string $destination, array $config = []): void
     {
-        $this->adapter->copy(
-            $this->pathNormalizer->normalizePath($source),
-            $this->pathNormalizer->normalizePath($destination),
-            $this->config->extend($config)
-        );
+        $from = $this->pathNormalizer->normalizePath($source);
+        $to = $this->pathNormalizer->normalizePath($destination);
+
+        if ($from === $to) {
+            throw UnableToCopyFile::sourceAndDestinationAreTheSame($source, $destination);
+        }
+
+        $this->adapter->copy($from, $to, $this->config->extend($config));
     }
 
     public function lastModified(string $path): int

src/FilesystemTest.php

@@ -480,6 +480,26 @@ public function publicUrl(string $path, Config $config): string
         self::assertSame('custom/file.txt', $filesystem->publicUrl('file.txt'));
     }
 
+    /**
+     * @test
+     */
+    public function copying_from_and_to_the_same_location_fails(): void
+    {
+        $this->expectExceptionObject(UnableToMoveFile::sourceAndDestinationAreTheSame('from.txt', 'from.txt'));
+
+        $this->filesystem->move('from.txt', 'from.txt');
+    }
+
+    /**
+     * @test
+     */
+    public function moving_from_and_to_the_same_location_fails(): void
+    {
+        $this->expectExceptionObject(UnableToCopyFile::sourceAndDestinationAreTheSame('from.txt', 'from.txt'));
+
+        $this->filesystem->copy('from.txt', 'from.txt');
+    }
+
     /**
      * @test
      */

src/UnableToCopyFile.php

@@ -41,6 +41,20 @@ public static function fromLocationTo(
         return $e;
     }
 
+    public static function sourceAndDestinationAreTheSame(string $source, string $destination): UnableToCopyFile
+    {
+        return UnableToCopyFile::because('Source and destination are the same', $source, $destination);
+    }
+
+    public static function because(string $reason, string $sourcePath, string $destinationPath): UnableToCopyFile
+    {
+        $e = new static("Unable to copy file from $sourcePath to $destinationPath, because $reason");
+        $e->source = $sourcePath;
+        $e->destination = $destinationPath;
+
+        return $e;
+    }
+
     public function operation(): string
     {
         return FilesystemOperationFailed::OPERATION_COPY;

src/UnableToMoveFile.php

@@ -19,6 +19,11 @@ final class UnableToMoveFile extends RuntimeException implements FilesystemOpera
      */
     private $destination;
 
+    public static function sourceAndDestinationAreTheSame(string $source, string $destination): UnableToMoveFile
+    {
+        return UnableToMoveFile::because('Source and destination are the same', $source, $destination);
+    }
+
     public function source(): string
     {
         return $this->source;
@@ -42,6 +47,19 @@ public static function fromLocationTo(
         return $e;
     }
 
+    public static function because(
+        string $reason,
+        string $sourcePath,
+        string $destinationPath,
+    ): UnableToMoveFile {
+        $message = "Unable to move file from $sourcePath to $destinationPath, because $reason";
+        $e = new static($message);
+        $e->source = $sourcePath;
+        $e->destination = $destinationPath;
+
+        return $e;
+    }
+
     public function operation(): string
     {
         return FilesystemOperationFailed::OPERATION_MOVE;