X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=inc%2Fmodules%2Fmember%2Fwhat-transfer.php;h=85bff3a9f43dd243b03cd32f6cf0deff26d6b606;hb=a226d81121cb96048b1ed85fd8584a87d435b0db;hp=e21e5b0b4e5544abf9645bf0b5fb7f9cc2e052da;hpb=e2148142f8b1a8f40fd6e7ca32185569c5a9083e;p=mailer.git diff --git a/inc/modules/member/what-transfer.php b/inc/modules/member/what-transfer.php index e21e5b0b4e..85bff3a9f4 100644 --- a/inc/modules/member/what-transfer.php +++ b/inc/modules/member/what-transfer.php @@ -18,6 +18,7 @@ * svn:keywords Date Revision" (autoprobset!) at least!!!!!! * * -------------------------------------------------------------------- * * Copyright (c) 2003 - 2009 by Roland Haeder * + * Copyright (c) 2009, 2010 by Mailer Developer Team * * For more information visit: http://www.mxchange.org * * * * This program is free software; you can redistribute it and/or modify * @@ -51,15 +52,9 @@ if ((!isExtensionActive('transfer')) && (!isAdmin())) { return; } // END - if -// Load data -if (!fetchUserData(getMemberId())) { - // Something really bad happened - debug_report_bug('No user account ' . getMemberId() . ' found.'); -} // END - if - // Check for mode in GET $mode = ''; -if (isGetRequestElementSet('mode')) $mode = getRequestElement('mode'); +if (isGetRequestParameterSet('mode')) $mode = getRequestParameter('mode'); // Check for "faker" if ((getUserData('opt_in') != 'Y') && ($mode == 'new')) $mode = ''; @@ -67,7 +62,7 @@ if ((getUserData('opt_in') != 'Y') && ($mode == 'new')) $mode = ''; switch ($mode) { case 'new': // Start new transfer // Get total points and subtract the balance amount from it = maximum transferable points - $total = countSumTotalData(getMemberId(), 'user_points', 'points') - countSumTotalData(getMemberId(), 'user_data', 'used_points'); + $total = getTotalPoints(getMemberId()); // Remember maximum value for template $content['max_transferable'] = round($total - getConfig('transfer_balance') - 0.5); @@ -76,24 +71,24 @@ switch ($mode) { // Add new transfer if (getConfig('transfer_code') > 0) { // Check for code - $code = generateRandomCode(getConfig('transfer_code'), postRequestElement('code_chk'), getMemberId(), $content['max_transferable']); - $valid_code = ($code == postRequestElement('code')); + $code = generateRandomCode(getConfig('transfer_code'), postRequestParameter('code_chk'), getMemberId(), $content['max_transferable']); + $valid_code = ($code == postRequestParameter('code')); } else { // Zero length (= disabled) is always valid! $valid_code = true; } // Test password - $valid_pass = ($pass == generateHash(postRequestElement('password'), $pass)); + $valid_pass = ($pass == generateHash(postRequestParameter('password'), $pass)); // Test transfer amount - $valid_amount = ((isPostRequestElementSet('points')) && (postRequestElement('points') <= $content['max_transferable'])); + $valid_amount = ((isPostRequestParameterSet('points')) && (postRequestParameter('points') <= $content['max_transferable'])); // Test reason for transfer - $valid_reason = (isPostRequestElementSet('reason')); + $valid_reason = (isPostRequestParameterSet('reason')); // Test if a recipient is selected - $valid_recipient = (postRequestElement('to_userid') > 0); + $valid_recipient = isValidUserId(postRequestParameter('to_userid')); // Check for nickname extension and set additional data // @TODO Rewrite this to a filter @@ -106,7 +101,7 @@ switch ($mode) { $result = SQL_QUERY_ESC("SELECT `userid`, `gender`, `surname`, `family`, `email`".$add." FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `userid` IN ('%s','%s') AND `status`='CONFIRMED' LIMIT 2", array( getMemberId(), - bigintval(postRequestElement('to_userid')) + bigintval(postRequestParameter('to_userid')) ), __FILE__, __LINE__); // Do we have two entries? @@ -125,17 +120,13 @@ switch ($mode) { $content['recipient']['nickname'] = ''; } // END - if - // Translate some data - $content['sender']['gender'] = translateGender($content['sender']['gender']); - $content['recipient']['gender'] = translateGender($content['recipient']['gender']); - // Prepare variables for testing $TEST_NICK_SENDER = $content['sender']['nickname']; $TEST_NICK_REC = $content['recipient']['nickname']; // Default is userids for subject line $SENDER = getMemberId(); - $RECIPIENT = bigintval(postRequestElement('to_userid')); + $RECIPIENT = bigintval(postRequestParameter('to_userid')); // If nickname is installed we can set the nickname // @TODO Rewrite this to a filter @@ -144,68 +135,68 @@ switch ($mode) { $SENDER = $content['sender']['nickname']; } - if (($TEST_NICK_REC != postRequestElement('to_userid')) && (!empty($TEST_NICK_REC))) { + if (($TEST_NICK_REC != postRequestParameter('to_userid')) && (!empty($TEST_NICK_REC))) { $RECIPIENT = $content['recipient']['nickname']; } } // END - if // Remember transfer reason and fancy date/time in constants - $content['reason'] = secureString(postRequestElement('reason')); + $content['reason'] = secureString(postRequestParameter('reason')); $content['expires'] = createFancyTime(getConfig('transfer_age')); // Generate tranafer id - $content['trans_id'] = bigintval(generateRandomCode('10', mt_rand(0, 99999), getMemberId(), postRequestElement('reason'))); + $content['trans_id'] = bigintval(generateRandomCode('10', mt_rand(0, 99999), getMemberId(), postRequestParameter('reason'))); // Add entries to both tables SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_user_transfers_in` (`userid`, `from_userid`, `points`, `reason`, `time_trans`, `trans_id`) VALUES ('%s','%s','%s','%s', UNIX_TIMESTAMP(),'%s')", - array(bigintval(postRequestElement('to_userid')), getMemberId(), bigintval(postRequestElement('points')), postRequestElement('reason'), $content['trans_id']), __FILE__, __LINE__); + array(bigintval(postRequestParameter('to_userid')), getMemberId(), bigintval(postRequestParameter('points')), postRequestParameter('reason'), $content['trans_id']), __FILE__, __LINE__); SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_user_transfers_out` (`userid`, `to_userid`, `points`, `reason`, `time_trans`, `trans_id`) VALUES ('%s','%s','%s','%s', UNIX_TIMESTAMP(),'%s')", - array(getMemberId(), bigintval(postRequestElement('to_userid')), bigintval(postRequestElement('points')), postRequestElement('reason'), $content['trans_id']), __FILE__, __LINE__); + array(getMemberId(), bigintval(postRequestParameter('to_userid')), bigintval(postRequestParameter('points')), postRequestParameter('reason'), $content['trans_id']), __FILE__, __LINE__); // Add points to account *directly* ... - addPointsDirectly('member_transfer', bigintval(postRequestElement('to_userid')), bigintval(postRequestElement('points'))); + addPointsDirectly('member_transfer', bigintval(postRequestParameter('to_userid')), bigintval(postRequestParameter('points'))); // ... and add it to current user's used points - subtractPoints('transfer', getMemberId(), postRequestElement('points')); + subtractPoints('transfer', getMemberId(), postRequestParameter('points')); // First send email to recipient - $message = loadEmailTemplate('member_transfer_recipient', $content, postRequestElement('to_userid')); - sendEmail($content['recipient']['email'], getMessage('TRANSFER_MEMBER_RECIPIENT_SUBJ') . ': ' . $SENDER, $message); + $message = loadEmailTemplate('member_transfer_recipient', $content, postRequestParameter('to_userid')); + sendEmail($content['recipient']['email'], '{--TRANSFER_MEMBER_RECIPIENT_SUBJECT--}' . ': ' . $SENDER, $message); // Second send email to sender $message = loadEmailTemplate('member_transfer_sender', $content, getMemberId()); - sendEmail($content['sender']['email'], getMessage('TRANSFER_MEMBER_SENDER_SUBJ') . ': ' . $RECIPIENT, $message); + sendEmail($content['sender']['email'], '{--TRANSFER_MEMBER_SENDER_SUBJECT--}' . ': ' . $RECIPIENT, $message); // At last send admin mail(s) - $ADMIN_SUBJ = sprintf("%s (%s->%s)", getMessage('TRANSFER_ADMIN_SUBJECT'), $SENDER, $RECIPIENT); - sendAdminNotification($ADMIN_SUBJ, 'admin_transfer_points', $content); + $adminSubject = sprintf("%s (%s->%s)", '{--TRANSFER_ADMIN_SUBJECT--}', $SENDER, $RECIPIENT); + sendAdminNotification($adminSubject, 'admin_transfer_points', $content); // Transfer is completed - loadTemplate('admin_settings_saved', false, getMessage('TRANSFER_COMPLETED') . "
{--TRANSFER_CONTINUE_OVERVIEW--}"); + loadTemplate('admin_settings_saved', false, '{--TRANSFER_COMPLETED--}' . '
{--TRANSFER_CONTINUE_OVERVIEW--}'); } elseif ($valid_code === false) { // Invalid Touring code! - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_CODE--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_CODE--}
'); + unsetPostRequestParameter('ok'); } elseif ($valid_pass === false) { // Wrong password entered - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_PASSWORD--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_PASSWORD--}
'); + unsetPostRequestParameter('ok'); } elseif ($valid_amount === false) { // Too much points entered - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_POINTS--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_POINTS--}
'); + unsetPostRequestParameter('ok'); } elseif ($valid_reason === false) { // No transfer reason entered - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_REASON--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_REASON--}
'); + unsetPostRequestParameter('ok'); } elseif ($valid_recipient === false) { // No recipient selected - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_RECIPIENT--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_RECIPIENT--}
'); + unsetPostRequestParameter('ok'); } elseif ($valid_data === false) { // No recipient/sender selected - loadTemplate('admin_settings_saved', false, "
{--TRANSFER_INVALID_DATA--}
"); - unsetPostRequestElement('ok'); + loadTemplate('admin_settings_saved', false, '
{--TRANSFER_INVALID_DATA--}
'); + unsetPostRequestParameter('ok'); } // Free result @@ -224,15 +215,15 @@ switch ($mode) { array(getMemberId()), __FILE__, __LINE__); } - if (SQL_NUMROWS($result) > 0) { + if (!SQL_HASZERONUMS($result)) { // Load list $OUT = " " . $img; + $content['captcha_code'] = ' ' . $img; } else { $code = '00000'; - $content['captcha_code'] = loadTemplate('admin_settings_saved', true, getMessage('TRANSFER_NO_CODE')); + $content['captcha_code'] = loadTemplate('admin_settings_saved', true, '{--TRANSFER_NO_CODE--}'); } // Init points/reason @@ -272,15 +263,12 @@ switch ($mode) { $content['reason'] = ''; // Transfer maybe already entered valued' - if (isGetRequestElementSet('ok')) { + if (isPostRequestParameterSet('ok')) { // Get values from form - $content['points'] = bigintval(postRequestElement('points')); - $content['reason'] = secureString(postRequestElement('reason')); + $content['points'] = postRequestParameter('points'); + $content['reason'] = postRequestParameter('reason'); } // END - if - // Translate some array elements for template - $content['max_transferable'] = translateComma($content['max_transferable']); - // Output form loadTemplate('member_transfer_new', false, $content); } // END - if @@ -292,17 +280,17 @@ switch ($mode) { $nothingMessage = ''; switch ($mode) { case 'list_in': - $sql = "SELECT `trans_id`, `from_userid` AS party_userid, `points`, `reason`, `time_trans` FROM `{?_MYSQL_PREFIX?}_user_transfers_in` WHERE `userid`=%s ORDER BY `time_trans` DESC LIMIT ".getConfig('transfer_max'); - $nothingMessage = getMessage('TRANSFER_NO_INCOMING_TRANSFERS'); - $content['balance'] = getMessage('TRANSFER_TOTAL_INCOMING'); - $content['title'] = getMessage('TRANSFER_LIST_INCOMING'); + $sql = "SELECT `trans_id`, `from_userid` AS party_userid, `points`, `reason`, `time_trans` FROM `{?_MYSQL_PREFIX?}_user_transfers_in` WHERE `userid`=%s ORDER BY `time_trans` DESC LIMIT {?transfer_max?}"; + $nothingMessage = '{--TRANSFER_NO_INCOMING_TRANSFERS--}'; + $content['balance'] = '{--TRANSFER_TOTAL_INCOMING--}'; + $content['title'] = '{--TRANSFER_LIST_INCOMING--}'; break; case 'list_out': - $sql = "SELECT `trans_id`, `to_userid` AS party_userid, `points`, `reason`, `time_trans` FROM `{?_MYSQL_PREFIX?}_user_transfers_out` WHERE `userid`=%s ORDER BY `time_trans` DESC LIMIT ".getConfig('transfer_max'); - $nothingMessage = getMessage('TRANSFER_NO_OUTGOING_TRANSFERS'); - $content['balance'] = getMessage('TRANSFER_TOTAL_OUTGOING'); - $content['title'] = getMessage('TRANSFER_LIST_OUTGOING'); + $sql = "SELECT `trans_id`, `to_userid` AS party_userid, `points`, `reason`, `time_trans` FROM `{?_MYSQL_PREFIX?}_user_transfers_out` WHERE `userid`=%s ORDER BY `time_trans` DESC LIMIT {?transfer_max?}"; + $nothingMessage = '{--TRANSFER_NO_OUTGOING_TRANSFERS--}'; + $content['balance'] = '{--TRANSFER_TOTAL_OUTGOING--}'; + $content['title'] = '{--TRANSFER_LIST_OUTGOING--}'; break; } // END - switch @@ -311,8 +299,8 @@ switch ($mode) { $result = SQL_QUERY_ESC($sql, array(getMemberId()), __FILE__, __LINE__); // Do we have entries? - if (SQL_NUMROWS($result) > 0) { - $OUT = ''; $SW = 2; + if (!SQL_HASZERONUMS($result)) { + $OUT = ''; // @TODO This should be somehow rewritten to $row = SQL_FETCHARRAY(), see switch() block above for SQL queries while (list($tid, $userid, $points, $reason, $stamp) = SQL_FETCHROW($result)) { // Rewrite points @@ -321,12 +309,11 @@ switch ($mode) { // Prepare content for template // @TODO Rewrite: tid->trans_id,stamp->time_trans $row = array( - 'sw' => $SW, 'trans_id' => $tid, 'stamp' => generateDateTime($stamp, 3), 'userid' => $userid, 'reason' => $reason, - 'points' => translateComma($points) + 'points' => $points ); // Load row template @@ -334,7 +321,6 @@ switch ($mode) { // Add points and switch color $totalPoints += $points; - $SW = 3 - $SW; } // END - while // Free memory @@ -380,7 +366,7 @@ KEY (`party_userid`) while ($DATA = SQL_FETCHROW($result)) { $DATA[] = 'IN'; $DATA = implode("','", $DATA); - $res_temp = SQL_QUERY("INSERT INTO `{?_MYSQL_PREFIX?}_transfers_tmp` (`trans_id`, `party_userid`, `points`, `reason`, `time_trans`, `trans_type`) VALUES ('".$DATA."')", __FILE__, __LINE__); + $res_temp = SQL_QUERY("INSERT INTO `{?_MYSQL_PREFIX?}_transfers_tmp` (`trans_id`, `party_userid`, `points`, `reason`, `time_trans`, `trans_type`) VALUES ('" . $DATA . "')", __FILE__, __LINE__); } // END - while // Free memory @@ -392,26 +378,25 @@ KEY (`party_userid`) while ($DATA = SQL_FETCHROW($result)) { $DATA[] = 'OUT'; $DATA = implode("','", $DATA); - $res_temp = SQL_QUERY("INSERT INTO `{?_MYSQL_PREFIX?}_transfers_tmp` (`trans_id`, `party_userid`, `points`, `reason`, `time_trans`, `trans_type`) VALUES ('".$DATA."')", __FILE__, __LINE__); + $res_temp = SQL_QUERY("INSERT INTO `{?_MYSQL_PREFIX?}_transfers_tmp` (`trans_id`, `party_userid`, `points`, `reason`, `time_trans`, `trans_type`) VALUES ('" . $DATA . "')", __FILE__, __LINE__); } // END - while // Free memory SQL_FREERESULT($result); $total = '0'; - if (SQL_NUMROWS($result) > 0) { + if (!SQL_HASZERONUMS($result)) { // Search for entries $result = SQL_QUERY("SELECT `party_userid`, `trans_id`, `points`, `reason`, `time_trans`, `trans_type` FROM `{?_MYSQL_PREFIX?}_transfers_tmp` ORDER BY `time_trans` DESC", __FILE__, __LINE__); // Output rows - $OUT = ''; $SW = 2; + $OUT = ''; while ($content = SQL_FETCHARRAY($result)) { // Rewrite points if ($content['trans_type'] == 'OUT') $content['points'] = '-'.$content['points'].""; // Prepare content for template - $content['sw'] = $SW; $content['time'] = generateDateTime($content['time_trans'], 3); $content['points'] = translateComma($content['points']); @@ -420,18 +405,17 @@ KEY (`party_userid`) // Add points and switch color $total += $content['points']; - $SW = 3 - $SW; } // END - while // Free memory SQL_FREERESULT($result); } else { // Nothing for in and out - $OUT = " - - ".loadTemplate('admin_settings_saved', true, getMessage('TRANSFER_NO_INOUT_TRANSFERS'))." + $OUT = ' + + ' . loadTemplate('admin_settings_saved', true, '{--TRANSFER_NO_INOUT_TRANSFERS--}') . ' -"; +'; } // ... and add them to a constant for the template @@ -441,10 +425,10 @@ KEY (`party_userid`) $content['total'] = translateComma($total); // Set title - $content['title'] = getMessage('TRANSFER_LIST_ALL'); + $content['title'] = '{--TRANSFER_LIST_ALL--}'; // Set "balance" word - $content['balance'] = getMessage('TRANSFER_TOTAL_BALANCE'); + $content['balance'] = '{--TRANSFER_TOTAL_BALANCE--}'; // At the end we don't need a temporary table in memory $result = SQL_QUERY("DROP TABLE IF EXISTS `{?_MYSQL_PREFIX?}_transfers_tmp`", __FILE__, __LINE__); @@ -461,7 +445,7 @@ KEY (`party_userid`) $total = countSumTotalData(getMemberId(), 'user_transfers_in', 'id', 'userid', true); $content['in_link'] = $total; if ($total > 0) { - $content['in_link'] = "".$total.""; + $content['in_link'] = '' . $total . ''; } // END - if // Check outgoing transfers @@ -472,25 +456,22 @@ KEY (`party_userid`) $content['out_link'] = $dmy; if ($dmy > 0) { - $content['out_link'] = "".$dmy.""; + $content['out_link'] = '' . $dmy . ''; } // END - if // Total transactions $content['all_link'] = $total; if ($total > 0) { - $content['all_link'] = "".$total.""; + $content['all_link'] = '' . $total . ''; } // END - if if (isFormSent()) { // Save settings SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `opt_in`='%s' WHERE `userid`=%s LIMIT 1", - array(postRequestElement('opt_in'), getMemberId()), __FILE__, __LINE__); - - // Rember for next switch() command - getUserData('opt_in') = substr(postRequestElement('opt_in'), 0, 1); + array(postRequestParameter('opt_in'), getMemberId()), __FILE__, __LINE__); // "Settings saved..." - loadTemplate('admin_settings_saved', false, "
{--SETTINGS_SAVED--}
"); + loadTemplate('admin_settings_saved', false, '
{--SETTINGS_SAVED--}
'); } // END - if // Init entries @@ -504,11 +485,11 @@ KEY (`party_userid`) // Set 'new transfer' link according to above option switch (getUserData('opt_in')) { case 'Y': - $content['new_link'] = "{--TRANSFER_NOW_LINK--}"; + $content['new_link'] = '{--TRANSFER_NOW_LINK--}'; break; case 'N': - $content['new_link'] = getMessage('TRANSFER_PLEASE_ALLOW_OPT_IN'); + $content['new_link'] = '{--TRANSFER_PLEASE_ALLOW_OPT_IN--}'; break; } // END - switch @@ -531,7 +512,7 @@ LIMIT 1", if (SQL_NUMROWS($result) == 1) { // Load newest transaction list($newest) = SQL_FETCHROW($result); - $content['settings'] = sprintf(getMessage('TRANSFER_LATEST_IS'), generateDateTime($newest, 3)); + $content['settings'] = getMaskedMessage('TRANSFER_LATEST_IS', generateDateTime($newest, 3)); } else { // Load template $content['settings'] = loadTemplate('member_transfer_settings', true, $content);