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

If user drag'n'drops a folder into a course, tell them to try zipping it first

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. (Purge all caches, if a version change hasn't already achieved this)
      2. View a course and turn editing on
      3. Drag and drop a folder onto the course
      4. verify the error message suggests that the user 'zip' the folder before uploading
      Show
      (Purge all caches, if a version change hasn't already achieved this) View a course and turn editing on Drag and drop a folder onto the course verify the error message suggests that the user 'zip' the folder before uploading
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-35288_dndupload_folder_warning

      Description

      If a user tries to drag and drop a folder into Moodle, they get an error message that says:

      "Error: The file ..... has no data in it - did you try to upload a folder?"

      However there's already functionality available in Moodle that if you upload a zipped folder, it will ask you what you want to do with it and the default option is to create a folder activity with the unzipped contents of the folder.

      That's really cool functionality, and would be more discoverable if the error message above said something like:

      ""Error: The file is empty or a folder. To upload folders zip them first"

        Attachments

          Activity

            People

            Assignee:
            davosmith Davo Smith
            Reporter:
            bawjaws David Scotson
            Integrator:
            Sam Hemelryk
            Tester:
            Rossiani Wijaya
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Barbara Ramiro, Bas Brands
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              12/Nov/12