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 Sam Hemelryk
            Tester:
            Rossiani Wijaya Rossiani Wijaya
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Huong Nguyen, Barbara Ramiro, Bas Brands, Mathew May
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

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