From 0a2c84851cf66e3c27642a7d9ffa03d4ea3e5e6b Mon Sep 17 00:00:00 2001
From: tjhunt <tjhunt>
Date: Thu, 13 Dec 2007 14:19:24 +0000
Subject: [PATCH] MDL-12517 - Move checking code into the user interface layer
 where it belongs. Merged from MOODLE_19_STABLE.

---
 grade/report/grader/lib.php | 14 ++++++++++++++
 lib/grade/grade_item.php    | 12 ------------
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/grade/report/grader/lib.php b/grade/report/grader/lib.php
index 08c1588c8d..d5f93b63b1 100644
--- a/grade/report/grader/lib.php
+++ b/grade/report/grader/lib.php
@@ -199,6 +199,20 @@ class grade_report_grader extends grade_report {
                     $finalgrade = unformat_float($postedvalue);
                 }
 
+                // Warn if the grade is out of bounds.
+                $errorstr = '';
+                if ($finalgrade < $grade_item->grademax) {
+                    $errorstr = 'lessthanmin';
+                } else if ($finalgrade > $grade_item->grademax) {
+                    $errorstr = 'morethanmax';
+                }
+                if ($errorstr) {
+                    $user = get_record('user', 'id', $userid,'','','','','id, firstname, lastname');
+                    $gradestr->username = fullname($user);
+                    $gradestr->itemname = $grade_item->get_name();
+                    notify(get_string($errorstr, 'grades', $gradestr)); 
+                }
+
             } else if ($data_type == 'feedback') {
                 $finalgrade = false;
                 $trimmed = trim($postedvalue);
diff --git a/lib/grade/grade_item.php b/lib/grade/grade_item.php
index e8d531d6dd..2794ff4134 100644
--- a/lib/grade/grade_item.php
+++ b/lib/grade/grade_item.php
@@ -1366,18 +1366,6 @@ class grade_item extends grade_object {
             if (is_null($finalgrade)) {
                 $grade->finalgrade = null;
             } else {
-                // MDL-12517, warn user if grade is out of bounds
-                if ($finalgrade < $this->grademin) {
-                    $user = get_record('user', 'id', $grade->userid,'','','','','id, firstname, lastname');
-                    $gradestr->username = fullname($user);
-                    $gradestr->itemname = $this->get_name();
-                    notify(get_string('lessthanmin', 'grades', $gradestr)); 
-                } else if ($finalgrade > $this->grademax) {
-                    $user = get_record('user', 'id', $grade->userid,'','','','','id, firstname, lastname');
-                    $gradestr->username = fullname($user);
-                    $gradestr->itemname = $this->get_name();
-                    notify(get_string('morethanmax', 'grades', $gradestr));
-                }
                 $grade->finalgrade = (float)bounded_number($this->grademin, $finalgrade, $this->grademax);
             }
         }
-- 
2.39.5