From 025bc8d0b125906c581d03e6c35e6bb8c79594d4 Mon Sep 17 00:00:00 2001
From: nicolasconnault <nicolasconnault>
Date: Wed, 18 Jul 2007 03:14:06 +0000
Subject: [PATCH] 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.
---
 grade/edit/item.php | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

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');
-- 
2.39.5