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

workflow_state_updated event not fired/logged when updating individual grade

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.7.3, 2.8.1, 2.9
    • 2.7.5, 2.8.3
    • Assignment
    • MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MDL-48574_workflow_state_updated_batch_single
    • Hide
      Test 1
      1. Create an assignment that uses Marking workflow.
      2. Submit an assignment.
      3. Using the single assignment grading page, change the workflow state.
        • Examine the logs for that assignment, and find a corresponding "The state of the workflow has been updated." log entry.
      4. Using quick grading, change the workflow state.
        • Examine the logs for that assignment, and find a corresponding "The state of the workflow has been updated." log entry.
      Test 2
      1. Run the unit tests in mod/assign/tests/events_test.php and ensure all passes.
      Show
      Test 1 Create an assignment that uses Marking workflow. Submit an assignment. Using the single assignment grading page, change the workflow state. Examine the logs for that assignment, and find a corresponding "The state of the workflow has been updated." log entry. Using quick grading, change the workflow state. Examine the logs for that assignment, and find a corresponding "The state of the workflow has been updated." log entry. Test 2 Run the unit tests in mod/assign/tests/events_test.php and ensure all passes.

    Description

      The workflow_state_updated event is only called from batch updates [process_set_batch_marking_workflow_state()].

      This means that changing workflow state using the individual grading page [apply_grade_to_user()] does not create a log entry (whilst using the quick grading batch interface does).

      Attachments

        Activity

          People

            balchd Dave Balch
            balchd Dave Balch
            Mark Nelson Mark Nelson
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Dave Cooper Dave Cooper
            Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              2/Feb/15