FileSystem::rename() fix renaming file/directory if only case changes (#155)

Author: Ciki

src/Utils/FileSystem.php

@@ -103,7 +103,9 @@ public static function rename(string $name, string $newName, bool $overwrite = t
 
 		} else {
 			static::createDir(dirname($newName));
-			static::delete($newName);
+			if (realpath($name) !== realpath($newName)) {
+				static::delete($newName);
+			}
 			if (!@rename($name, $newName)) { // @ is escalated to exception
 				throw new Nette\IOException("Unable to rename file or directory '$name' to '$newName'.");
 			}

tests/Utils/FileSystem.phpt

@@ -133,16 +133,18 @@ test(function () { // rename
 	FileSystem::rename(TEMP_DIR . '/10', TEMP_DIR . '/9');
 	Assert::false(file_exists(TEMP_DIR . '/9/x/file'));
 	Assert::false(file_exists(TEMP_DIR . '/10'));
+
+	FileSystem::createDir(TEMP_DIR . '/11/');
+	FileSystem::rename(TEMP_DIR . '/11', TEMP_DIR . '/11');
+	Assert::true(file_exists(TEMP_DIR . '/11'));
+	FileSystem::rename(TEMP_DIR . '/11', TEMP_DIR . '/11/');
+	Assert::true(file_exists(TEMP_DIR . '/11'));
 });
 
 Assert::exception(function () {
 	FileSystem::rename(TEMP_DIR . '/10', TEMP_DIR . '/9');
 }, Nette\IOException::class, "File or directory '%S%' not found.");
 
-Assert::exception(function () {
-	FileSystem::rename(TEMP_DIR . '/9', TEMP_DIR . '/9');
-}, Nette\IOException::class, "Unable to rename file or directory '%a%' to '%a%'.");
-
 
 test(function () { // isAbsolute
 	Assert::false(FileSystem::isAbsolute(''));