Browse code

Try to not overwrite existing files

Benjamin Roth authored on12/10/2023 15:11:12
Showing1 changed files
... ...
@@ -196,7 +196,8 @@ class SecureDownloadsJob
196 196
                 }
197 197
 
198 198
                 // Remove fragments from file name
199
-                $sanitizedFilename = preg_replace('/'.$objRoots->secureDownloadsRegExp.'/U','',$objFile->filename).'.'.$objFile->extension;
199
+                $baseFilename = preg_replace('/'.$objRoots->secureDownloadsRegExp.'/U','',$objFile->filename);
200
+                $sanitizedFilename = $baseFilename.'.'.$objFile->extension;
200 201
 
201 202
                 // Check or create folder structure
202 203
                 $strSubFolder = sprintf("%03d",ceil(($objMember->id+1)/100)-1).'/'.$objMember->id;
... ...
@@ -216,10 +217,9 @@ class SecureDownloadsJob
216 217
 
217 218
                 // Do not overwrite existing files
218 219
                 $copyId = 1;
219
-                $baseFilename = $sanitizedFilename;
220 220
                 while(file_exists(TL_ROOT . "/".$strTargetFolder.'/'.$sanitizedFilename))
221 221
                 {
222
-                    $sanitizedFilename = $baseFilename.'_'.$copyId++;
222
+                    $sanitizedFilename = $baseFilename.'_'.$copyId++.'.'.$objFile->extension;
223 223
                 }
224 224
 
225 225
                 // Try to copy file