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

mod_assign: assignment submission event is not triggered when submit button is not required

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.2
    • Fix Version/s: 2.4
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide

      The plagiarism API is not directly testable at the moment so this minimal test should be sufficient.

      1. Create an assignment with "Require students click submit button" set to off and "Online Text" submissions on.
      2. Login as a student and make a submission to the assignment
      3. Verify there are no errors reported in the page
      Show
      The plagiarism API is not directly testable at the moment so this minimal test should be sufficient. Create an assignment with "Require students click submit button" set to off and "Online Text" submissions on. Login as a student and make a submission to the assignment Verify there are no errors reported in the page
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:

      Description

      When pressing submit button is not required (assignment settings), submission event (currently assessable_content_done, see related discussion at MDL-35710) will not be trigged (while the assignment will still be submitted). The problem is related to the fact that event trigger is not called from process_save_submission function.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              kabalin Ruslan Kabalin
              Reporter:
              kabalin Ruslan Kabalin
              Peer reviewer:
              Damyon Wiese
              Integrator:
              Dan Poltawski
              Tester:
              Ankit Agarwal
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                3/Dec/12