Changeset 4008

Timestamp:

Oct 9, 2009, 5:56:21 PM (15 years ago)

Author:

Eric

Message:

[Bug 1041] Switchable double/single password input with text or password type in admin using new $conf

Coding guidelines improvement

File:

• trunk/admin/user_list.php (modified) (1 diff)

trunk/admin/user_list.php

@@ -188 +188 @@
 if ($conf['double_password_type_in_admin'] == true)
 {
-        if (isset($_POST['submit_add']))
-        {
-                if(empty($_POST['password']))
-                {
-                        array_push($page['errors'], l10n('Password is missing'));
-                }
-                else if(empty($_POST['password_conf']))
-                {
-                        array_push($page['errors'], l10n('Password confirmation is missing'));
-                }
-                else if(empty($_POST['email']))
-                {
-                        array_push($page['errors'], l10n('Email address is missing'));
-                }
-                else if ($_POST['password'] != $_POST['password_conf'])
-                {
-                        array_push($page['errors'], l10n('Password confirmation error'));
-                }
-                else
-                {
-                $page['errors'] = register_user(
-                $_POST['login'], $_POST['password'], $_POST['email'], false);
-
-                        if (count($page['errors']) == 0)
-                {
-                array_push(
-                        $page['infos'],
-                        sprintf(
-                                l10n('user "%s" added'),
-                                $_POST['login']
-                        )
-                );
-                }
-                }
-        }
+  if (isset($_POST['submit_add']))
+  {
+    if(empty($_POST['password']))
+    {
+      array_push($page['errors'], l10n('Password is missing'));
+    }
+    else if(empty($_POST['password_conf']))
+    {
+      array_push($page['errors'], l10n('Password confirmation is missing'));
+    }
+    else if(empty($_POST['email']))
+    {
+      array_push($page['errors'], l10n('Email address is missing'));
+    }
+    else if ($_POST['password'] != $_POST['password_conf'])
+    {
+      array_push($page['errors'], l10n('Password confirmation error'));
+    }
+    else
+    {
+      $page['errors'] = register_user(
+        $_POST['login'], $_POST['password'], $_POST['email'], false);
+
+      if (count($page['errors']) == 0)
+      {
+        array_push(
+          $page['infos'],
+          sprintf(
+            l10n('user "%s" added'),
+            $_POST['login']
+          )
+        );
+      }
+    }
+  }
 }
 else if ($conf['double_password_type_in_admin'] == false)
 {
-        if (isset($_POST['submit_add']))
-        {
-        $page['errors'] = register_user(
-        $_POST['login'], $_POST['password'], $_POST['email'], false);
-
-        if (count($page['errors']) == 0)
-        {
-        array_push(
-        $page['infos'],
-        sprintf(
-                l10n('user "%s" added'),
-                $_POST['login']
-                )
-        );
-        }
+  if (isset($_POST['submit_add']))
+  {
+    $page['errors'] = register_user(
+      $_POST['login'], $_POST['password'], $_POST['email'], false);
+
+    if (count($page['errors']) == 0)
+    {
+      array_push(
+        $page['infos'],
+        sprintf(
+          l10n('user "%s" added'),
+          $_POST['login']
+          )
+        );
+    }
   }
 }