mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-07-16 20:06:30 +02:00
Code Issues Projects Releases Wiki Activity

MDL-9506 Resolved a small bug in grade_category that failed the set_as_parent method.

Browse Source
This commit is contained in:
nicolasconnault
2007-05-21 02:22:12 +00:00
parent 83bc64dba4
commit 750b0550d9
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/grade/grade_category.php
View File

@@ -744,9 +744,10 @@ class grade_category extends grade_object {
return false;
}
$query = "UPDATE {$CFG->prefix}grade_items SET sortorder = sortorder + 1 WHERE sortorder >= $this->grade_item->sortorder";
$query = "UPDATE {$CFG->prefix}grade_items SET sortorder = sortorder + 1 WHERE sortorder >= {$this->grade_item->sortorder}";
if (!execute_sql($query)) {
debugging("Could not update the sortorder of grade_items listed after this category.");
return false;
} else {
return true;
}

3
lib/simpletest/grade/simpletest/testgradecategory.php
View File

@@ -223,6 +223,7 @@ class grade_category_test extends gradelib_test {
$grade_category = new grade_category();
$grade_category->fullname = 'new topcategory';
$grade_category->courseid = $this->courseid;
$grade_category->insert();
// 1. mixed types of children
$child1 = new grade_item();
@@ -254,6 +255,8 @@ class grade_category_test extends gradelib_test {
$child2->itemname = 'new grade_item';
$child1->sortorder = 1;
$child2->sortorder = 2;
$child1->insert();
$child2->insert();
$this->assertTrue($grade_category->set_as_parent(array($child1, $child2)));
}
}