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

Private files area quota applies when unzipping to non-private file areas.

XMLWordPrintable

    • MOODLE_401_STABLE, MOODLE_402_STABLE, MOODLE_403_STABLE, MOODLE_404_STABLE
    • MOODLE_401_STABLE, MOODLE_402_STABLE, MOODLE_403_STABLE
    • MDL-81060-401-fix
    • MDL-81060-403-fix
    • MDL-81060-main-fix
    • Hide

      1. Testing SCORM

      1. Login as admin
      2. Set Private files space (userquota) to 0 MB on the site administrator.
      3. Create a course with the Maximum upload size is 500 KB
      4. Assign user as a teacher role to the Course
      5. Logout
      6. Login as the teacher
      7. Add SCORM package activity
      8. Specify the name with "SCORM Test"
      9. Download the SCORM sample file mdl81060_scorm_sample.zip
      10. In the Package file, upload the SCORM sample file
      11. Click the Save and Display button
      12. VERIFY the SCORM created successfully without any errors
      13. VERIFY the SCORM can be previewed.

      2. Testing Private Files

      1. Please follow the MDL-74641 testing instructions
        • Note: Ignore the expected error message at step 12. What we instead expect to see is an error saying the zip file contents cannot be read.
      2. Verify the testing is passed without any errors.
      Show
      1. Testing SCORM Login as admin Set Private files space (userquota) to 0 MB on the site administrator. Create a course with the Maximum upload size is 500 KB Assign user as a teacher role to the Course Logout Login as the teacher Add SCORM package activity Specify the name with "SCORM Test" Download the SCORM sample file mdl81060_scorm_sample.zip In the Package file, upload the SCORM sample file Click the Save and Display button VERIFY the SCORM created successfully without any errors VERIFY the SCORM can be previewed. 2. Testing Private Files Please follow the MDL-74641 testing instructions Note: Ignore the expected error message at step 12. What we instead expect to see is an error saying the zip file contents cannot be read. Verify the testing is passed without any errors.
    • 6
    • Team Hedgehog 2024 Sprint 1.2, Team Hedgehog 2024 Sprint 1.3, Team Hedgehog 2024 Sprint 1.4

      Plugins such as mod_scorm use zips, which are unzipped and stored in the mdl_files when saved.

      Since MDL-74641, there is a check added here that the unzipped file size does not exceed the "Private files space" config:

      https://github.com/moodle/moodle/blob/e567c21d6edb902e652fcbf1aaf4a49a82579f84/lib/filestorage/zip_packer.php#L457-L462

      Two issues with this:

      1. This is confusing for end users, since mod_scorm doesn't touch the private files area, but this setting "Private files space" affects it. I believe this should be at a minimum renamed to "unzip file area quota"
      2. It checks the existing usage of files in the users private file area, regardless if the file being unzipped is in the private file area (in scorm, they are not)

      A better solution would be being able to define max file sizes per-area (so for example, scorm could be 200mb but private files could be 1mb, etc...), and then check the respective area for existing usage.

        1. Screenshot 2024-03-18 at 7.38.01 am.png
          Screenshot 2024-03-18 at 7.38.01 am.png
          63 kB
        2. not_exapanded.png
          not_exapanded.png
          20 kB
        3. mdl81060_warningmessage.png
          mdl81060_warningmessage.png
          19 kB
        4. mdl81060_scorm_sample.zip
          245 kB
        5. mdl81060_imscp_sample.zip
          22 kB
        6. mdl81060_bookchapter_sample.zip
          2 kB
        7. main - 2.16 teacher.png
          main - 2.16 teacher.png
          380 kB
        8. main - 2.16 student.png
          main - 2.16 student.png
          392 kB
        9. main - 1.13.png
          main - 1.13.png
          529 kB
        10. expanded.png
          expanded.png
          50 kB

            meirza.arson@moodle.com Meirza
            matthewhilton Matthew Hilton
            Stevani Andolo Stevani Andolo
            Jake Dallimore Jake Dallimore
            Glyn (Mathew) May Glyn (Mathew) May
            Votes:
            3 Vote for this issue
            Watchers:
            13 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 week, 1 hour, 30 minutes
                1w 1h 30m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.