From: nicolasconnault <nicolasconnault>
Date: Wed, 18 Jul 2007 03:14:06 +0000 (+0000)
Subject: MDL-10477 Fixed a small bug in grade/edit/item.php , where I was testing for
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=025bc8d0b125906c581d03e6c35e6bb8c79594d4;p=moodle.git

MDL-10477 Fixed a small bug in grade/edit/item.php , where I was testing for
if (!empty($data->pref_gradedisplaytype))

which returned false if the value was set to 0. I just changed !empty to isset and it fixed it.
---

diff --git a/grade/edit/item.php b/grade/edit/item.php
index 4da1c2f04e..41cd091ca1 100644
--- a/grade/edit/item.php
+++ b/grade/edit/item.php
@@ -41,8 +41,6 @@ if ($item = get_record('grade_items', 'id', $id, 'courseid', $course->id)) {
 }
 
 if ($data = $mform->get_data()) {
-    $errors = array();
-
     if (array_key_exists('calculation', $data)) {
         $data->calculation = grade_item::normalize_formula($data->calculation, $course->id);
     }
@@ -59,19 +57,19 @@ if ($data = $mform->get_data()) {
     }
 
     // Handle user preferences
-    if (!empty($data->pref_gradedisplaytype)) {
+    if (isset($data->pref_gradedisplaytype)) {
         if (!grade_report::set_pref('gradedisplaytype', $data->pref_gradedisplaytype, $grade_item->id)) {
             error("Could not set preference gradedisplaytype to $value for this grade item");
         }
     }
 
-    if (!empty($data->pref_decimalpoints)) {
+    if (isset($data->pref_decimalpoints)) {
         if (!grade_report::set_pref('decimalpoints', $data->pref_decimalpoints, $grade_item->id)) {
-            errors("Could not set preference decimalpoints to $value for this grade item");
+            error("Could not set preference decimalpoints to $value for this grade item");
         }
     }
 
-    redirect($returnurl, 'temporary debug delay', 10);
+    redirect($returnurl, 'temporary debug delay', 50);
 }
 
 $strgrades       = get_string('grades');