diff --git a/admin/roles/manage.html b/admin/roles/manage.html index 5d6845359d8..188907e509a 100755 --- a/admin/roles/manage.html +++ b/admin/roles/manage.html @@ -24,7 +24,7 @@ : - name); ?> + name)); ?> : @@ -32,11 +32,11 @@ : - description); $usehtmleditor = false; ?> + description, FORMAT_HTML)); $usehtmleditor = false; ?> : - legacytype)) { print_string('none'); diff --git a/backup/restore_form.html b/backup/restore_form.html index ae71036e90f..d5e11a0f243 100644 --- a/backup/restore_form.html +++ b/backup/restore_form.html @@ -206,7 +206,7 @@ function selectItemInCheckboxByName(formId, checkName, checked ) { * if user has course:create in any category, we show * newcourse */ - + // Non-cached - get accessinfo if (isset($USER->access)) { $accessinfo = $USER->access; @@ -529,7 +529,7 @@ function selectItemInCheckboxByName(formId, checkName, checked ) { echo ""; } echo ""; - + // do you want grade histories to be restored? if (empty($CFG->disablegradehistory)) { echo ""; @@ -661,7 +661,7 @@ if ($info->backup_moodle_version < 2006092801) { $mappableroles = $siteroleschoicearray; echo (''); - echo ''; + echo ''; echo (''); /// first, we see if any exact role definition is found @@ -671,7 +671,7 @@ if ($info->backup_moodle_version < 2006092801) { $matchrole = $samerole->id; // if an exact role is found, it does not matter whether this user can assign this role or not, // this will be presented as a valid option regardless - $mappableroles[$samerole->id] = $allroles[$samerole->id]->name." (". $allroles[$samerole->id]->shortname.")"; + $mappableroles[$samerole->id] = format_string($allroles[$samerole->id]->name)." (". $allroles[$samerole->id]->shortname.")"; } else { // no exact role found, let's try to match shortname // this is useful in situations where basic roles differ slightly in definition @@ -712,9 +712,9 @@ echo (''); // end of role mappings table /** * Look for a role in the database with exactly the same definition as the one in the backup file. - * + * * @param integer $roleid the id that the role in the backup files had on the old server. - * @param object $role the rest of the definition of the role from the backup file. + * @param object $role the rest of the definition of the role from the backup file. */ function restore_samerole($roleid, $rolefromxml) { global $CFG; @@ -724,13 +724,13 @@ function restore_samerole($roleid, $rolefromxml) { // First guess, try let's use the id if (restore_is_samerole($roleid, $rolefromxml)) { - return get_record('role', 'id', $roleid); + return get_record('role', 'id', $roleid); } // Second guess, try the shortname $testroleid = get_field('role', 'id', 'shortname', $rolefromxml->shortname); if ($testroleid && restore_is_samerole($testroleid, $rolefromxml)) { - return get_record('role', 'id', $testroleid); + return get_record('role', 'id', $testroleid); } // Finally, search all other roles. In orter to speed things up, we exclude the ones we have @@ -765,7 +765,7 @@ function restore_samerole($roleid, $rolefromxml) { */ function restore_is_samerole($testroleid, $rolefromxml) { // Load the role definition from the databse. - $rolefromdb = get_records('role_capabilities', 'roleid', $testroleid, '', 'capability,permission'); + $rolefromdb = get_records('role_capabilities', 'roleid', $testroleid, '', 'capability,permission'); if (!$rolefromdb) { return false; } diff --git a/lib/accesslib.php b/lib/accesslib.php index f5f7032a1ef..1a5143ecf96 100755 --- a/lib/accesslib.php +++ b/lib/accesslib.php @@ -5212,7 +5212,7 @@ function role_fix_names($roleoptions, $context, $rolenamedisplay=ROLENAME_ALIAS) } } foreach ($roleoptions as $rid => $name) { - $roleoptions[$rid] = strip_tags($name); + $roleoptions[$rid] = strip_tags(format_string($name)); } return $roleoptions; }