MDL-25778 fix default country and city on user upload form and revert change in auth...
authorPetr Skoda <commits@skodak.org>
Tue, 15 Feb 2011 07:59:53 +0000 (08:59 +0100)
committerPetr Skoda <commits@skodak.org>
Tue, 15 Feb 2011 08:13:33 +0000 (09:13 +0100)
Auth plugins should have separate defaults from site defaults).

admin/uploaduser.php
admin/uploaduser_form.php
auth/db/auth.php
auth/ldap/auth.php

index b25f609..5ee0e8a 100755 (executable)
@@ -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';
             }
index 3024874..956ecdb 100644 (file)
@@ -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();
index 29befd9..4f875b5 100644 (file)
@@ -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
index d2d8b01..d9cd436 100644 (file)
@@ -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)) {