Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-33296

Remove the base directory '0' when downloading all as a zip.

    Details

    • Affected Branches:
      MOODLE_23_STABLE

      Description

      In My Private files, when downloading the zip file with the "Download all" link, a zip is successfully created.
      When opening this zip file there is a folder that shouldn't exist.

      Steps to reproduce:
      Navigate to My Private Files
      Create a folder
      Upload a file to this folder
      Click on "Save changes"
      Navigate to the main folder
      Click on "Download all"
      Open the zip file
      Navigate to the created folder
      There is another folder in the zip file that shouldn't exist.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              tedvdb Ted van den Brink created issue -
              tedvdb Ted van den Brink made changes -
              Field Original Value New Value
              Labels mdlqa
              tedvdb Ted van den Brink made changes -
              Attachment result zip.png [ 28292 ]
              Attachment Folder contents.PNG [ 28293 ]
              tedvdb Ted van den Brink made changes -
              Link This issue blocks MDLQA-2367 [ MDLQA-2367 ]
              moodle.com moodle.com made changes -
              Assignee moodle.com [ moodle.com ] Adrian Greeve [ abgreeve ]
              Hide
              abgreeve Adrian Greeve added a comment -

              Thanks Ted for spotting this issue. I've changed the status of this from a bug to an improvement so as to not block the completion of QA testing. The file system does remain intact and the files are not corrupted, it would be nice however to remove the base folder.

              Thanks again,

              Adrian.

              Show
              abgreeve Adrian Greeve added a comment - Thanks Ted for spotting this issue. I've changed the status of this from a bug to an improvement so as to not block the completion of QA testing. The file system does remain intact and the files are not corrupted, it would be nice however to remove the base folder. Thanks again, Adrian.
              abgreeve Adrian Greeve made changes -
              Summary Non existing folder in download all files zip Download all as a zip creates a base directory '0'
              Issue Type Bug [ 1 ] Improvement [ 4 ]
              Description In My Private files, when downloading the zip file with the "Download all" link, a zip is succefully created.
              When opening this zip file there is a folder that shouldn't exist.

              Steps to reproduce:
              Navigate to My Private Files
              Create a folder
              Upload a file to this folder
              Click on "Save changes"
              Navigate to the main folder
              Click on "Download all"
              Open the zip file
              Navigate to the created folder
              There is another folder in the zip file that shouldn't exist.
              In My Private files, when downloading the zip file with the "Download all" link, a zip is successfully created.
              When opening this zip file there is a folder that shouldn't exist.

              Steps to reproduce:
              Navigate to My Private Files
              Create a folder
              Upload a file to this folder
              Click on "Save changes"
              Navigate to the main folder
              Click on "Download all"
              Open the zip file
              Navigate to the created folder
              There is another folder in the zip file that shouldn't exist.
              abgreeve Adrian Greeve made changes -
              Summary Download all as a zip creates a base directory '0' Remove the base directory '0' when downloading all as a zip.
              Hide
              salvetore Michael de Raadt added a comment -

              Thanks for suggesting this, Ted.

              I've put that on the backlog so we can work in this later.

              Feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it.

              The test associated with this issue has been reset. It would be good if you could re-run the test.

              Show
              salvetore Michael de Raadt added a comment - Thanks for suggesting this, Ted. I've put that on the backlog so we can work in this later. Feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it. The test associated with this issue has been reset. It would be good if you could re-run the test.
              salvetore Michael de Raadt made changes -
              Fix Version/s STABLE backlog [ 10463 ]
              Labels mdlqa triaged
              marina Marina Glancy made changes -
              Link This issue has a non-specific relationship to MDL-32639 [ MDL-32639 ]
              Hide
              marina Marina Glancy added a comment -

              I'm linking it to MDL-32639, which is more complete list of problems when downloading a file

              Show
              marina Marina Glancy added a comment - I'm linking it to MDL-32639 , which is more complete list of problems when downloading a file
              Hide
              vadimon Vadim Dvorovenko added a comment -

              Take a look at MDL-32639 - i've added pathces there, and testing instructions

              Show
              vadimon Vadim Dvorovenko added a comment - Take a look at MDL-32639 - i've added pathces there, and testing instructions
              vadimon Vadim Dvorovenko made changes -
              Link This issue has a non-specific relationship to MDL-36122 [ MDL-36122 ]
              fred Frédéric Massart made changes -
              Link This issue has been marked as being related by MDL-29203 [ MDL-29203 ]
              Hide
              fred Frédéric Massart added a comment -

              Closing as resolved by MDL-32639.

              Show
              fred Frédéric Massart added a comment - Closing as resolved by MDL-32639 .
              fred Frédéric Massart made changes -
              Status Open [ 1 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              fred Frédéric Massart made changes -
              Link This issue will be resolved by MDL-32639 [ MDL-32639 ]
              fred Frédéric Massart made changes -
              Link This issue has a non-specific relationship to MDL-32639 [ MDL-32639 ]

                People

                • Votes:
                  1 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: