mirror of
https://github.com/moodle/moodle.git
synced 2025-01-31 04:33:13 +01:00
MDL-25778 fix default country and city on user upload form and revert change in auth plugins
Auth plugins should have separate defaults from site defaults).
This commit is contained in:
parent
fa7f750c60
commit
9449d0c5e8
@ -572,10 +572,6 @@ if ($formdata = $mform->is_cancelled()) {
|
||||
$user->timemodified = time();
|
||||
$user->timecreated = time();
|
||||
|
||||
if (!empty($CFG->defaultcity) and !property_exists($user, 'city')) {
|
||||
$user->city = $CFG->defaultcity;
|
||||
}
|
||||
|
||||
if (isset($user->auth) && empty($user->auth)) {
|
||||
$user->auth = 'manual';
|
||||
}
|
||||
|
@ -206,10 +206,18 @@ class admin_uploaduser_form2 extends moodleform {
|
||||
|
||||
$mform->addElement('text', 'city', get_string('city'), 'maxlength="100" size="25"');
|
||||
$mform->setType('city', PARAM_MULTILANG);
|
||||
$mform->setDefault('city', $templateuser->city);
|
||||
if (empty($CFG->defaultcity)) {
|
||||
$mform->setDefault('city', $templateuser->city);
|
||||
} else {
|
||||
$mform->setDefault('city', $CFG->defaultcity);
|
||||
}
|
||||
|
||||
$mform->addElement('select', 'country', get_string('selectacountry'), get_string_manager()->get_list_of_countries());
|
||||
$mform->setDefault('country', $templateuser->country);
|
||||
if (empty($CFG->country)) {
|
||||
$mform->setDefault('country', $templateuser->country);
|
||||
} else {
|
||||
$mform->setDefault('country', $CFG->country);
|
||||
}
|
||||
$mform->setAdvanced('country');
|
||||
|
||||
$choices = get_list_of_timezones();
|
||||
|
@ -358,10 +358,6 @@ class auth_plugin_db extends auth_plugin_base {
|
||||
echo "\t"; print_string('auth_dbreviveduser', 'auth_db', array('name'=>$user->username, 'id'=>$user->id)); echo "\n";
|
||||
|
||||
} else {
|
||||
if (!empty($CFG->defaultcity) and !property_exists($user, 'city')) {
|
||||
$user->city = $CFG->defaultcity;
|
||||
}
|
||||
|
||||
$id = $DB->insert_record ('user',$user); // it is truly a new user
|
||||
echo "\t"; print_string('auth_dbinsertuser','auth_db',array('name'=>$user->username, 'id'=>$id)); echo "\n";
|
||||
// if relevant, tag for password generation
|
||||
|
@ -490,10 +490,6 @@ class auth_plugin_ldap extends auth_plugin_base {
|
||||
print_error('auth_ldap_create_error', 'auth_ldap');
|
||||
}
|
||||
|
||||
if (!empty($CFG->defaultcity) and !property_exists($user, 'city')) {
|
||||
$user->city = $CFG->defaultcity;
|
||||
}
|
||||
|
||||
$user->id = $DB->insert_record('user', $user);
|
||||
|
||||
// Save any custom profile field information
|
||||
@ -853,10 +849,6 @@ class auth_plugin_ldap extends auth_plugin_base {
|
||||
$user->lang = $CFG->lang;
|
||||
}
|
||||
|
||||
if (!empty($CFG->defaultcity) and !property_exists($user, 'city')) {
|
||||
$user->city = $CFG->defaultcity;
|
||||
}
|
||||
|
||||
$id = $DB->insert_record('user', $user);
|
||||
echo "\t"; print_string('auth_dbinsertuser', 'auth_db', array('name'=>$user->username, 'id'=>$id)); echo "\n";
|
||||
if (!empty($this->config->forcechangepassword)) {
|
||||
|
Loading…
x
Reference in New Issue
Block a user