MDL-27931 admin bulk user upload: convert city as required field. Cre…

…dit goes to Charles Fulton for supplying the patch.
moodle · Jul 11, 2011 · 3372846 · 3372846
1 parent 661dd2c
commit 3372846
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/admin/uploaduser.php b/admin/uploaduser.php
@@ -915,6 +915,14 @@
             $rowcols['status'][] = $stremailduplicate;
         }
     }
+
+    if (isset($rowcols['city'])) {
+        $rowcols['city'] = trim($rowcols['city']);
+        if (empty($rowcols['city'])) {
+            $rowcols['status'][] = get_string('fieldrequired', 'error', 'city');
+        }
+    }
+
     $rowcols['status'] = implode('<br />', $rowcols['status']);
     $data[] = $rowcols;
 }

diff --git a/admin/uploaduser_form.php b/admin/uploaduser_form.php
@@ -264,6 +264,7 @@ function definition () {
         } else {
             $mform->setDefault('city', $CFG->defaultcity);
         }
+        $mform->addRule('city', get_string('required'), 'required');
 
         $mform->addElement('select', 'country', get_string('selectacountry'), get_string_manager()->get_list_of_countries());
         if (empty($CFG->country)) {
@@ -408,10 +409,6 @@ function validation($data, $files) {
             if (!in_array('email', $columns) and empty($data['email'])) {
                 $errors['email'] = get_string('requiredtemplate', 'admin');
             }
-
-            if (!in_array('city', $columns) and empty($data['city'])) {
-                $errors['city'] = get_string('required');
-            }
         }
 
         return $errors;