diff --git a/admin/settings/appearance.php b/admin/settings/appearance.php
index 067c62e6c93..f00c04b35cd 100644
--- a/admin/settings/appearance.php
+++ b/admin/settings/appearance.php
@@ -119,7 +119,7 @@ $temp->add(new admin_setting_configcheckbox('mymoodleredirect', get_string('mymo
$ADMIN->add('appearance', $temp);
// new CFG variable for gradebook (what roles to display)
-$temp = new admin_settingpage('gradebook_roles', get_string('graderoles', 'admin'));
+$temp = new admin_settingpage('gradebook', get_string('gradebook', 'admin'));
$temp->add(new admin_setting_special_gradebook_roles());
$ADMIN->add('appearance', $temp);
diff --git a/lang/en_utf8/admin.php b/lang/en_utf8/admin.php
index 4ae9a32545f..17da1929b60 100644
--- a/lang/en_utf8/admin.php
+++ b/lang/en_utf8/admin.php
@@ -3,7 +3,9 @@
// begin {added with admin cleanup}
-
+$string['gradebook'] = 'Gradebook';
+$string['gradebookroles'] = 'Graded roles';
+$string['configgradebookroles'] = 'Which roles are gradable?';
$string['enableajax'] = 'Enable AJAX';
$string['configenableajax'] = 'If you enable AJAX features users can disable them in their profile, otherwise AJAX is disabled for everybody.';
$string['autologinguests'] = 'Auto-login guests';
diff --git a/lib/adminlib.php b/lib/adminlib.php
index acee7222b74..f87546f0db3 100644
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -2303,21 +2303,21 @@ class admin_setting_special_calendar_weekend extends admin_setting {
}
-/*
- * this is used in config->appearance->gradeconfig
+/*
+ * this is used in config->appearance->gradeconfig
*/
class admin_setting_special_gradebook_roles extends admin_setting {
function admin_setting_special_gradebook_roles() {
$name = 'gradebook_roles';
- $visiblename = get_string('gradebook_roles', 'admin');
- $description = get_string('gradebook_roles', 'admin');
-
+ $visiblename = get_string('gradebookroles', 'admin');
+ $description = get_string('configgradebookroles', 'admin');
+
$value = array();
-
- if ($studentroles = get_roles_with_capability('moodle/legacy:student', CAP_ALLOW)) {
+
+ if ($studentroles = get_roles_with_capability('moodle/legacy:student', CAP_ALLOW)) {
foreach ($studentroles as $roleid=>$studentrole) {
- $value[$roleid] = 1;
+ $value[$roleid] = 1;
}
}
@@ -2343,8 +2343,8 @@ class admin_setting_special_gradebook_roles extends admin_setting {
}
return set_config($this->name, rtrim($str, ","))?'':get_string('errorsetting', 'admin') . $this->visiblename . '
';
} else {
- return set_config($this->name, '')?'':get_string('errorsetting', 'admin') . $this->visiblename . '
';
- }
+ return set_config($this->name, '')?'':get_string('errorsetting', 'admin') . $this->visiblename . '
';
+ }
}
function output_html() {
@@ -2354,24 +2354,27 @@ class admin_setting_special_gradebook_roles extends admin_setting {
} else {
$currentsetting = $this->get_setting();
}
-
+
// from to process which roles to display
if ($roles = get_records('role')) {
- $return = '
| '.get_string('showroles','grades').': |
| '; - $return .= ''.$role->name; - $return .= ' |