moodle/moodle
3
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-08-04 16:36:37 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

do not use role functions in upgrade scripts if roles not setup yet

Browse source
This commit is contained in:
skodak 2007-07-10 07:46:19 +00:00
parent 197ba19cae
commit 8c6c185a1c
1 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
lib/db/upgrade.php
View file

@ -122,7 +122,8 @@ function xmldb_main_upgrade($oldversion=0) {
execute_sql("DELETE FROM {$CFG->prefix}config where name='secureforms'", true);
}
if ($oldversion < 2006120700) { // add moodle/user:viewdetails to all roles!
if (!empty($CFG->rolesactive) && $oldversion < 2006120700) { // add moodle/user:viewdetails to all roles!
// note: use of assign_capability() is discouraged in upgrade script!
if ($roles = get_records('role')) {
$context = get_context_instance(CONTEXT_SYSTEM);
foreach ($roles as $roleid=>$role) {
@ -861,8 +862,9 @@ function xmldb_main_upgrade($oldversion=0) {
}
}
if ($result && $oldversion < 2007051801) {
if (!empty($CFG->rolesactive) && $result && $oldversion < 2007051801) {
// Get the role id of the "Auth. User" role and check if the default role id is different
// note: use of assign_capability() is discouraged in upgrade script!
$userrole = get_record( 'role', 'shortname', 'user' );
$defaultroleid = $CFG->defaultuserroleid;