MDL-32639 files: Zipping a folder does not overwrite existing files

diff --git a/repository/draftfiles_ajax.php b/repository/draftfiles_ajax.php
index 5e89450..31b7e15 100644 (file)
--- a/repository/draftfiles_ajax.php
+++ b/repository/draftfiles_ajax.php
@@ -222,8 +222,9 @@ switch ($action) {
 
         $filepath = explode('/', trim($file->get_filepath(), '/'));
         $filepath = array_pop($filepath);
 
         $filepath = explode('/', trim($file->get_filepath(), '/'));
         $filepath = array_pop($filepath);

+        $zipfile = repository::get_unused_filename($draftid, $parent_path, $filepath . '.zip');

 
 

-        if ($newfile = $zipper->archive_to_storage(array($filepath => $file), $user_context->id, 'user', 'draft', $draftid, $parent_path, $filepath.'.zip', $USER->id)) {
+        if ($newfile = $zipper->archive_to_storage(array($filepath => $file), $user_context->id, 'user', 'draft', $draftid, $parent_path, $zipfile, $USER->id)) {

             $return = new stdClass();
             $return->filepath = $parent_path;
             echo json_encode($return);
             $return = new stdClass();
             $return->filepath = $parent_path;
             echo json_encode($return);

diff --git a/repository/draftfiles_manager.php b/repository/draftfiles_manager.php
index fa966fc..7f4dee7 100644 (file)
--- a/repository/draftfiles_manager.php
+++ b/repository/draftfiles_manager.php
@@ -168,6 +168,7 @@ case 'zip':
         $filename = $filepath.'.zip';
     }
 
         $filename = $filepath.'.zip';
     }
 

+    $filename = repository::get_unused_filename($itemid, $parent_path, $filename);

     $newfile = $zipper->archive_to_storage(array($filepath => $file), $user_context->id, 'user', 'draft', $itemid, $parent_path, $filename, $USER->id);
 
     $home_url->param('action', 'browse');
     $newfile = $zipper->archive_to_storage(array($filepath => $file), $user_context->id, 'user', 'draft', $itemid, $parent_path, $filename, $USER->id);
 
     $home_url->param('action', 'browse');