]> git.mxchange.org Git - mailer.git/blobdiff - inc/modules/member/what-transfer.php
Typo in function name fixed, double->single quotes, some HTML fixes
[mailer.git] / inc / modules / member / what-transfer.php
index 5ce05ad5678e9bee4f0543009ebe8ab877196c0b..ff3034a77eccd2706ce629ddad5e7fe161b13d13 100644 (file)
@@ -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')."<br /><a href=\"{!URL!}/modules.php?module=login&amp;what=transfer\">{--TRANSFER_CONTINUE_OVERVIEW--}</a>");
@@ -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', "<input type=\"hidden\" name=\"code_chk\" value=\"".$rand."\" /><input type=\"text\" name=\"code\" class=\"member_normal\" size=\"5\" maxlength=\"7\"{!__TRANSFER_TO_DISABLED!} />&nbsp;".$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, "<div class=\"member_done\">{--SETTINGS_SAVED--}</div>");
-                               }
+                               } // 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));