Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.0.5, 2.1.2, 2.2
-
Component/s: Assignment (2.2)
-
Database:Any
-
Testing Instructions:
-
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:
- Create a new assignment.
- 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).
- Click on the 'Prevent submission updates' button
Gliffy Diagrams
Activity
I tried to replicate this on a fresh install, but the problem didn't occur for me.
Could you have a look at what might be different about your instance that is causing this?
Checked this from a clean install (today) against 2.0.3+ (Build 20110525). Definitely still there.
Extra info:
Installation was Ubuntu, Postgres 8.4, Php 5.3.5
The install had an empty database, only 1 course created for the testing, 1 assignment for the same and a manual test user.
Can provide screencast or screenshots of the process if needed.
An important note is that the user must have never submitted. Otherwise a record will exist that can be updated.
just adding that i've replicated this on 2.0.3+
Hi Jason and Gerwood,
Not sure if am doing anything wrong, but I am not able to replicate this.
Anyways about the patch this needs to be adjusted based on Moodle Coding guidelines..
Ifs should be properly intended.
Rest looks good.
Thanks
The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.
TIA and ciao 
Hi,
Testers please note that this is specific to "advance uploading of files" type assignment.
Thanks
i can't replicate this now - am i missing something? (could this have already been fixed in latest 2.0.x?)
Thanks for fixing up the test instructions Jason. This has been integrated and is up for testing.
Thanks for clear test instructions Jason. It was easy to replicate the issue on stable branches and testing on integration branches
Works as mentioned.
Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks!
Ciao
Have a patch available for this issue here: git://github.com/dogma/moodle.git commit: 1dd0ab5 (
MDL-27521branch)This is an atomic commit which should fix the issue and can be cherry-picked.
It corrects the issue by changing can_finalize and can_unfinalize to recognize when a boolean is passed instead of a submission object. This results in the methods returning false, which removes the button on the submission/grade page.
G