From 750b0550d9c65f5c87601ca6be3a21c74e2ee01c Mon Sep 17 00:00:00 2001
From: nicolasconnault <nicolasconnault>
Date: Mon, 21 May 2007 02:22:12 +0000
Subject: [PATCH] MDL-9506 Resolved a small bug in grade_category that failed
 the set_as_parent method.

---
 lib/grade/grade_category.php                          | 5 +++--
 lib/simpletest/grade/simpletest/testgradecategory.php | 3 +++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/lib/grade/grade_category.php b/lib/grade/grade_category.php
index d39ed115b9..a0195837c6 100644
--- a/lib/grade/grade_category.php
+++ b/lib/grade/grade_category.php
@@ -743,10 +743,11 @@ class grade_category extends grade_object {
             debugging("Could not update this category's sortorder in DB.");
             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;
         }
diff --git a/lib/simpletest/grade/simpletest/testgradecategory.php b/lib/simpletest/grade/simpletest/testgradecategory.php
index 22cf81d369..73b4a9ffb4 100755
--- a/lib/simpletest/grade/simpletest/testgradecategory.php
+++ b/lib/simpletest/grade/simpletest/testgradecategory.php
@@ -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)));
     }
 } 
-- 
2.39.5