mahara-contributors team mailing list archive

[Mahara Bot <1904946@xxxxxxxxxxxxxxxxxx>]

Patch for "master" branch: https://reviews.mahara.org/11744

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1904946

Title:
  Filebrowser - Uploading number of files exceeding limit gives never-
  ending spinner

Status in Mahara:
  Fix Released

Bug description:
  In the /etc/php/7.2/apache2/php.ini files it states max_file_uploads =
  20

  Trying to upload more than max_file_uploads in one request will cause
  the wrong error messages to appear.

  Reproduce:
  1. Create -> Files -> Browse
  2. Select 21 files and click 'ok'

  Expect:
  - Info messages appear for each upload-file saying it's being uploaded
  - Once uploaded, the matching info message turns to a temp success message (green) and disappears
  - All the selected files have been uploaded and appear in the user's list of files

  Actual:
  - Info messages appear for each upload-file saying it's being uploaded
  - The the 21st file does not get uploaded due to the restriction but the user doesn't know about the restriction
  - The info message matching the 1st file in the selected upload-list stays forever spinning.

  Notes:
  - Even if a message stays spinning forever because there was a file(s) that didn't get uploaded, the text in the message should match the non-uploaded file
  - There should be a message informing the user of the max number of files they're allowed to upload at any one time
  - The filebrowser could prevent the user from selecting more than the max_file_uploads number

  Todo:
  - (Maximum upload size: [SPACE AVAILABLE]MB, maximum file uploads: [VALLUE FROM PHP.INI FILE], check what happens when PHP.ini doesn't have a value for max_file_uploads)
  - Error message: replace info message with an error message for the same file.
  - (nice to have)check upload_validate js function to check if we can alert the user before files are uploaded

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1904946/+subscriptions