MDL-10702 fixing handling of slashes in grade forms
This commit is contained in:
@@ -39,7 +39,7 @@ if ($mform->is_cancelled()) {
|
||||
$calculation = grade_item::denormalize_formula($grade_item->calculation, $grade_item->courseid);
|
||||
$mform->set_data(array('courseid'=>$grade_item->courseid, 'calculation'=>$calculation, 'id'=>$grade_item->id, 'itemname'=>$grade_item->itemname));
|
||||
|
||||
if ($data = $mform->get_data()) {
|
||||
if ($data = $mform->get_data(false)) {
|
||||
$grade_item->set_calculation($data->calculation);
|
||||
redirect($returnurl);
|
||||
}
|
||||
|
||||
@@ -34,7 +34,7 @@ if ($category = get_record('grade_categories', 'id', $id, 'courseid', $course->i
|
||||
if ($mform->is_cancelled()) {
|
||||
redirect($returnurl);
|
||||
|
||||
} else if ($data = $mform->get_data()) {
|
||||
} else if ($data = $mform->get_data(false)) {
|
||||
$grade_category = new grade_category(array('id'=>$id, 'courseid'=>$course->id));
|
||||
grade_category::set_properties($grade_category, $data);
|
||||
|
||||
|
||||
@@ -82,7 +82,7 @@ if ($mform->is_cancelled()) {
|
||||
redirect($returnurl);
|
||||
|
||||
// form processing
|
||||
} else if ($data = $mform->get_data()) {
|
||||
} else if ($data = $mform->get_data(false)) {
|
||||
$old_grade_grade = new grade_grade(array('userid'=>$data->userid, 'itemid'=>$grade_item->id), true); //might not exist yet
|
||||
|
||||
// update final grade or feedback
|
||||
@@ -90,7 +90,7 @@ if ($mform->is_cancelled()) {
|
||||
|
||||
$grade_grade = grade_grade::fetch(array('userid'=>$data->userid, 'itemid'=>$grade_item->id));
|
||||
|
||||
$grade_grade->set_hidden($data->hidden); // TODO: this is wrong!
|
||||
$grade_grade->set_hidden($data->hidden); // TODO: this is wrong - hidden might be a data to hide until
|
||||
|
||||
// ignore overridden flag when changing final grade
|
||||
if ($old_grade_grade->finalgrade == $grade_grade->finalgrade) {
|
||||
|
||||
@@ -41,7 +41,7 @@ if ($item = get_record('grade_items', 'id', $id, 'courseid', $course->id)) {
|
||||
$mform->set_data($item);
|
||||
}
|
||||
|
||||
if ($data = $mform->get_data()) {
|
||||
if ($data = $mform->get_data(false)) {
|
||||
if (array_key_exists('calculation', $data)) {
|
||||
$data->calculation = grade_item::normalize_formula($data->calculation, $course->id);
|
||||
}
|
||||
|
||||
@@ -46,7 +46,7 @@ if ($item = get_record('grade_items', 'id', $id, 'courseid', $course->id)) {
|
||||
$mform->set_data($item);
|
||||
}
|
||||
|
||||
if ($data = $mform->get_data()) {
|
||||
if ($data = $mform->get_data(false)) {
|
||||
if (array_key_exists('calculation', $data)) {
|
||||
$data->calculation = grade_item::normalize_formula($data->calculation, $course->id);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user