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

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

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: STABLE backlog
    • Component/s: My home
    • Labels:
    • 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

          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: