From baacc970b90645f3043ed1f05cc05946720f9d59 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Mon, 21 May 2007 07:37:55 +0000
Subject: [PATCH] MDL-4325 ... fix the order of the name fields is
 lastname/firstname are reversed - thanks Kita!

---
 login/signup_form.php | 12 ++++++++++--
 user/editlib.php      | 14 +++++++++++---
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/login/signup_form.php b/login/signup_form.php
index 51508d8929..218731e3c0 100644
--- a/login/signup_form.php
+++ b/login/signup_form.php
@@ -29,11 +29,19 @@ class login_signup_form extends moodleform {
         $mform->setType('email2', PARAM_NOTAGS);
         $mform->addRule('email2', get_string('missingemail'), 'required', null, 'server');
 
-        $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="25"');
+        $nameordercheck->firstname = 'a';
+        $nameordercheck->lastname  = 'b';
+        if (fullname($nameordercheck) == 'b a' ) {  // See MDL-4325
+            $mform->addElement('text', 'lastname',  get_string('lastname'),  'maxlength="100" size="30"');
+            $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="30"');
+        } else {
+            $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="30"');
+            $mform->addElement('text', 'lastname',  get_string('lastname'),  'maxlength="100" size="30"');
+        }
+
         $mform->setType('firstname', PARAM_TEXT);
         $mform->addRule('firstname', get_string('missingfirstname'), 'required', null, 'server');
 
-        $mform->addElement('text', 'lastname', get_string('lastname'), 'maxlength="100" size="25"');
         $mform->setType('lastname', PARAM_TEXT);
         $mform->addRule('lastname', get_string('missinglastname'), 'required', null, 'server');
 
diff --git a/user/editlib.php b/user/editlib.php
index dfb1f5b9bc..54e0b078d6 100644
--- a/user/editlib.php
+++ b/user/editlib.php
@@ -59,11 +59,19 @@ function useredit_shared_definition(&$mform) {
 
     $strrequired = get_string('required');
 
-    $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="30"');
+    $nameordercheck->firstname = 'a';
+    $nameordercheck->lastname  = 'b';
+    if (fullname($nameordercheck) == 'b a' ) {  // See MDL-4325
+        $mform->addElement('text', 'lastname',  get_string('lastname'),  'maxlength="100" size="30"');
+        $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="30"');
+    } else {
+        $mform->addElement('text', 'firstname', get_string('firstname'), 'maxlength="100" size="30"');
+        $mform->addElement('text', 'lastname',  get_string('lastname'),  'maxlength="100" size="30"');
+    }
+
     $mform->addRule('firstname', $strrequired, 'required', null, 'client');
     $mform->setType('firstname', PARAM_NOTAGS);
 
-    $mform->addElement('text', 'lastname', get_string('lastname'), 'maxlength="100" size="30"');
     $mform->addRule('lastname', $strrequired, 'required', null, 'client');
     $mform->setType('lastname', PARAM_NOTAGS);
 
@@ -249,4 +257,4 @@ function useredit_shared_definition(&$mform) {
     $mform->setType('address', PARAM_MULTILANG);
 }
 
-?>
\ No newline at end of file
+?>
-- 
2.39.5