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

Question types using uploaded files in responses cannot be automatically graded

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.4, 2.3.1, 2.4
    • Fix Version/s: 2.5
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide
      1. Run all the unit tests.
      2. Create a quiz containing an essay questions that allows files in the HTML editor, and attachments.
      3. Attempt the quiz as a student, and make sure you can embed an image in the editor, and attach a file, and then Submit all and finish without errors.
      4. Make sure that when you review the quiz as teacher, you can see both the image and the attached file.
      Show
      Run all the unit tests. Create a quiz containing an essay questions that allows files in the HTML editor, and attachments. Attempt the quiz as a student, and make sure you can embed an image in the editor, and attach a file, and then Submit all and finish without errors. Make sure that when you review the quiz as teacher, you can see both the image and the attached file.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      There is no way to get the files in the grade_responses method, either initially when the student submits their response, or later when the question is re-graded.

      See http://moodle.org/mod/forum/discuss.php?d=208009#p906877.

      Two things need to be done:

      1. add a get_files method to question_file_saver
      2. in question_attempt_step::load_from_records, we need to create an instance of a new class question_file_loader, which also implements the get_files and __toString methods, for those data values that have files.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                timhunt Tim Hunt
                Reporter:
                timhunt Tim Hunt
                Peer reviewer:
                Tim Hunt
                Integrator:
                Dan Poltawski
                Tester:
                Mark Nelson
                Participants:
                Component watchers:
                Tim Hunt, Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
              • Votes:
                6 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13