Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Blocker
-
Resolution: Fixed
-
Affects Version/s: 2.5
-
Fix Version/s: 2.5
-
Component/s: Assignment
-
Labels:
-
Testing Instructions:
-
Affected Branches:MOODLE_25_STABLE
-
Fixed Branches:MOODLE_25_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-38973-master -
Pull Master Diff URL:
Description
I found that past attempts were not appearing when blind marking was turned on in an assignment.
Replication steps:
- Set up a new assignment
- File submissions on
- Attempts reopened set to Manually
- Blind marking set to Yes
- Log in as a student and submit
- Log in as a teacher and grade the submission and allow a reattempt
Expected result:
- The previous attempt should appear for the student
- The grade on the previous attempt should appear to the student
- The previous attempt should appear for the teacher on the grading sheet
Actual result: Negate the above
Gliffy Diagrams
Issue Links
Activity
Mark, giving you this one, in return for taking MDL-38750 off you.
Show
Dan Poltawski
added a comment - Mark, giving you this one, in return for taking MDL-38750 off you.
Mark has tested it and says its working. Yay.
Show
Dan Poltawski
added a comment - Mark has tested it and says its working. Yay.
Your awesome contributions are now part of Moodle, your fav LMS out there.
Closing this as fixed.
Many thanks for all the hard work, ciao 
Show
Eloy Lafuente (stronk7)
added a comment - Your awesome contributions are now part of Moodle, your fav LMS out there.
Closing this as fixed.
Many thanks for all the hard work, ciao
Hello,
this commit introduces a bug when you have a submission with status ASSIGN_SUBMISSION_STATUS_REOPENED and just one attempt.
Here is the code that fails mod/assign/gradingtable.php:
976 if ($submission->status == ASSIGN_SUBMISSION_STATUS_REOPENED) {
|
977 // For a newly reopened submission - we want to show the previous submission in the table.
|
978 $this->get_group_and_submission($row->id, $group, $submission, $submission->attemptnumber-1);
|
979 }
|
Here is the $row variable record that causes the bug:
stdClass Object
|
(
|
[id] => 2267
|
[picture] => 103145
|
[firstname] => Foo bar
|
[lastname] => (-----)
|
[imagealt] =>
|
[email] => mmmm@mmm.com
|
[userid] => 2267
|
[status] => reopened
|
[submissionid] => 3877
|
[firstsubmission] => 1379348815
|
[timesubmitted] => 1379348815
|
[attemptnumber] => 1
|
[gradeid] => 2401
|
[grade] =>
|
[timemarked] => 1379348810
|
[firstmarked] => 1379348810
|
[mailed] => 1
|
[locked] => 0
|
[extensionduedate] => 0
|
)
|
Here is the obvius (but not exactly the best) fix:
--- a/mod/assign/gradingtable.php
|
+++ b/mod/assign/gradingtable.php
|
@@ -988,7 +988,11 @@ class assign_grading_table extends table_sql implements renderable {
|
} else if ($row->submissionid) {
|
if ($row->status == ASSIGN_SUBMISSION_STATUS_REOPENED) {
|
// For a newly reopened submission - we want to show the previous submission in the table.
|
- $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber - 1);
|
+ if ($row->attemptnumber > 1) {
|
+ $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber - 1)
|
+ } else {
|
+ $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber);
|
+ }
|
} else {
|
$submission = new stdClass();
|
$submission->id = $row->submissionid;
|
|
Would appreciate very much if someone could take a look.
Kind regards,
Daniel
Show
Daniel Neis
added a comment - Hello,
this commit introduces a bug when you have a submission with status ASSIGN_SUBMISSION_STATUS_REOPENED and just one attempt.
Here is the code that fails mod/assign/gradingtable.php:
976 if ($submission->status == ASSIGN_SUBMISSION_STATUS_REOPENED) {
977 // For a newly reopened submission - we want to show the previous submission in the table.
978 $this->get_group_and_submission($row->id, $group, $submission, $submission->attemptnumber-1);
979 }
Here is the $row variable record that causes the bug:
stdClass Object
(
[id] => 2267
[picture] => 103145
[firstname] => Foo bar
[lastname] => (-----)
[imagealt] =>
[email] => mmmm@mmm.com
[userid] => 2267
[status] => reopened
[submissionid] => 3877
[firstsubmission] => 1379348815
[timesubmitted] => 1379348815
[attemptnumber] => 1
[gradeid] => 2401
[grade] =>
[timemarked] => 1379348810
[firstmarked] => 1379348810
[mailed] => 1
[locked] => 0
[extensionduedate] => 0
)
Here is the obvius (but not exactly the best) fix:
--- a/mod/assign/gradingtable.php
+++ b/mod/assign/gradingtable.php
@@ -988,7 +988,11 @@ class assign_grading_table extends table_sql implements renderable {
} else if ($row->submissionid) {
if ($row->status == ASSIGN_SUBMISSION_STATUS_REOPENED) {
// For a newly reopened submission - we want to show the previous submission in the table.
- $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber - 1);
+ if ($row->attemptnumber > 1) {
+ $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber - 1)
+ } else {
+ $submission = $this->assignment->get_user_submission($row->userid, false, $row->attemptnumber);
+ }
} else {
$submission = new stdClass();
$submission->id = $row->submissionid;
Would appreciate very much if someone could take a look.
Kind regards,
Daniel
Hi Daniel,
Thanks for investigating this.
If you could create a separate issue to resolve this and link it here that would be very much appreciated.
Regards,
Mark
Show
Mark Nelson
added a comment - Hi Daniel,
Thanks for investigating this.
If you could create a separate issue to resolve this and link it here that would be very much appreciated.
Regards,
Mark
Doh, i'd integrated this but not updated the status.