[Media] Dont allow extensions to be changed

thedevdojo · Sep 1, 2021 · 523fa3c · 523fa3c
1 parent b090aa2
commit 523fa3c
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/publishable/lang/de/media.php b/publishable/lang/de/media.php
@@ -14,6 +14,7 @@
     'error_deleting_folder'  => 'Beim Versuch diesen Ordner zu löschen ist ein Fehler aufgetreten. Stellen Sie sicher, dass Sie ausreichende Zugriffsrechte dafür haben',
     'error_may_exist'        => 'Datei oder Ordner unter diesem Namen können bereits existieren. Wählen Sie einen anderen Namen oder löschen Sie die andere Datei.',
     'error_moving'           => 'Beim Versuch diese Datei bzw. Ordner zu verschieben ist ein Fehler aufgetreten. Stellen Sie sicher, dass Sie ausreichende Zugriffsrechte dafür haben.',
+    'error_renaming_ext'     => 'Die Erweiterung einer Datei kann nicht geändert werden!',
     'error_uploading'        => 'Hochladen fehlgeschlagen: Unbekannter Fehler aufgetreten!',
     'files'                  => '1 Datei|:count Dateien',
     'files_more'             => 'Und :count Weitere',

diff --git a/publishable/lang/en/media.php b/publishable/lang/en/media.php
@@ -14,6 +14,7 @@
     'error_deleting_folder'  => 'Sorry something seems to have gone wrong when deleting this folder, please check your permissions',
     'error_may_exist'        => 'File or Folder may already exist with that name. Please choose another name or delete the other file.',
     'error_moving'           => 'Sorry there seems to be a problem moving that file/folder, please make sure you have the correct permissions.',
+    'error_renaming_ext'     => 'The extension of a file can not be changed!',
     'error_uploading'        => 'Upload Fail: Unknown error occurred!',
     'files'                  => '1 file|:count files',
     'files_more'             => 'And :count more',

diff --git a/src/Http/Controllers/VoyagerMediaController.php b/src/Http/Controllers/VoyagerMediaController.php
@@ -190,20 +190,24 @@ public function rename(Request $request)
         $success = false;
         $error = false;
 
-        if (is_array($folderLocation)) {
-            $folderLocation = rtrim(implode('/', $folderLocation), '/');
-        }
+        if (pathinfo($filename)['extension'] !== pathinfo($newFilename)['extension']) {
+            $error = __('voyager::media.error_renaming_ext');
+        } else {
+            if (is_array($folderLocation)) {
+                $folderLocation = rtrim(implode('/', $folderLocation), '/');
+            }
 
-        $location = "{$this->directory}/{$folderLocation}";
+            $location = "{$this->directory}/{$folderLocation}";
 
-        if (!Storage::disk($this->filesystem)->exists("{$location}/{$newFilename}")) {
-            if (Storage::disk($this->filesystem)->move("{$location}/{$filename}", "{$location}/{$newFilename}")) {
-                $success = true;
+            if (!Storage::disk($this->filesystem)->exists("{$location}/{$newFilename}")) {
+                if (Storage::disk($this->filesystem)->move("{$location}/{$filename}", "{$location}/{$newFilename}")) {
+                    $success = true;
+                } else {
+                    $error = __('voyager::media.error_moving');
+                }
             } else {
-                $error = __('voyager::media.error_moving');
+                $error = __('voyager::media.error_may_exist');
             }
-        } else {
-            $error = __('voyager::media.error_may_exist');
         }
 
         return compact('success', 'error');