A lot rewrites from double-quote to single-quote, some fixes for extension handling...

[mailer.git] / inc / modules / member / what-transfer.php

diff --git a/inc/modules/member/what-transfer.php b/inc/modules/member/what-transfer.php
index 7d082789ea3f3345824252ace8e4c1c6303057b5..04d2c686e831ad94cf503dbb0156a8887cb74734 100644 (file)
--- a/inc/modules/member/what-transfer.php
+++ b/inc/modules/member/what-transfer.php
@@ -41,14 +41,14 @@ if (!defined('__SECURITY')) {
        $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
        require($INC);
 } elseif (!IS_MEMBER()) {
-       LOAD_URL("modules.php?module=index");
-} elseif ((!EXT_IS_ACTIVE("transfer")) && (!IS_ADMIN())) {
-       addFatalMessage(__FILE__, __LINE__, getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), "transfer");
+       LOAD_URL('modules.php?module=index');
+} elseif ((!EXT_IS_ACTIVE('transfer')) && (!IS_ADMIN())) {
+       addFatalMessage(__FILE__, __LINE__, getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), 'transfer');
        return;
 }
 
 // Add description as navigation point
-ADD_DESCR("member", __FILE__);
+ADD_DESCR('member', __FILE__);
 
 // Load data
 $result = SQL_QUERY_ESC("SELECT opt_in FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s LIMIT 1",
@@ -58,11 +58,11 @@ list($opt_in) = SQL_FETCHROW($result);
 // Free memory
 SQL_FREERESULT($result);
 
-$MODE = "";
+$MODE = '';
 if (REQUEST_ISSET_GET(('mode'))) $MODE = REQUEST_GET('mode');
 
 // Check for "faker"
-if (($opt_in == "N") && ($MODE == "new")) $MODE = "";
+if (($opt_in == 'N') && ($MODE == "new")) $MODE = '';
 
 switch ($MODE)
 {
@@ -98,7 +98,7 @@ case "new": // Start new transfer
 
                // Check for nickname extension and set additional data
                $nick = false; $add = ", userid";
-               if (EXT_IS_ACTIVE("nickname")) {
+               if (EXT_IS_ACTIVE('nickname')) {
                        $add = ", nickname";
                        $nick = true;
                }
@@ -185,14 +185,14 @@ case "new": // Start new transfer
                        ADD_POINTS_REFSYSTEM("member_transfer", bigintval(REQUEST_POST('to_uid')), bigintval(REQUEST_POST('points')), false, "0", false, "direct");
 
                        // ... and add it to current user's used points
-                       SUB_POINTS("transfer", getUserId(), REQUEST_POST('points'));
+                       SUB_POINTS('transfer', getUserId(), REQUEST_POST('points'));
 
                        // First send email to recipient
-                       $msg = LOAD_EMAIL_TEMPLATE("member_transfer_recipient", "", constant('__RECIPIENT_UID'));
+                       $msg = LOAD_EMAIL_TEMPLATE("member_transfer_recipient", '', constant('__RECIPIENT_UID'));
                        SEND_EMAIL(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'));
+                       $msg = LOAD_EMAIL_TEMPLATE("member_transfer_sender", '', constant('__SENDER_UID'));
                        SEND_EMAIL(constant('__SENDER_EMAIL'), getMessage('TRANSFER_MEMBER_SENDER_SUBJ').": ".$RECIPIENT, $msg);
 
                        // At last send admin mail(s)
@@ -200,37 +200,37 @@ case "new": // Start new transfer
                        SEND_ADMIN_NOTIFICATION($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>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, getMessage('TRANSFER_COMPLETED')."<br /><a href=\"{!URL!}/modules.php?module=login&amp;what=transfer\">{--TRANSFER_CONTINUE_OVERVIEW--}</a>");
                } elseif (!$valid_code) {
                        // Invalid Touring code!
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_CODE--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_CODE--}</div>");
                        REQUEST_UNSET_POST('ok');
                } elseif (!$valid_pass) {
                        // Wrong password entered
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_PASSWORD--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_PASSWORD--}</div>");
                        REQUEST_UNSET_POST('ok');
                } elseif (!$valid_amount) {
                        // Too much points entered
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_POINTS--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_POINTS--}</div>");
                        REQUEST_UNSET_POST('ok');
                } elseif (!$valid_reason) {
                        // No transfer reason entered
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_REASON--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_REASON--}</div>");
                        REQUEST_UNSET_POST('ok');
                } elseif (!$valid_recipient) {
                        // No recipient selected
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_RECIPIENT--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_RECIPIENT--}</div>");
                        REQUEST_UNSET_POST('ok');
                } elseif (!$valid_data) {
                        // No recipient selected
-                       LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_note\">{--TRANSFER_INVALID_DATA--}</div>");
+                       LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_note\">{--TRANSFER_INVALID_DATA--}</div>");
                        REQUEST_UNSET_POST('ok');
                }
        }
 
        if (!IS_FORM_SENT()) {
                // Load member list
-               if (EXT_IS_ACTIVE("nickname")) {
+               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",
                         array(getUserId()), __FILE__, __LINE__);
@@ -259,7 +259,7 @@ case "new": // Start new transfer
                                $OUT .= "</option>\n";
                        }
                        $OUT .= "</select>\n";
-                       define('__TRANSFER_TO_DISABLED', "");
+                       define('__TRANSFER_TO_DISABLED', '');
 
                        // Free memory
                        SQL_FREERESULT($result);
@@ -290,8 +290,8 @@ case "new": // Start new transfer
                        define('__TRANSFER_REASON_VALUE', strip_tags(REQUEST_POST('reason')));
                } else {
                        // Set empty values
-                       define('__TRANSFER_POINTS_VALUE', "");
-                       define('__TRANSFER_REASON_VALUE', "");
+                       define('__TRANSFER_POINTS_VALUE', '');
+                       define('__TRANSFER_REASON_VALUE', '');
                }
 
                // Output form
@@ -325,7 +325,7 @@ case "list_out": // List only outgoing transactions
        $total = 0;
        $result = SQL_QUERY_ESC($sql, array(getUserId()), __FILE__, __LINE__);
        if (SQL_NUMROWS($result) > 0) {
-               $OUT = ""; $SW = 2;
+               $OUT = ''; $SW = 2;
                // @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
@@ -355,7 +355,7 @@ case "list_out": // List only outgoing transactions
                // Nothing for in or out
                $OUT = "<tr>
   <td colspan=\"5\" align=\"center\" class=\"bottom2\" height=\"70\">
-    ".LOAD_TEMPLATE("admin_settings_saved", true, $NOTHING)."
+    ".LOAD_TEMPLATE('admin_settings_saved', true, $NOTHING)."
   </td>
 </tr>";
        }
@@ -418,7 +418,7 @@ array(getUserId(), getConfig('transfer_max')), __FILE__, __LINE__);
                        __FILE__, __LINE__);
 
                // Output rows
-               $OUT = ""; $SW = 2;
+               $OUT = ''; $SW = 2;
                while ($content = SQL_FETCHARRAY($result)) {
                        // Rewrite points
                        if ($content['trans_type'] == "OUT") $content['points'] = "-".$content['points']."";
@@ -442,7 +442,7 @@ array(getUserId(), getConfig('transfer_max')), __FILE__, __LINE__);
                // Nothing for in and out
                $OUT = "<tr>
   <td colspan=\"5\" align=\"center\" class=\"bottom2\" height=\"70\">
-    ".LOAD_TEMPLATE("admin_settings_saved", true, getMessage('TRANSFER_NO_INOUT_TRANSFERS'))."
+    ".LOAD_TEMPLATE('admin_settings_saved', true, getMessage('TRANSFER_NO_INOUT_TRANSFERS'))."
   </td>
 </tr>";
        }
@@ -511,20 +511,20 @@ case "": // Overview page
                $opt_in = REQUEST_POST('opt_in');
 
                // "Settings saved..."
-               LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"member_done\">{--SETTINGS_SAVED--}</div>");
+               LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"member_done\">{--SETTINGS_SAVED--}</div>");
        }
 
        switch ($opt_in)
        {
-       case "Y":
-               define('__TRANSFER_ALLOW_Y', " checked=\"checked\"");
-               define('__TRANSFER_ALLOW_N', "");
+       case 'Y':
+               define('__TRANSFER_ALLOW_Y', ' chkecked="checked"');
+               define('__TRANSFER_ALLOW_N', '');
                define('__TRANSFER_NEW_LINK', "<a href=\"{!URL!}/modules.php?module=login&amp;what=transfer&amp;mode=new\">{--TRANSFER_NOW_LINK--}</a>");
                break;
 
-       case "N":
-               define('__TRANSFER_ALLOW_Y', "");
-               define('__TRANSFER_ALLOW_N', " checked=\"checked\"");
+       case 'N':
+               define('__TRANSFER_ALLOW_Y', '');
+               define('__TRANSFER_ALLOW_N', ' chkecked="checked"');
                define('__TRANSFER_NEW_LINK', getMessage('TRANSFER_PLEASE_ALLOW_OPT_IN'));
                break;
        }