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

disabled ajax and 'quick feedback' in grader report causes grades to become 'overridden' with multiline module feedback

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Have a course with multiple students enrolled, and a few offline assignments
      2. Grade some student submissions with multiline feedback.
      3. In the grader report settings, turn AJAX off (grade_report_enableajax) and Quick Feedback on (grade_report_showquickfeedback). (This means you should see feedback boxes too)
      4. In the grader report, turn editing on. Hit save down at the bottom.
      5. Confirm that non of the cells have turned colors to indicate overridden.
      Show
      Have a course with multiple students enrolled, and a few offline assignments Grade some student submissions with multiline feedback. In the grader report settings, turn AJAX off (grade_report_enableajax) and Quick Feedback on (grade_report_showquickfeedback). (This means you should see feedback boxes too) In the grader report, turn editing on. Hit save down at the bottom. Confirm that non of the cells have turned colors to indicate overridden.
    • Workaround:
      Hide

      Disable Quick Grading

      Show
      Disable Quick Grading
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-40626-master

      Description

      This is another fix raised in https://tracker.moodle.org/browse/MDL-34363.

      When you are in the viewing the gradebook in 'grader report' and have 'show quick feedback' enabled so you have the feedback box and disable 'Enable AJAX'.

      Then go into edit mode in the grader report and click 'update'. All fields with multiline feedback from a module like assign / assignment are now set to overridden.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              tlock Tim Lock
              Reporter:
              tlock Tim Lock
              Peer reviewer:
              Dan Poltawski
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              David Monllaó
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Sep/13