mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-03-18 22:50:19 +01:00
Code Issues Projects Releases Wiki Activity

MDL-48235 mod_assign: Fix reopen until pass behavior

Browse Source
This commit is contained in:
Matt Petro 2014-12-02 12:59:52 -05:00
parent d87bcfb325
commit e77bacab4f
2 changed files with 177 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
mod/assign/locallib.php
View File

@ -1882,20 +1882,14 @@ class assign {
// Set it to the default.
$grade->attemptnumber = 0;
}
$result = $DB->update_record('assign_grades', $grade);
$DB->update_record('assign_grades', $grade);
// If the conditions are met, allow another attempt.
$submission = null;
if ($this->get_instance()->teamsubmission) {
$submission = $this->get_group_submission($grade->userid, 0, false);
} else {
$submission = $this->get_user_submission($grade->userid, false);
}
if ($submission && $submission->attemptnumber == $grade->attemptnumber) {
$this->reopen_submission_if_required($grade->userid,
$submission,
$reopenattempt);
}
// Only push to gradebook if the update is for the latest attempt.
// Not the latest attempt.
@ -1903,11 +1897,16 @@ class assign {
return true;
}
if ($result) {
$this->gradebook_item_update(null, $grade);
\mod_assign\event\submission_graded::create_from_grade($this, $grade)->trigger();
$this->gradebook_item_update(null, $grade);
// If the conditions are met, allow another attempt.
if ($submission) {
$this->reopen_submission_if_required($grade->userid,
$submission,
$reopenattempt);
}
return $result;
\mod_assign\event\submission_graded::create_from_grade($this, $grade)->trigger();
return true;
}
/**
@ -6630,7 +6629,8 @@ class assign {
if ($gradeitem) {
$gradegrade = grade_grade::fetch(array('userid' => $userid, 'itemid' => $gradeitem->id));
if ($gradegrade && !$gradegrade->is_passed()) {
// Do not reopen if is_passed returns null, e.g. if there is no pass criterion set.
if ($gradegrade && ($gradegrade->is_passed() === false)) {
$shouldreopen = true;
}
}

165
mod/assign/tests/locallib_test.php
View File

@ -1498,6 +1498,171 @@ class mod_assign_locallib_testcase extends mod_assign_base_testcase {
}
/**
* Test reopen behavior when in "Reopen until pass" mode.
*/
public function test_attempt_reopen_method_untilpass() {
global $PAGE;
$this->setUser($this->editingteachers[0]);
$assign = $this->create_instance(array('attemptreopenmethod' => ASSIGN_ATTEMPT_REOPEN_METHOD_UNTILPASS,
'maxattempts' => 3,
'submissiondrafts' => 1,
'assignsubmission_onlinetext_enabled' => 1));
$PAGE->set_url(new moodle_url('/mod/assign/view.php', array('id' => $assign->get_course_module()->id)));
// Set grade to pass to 80.
$gradeitem = $assign->get_grade_item();
$gradeitem->gradepass = '80.0';
$gradeitem->update();
// Student should be able to see an add submission button.
$this->setUser($this->students[0]);
$output = $assign->view_student_summary($this->students[0], true);
$this->assertNotEquals(false, strpos($output, get_string('addsubmission', 'assign')));
// Add a submission.
$now = time();
$submission = $assign->get_user_submission($this->students[0]->id, true);
$data = new stdClass();
$data->onlinetext_editor = array('itemid' => file_get_unused_draft_itemid(),
'text' => 'Submission text',
'format' => FORMAT_MOODLE);
$plugin = $assign->get_submission_plugin_by_type('onlinetext');
$plugin->save($submission, $data);
// And now submit it for marking.
$submission->status = ASSIGN_SUBMISSION_STATUS_SUBMITTED;
$assign->testable_update_submission($submission, $this->students[0]->id, true, false);
// Verify the student cannot make a new attempt.
$output = $assign->view_student_summary($this->students[0], true);
$this->assertEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
// Mark the submission as non-passing.
$this->setUser($this->teachers[0]);
$data = new stdClass();
$data->grade = '50.0';
$assign->testable_apply_grade_to_user($data, $this->students[0]->id, 0);
// Check the student can see the grade.
$this->setUser($this->students[0]);
$output = $assign->view_student_summary($this->students[0], true);
$this->assertNotEquals(false, strpos($output, '50.0'));
// Check that the student now has a button for Add a new attempt.
$output = $assign->view_student_summary($this->students[0], true);
$this->assertNotEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
// Check that the student now does not have a button for Submit.
$this->assertEquals(false, strpos($output, get_string('submitassignment', 'assign')));
// Check that the student now has a submission history.
$this->assertNotEquals(false, strpos($output, get_string('attempthistory', 'assign')));
// Add a second submission.
$now = time();
$submission = $assign->get_user_submission($this->students[0]->id, true, 1);
$data = new stdClass();
$data->onlinetext_editor = array('itemid' => file_get_unused_draft_itemid(),
'text' => 'Submission text',
'format' => FORMAT_MOODLE);
$plugin = $assign->get_submission_plugin_by_type('onlinetext');
$plugin->save($submission, $data);
// And now submit it for marking.
$submission->status = ASSIGN_SUBMISSION_STATUS_SUBMITTED;
$assign->testable_update_submission($submission, $this->students[0]->id, true, false);
// Mark the submission as passing.
$this->setUser($this->teachers[0]);
$data = new stdClass();
$data->grade = '80.0';
$assign->testable_apply_grade_to_user($data, $this->students[0]->id, 1);
// Check that the student does not have a button for Add a new attempt.
$this->setUser($this->students[0]);
$output = $assign->view_student_summary($this->students[0], true);
$this->assertEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
// Re-mark the submission as not passing.
$this->setUser($this->teachers[0]);
$data = new stdClass();
$data->grade = '50.0';
$assign->testable_apply_grade_to_user($data, $this->students[0]->id, 1);
// Check that the student now has a button for Add a new attempt.
$this->setUser($this->students[0]);
$output = $assign->view_student_summary($this->students[0], true);
$this->assertNotEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
// Add a submission as a second student.
$this->setUser($this->students[1]);
$now = time();
$submission = $assign->get_user_submission($this->students[1]->id, true);
$data = new stdClass();
$data->onlinetext_editor = array('itemid' => file_get_unused_draft_itemid(),
'text' => 'Submission text',
'format' => FORMAT_MOODLE);
$plugin = $assign->get_submission_plugin_by_type('onlinetext');
$plugin->save($submission, $data);
// And now submit it for marking.
$submission->status = ASSIGN_SUBMISSION_STATUS_SUBMITTED;
$assign->testable_update_submission($submission, $this->students[1]->id, true, false);
// Mark the submission as passing.
$this->setUser($this->teachers[0]);
$data = new stdClass();
$data->grade = '100.0';
$assign->testable_apply_grade_to_user($data, $this->students[1]->id, 0);
// Check the student can see the grade.
$this->setUser($this->students[1]);
$output = $assign->view_student_summary($this->students[1], true);
$this->assertNotEquals(false, strpos($output, '100.0'));
// Check that the student does not have a button for Add a new attempt.
$output = $assign->view_student_summary($this->students[1], true);
$this->assertEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
// Set grade to pass to 0, so that no attempts should reopen.
$gradeitem = $assign->get_grade_item();
$gradeitem->gradepass = '0';
$gradeitem->update();
// Add another submission.
$this->setUser($this->students[2]);
$now = time();
$submission = $assign->get_user_submission($this->students[2]->id, true);
$data = new stdClass();
$data->onlinetext_editor = array('itemid' => file_get_unused_draft_itemid(),
'text' => 'Submission text',
'format' => FORMAT_MOODLE);
$plugin = $assign->get_submission_plugin_by_type('onlinetext');
$plugin->save($submission, $data);
// And now submit it for marking.
$submission->status = ASSIGN_SUBMISSION_STATUS_SUBMITTED;
$assign->testable_update_submission($submission, $this->students[2]->id, true, false);
// Mark the submission as graded.
$this->setUser($this->teachers[0]);
$data = new stdClass();
$data->grade = '0.0';
$assign->testable_apply_grade_to_user($data, $this->students[2]->id, 0);
// Check the student can see the grade.
$this->setUser($this->students[2]);
$output = $assign->view_student_summary($this->students[2], true);
$this->assertNotEquals(false, strpos($output, '0.0'));
// Check that the student does not have a button for Add a new attempt.
$output = $assign->view_student_summary($this->students[2], true);
$this->assertEquals(false, strpos($output, get_string('addnewattempt', 'assign')));
}
public function test_markingworkflow() {
global $PAGE;