Even more rewritten

[mailer.git] / inc / modules / admin / what-edit_user.php

diff --git a/inc/modules/admin/what-edit_user.php b/inc/modules/admin/what-edit_user.php
index 32d8a67535651c7c919b32f6ae42fa0e6f1e777d..b81215da76fb47a11e678d188f9034083e5e6df4 100644 (file)
--- a/inc/modules/admin/what-edit_user.php
+++ b/inc/modules/admin/what-edit_user.php
@@ -10,7 +10,12 @@
  * -------------------------------------------------------------------- *
  * Kurzbeschreibung  : Mitgliederprofile aendern                        *
  * -------------------------------------------------------------------- *
- *                                                                      *
+ * $Revision::                                                        $ *
+ * $Date::                                                            $ *
+ * $Tag:: 0.2.1-FINAL                                                 $ *
+ * $Author::                                                          $ *
+ * Needs to be in all Files and every File needs "svn propset           *
+ * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2008 by Roland Haeder                           *
  * For more information visit: http://www.mxchange.org                  *
@@ -33,51 +38,51 @@
 
 // Some security stuff...
 if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
+       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
        require($INC);
 }
 
 // Add description as navigation point
-ADD_DESCR("admin", __FILE__);
+ADD_DESCR('admin', __FILE__);
 
 // Fix a notice
 $result_main = false;
-if (isset($_GET['u_id'])) {
+if (REQUEST_ISSET_GET('uid')) {
        //                                    0      1        2         3      4     5      6       7         8          9           10         11
        $result_main = SQL_QUERY_ESC("SELECT gender, surname, family, street_nr, zip, city, country, email, birth_day, birth_month, birth_year, max_mails
-FROM `{!MYSQL_PREFIX!}_user_data`
+FROM `{!_MYSQL_PREFIX!}_user_data`
 WHERE userid=%s
 LIMIT 1",
-               array(bigintval($_GET['u_id'])), __FILE__, __LINE__);
+               array(bigintval(REQUEST_GET('uid'))), __FILE__, __LINE__);
 }
 
-if ((SQL_NUMROWS($result_main) == 1) || (empty($_GET['u_id'])))
+if ((SQL_NUMROWS($result_main) == 1) || (!REQUEST_ISSET_GET('uid')))
 {
        // User found
-       if (empty($_GET['u_id']))
+       if (!REQUEST_ISSET_GET('uid'))
        {
                // Output selection form with all confirmed user accounts listed
                ADD_MEMBER_SELECTION_BOX();
        }
-        elseif (isset($_POST['edit']))
+        elseif (REQUEST_ISSET_POST('edit'))
        {
                // Ok, change the account...
-               $PASS = false; $ADD = "";
-               if ((empty($_POST['pass1'])) && (empty($_POST['pass2'])))
+               $PASS = false; $add = '';
+               if ((!REQUEST_ISSET_POST(('pass1'))) && (!REQUEST_ISSET_POST(('pass2'))))
                {
                        // Don't change the password
                        $PASS = true;
                }
-                elseif (($_POST['pass1'] == $_POST['pass2']))
+                elseif ((REQUEST_POST('pass1') == REQUEST_POST('pass2')))
                {
                        // Change the password
                        $PASS = true;
-                       $ADD = ", password='".generateHash($_POST['pass1'])."'";
+                       $add = ", password='".generateHash(REQUEST_POST('pass1'))."'";
                }
                if ($PASS)
                {
                        // We have to add the following things: birthday and max receive mails
-                       SQL_QUERY_ESC("UPDATE `{!MYSQL_PREFIX!}_user_data` SET
+                       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET
 gender='%s',
 surname='%s',
 family='%s',
@@ -86,18 +91,18 @@ country='%s',
 zip=%s,
 city='%s',
 email='%s'
-".$ADD."
+".$add."
 WHERE userid=%s LIMIT 1",
  array(
-       substr($_POST['gender'], 0, 1),
-       $_POST['surname'],
-       $_POST['family_name'],
-       $_POST['street_nr'],
-       $_POST['country'],
-       bigintval($_POST['zip']),
-       $_POST['city'],
-       $_POST['email'],
-       bigintval($_GET['u_id']),
+       substr(REQUEST_POST('gender'), 0, 1),
+       REQUEST_POST('surname'),
+       REQUEST_POST('family'),
+       REQUEST_POST('street_nr'),
+       REQUEST_POST('country'),
+       bigintval(REQUEST_POST('zip')),
+       REQUEST_POST('city'),
+       REQUEST_POST('email'),
+       bigintval(REQUEST_GET('uid')),
 ), __FILE__, __LINE__);
                        $content = USER_ACCOUNT_SAVED;
                }
@@ -108,7 +113,7 @@ WHERE userid=%s LIMIT 1",
                }
 
                // Load template
-               LOAD_TEMPLATE("admin_settings_saved", false, $content);
+               LOAD_TEMPLATE('admin_settings_saved', false, $content);
        }
         else
        {
@@ -120,21 +125,21 @@ WHERE userid=%s LIMIT 1",
                switch ($gender)
                {
                case "M":
-                       define('_GENDER_M', " selected=\"selected\"");
-                       define('_GENDER_F', "");
-                       define('_GENDER_C', "");
+                       define('_GENDER_M', ' selected="selected"');
+                       define('_GENDER_F', '');
+                       define('_GENDER_C', '');
                        break;
 
                case "F":
-                       define('_GENDER_M', "");
-                       define('_GENDER_F', " selected=\"selected\"");
-                       define('_GENDER_C', "");
+                       define('_GENDER_M', '');
+                       define('_GENDER_F', ' selected="selected"');
+                       define('_GENDER_C', '');
                        break;
 
                case "C":
-                       define('_GENDER_M', "");
-                       define('_GENDER_F', "");
-                       define('_GENDER_C', " selected=\"selected\"");
+                       define('_GENDER_M', '');
+                       define('_GENDER_F', '');
+                       define('_GENDER_C', ' selected="selected"');
                        break;
                }
 
@@ -143,13 +148,11 @@ WHERE userid=%s LIMIT 1",
                define('_COUNTRY', $country); define('_EMAIL' , $email);
 
                // Load template
-               LOAD_TEMPLATE("admin_edit_user", false, bigintval($_GET['u_id']));
+               LOAD_TEMPLATE("admin_edit_user", false, bigintval(REQUEST_GET('uid')));
        }
-}
- else
-{
+} else {
        // Account does not exists!
-       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"admin_failed\">".ADMIN_MEMBER_404_1.$_GET['u_id'].ADMIN_MEMBER_404_2."</div>");
+       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"admin_failed\">".sprintf(getMessage('ADMIN_MEMBER_404'), REQUEST_GET('uid'))."</div>");
 }
 
 //