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

If the user drags and drops more files than the forum allows to be attached to a discussion some of them are accepted while others are rejected

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Forms Library
    • Labels:

      Description

      Go into a forum and go to the screen used to create a new discussion.

      If the user drags in more files than than the maximum number of attachments (max attachments is in the forum settings) a random subset of files get attached. If I am a teacher and I have 2 documents (say, the exam location announcement and the practice exam) to distribute to my students but I'm only allowed 1 attachment its unlikely that I'll say "ah well, that'll have to do" when only 1 of those documents attach while the other vanishes. We're just forcing the user to then delete any files that did upload before starting over ie putting those files into a zip then uploading that or similar.

      We should either accept all of the files, or if we can't, then display an error and reject the whole lot.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              davosmith Davo Smith added a comment -

              Agreed - I'll work on this

              Show
              davosmith Davo Smith added a comment - Agreed - I'll work on this
              Hide
              salvetore Michael de Raadt added a comment -

              Hi, Davo.

              I've just triaged this issue, but please continue working on it.

              Show
              salvetore Michael de Raadt added a comment - Hi, Davo. I've just triaged this issue, but please continue working on it.
              Hide
              davosmith Davo Smith added a comment -

              Small change to the code to reject an entire group of files if this would cause the total count to exceed the specified limit for the filemanager.

              A filepicker element will continue to work as before - uploading the first file in the group - as it is easy to drag and drop another file instead (it will replace the original, without having to stop and delete the file). A pop-up message would, that the user has to 'OK' would be more of an inconvenience in this case.

              Show
              davosmith Davo Smith added a comment - Small change to the code to reject an entire group of files if this would cause the total count to exceed the specified limit for the filemanager. A filepicker element will continue to work as before - uploading the first file in the group - as it is easy to drag and drop another file instead (it will replace the original, without having to stop and delete the file). A pop-up message would, that the user has to 'OK' would be more of an inconvenience in this case.
              Hide
              poltawski Dan Poltawski added a comment -

              Makes sense +1

              I'm a parenthesis lover so i'd add an extra one to make the two statements clearer but thats just my personal preference

              Show
              poltawski Dan Poltawski added a comment - Makes sense +1 I'm a parenthesis lover so i'd add an extra one to make the two statements clearer but thats just my personal preference
              Hide
              davosmith Davo Smith added a comment -

              @Dan - I've added 2 more sets of brackets (just to make it a bit clearer), if all is OK, could you put it forward for integration review?

              Show
              davosmith Davo Smith added a comment - @Dan - I've added 2 more sets of brackets (just to make it a bit clearer), if all is OK, could you put it forward for integration review?
              Hide
              poltawski Dan Poltawski added a comment -

              Heh, you put more than i'd have added too

              Show
              poltawski Dan Poltawski added a comment - Heh, you put more than i'd have added too
              Hide
              davosmith Davo Smith added a comment -

              Better safe than sorry... Thanks

              Show
              davosmith Davo Smith added a comment - Better safe than sorry... Thanks
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              davosmith Davo Smith added a comment -

              Rebase done (without conflicts)

              Show
              davosmith Davo Smith added a comment - Rebase done (without conflicts)
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks Davo, this has been integrated now

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks Davo, this has been integrated now
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              This is working great!
              Passing
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - This is working great! Passing Thanks
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!).

              icao_reverse('arreis olik rebemevon afla letoh ognat');

              Closing, ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!). icao_reverse('arreis olik rebemevon afla letoh ognat'); Closing, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12