mirror of
https://github.com/moodle/moodle.git
synced 2025-08-04 08:26:37 +02:00
MDL-18095 gradebook: Make sure grade override flag is cleared whenever required
The check for feedback and grade equality between recorded values and the current form ones before removing override flag is not really required.
This commit is contained in:
parent
8075bd8d8a
commit
ed88a0b815
1 changed files with 5 additions and 7 deletions
|
@ -193,21 +193,19 @@ if ($mform->is_cancelled()) {
|
|||
$data->feedbackformat = $old_grade_grade->feedbackformat;
|
||||
}
|
||||
// update final grade or feedback
|
||||
// when we set override grade the first time, it happens here
|
||||
$grade_item->update_final_grade($data->userid, $data->finalgrade, 'editgrade', $data->feedback, $data->feedbackformat);
|
||||
|
||||
$grade_grade = new grade_grade(array('userid'=>$data->userid, 'itemid'=>$grade_item->id), true);
|
||||
$grade_grade->grade_item =& $grade_item; // no db fetching
|
||||
|
||||
if (has_capability('moodle/grade:manage', $context) or has_capability('moodle/grade:edit', $context)) {
|
||||
if (!grade_floats_different($data->finalgrade, $old_grade_grade->finalgrade)
|
||||
and $data->feedback === $old_grade_grade->feedback) {
|
||||
// change overridden flag only if grade or feedback not changed
|
||||
// change overridden flag
|
||||
if (!isset($data->overridden)) {
|
||||
$data->overridden = 0; // checkbox
|
||||
$data->overridden = 0; // checkbox unticked
|
||||
}
|
||||
$grade_grade->set_overridden($data->overridden);
|
||||
}
|
||||
}
|
||||
|
||||
if (has_capability('moodle/grade:manage', $context) or has_capability('moodle/grade:hide', $context)) {
|
||||
$hidden = empty($data->hidden) ? 0: $data->hidden;
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue