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

misleading error message during file upload

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.6.3, 1.9
    • Fix Version/s: None
    • Component/s: Antivirus, Libraries
    • Labels:
      None
    • Environment:
      Ubuntu 6.06, Apache 2.0.55, MySQL 5.0.22, PHP 5.1.2, Moodle 1.6.3 (2006050530)
    • Affected Branches:
      MOODLE_16_STABLE, MOODLE_19_STABLE
    • URL:
      \lib\uploadlib.php

      Description

      When uploading a file beyond the file max with ClamAV installed I get the error "A required parameter (id) was missing". I consider this misleading as it vaguely refers to something with the code; however, I suspect that what is happening is that Moodle attempts to upload the file and pass it to ClamAV for analysis; however, because it is too large it fails and the id does not get passed. What I think should happen is that Moodle ought to check the file size limit and compare it to the file size of the file. If it is too large then Moodle should report that the file is too large to be uploaded. If ClamAV is not being used then Moodle reports that the file was too large. This lets the user know the reason for the problem is a file that is too large. I would like for it to do the same if ClamAV is being used.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              aborrow Anthony Borrow
              Reporter:
              aborrow Anthony Borrow
              Tester:
              Nobody Nobody
              Participants:
              Component watchers:
              Ruslan Kabalin, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              8 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: