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

Clicking Prevent submission updates without a submission causes page to crash

    XMLWordPrintable

    Details

    • Database:
      Any
    • Testing Instructions:
      Hide
      1. Create a new "Advanced uploading of files" assignment.
      2. Ensure there are students enrolled in the course.
      3. Go to the grades page for the assignment (View submissions link on assignment page).
      4. Click on the 'Grade' button in the status column for a user (that hasn't submitted yet).

      Result:
      the 'Prevent submission updates' button should not be visible

      Show
      Create a new "Advanced uploading of files" assignment. Ensure there are students enrolled in the course. Go to the grades page for the assignment (View submissions link on assignment page). Click on the 'Grade' button in the status column for a user (that hasn't submitted yet). Result: the 'Prevent submission updates' button should not be visible
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-mdl-27521-master
    • Pull Master Diff URL:

      Description

      The 'Prevent submission updates' button on the grade page for a single user doesn't check for an existing submission when it tries to update the submission record to toggle finalize/unfinalize. This causes a db exception (as the record doesn't exist).

      Replication instructions:

      1. Create a new assignment.
      2. Go to the grades page for the assignment (View submissions link on assignment page).
      3. Click on the 'Grade' button in the status column for a user (that hasn't submitted yet).
      4. Click on the 'Prevent submission updates' button

        Attachments

          Activity

            People

            Assignee:
            phalacee Jason Fowler
            Reporter:
            gerwood Gerwood Stewart
            Peer reviewer:
            Ankit Agarwal
            Integrator:
            Aparup Banerjee
            Tester:
            Rajesh Taneja
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              28/Nov/11