From: nicolasconnault Date: Thu, 13 Nov 2008 15:44:01 +0000 (+0000) Subject: MDL-17226 Hiding the group show/hide settings and toggle for grader report. Merging... X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=7f304262baba31b4107e7f281f8b14ce3e2fe681;p=moodle.git MDL-17226 Hiding the group show/hide settings and toggle for grader report. Merging from MOODLE_19_STABLE --- diff --git a/grade/report/grader/lib.php b/grade/report/grader/lib.php index 1cbcdeb0c2..bcc868305d 100644 --- a/grade/report/grader/lib.php +++ b/grade/report/grader/lib.php @@ -400,7 +400,7 @@ class grade_report_grader extends grade_report { * @return string HTML code */ public function get_toggles_html() { - global $CFG, $USER; + global $CFG, $USER, $COURSE; $html = '
'; if ($USER->gradeediting[$this->courseid]) { @@ -427,7 +427,7 @@ class grade_report_grader extends grade_report { if (has_capability('moodle/grade:viewall', $this->context) and has_capability('moodle/site:accessallgroups', $this->context) - and $course_has_groups = true) { // TODO replace that last condition with proper check + and $COURSE->groupmode != NOGROUPS) { // TODO replace that last condition with proper check $html .= $this->print_toggle('groups', true); } diff --git a/grade/report/grader/preferences_form.php b/grade/report/grader/preferences_form.php index a32a63f828..60c229e7a4 100644 --- a/grade/report/grader/preferences_form.php +++ b/grade/report/grader/preferences_form.php @@ -72,7 +72,10 @@ class grader_report_preferences_form extends moodleform { if ($canviewhidden) { $preferences['prefshow']['showaverages'] = $checkbox_default; } - $preferences['prefshow']['showgroups'] = $checkbox_default; + if ($course->groupmode != NOGROUPS) { + $preferences['prefshow']['showgroups'] = $checkbox_default; + } + $preferences['prefshow']['showlocks'] = $checkbox_default; $preferences['prefrows'] = array( @@ -156,7 +159,7 @@ class grader_report_preferences_form extends moodleform { if (!empty($CFG->{$full_pref})) { $course_value = grade_get_setting($course->id, $pref, $CFG->{$full_pref}); } - + if ($pref == 'aggregationposition') { if (!empty($options[$course_value])) { $default = $options[$course_value];