X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Flibs%2Fsponsor_functions.php;h=180086fc8c1832db9df087f895bf9978418c30a6;hb=e1a62db0c2a7a8b56e066aa3a2a032cfabf08267;hp=ee59586f50d798230b1e80d1961af8d6f1b4672d;hpb=cb20090aa0558f1b74f0e9f925f29819e8bb3a45;p=mailer.git

diff --git a/inc/libs/sponsor_functions.php b/inc/libs/sponsor_functions.php
index ee59586f50..180086fc8c 100644
--- a/inc/libs/sponsor_functions.php
+++ b/inc/libs/sponsor_functions.php
@@ -162,14 +162,14 @@ function handlSponsorRequest (&$postData, $update=false, $messageArray=array(),
 				}
 
 				// Implode all data into strings
-				$KEYS   = implode("`, `"  , $DATA['keys']);
-				$valueS = str_repeat("%s', '", count($DATA['values']) - 1);
+				$keyArray   = implode("`, `"  , $DATA['keys']);
+				$valueArray = str_repeat("%s', '", count($DATA['values']) - 1);
 
 				// Generate string
-				$sql = "INSERT INTO `{?_MYSQL_PREFIX?}_sponsor_data` (`".$KEYS."`) VALUES ('".$valueS."%s')";
+				$sql = "INSERT INTO `{?_MYSQL_PREFIX?}_sponsor_data` (`" . $keyArray . "`) VALUES ('" . $valueArray . "%s')";
 
 				// Generate message
-				$message = getMessageFromIndexedArray(getMessage('ADMIN_SPONSOR_ADDED'), "added", $messageArray);
+				$message = getMessageFromIndexedArray(getMessage('ADMIN_SPONSOR_ADDED'), 'added', $messageArray);
 				$ret = 'added';
 			} elseif (($update === true) && (isAdmin())) {
 				// Add all data as hidden data
@@ -434,13 +434,12 @@ function saveSponsorData ($postData, $content) {
 	$sql = substr($sql, 0, -1);
 
 	// Add SQL tail data
-	$sql .= " WHERE `id`='%s' AND password='%s' LIMIT 1";
+	$sql .= " WHERE `id`=%s AND password='%s' LIMIT 1";
 	$DATA[] = bigintval(getSession('sponsorid'));
 	$DATA[] = getSession('sponsorpass');
 
 	// Saving data was completed... ufff...
-	switch (getWhat())
-	{
+	switch (getWhat()) {
 		case 'account': // Change account data
 			if ($EMAIL === true) {
 				$message   = getMessage('SPONSOR_ACCOUNT_EMAIL_CHANGED');
@@ -469,9 +468,11 @@ function saveSponsorData ($postData, $content) {
 			$message = getMaskedMessage('SPONSOR_UNKNOWN_WHAT', getWhat());
 			$templ = ''; $subj = '';
 			break;
-	}
+	} // END - switch
 
+	// Has an entry updated?
 	if (SQL_AFFECTEDROWS() == 1) {
+		// Template and subject are set?
 		if (!empty($templ) && !empty($subj)) {
 			// Run SQL command and check for success
 			$result = SQL_QUERY_ESC($sql, $DATA, __FUNCTION__, __LINE__);
@@ -479,23 +480,24 @@ function saveSponsorData ($postData, $content) {
 			// Add all data to content
 			$content['new_data'] = $postData;
 
-			// Change some data
-			if (isset($content['gender']))             $content['gender']                  = translateGender($content['gender']);
-			if (isset($content['new_data']['gender'])) $content['new_data']['gender']      = translateGender($content['new_data']['gender']);
-			if (isset($content['receive_warnings']))   $content['new_data']['receive']     = translateYesNo($content['new_data']['receive_warnings']);
-			if (isset($content['warning_interval']))   $content['new_data']['interval']    = createFancyTime($content['new_data']['warning_interval']);
+			// Translate some data
+			if (isset($content['gender']))             $content['gender']               = translateGender($content['gender']);
+			if (isset($content['new_data']['gender'])) $content['new_data']['gender']   = translateGender($content['new_data']['gender']);
+			if (isset($content['receive_warnings']))   $content['new_data']['receive']  = translateYesNo($content['new_data']['receive_warnings']);
+			if (isset($content['warning_interval']))   $content['new_data']['interval'] = createFancyTime($content['new_data']['warning_interval']);
 
 			// Send email to admins
 			sendAdminNotification($subj, $templ, $content);
 
 			// Shall we send mail to the sponsor's new email address?
 			if ($content['receive_warnings'] == 'Y') {
-				// Okay send email with confirmation link to new address and with no confirmation link
-				// to the old address
+				/*
+				 * Okay send email with confirmation link to new address and with no confirmation link
+				 * to the old address.
+				 */
 
 				// First to old address
-				switch (getWhat())
-				{
+				switch (getWhat()) {
 					case 'account': // Change account data
 						$email_msg = loadEmailTemplate('sponsor_change_data', $content);
 						sendEmail($content['email'], getMessage('SPONSOR_ACC_DATA_SUBJ'), $email_msg);
@@ -507,7 +509,7 @@ function saveSponsorData ($postData, $content) {
 							// Second mail goes to the new address
 							$email_msg = loadEmailTemplate('sponsor_change_email', $content);
 							sendEmail($content['email'], getMessage('SPONSOR_ACC_EMAIL_SUBJ'), $email_msg);
-						}
+						} // END - if
 						break;
 
 					case 'settings': // Change settings
@@ -515,7 +517,7 @@ function saveSponsorData ($postData, $content) {
 						$email_msg = loadEmailTemplate('sponsor_settings', $content);
 						sendEmail($content['email'], getMessage('SPONSOR_SETTINGS_SUBJ'), $email_msg);
 						break;
-				}
+				} // END - switch
 			} // END - if
 		} // END - if
 	} // END - if