From: skodak <skodak>
Date: Mon, 10 Mar 2008 20:25:29 +0000 (+0000)
Subject: MDL-13878 use ints for old values of scales in grader report; merged from MOODLE_19_S... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=99ccfda8e290419eaee88c52b30307395f187181;p=moodle.git

MDL-13878 use ints for old values of scales in grader report; merged from MOODLE_19_STABLE
---

diff --git a/grade/report/grader/lib.php b/grade/report/grader/lib.php
index cf8bd36d6a..43388bb815 100644
--- a/grade/report/grader/lib.php
+++ b/grade/report/grader/lib.php
@@ -175,7 +175,7 @@ class grade_report_grader extends grade_report {
             $oldvalue = $data->{'old'.$varname};
 
             // was change requested?
-            if ($oldvalue == $postedvalue) {
+            if ($oldvalue == $postedvalue) { // string comparison
                 continue;
             }
 
@@ -776,7 +776,7 @@ class grade_report_grader extends grade_report {
 
                     if ($item->scaleid && !empty($scales_array[$item->scaleid])) {
                         $scale = $scales_array[$item->scaleid];
-
+                        $gradeval = (int)$gradeval; // scales use only integers
                         $scales = explode(",", $scale->scale);
                         // reindex because scale is off 1
 
@@ -805,7 +805,7 @@ class grade_report_grader extends grade_report {
                             $scales = explode(",", $scale->scale);
 
                             // invalid grade if gradeval < 1
-                            if ((int) $gradeval < 1) {
+                            if ($gradeval < 1) {
                                 $studentshtml .= '<span class="gradevalue'.$hidden.$gradepass.'">-</span>';
                             } else {
                                 $gradeval = (int)bounded_number($grade->grade_item->grademin, $gradeval, $grade->grade_item->grademax); //just in case somebody changes scale