MDL-64161 gradereport_singleview: Stop updating grades with errors.

robjoyce · Nov 26, 2018 · f246252 · f246252
1 parent 599703e
commit f246252
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/grade/report/singleview/classes/local/screen/screen.php b/grade/report/singleview/classes/local/screen/screen.php
@@ -355,6 +355,10 @@ public function process($data) {
             // Optional type.
             if (!empty($msg)) {
                 $warnings[] = $msg;
+                if ($element instanceof \gradereport_singleview\local\ui\finalgrade) {
+                    // This item wasn't changed so don't add to the changecount.
+                    continue;
+                }
             }
             if (preg_match('/_(\d+)_(\d+)/', $varname, $matchelement)) {
                 $changecount[$matchelement[0]] = 1;

diff --git a/grade/report/singleview/classes/local/ui/finalgrade.php b/grade/report/singleview/classes/local/ui/finalgrade.php
@@ -170,9 +170,11 @@ public function set($value) {
             }
             $gradestr->itemname = $this->grade->grade_item->get_name();
             $errorstr = get_string($errorstr, 'grades', $gradestr);
+            return $errorstr;
         }
 
+        // Only update grades if there are no errors.
         $gradeitem->update_final_grade($userid, $finalgrade, 'singleview', $feedback, FORMAT_MOODLE);
-        return $errorstr;
+        return '';
     }
 }