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

Assign grader doesn't correctly detect changes

XMLWordPrintable

    • MOODLE_401_STABLE

      Background

      When a student submits a file to an assignment and later overrides it, the following should happen (as per MDL-45580):

      1. Teacher grades it, adding annotations
      2. The student will have a "View annotated PDF" link available, which shows the annotated PDF
      3. The student edits the submission, adding another file
      4. The "View annotated PDF" link should still display the old annotated PDF
      5. Teacher grades again, they should see the new PDF, and be able to annotate it
      6. The student's "View annotated PDF" link will now show the newly annotated PDF

      While working on MDL-75898 I found a situation that breaks this. In normal day to day use it seems extremely unlikely this could happen, but it does cause problems for behat tests (where the exact same sequence of "user inputs" can occur).

      Prerequisites

      1. Ensure the patch from MDL-75898 is applied
      2. A course with an assignment
      3. A student in the course
      4. A macro recorder, I used atbswp

      Reproduction steps

      1. Log in as the student and submit the file "Submission 1.pdf" (attached to this ticket)
      2. As admin, grade the student in the PDF editor:
        1. Start the macro recorder
        2. Click the line tool in the PDF editor
        3. Draw a line on the PDF
        4. Stop recording
      3. Press the "Save changes" button
      4. As the student, edit the submission, deleting "Submission 1.pdf" and adding, "Submission 2.pdf" (attached to this ticket)
      5. As admin, grade the student in the PDF editor, it should be the new PDF and there should be no annotations
      6. Play back the macro, this ensures that the new line is in the exact same place as the original one
      7. As the student, go to the assignment and press the "View annotated PDF"
      8. The PDF that displays is the original one instead of the newly annotated one

      It seems as though the code believes annotations haven't changed or something like that, so it never copies the new PDF over to the readonlypages file area.

            Unassigned Unassigned
            cameron1729 cameron1729
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

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