From 05766b50c721486cb9227b0a2a263af4f9430da4 Mon Sep 17 00:00:00 2001
From: nicolasconnault <nicolasconnault>
Date: Wed, 27 Feb 2008 07:51:38 +0000
Subject: [PATCH] MDL-13579 Corrected a bug in default value selection for the
 grader report preferences form. Merging from MOODLE_19_STABLE

---
 grade/report/grader/lib.php              | 2 +-
 grade/report/grader/preferences_form.php | 8 +++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/grade/report/grader/lib.php b/grade/report/grader/lib.php
index 30de950d5f..9d18e10afe 100644
--- a/grade/report/grader/lib.php
+++ b/grade/report/grader/lib.php
@@ -114,7 +114,7 @@ class grade_report_grader extends grade_report {
 
         // if user report preference set or site report setting set use it, otherwise use course or site setting
         $switch = $this->get_pref('aggregationposition');
-        if (empty($switch)) {
+        if ($switch == '') {
             $switch = grade_get_setting($this->courseid, 'aggregationposition', $CFG->grade_aggregationposition);
         }
 
diff --git a/grade/report/grader/preferences_form.php b/grade/report/grader/preferences_form.php
index 797c9781fb..5c5df2711f 100644
--- a/grade/report/grader/preferences_form.php
+++ b/grade/report/grader/preferences_form.php
@@ -153,8 +153,14 @@ class grader_report_preferences_form extends moodleform {
                     $type = 'select';
                     // MDL-11478
                     // get default aggregationposition from grade_settings
+                    $course_value = grade_get_setting($course->id, $pref, $CFG->{$full_pref});
+                    
                     if ($pref == 'aggregationposition') {
-                        $default = $options[grade_get_setting($course->id, $pref, $CFG->{$full_pref})];
+                        if (!empty($options[$course_value])) {
+                            $default = $options[$course_value];
+                        } else {
+                            $default = $options[$CFG->grade_aggregationposition];
+                        }
                     } elseif (isset($options[$CFG->{$full_pref}])) {
                         $default = $options[$CFG->{$full_pref}];
                     } else {
-- 
2.39.5