X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-transfer.php;h=c094189b334be9f17a0cb7acbbac893e6904a5f7;hb=885637e9f61f315ba051500061ec193ebc937f7b;hp=5ce05ad5678e9bee4f0543009ebe8ab877196c0b;hpb=3afdce4fe00b4af570122ce7b8158ced44aec7d3;p=mailer.git diff --git a/inc/modules/member/what-transfer.php b/inc/modules/member/what-transfer.php index 5ce05ad567..c094189b33 100644 --- a/inc/modules/member/what-transfer.php +++ b/inc/modules/member/what-transfer.php @@ -43,7 +43,7 @@ if (!defined('__SECURITY')) { } elseif (!IS_MEMBER()) { redirectToUrl('modules.php?module=index'); } elseif ((!EXT_IS_ACTIVE('transfer')) && (!IS_ADMIN())) { - addFatalMessage(__FILE__, __LINE__, sprintf(getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), 'transfer')); + addFatalMessage(__FILE__, __LINE__, generateExtensionInactiveNotInstalledMessage('transfer')); return; } @@ -59,7 +59,7 @@ list($opt_in) = SQL_FETCHROW($result); SQL_FREERESULT($result); $mode = ''; -if (REQUEST_ISSET_GET(('mode'))) $mode = REQUEST_GET('mode'); +if (REQUEST_ISSET_GET('mode')) $mode = REQUEST_GET('mode'); // Check for "faker" if (($opt_in == 'N') && ($mode == "new")) $mode = ''; @@ -77,7 +77,7 @@ switch ($mode) // Add new transfer if (getConfig('transfer_code') > 0) { // Check for code - $code = generateRandomCodde(getConfig('transfer_code'), REQUEST_POST('code_chk'), getUserId(), constant('__TRANSFER_MAX_VALUE')); + $code = generateRandomCode(getConfig('transfer_code'), REQUEST_POST('code_chk'), getUserId(), constant('__TRANSFER_MAX_VALUE')); $valid_code = ($code == REQUEST_POST('code')); } else { // Zero length (= disabled) is always valid! @@ -171,7 +171,7 @@ switch ($mode) define('__TRANSFER_EXPIRES', createFancyTime(getConfig('transfer_age'))); // Generate tranafer id - define('__TRANS_ID', bigintval(generateRandomCodde("10", mt_rand(0, 99999), getUserId(), REQUEST_POST('reason')))); + define('__TRANS_ID', bigintval(generateRandomCode("10", mt_rand(0, 99999), getUserId(), REQUEST_POST('reason')))); // Add entries to both tables SQL_QUERY_ESC("INSERT INTO `{!_MYSQL_PREFIX!}_user_transfers_in` (userid, from_uid, points, reason, time_trans, trans_id) VALUES ('%s','%s','%s','%s', UNIX_TIMESTAMP(),'%s')", @@ -188,16 +188,16 @@ switch ($mode) SUB_POINTS('transfer', getUserId(), REQUEST_POST('points')); // First send email to recipient - $msg = LOAD_EMAIL_TEMPLATE("member_transfer_recipient", '', constant('__RECIPIENT_UID')); - sendEmail(constant('__RECIPIENT_EMAIL'), getMessage('TRANSFER_MEMBER_RECIPIENT_SUBJ').": ".$SENDER, $msg); + $msg = LOAD_EMAIL_TEMPLATE('member_transfer_recipient', '', constant('__RECIPIENT_UID')); + sendEmail(constant('__RECIPIENT_EMAIL'), getMessage('TRANSFER_MEMBER_RECIPIENT_SUBJ') . ': ' . $SENDER, $msg); // Second send email to sender - $msg = LOAD_EMAIL_TEMPLATE("member_transfer_sender", '', constant('__SENDER_UID')); - sendEmail(constant('__SENDER_EMAIL'), getMessage('TRANSFER_MEMBER_SENDER_SUBJ').": ".$RECIPIENT, $msg); + $msg = LOAD_EMAIL_TEMPLATE('member_transfer_sender', '', constant('__SENDER_UID')); + sendEmail(constant('__SENDER_EMAIL'), getMessage('TRANSFER_MEMBER_SENDER_SUBJ') . ': ' . $RECIPIENT, $msg); // At last send admin mail(s) $ADMIN_SUBJ = sprintf("%s (%s->%s)", getMessage('TRANSFER_ADMIN_SUBJECT'), $SENDER, $RECIPIENT); - sendAdminNotification($ADMIN_SUBJ, "admin_transfer_points"); + sendAdminNotification($ADMIN_SUBJ, 'admin_transfer_points'); // Transfer is completed LOAD_TEMPLATE('admin_settings_saved', false, getMessage('TRANSFER_COMPLETED')."
{--TRANSFER_CONTINUE_OVERVIEW--}"); @@ -233,11 +233,11 @@ switch ($mode) if (EXT_IS_ACTIVE('nickname')) { // Load userid and nickname $result = SQL_QUERY_ESC("SELECT userid, nickname FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' AND opt_in='Y' AND userid != '%s' ORDER BY `userid` ASC", - array(getUserId()), __FILE__, __LINE__); + array(getUserId()), __FILE__, __LINE__); } else { // Load only userid $result = SQL_QUERY_ESC("SELECT userid, userid FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' AND opt_in='Y' AND userid != '%s' ORDER BY `userid` ASC", - array(getUserId()), __FILE__, __LINE__); + array(getUserId()), __FILE__, __LINE__); } if (SQL_NUMROWS($result) > 0) { @@ -266,7 +266,7 @@ switch ($mode) } else { // No one else is opt-in $OUT = getMessage('TRANSFER_NO_ONE_ELSE_OPT_IN'); - define('__TRANSFER_TO_DISABLED', " disabled"); + define('__TRANSFER_TO_DISABLED', ' disabled="disabled"'); } // Transfer output to constant for the template @@ -275,11 +275,11 @@ switch ($mode) // Generate Code if (getConfig('transfer_code') > 0) { $rand = mt_rand(0, 99999); - $code = generateRandomCodde(getConfig('transfer_code'), $rand, getUserId(), constant('__TRANSFER_MAX_VALUE')); + $code = generateRandomCode(getConfig('transfer_code'), $rand, getUserId(), constant('__TRANSFER_MAX_VALUE')); $img = GENERATE_IMAGE($code, false); define('__TRANSFER_IMAGE_INPUT', " ".$img); } else { - $code = "00000"; + $code = '00000'; define('__TRANSFER_IMAGE_INPUT', getMessage('TRANSFER_NO_CODE')); } @@ -295,16 +295,16 @@ switch ($mode) } // Output form - LOAD_TEMPLATE("member_transfer_new"); + LOAD_TEMPLATE('member_transfer_new'); } break; - case "list_in": // List only incoming transactions - case "list_out": // List only outgoing transactions + case 'list_in': // List only incoming transactions + case 'list_out': // List only outgoing transactions // As you can see I put list_in and list_out together. I now do a switch() again on it for the right SQL command switch ($mode) { - case "list_in": + case 'list_in': $sql = "SELECT trans_id, from_uid, points, reason, time_trans FROM `{!_MYSQL_PREFIX!}_user_transfers_in` WHERE userid=%s ORDER BY time_trans DESC LIMIT ".getConfig('transfer_max'); // @TODO Rewrite these constants $NOTHING = getMessage('TRANSFER_NO_INCOMING_TRANSFERS'); @@ -312,7 +312,7 @@ switch ($mode) define('__TRANSFER_TITLE', getMessage('TRANSFER_LIST_INCOMING')); break; - case "list_out": + case 'list_out': $sql = "SELECT trans_id, to_uid, points, reason, time_trans FROM `{!_MYSQL_PREFIX!}_user_transfers_out` WHERE userid=%s ORDER BY time_trans DESC LIMIT ".getConfig('transfer_max'); // @TODO Rewrite these constants $NOTHING = getMessage('TRANSFER_NO_OUTGOING_TRANSFERS'); @@ -329,7 +329,7 @@ switch ($mode) // @TODO This should be somehow rewritten to $content = SQL_FETCHARRAY(), see switch() block above for SQL queries while (list($tid, $uid, $points, $reason, $stamp) = SQL_FETCHROW($result)) { // Rewrite points - if ($type == "OUT") $points = $points.'-'; + if ($type == 'OUT') $points = $points.'-'; // Prepare content for template $content = array( @@ -342,12 +342,12 @@ switch ($mode) ); // Load row template - $OUT .= LOAD_TEMPLATE("member_transfer_row2", true, $content); + $OUT .= LOAD_TEMPLATE('member_transfer_row2', true, $content); // Add points and switch color $total += $points; $SW = 3 - $SW; - } + } // END - while // Free memory SQL_FREERESULT($result); @@ -368,10 +368,10 @@ switch ($mode) define('__TRANSFER_TOTAL_VALUE', $total); // Load final template - LOAD_TEMPLATE("member_transfer_list"); + LOAD_TEMPLATE('member_transfer_list'); break; - case "list_all": // List all transactions + case 'list_all': // List all transactions // We fill a temporary table with data from both tables. This is much easier // to code and unstand by you as sub-SELECT queries. I know this is not the // fastest way but it shall be fine for now. @@ -391,7 +391,7 @@ KEY(party_uid) $result = SQL_QUERY_ESC("SELECT trans_id, from_uid, points, reason, time_trans FROM `{!_MYSQL_PREFIX!}_user_transfers_in` WHERE userid=%s ORDER BY `id` LIMIT %s", array(getUserId(), getConfig('transfer_max')), __FILE__, __LINE__); while ($DATA = SQL_FETCHROW($result)) { - $DATA[] = "IN"; + $DATA[] = 'IN'; $DATA = implode("','", $DATA); $res_temp = SQL_QUERY("INSERT INTO `{!_MYSQL_PREFIX!}_transfers_tmp` (trans_id, party_uid, points, reason, time_trans, trans_type) VALUES ('".$DATA."')", __FILE__, __LINE__); } @@ -403,7 +403,7 @@ KEY(party_uid) $result = SQL_QUERY_ESC("SELECT trans_id, to_uid, points, reason, time_trans FROM `{!_MYSQL_PREFIX!}_user_transfers_out` WHERE userid=%s ORDER BY `id` LIMIT %s", array(getUserId(), getConfig('transfer_max')), __FILE__, __LINE__); while ($DATA = SQL_FETCHROW($result)) { - $DATA[] = "OUT"; + $DATA[] = 'OUT'; $DATA = implode("','", $DATA); $res_temp = SQL_QUERY("INSERT INTO `{!_MYSQL_PREFIX!}_transfers_tmp` (trans_id, party_uid, points, reason, time_trans, trans_type) VALUES ('".$DATA."')", __FILE__, __LINE__); } @@ -421,7 +421,7 @@ KEY(party_uid) $OUT = ''; $SW = 2; while ($content = SQL_FETCHARRAY($result)) { // Rewrite points - if ($content['trans_type'] == "OUT") $content['points'] = '-'.$content['points'].""; + if ($content['trans_type'] == 'OUT') $content['points'] = '-'.$content['points'].""; // Prepare content for template $content['sw'] = $SW; @@ -434,7 +434,7 @@ KEY(party_uid) // Add points and switch color $total += $content['points']; $SW = 3 - $SW; - } + } // END - while // Free memory SQL_FREERESULT($result); @@ -461,7 +461,7 @@ KEY(party_uid) define('__TRANSFER_SUM', getMessage('TRANSFER_TOTAL_BALANCE')); // Load final template - LOAD_TEMPLATE("member_transfer_list"); + LOAD_TEMPLATE('member_transfer_list'); // 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__); @@ -470,7 +470,7 @@ KEY(party_uid) SQL_FREERESULT($result); break; - case "": // Overview page + case '': // Overview page // Check incoming transfers $result = SQL_QUERY_ESC("SELECT COUNT(id) FROM `{!_MYSQL_PREFIX!}_user_transfers_in` WHERE userid=%s", array(getUserId()), __FILE__, __LINE__); list($dmy) = SQL_FETCHROW($result); @@ -512,10 +512,9 @@ KEY(party_uid) // "Settings saved..." LOAD_TEMPLATE('admin_settings_saved', false, "
{--SETTINGS_SAVED--}
"); - } + } // END - if - switch ($opt_in) - { + switch ($opt_in) { case 'Y': define('__TRANSFER_ALLOW_Y', ' checked="checked"'); define('__TRANSFER_ALLOW_N', ''); @@ -527,14 +526,15 @@ KEY(party_uid) define('__TRANSFER_ALLOW_N', ' checked="checked"'); define('__TRANSFER_NEW_LINK', getMessage('TRANSFER_PLEASE_ALLOW_OPT_IN')); break; - } + } // END - switch // Check for latest out-transfers $result = SQL_QUERY_ESC("SELECT time_trans FROM `{!_MYSQL_PREFIX!}_user_transfers_out` WHERE time_trans > (UNIX_TIMESTAMP() - %s) AND `userid`=%s ORDER BY time_trans DESC -LIMIT 1", array(getConfig('transfer_timeout'), getUserId()), __FILE__, __LINE__); +LIMIT 1", + array(getConfig('transfer_timeout'), getUserId()), __FILE__, __LINE__); if (SQL_NUMROWS($result) == 0) { // Load template define('__TRANSFER_SETTINGS_CONTENT', LOAD_TEMPLATE("member_transfer_settings", true));