X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-edit_user.php;h=6922dbedb93b0ab40773e97c5b2c79d110030faf;hb=01f61fde8c4a4e040dc089ccb274441859487e1d;hp=e8d3fbd99fc77b95ddd540922bca45de037017f3;hpb=5071030af40e69ca4284642f44758964e18f5be8;p=mailer.git

diff --git a/inc/modules/admin/what-edit_user.php b/inc/modules/admin/what-edit_user.php
index e8d3fbd99f..6922dbedb9 100644
--- a/inc/modules/admin/what-edit_user.php
+++ b/inc/modules/admin/what-edit_user.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 09/28/2003 *
- * ===============                              Last change: 06/10/2004 *
+ * Mailer v0.2.1-FINAL                                Start: 09/28/2003 *
+ * ===================                          Last change: 06/10/2004 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : what-edit_user.php                               *
@@ -17,7 +17,9 @@
  * 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                           *
+ * @TODO Add support for ext-country                                    *
+ * -------------------------------------------------------------------- *
+ * Copyright (c) 2003 - 2009 by Roland Haeder                           *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -37,123 +39,107 @@
  ************************************************************************/
 
 // Some security stuff...
-if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
-	require($INC);
-}
+if ((!defined('__SECURITY')) || (!isAdmin())) {
+	die();
+} // END - if
 
 // Add description as navigation point
-ADD_DESCR('admin', __FILE__);
+addMenuDescription('admin', __FILE__);
 
 // Fix a notice
 $result_main = false;
-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`
-WHERE userid=%s
+if (isGetRequestParameterSet('userid')) {
+	// Load user entry
+	$result_main = SQL_QUERY_ESC("SELECT
+	`userid`, `gender`, `surname`, `family`, `street_nr`, `zip`, `city`, `country`,
+	`email`,
+	`birth_day`, `birth_month`, `birth_year`,
+	`max_mails`
+FROM
+	`{?_MYSQL_PREFIX?}_user_data`
+WHERE
+	`userid`=%s
 LIMIT 1",
-	array(bigintval(REQUEST_GET('uid'))), __FILE__, __LINE__);
-}
+		array(
+			bigintval(getRequestParameter('userid'))
+		), __FILE__, __LINE__);
+} // END - if
 
-if ((SQL_NUMROWS($result_main) == 1) || (!REQUEST_ISSET_GET('uid')))
-{
+if ((SQL_NUMROWS($result_main) == 1) || (!isGetRequestParameterSet('userid'))) {
 	// User found
-	if (!REQUEST_ISSET_GET('uid'))
-	{
+	if (!isGetRequestParameterSet('userid')) {
 		// Output selection form with all confirmed user accounts listed
-		ADD_MEMBER_SELECTION_BOX();
-	}
-	elseif (REQUEST_ISSET_POST('edit'))
-	{
+		addMemberSelectionBox();
+	} elseif (isPostRequestParameterSet('edit')) {
 		// Ok, change the account...
-		$PASS = false; $add = '';
-		if ((!REQUEST_ISSET_POST(('pass1'))) && (!REQUEST_ISSET_POST(('pass2'))))
-		{
+		$updateUser = false; $add = '';
+		if ((!isPostRequestParameterSet('pass1')) && (!isPostRequestParameterSet('pass2'))) {
 			// Don't change the password
-			$PASS = true;
-		}
-		elseif ((REQUEST_POST('pass1') == REQUEST_POST('pass2')))
-		{
+			$updateUser = true;
+		} elseif ((postRequestParameter('pass1') == postRequestParameter('pass2'))) {
 			// Change the password
-			$PASS = true;
-			$add = ", password='".generateHash(REQUEST_POST('pass1'))."'";
+			$updateUser = true;
+			$add = ", `password`='" . generateHash(postRequestParameter('pass1')) . "'";
 		}
-		if ($PASS)
-		{
+
+		if ($updateUser === true) {
 			// We have to add the following things: birthday and max receive mails
-			SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET
-gender='%s',
-surname='%s',
-family='%s',
-street_nr='%s',
-country='%s',
-zip=%s,
-city='%s',
-email='%s'
-".$add."
-WHERE userid=%s LIMIT 1",
+			SQL_QUERY_ESC("UPDATE
+	`{?_MYSQL_PREFIX?}_user_data`
+SET
+	`gender`='%s',
+	`surname`='%s',
+	`family`='%s',
+	`street_nr`='%s',
+	`country`='%s',
+	`zip`=%s,
+	`city`='%s',
+	`email`='%s'
+	".$add."
+WHERE
+	`userid`=%s
+LIMIT 1",
 			array(
-			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')),
+				substr(postRequestParameter('gender'), 0, 1),
+				postRequestParameter('surname'),
+				postRequestParameter('family'),
+				postRequestParameter('street_nr'),
+				postRequestParameter('country'),
+				bigintval(postRequestParameter('zip')),
+				postRequestParameter('city'),
+				postRequestParameter('email'),
+				bigintval(getRequestParameter('userid')),
 			), __FILE__, __LINE__);
-			$content = USER_ACCOUNT_SAVED;
-		}
-		else
-		{
+			$content = getMessage('USER_ACCOUNT_SAVED');
+		} else {
 			// Problem while saving data
-			$content = USER_ACCOUNT_NOT_SAVED;
+			$content = getMessage('USER_ACCOUNT_NOT_SAVED');
 		}
 
 		// Load template
-		LOAD_TEMPLATE('admin_settings_saved', false, $content);
-	}
-	else
-	{
-		// Display form to edit
-		list($gender, $surname, $family, $street, $zip, $city, $country, $email, $bday, $bmonth, $byear, $max) = SQL_FETCHROW($result_main);
-		SQL_FREERESULT($result_main);
-
-		// Transfer data to constants for the template
-		switch ($gender)
-		{
-			case "M":
-				define('_GENDER_M', ' selected="selected"');
-				define('_GENDER_F', '');
-				define('_GENDER_C', '');
-				break;
+		loadTemplate('admin_settings_saved', false, $content);
+	} else {
+		// Load entry
+		$content = SQL_FETCHARRAY($result_main);
 
-			case "F":
-				define('_GENDER_M', '');
-				define('_GENDER_F', ' selected="selected"');
-				define('_GENDER_C', '');
-				break;
+		// Init entries
+		foreach (array('gender_m','gender_f','gender_c') as $entry) {
+			$content[$entry] = '';
+		} // END - foreach
 
-			case "C":
-				define('_GENDER_M', '');
-				define('_GENDER_F', '');
-				define('_GENDER_C', ' selected="selected"');
-				break;
-		}
-
-		define('_SURNAME', $surname); define('_FAMILY', $family); define('_CITY'     , $city);
-		define('_STREET' , $street);  define('_ZIP'   , $zip);    define('_MAX_MAILS', $max);
-		define('_COUNTRY', $country); define('_EMAIL' , $email);
+		// Prepare selections
+		$content['gender_' . strtolower($content['gender'])] = ' selected="selected"';
 
 		// Load template
-		LOAD_TEMPLATE("admin_edit_user", false, bigintval(REQUEST_GET('uid')));
+		loadTemplate('admin_edit_user', false, $content);
 	}
 } else {
 	// Account does not exists!
-	LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"admin_failed\">".sprintf(getMessage('ADMIN_MEMBER_404'), REQUEST_GET('uid'))."</div>");
+	loadTemplate('admin_settings_saved', false, '<div class="admin_failed">' . getMaskedMessage('ADMIN_MEMBER_404', getRequestParameter('userid')) . '</div>');
 }
 
-//
+// Free the result
+SQL_FREERESULT($result_main);
+
+// [EOF]
 ?>