Ticket resolved, code rewrites and caching:
authorRoland Häder <roland@mxchange.org>
Sun, 22 Feb 2009 20:40:00 +0000 (20:40 +0000)
committerRoland Häder <roland@mxchange.org>
Sun, 22 Feb 2009 20:40:00 +0000 (20:40 +0000)
- Resolves ticket #90
- Caching added to prevent some queries
- Some code cleanups
- TODOs added

31 files changed:
beg.php
birthday_confirm.php
inc/autopurge/purge-general.php
inc/autopurge/purge-inact.php
inc/autopurge/purge-mails.php
inc/autopurge/purge-tsks.php
inc/autopurge/purge-unconfirmed.php
inc/doubler_send.php
inc/extensions/ext-autopurge.php
inc/extensions/ext-bonus.php
inc/extensions/ext-newsletter.php
inc/extensions/ext-profile.php
inc/functions.php
inc/libs/beg_functions.php
inc/libs/bonus_functions.php
inc/libs/wernis_functions.php
inc/load_extensions.php
inc/modules/admin/what-config_autopurge.php
inc/modules/admin/what-config_order.php
inc/modules/admin/what-config_other.php
inc/modules/admin/what-unlock_emails.php
inc/modules/guest/what-register.php
inc/modules/member/what-order.php
inc/modules/member/what-points.php
inc/modules/member/what-refback.php
inc/monthly/monthly_beg.php
inc/mysql-manager.php
inc/profile-updte.php
inc/stylesheet.php
mailid_top.php
modules.php

diff --git a/beg.php b/beg.php
index ccf6edf..5f68fbc 100644 (file)
--- a/beg.php
+++ b/beg.php
@@ -191,6 +191,7 @@ if (isBooleanConstantAndTrue('mxchange_installed')) {
                        // Tracker code enabled? (We don't track users here!
                        if ((getConfig('beg_pay_mode') != "NONE") && ($pay)) {
                                // Prepare content for template
+                               // @TODO Opps, what is missing here???
                                $content = array(
                                );
 
index ca7d824..25e7371 100644 (file)
@@ -76,7 +76,7 @@ WHERE b.userid=%s AND b.chk_value='%s' LIMIT 1",
                if ($data['status'] == "CONFIRMED") {
                        // Set mode depending on how many mails the member has to confirm
                        $locked = false;
-                       if (($data['ref_payout'] > 0) && (getConfig('allow_direct_pay') == "N")) $locked = true;
+                       if (($data['ref_payout'] > 0) && (getConfig('allow_direct_pay') != "Y")) $locked = true;
 
                        // Add points to account
                        unset($GLOBALS['ref_level']);
index a11fc14..782b69d 100644 (file)
@@ -39,7 +39,7 @@ if (!defined('__SECURITY')) {
 }
 
 // Abort if autopurge is not active or disabled by admin
-if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') == "N")) {
+if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') != "Y")) {
        // Abort here
        return false;
 }
index bcf5a92..c015fc5 100644 (file)
@@ -38,7 +38,7 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Abort if autopurge is not active or disabled by admin
-if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') == "N")) {
+if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') != "Y")) {
        // Abort here
        return false;
 } // END - if
index d1aeb1d..7d5de27 100644 (file)
@@ -38,7 +38,7 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Abort if autopurge is not active or disabled by admin
-if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') == "N")) {
+if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') != "Y")) {
        // Abort here
        return false;
 } // END - if
index 07dbc70..dde1e7c 100644 (file)
@@ -38,7 +38,7 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Abort if autopurge is not active or disabled by admin
-if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') == "N")) {
+if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') != "Y")) {
        // Abort here
        return false;
 } // END - if
index c8e1c68..baa56db 100644 (file)
@@ -38,7 +38,7 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // Abort if autopurge is not active or disabled by admin
-if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') == "N")) {
+if ((!EXT_IS_ACTIVE("autopurge")) || (getConfig('auto_purge_active') != "Y")) {
        // Abort here
        return false;
 } // END - if
index 1fd5057..75e73ca 100644 (file)
@@ -72,7 +72,7 @@ ORDER BY d.timemark
 LIMIT %d", array($DOUBLER_POINTS, $min, getConfig('doubler_max_sent')), __FILE__, __LINE__);
 
 // Do we have entries found?
-if (((SQL_NUMROWS($result_total) > 0) && (getConfig('doubler_sent_all') == "Y")) || ((SQL_NUMROWS($result_main) == getConfig('doubler_group_sent')) && (getConfig('doubler_sent_all') == "N"))) {
+if (((SQL_NUMROWS($result_total) > 0) && (getConfig('doubler_sent_all') == "Y")) || ((SQL_NUMROWS($result_main) == getConfig('doubler_group_sent')) && (getConfig('doubler_sent_all') != "Y"))) {
        // Switch to matching SQL resource
        $result_load = $result_main;
        if ((SQL_NUMROWS($result_total) > 0) && (getConfig('doubler_sent_all') == "Y")) $result_load = $result_total;
index 9c46b91..d876c01 100644 (file)
@@ -254,7 +254,7 @@ case "test": // For testing purposes. For details see file inc/modules/admin/wha
 
 default: // Do stuff when extension is loaded
        // Do we have a daily-reset-run?
-       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!DEBUG_MODE) && ($CSS != 1))
+       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!isBooleanConstantAndTrue('DEBUG_MODE')) && ($CSS != 1))
        {
                // Yes, we have. So let's auto-purge some campaigns, inactive users and unconfirmed accounts
                $INC_POOL[] = sprintf("%sinc/autopurge.php", constant('PATH'));
index 0bcddc7..8fad3e4 100644 (file)
@@ -546,7 +546,7 @@ case "test": // For testing purposes. For details see file inc/modules/admin/wha
        break;
 
 default: // Do stuff when extension is loaded
-       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!DEBUG_MODE) && ($CSS != 1)) {
+       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!isBooleanConstantAndTrue('DEBUG_MODE')) && ($CSS != 1)) {
                // Daily reset was run so let's check if active rallye is activated
                if (getConfig('bonus_active') == "Y") {
                        // Run active rallye
index 5810b4e..daaf756 100644 (file)
@@ -168,7 +168,7 @@ default: // Do stuff when extension is loaded
                '2','3','4','5','6','7','8','9'
        );
 
-       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!DEBUG_MODE) && ($CSS != 1)) {
+       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!isBooleanConstantAndTrue('DEBUG_MODE')) && ($CSS != 1)) {
                // Daily reset was run so let's check out for expired newsletter orders
                $INC_POOL[] = sprintf("%sinc/monthly/monthly_newsletter.php", constant('PATH'));
        }
index ae60910..6d28057 100644 (file)
@@ -123,7 +123,7 @@ case "test": // For testing purposes. For details see file inc/modules/admin/wha
 
 default: // Do stuff when extension is loaded
        // Do we have a daily-reset-run?
-       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!DEBUG_MODE) && ($CSS != 1)) {
+       if (isBooleanConstantAndTrue('__DAILY_RESET') && (!isBooleanConstantAndTrue('DEBUG_MODE')) && ($CSS != 1)) {
                // So let's check for profiles which needs an update
                $INC_POOL[] = sprintf("%sinc/profile-updte.php", constant('PATH'));
        }
index 2ee2e1e..74cef54 100644 (file)
@@ -145,7 +145,7 @@ function OUTPUT_HTML ($HTML, $newLine = true) {
                        // Was that eval okay?
                        if (empty($newContent)) {
                                // Something went wrong!
-                               die("Evaluation error:<pre>".htmlentities($eval)."</pre>");
+                               mxchange_die("Evaluation error:<pre>".htmlentities($eval)."</pre>");
                        } // END - if
                        $OUTPUT = $newContent;
                } // END - while
@@ -853,22 +853,22 @@ function LOAD_EMAIL_TEMPLATE($template, $content=array(), $UID="0") {
                $tmpl_file = addslashes($tmpl_file);
 
                // Run code
-               $tmpl_file = "\$newContent=html_entity_decode(\"".COMPILE_CODE($tmpl_file)."\");";
+               $tmpl_file = "\$newContent = html_entity_decode(\"".COMPILE_CODE($tmpl_file)."\");";
                @eval($tmpl_file);
        } elseif (!empty($template)) {
                // Template file not found!
-               $newContent = TEMPLATE_404.": ".$template."<br />
-".TEMPLATE_CONTENT."
+               $newContent = "{!TEMPLATE_404!}: ".$template."<br />
+{!TEMPLATE_CONTENT!}
 <pre>".print_r($content, true)."</pre>
-".TEMPLATE_DATA."
+{!TEMPLATE_DATA!}
 <pre>".print_r($DATA, true)."</pre>
 <br /><br />";
 
                // Debug mode not active? Then remove the HTML tags
-               if (!DEBUG_MODE) $newContent = strip_tags($newContent);
+               if (!isBooleanConstantAndTrue('DEBUG_MODE')) $newContent = strip_tags($newContent);
        } else {
                // No template name supplied!
-               $newContent = NO_TEMPLATE_SUPPLIED;
+               $newContent = getMessage('NO_TEMPLATE_SUPPLIED');
        }
 
        // Is there some content?
@@ -2638,7 +2638,7 @@ function CACHE_PURGE_ADMIN_MENU ($id=0, $action="", $what="", $str="") {
                // No cache instance!
                DEBUG_LOG(__FUNCTION__, __LINE__, " No cache instance found.");
                return false;
-       } elseif ((!isConfigEntrySet('cache_admin_menu')) || (getConfig('cache_admin_menu') == "N")) {
+       } elseif ((!isConfigEntrySet('cache_admin_menu')) || (getConfig('cache_admin_menu') != "Y")) {
                // Caching disabled (currently experiemental!)
                return false;
        }
index fdadd1a..62e5949 100644 (file)
@@ -43,7 +43,7 @@ function BEG_ADD_POINTS ($uid, $points) {
 
        // Set mode depending on how many mails the member has to confirm
        $locked = false;
-       if ((getConfig('ref_payout') > 0) && (getConfig('allow_direct_pay') == "N")) $locked = true;
+       if ((getConfig('ref_payout') > 0) && (getConfig('allow_direct_pay') != "Y")) $locked = true;
 
        // Is begging rallye active?
        if (getConfig('beg_rallye') == "Y") {
index 5ceee13..b5ea9b2 100644 (file)
@@ -42,7 +42,7 @@ function BONUS_ADD_TURBO_POINTS($mid, $uid, $type) {
        global $_CONFIG;
 
        // Shall we add bonus points?
-       if (getConfig('bonus_active') == "N") return false;
+       if (getConfig('bonus_active') != "Y") return false;
 
        // Init variables
        $SQL = ""; $bonys = 0; $mail = 0; $column = "";
@@ -191,7 +191,7 @@ function BONUS_POINTS_HANDLER ($MODE) {
        global $_CONFIG;
 
        // Shall we add bonus points?
-       if (getConfig('bonus_active') == "N") return;
+       if (getConfig('bonus_active') != "Y") return;
 
        // Switch to jackpot-mode when no UID is supplied but userid-mode is selected
        if ((getConfig('bonus_mode') == "UID") && (getConfig('bonus_uid') == "0")) {
index 7809ce6..e8e6d9a 100644 (file)
@@ -210,7 +210,7 @@ function WERNIS_TEST_API () {
 // Widthdraw this amount
 function WERNIS_EXECUTE_WITHDRAW ($wdsId, $userMd5, $amount) {
        // Is the sponsor extension installed?
-       if (getConfig('wernis_withdraw_active') == "N") {
+       if (getConfig('wernis_withdraw_active') != "Y") {
                if (!EXT_IS_ACTIVE("sponsor")) {
                        // No, abort here
                        return false;
index f77b7c7..dd92ec8 100644 (file)
@@ -72,7 +72,7 @@ if (EXT_IS_ACTIVE("cache")) {
                // Init cache file
                $cacheInstance->init("EXTENSIONS");
                $cacheInstance->storeExtensionVersion("sql_patches");
-       } elseif (getConfig('cache_exts') == "N") {
+       } elseif (getConfig('cache_exts') != "Y") {
                // Cache will not be created for extensions
                $cacheMode = "skip";
        }
index b55200c..cea9d4c 100644 (file)
@@ -65,7 +65,7 @@ if (isset($_POST['ok'])) {
 
        // Mail confirmation links
        define('_CFG_AUTO_PURGE', CREATE_TIME_SELECTIONS(getConfig('auto_purge'), "auto_purge", "MWD"));
-       if (getConfig('auto_purge_active') == "N") {
+       if (getConfig('auto_purge_active') != "Y") {
                define('_CFG_AP_ACTIVE_N', " checked=\"checked\"");
                define('_CFG_AP_ACTIVE_Y', "");
        } else {
index 04febbd..dbf9e30 100644 (file)
@@ -79,7 +79,7 @@ if (isset($_POST['ok'])) {
        define('_CFG_UNCONFIRMED'   , getConfig('unconfirmed'));
        define('_CFG_MAX_TLENGTH'   , getConfig('max_tlength'));
 
-       if (getConfig('test_text') == "N") {
+       if (getConfig('test_text') != "Y") {
                define('_CFG_TEST_TEXT_N', " checked=\"checked\"");
                define('_CFG_TEST_TEXT_Y', "");
        } else {
@@ -107,7 +107,7 @@ if (isset($_POST['ok'])) {
                        break;
        }
 
-       if (getConfig('test_subj') == "N") {
+       if (getConfig('test_subj') != "Y") {
                define('_CFG_TEST_SUBJ_N', " checked=\"checked\"");
                define('_CFG_TEST_SUBJ_Y', "");
        } else {
@@ -115,7 +115,7 @@ if (isset($_POST['ok'])) {
                define('_CFG_TEST_SUBJ_Y', " checked=\"checked\"");
        }
 
-       if (getConfig('url_blacklist') == "N") {
+       if (getConfig('url_blacklist') != "Y") {
                define('_CFG_URL_BLIST_N', " checked=\"checked\"");
                define('_CFG_URL_BLIST_Y', "");
        } else {
index 58fe215..d73a2f7 100644 (file)
@@ -86,7 +86,7 @@ if (isset($_POST['ok'])) {
        define('_CFG_PROFILE_UPDATE', CREATE_TIME_SELECTIONS(getConfig('profile_update'), "profile_update"       , "YM"));
        define('_CFG_PROF_REUPDATE' , CREATE_TIME_SELECTIONS(getConfig('resend_profile_update') , "resend_profile_update", "MWD"));
 
-       if (getConfig('order_multi_page') == "N") {
+       if (getConfig('order_multi_page') != "Y") {
                define('_CFG_ORDER_MULTI_N', " checked=\"checked\"");
                define('_CFG_ORDER_MULTI_Y', "");
        } else {
@@ -94,7 +94,7 @@ if (isset($_POST['ok'])) {
                define('_CFG_ORDER_MULTI_Y', " checked=\"checked\"");
        }
 
-       if (getConfig('autosend_active') == "N") {
+       if (getConfig('autosend_active') != "Y") {
                define('_CFG_AUTOSEND_ACTIVE_N', " checked=\"checked\"");
                define('_CFG_AUTOSEND_ACTIVE_Y', "");
        } else {
@@ -102,7 +102,7 @@ if (isset($_POST['ok'])) {
                define('_CFG_AUTOSEND_ACTIVE_Y', " checked=\"checked\"");
        }
 
-       if (getConfig('send_prof_update') == "N") {
+       if (getConfig('send_prof_update') != "Y") {
                define('_CFG_SEND_UPDATE_N', " checked=\"checked\"");
                define('_CFG_SEND_UPDATE_Y', "");
        } else {
@@ -110,7 +110,7 @@ if (isset($_POST['ok'])) {
                define('_CFG_SEND_UPDATE_Y', " checked=\"checked\"");
        }
 
-       if (getConfig('admin_notify') == "N") {
+       if (getConfig('admin_notify') != "Y") {
                define('_CFG_ADMIN_NOTIFY_N', " checked=\"checked\"");
                define('_CFG_ADMIN_NOTIFY_Y', "");
        } else {
index b47c51c..e1f53a7 100644 (file)
@@ -227,7 +227,7 @@ LIMIT 1",
 
                // Load main template
                LOAD_TEMPLATE("admin_unlock_emails");
-       } elseif ((isset($_POST['lock'])) && (getConfig('url_blacklist') == "N")) {
+       } elseif ((isset($_POST['lock'])) && (getConfig('url_blacklist') != "Y")) {
                // URL blacklist not activated
                LOAD_TEMPLATE("admin_settings_saved", false, getMessage('ADMIN_URL_BLACKLIST_DISABLED'));
        } else {
index 7cf8158..ba46619 100644 (file)
@@ -195,11 +195,12 @@ if ((isset($_POST['ok'])) && ((!$FAILED) || (IS_ADMIN()))) {
        } // END - if
 
        // Check if I shall disable sending mail to newly registered members out about active/begging rallye
+       // @TODO Rewrite this to a single filter
        //
        // First comes first: begging rallye
        if (GET_EXT_VERSION("beg") >= "0.1.7") {
                // Okay, shall I disable now?
-               if (getConfig('beg_new_mem_notify') == "N") {
+               if (getConfig('beg_new_mem_notify') != "Y") {
                        $ADD1 .= ", beg_ral_notify, beg_ral_en_notify";
                        $ADD2 .= ", UNIX_TIMESTAMP(), UNIX_TIMESTAMP()";
                } // END - if
@@ -208,7 +209,7 @@ if ((isset($_POST['ok'])) && ((!$FAILED) || (IS_ADMIN()))) {
        // Second: active rallye
        if (GET_EXT_VERSION("bonus") >= "0.7.7") {
                // Okay, shall I disable now?
-               if (getConfig('bonus_new_mem_notify') == "N") {
+               if (getConfig('bonus_new_mem_notify') != "Y") {
                        $ADD1 .= ", bonus_ral_notify, bonus_ral_en_notify";
                        $ADD2 .= ", UNIX_TIMESTAMP(), UNIX_TIMESTAMP()";
                } // END - if
index ae81c82..c82794c 100644 (file)
@@ -147,7 +147,7 @@ WHERE sender=%s AND url='%s' AND timestamp > (UNIX_TIMESTAMP() - %s) LIMIT 1",
                                list($blist) = SQL_FETCHROW($result);
 
                                // Create redirect-URL
-                               $URL = "modules.php?module=login&amp;what=order&amp;msg=".constant('CODE_BLIST_URL')."&blist=".$blist;
+                               $URL = "modules.php?module=login&amp;what=order&amp;msg=".constant('CODE_BLIST_URL')."&amp;blist=".$blist;
                        } // END - if
 
                        // Free result
@@ -616,8 +616,8 @@ LIMIT 1", array(bigintval($ucat)), __FILE__, __LINE__);
                                // Free result
                                SQL_FREERESULT($result);
 
-                               // 01      2              21    12                                   2    23         443    3          3210
-                               if ((!empty($_POST['data'])) || ((getConfig('order_multi_page') == "N") && ((!IS_ADMIN()) && (!EXT_IS_ACTIVE("html_mail"))))) {
+                               // 01      2              21    12                                    2    23         443    3          3210
+                               if ((!empty($_POST['data'])) || ((getConfig('order_multi_page') != "Y") && ((!IS_ADMIN()) && (!EXT_IS_ACTIVE("html_mail"))))) {
                                        // Pre-output categories
                                        $CAT = "";
                                        foreach ($CATS['id'] as $key => $value) {
index da7b330..ebbf566 100644 (file)
@@ -168,7 +168,7 @@ if ((GET_EXT_VERSION("bonus") >= "0.2.2") && (EXT_IS_ACTIVE("bonus")) && (getCon
 
        // Output rows
        define('__SPECIAL_ROWS', LOAD_TEMPLATE("member_points_bonus_rows", true));
-} elseif (getConfig('bonus_active') == "N") {
+} elseif (getConfig('bonus_active') != "Y") {
        // Bonus active rallye deactivated
        define('__SPECIAL_ROWS', LOAD_TEMPLATE("member_points_bonus_disabled", true));
 } elseif ((IS_ADMIN()) && (EXT_VERSION_IS_OLDER("bonus", "0.2.2")) && (EXT_IS_ACTIVE("bonus"))) {
index e751a5a..687b45b 100644 (file)
@@ -44,7 +44,7 @@ if (!defined('__SECURITY')) {
 }
 
 // Is the refback system enabled?
-if (getConfig('refback_enabled') == "N") {
+if (getConfig('refback_enabled') != "Y") {
        // Output message
        LOAD_TEMPLATE("admin_settings_saved", false, getMessage('MEMBER_REFBACK_DISABLED'));
        // Abort here
index 17a38d2..292eb5b 100644 (file)
@@ -40,7 +40,7 @@ if (!defined('__SECURITY')) {
 }
 
 // Do not execute when script is in CSS mode or no daily reset
-if (($CSS == 1) || (!defined('__DAILY_RESET')) || (getConfig('beg_rallye') == "N")) return;
+if (($CSS == 1) || (!defined('__DAILY_RESET')) || (getConfig('beg_rallye') != "Y")) return;
 //* DEBUG: */ echo basename(__FILE__)."<br />\n";
 
 // Get current month (2 digits)
@@ -53,7 +53,7 @@ if (($curr != getConfig('last_month')) && (getConfig('last_month') > 0) && (getC
        $whereStatement2 = bigintval(getConfig('beg_ranks'));
 
        // Shall we exclude webmaster's own userid?
-       if ((getConfig('beg_include_own') == "N") && (getConfig('beg_uid') > 0)) {
+       if ((getConfig('beg_include_own') != "Y") && (getConfig('beg_uid') > 0)) {
                // Exclude it
                $whereStatement1 = sprintf(" AND userid != %s", bigintval(getConfig('beg_uid')));
        } // END - if
index 9484506..4e10c88 100644 (file)
@@ -37,10 +37,13 @@ if (!defined('__SECURITY')) {
        require($INC);
 }
 
-//
-function ADD_MODULE_TITLE($mod) {
+// Returns the title for given module
+function ADD_MODULE_TITLE ($mod) {
        global $cacheArray;
-       $name = ""; $result = false;
+
+       // Init variables
+       $name = "";
+       $result = false;
 
        // Is the script installed?
        if (isBooleanConstantAndTrue('mxchange_installed')) {
@@ -65,7 +68,7 @@ function ADD_MODULE_TITLE($mod) {
        // Still no luck or empty title?
        if (empty($name)) {
                // No name found
-               $name = sprintf("%s (%s)", LANG_UNKNOWN_MODULE, $mod);
+               $name = sprintf("%s (%s)", getMessage('LANG_UNKNOWN_MODULE'), $mod);
                if (SQL_NUMROWS($result) == 0) {
                        // Add module to database
                        $dummy = CHECK_MODULE($mod);
@@ -133,7 +136,7 @@ function CHECK_MODULE ($mod) {
        }
 
        // Is the module found?
-       if ($found) {
+       if ($found === true) {
                // Check returned values against current access permissions
                //
                //  Admin access            ----- Guest access -----           --- Guest   or   member? ---
@@ -204,6 +207,7 @@ function CHECK_MODULE ($mod) {
 // Add menu description pending on given file name (without path!)
 function ADD_DESCR ($ACC_LVL, $file, $return = false, $output = true) {
        global $NAV_DEPTH;
+
        // Use only filename of the file ;)
        $file = basename($file);
 
@@ -322,6 +326,7 @@ function ADD_DESCR ($ACC_LVL, $file, $return = false, $output = true) {
                return $OUT;
        }
 }
+
 //
 function ADD_MENU ($MODE, $act, $wht) {
        // Init some variables
@@ -437,6 +442,7 @@ function ADD_MENU ($MODE, $act, $wht) {
                OUTPUT_HTML("</table>");
        }
 }
+
 // This patched function will reduce many SELECT queries for the specified or current admin login
 function IS_ADMIN ($admin="") {
        global $cacheArray;
@@ -458,14 +464,14 @@ function IS_ADMIN ($admin="") {
                // Use cached string
                $valPass = $cacheArray['admin_hash'];
        } elseif ((!empty($passCookie)) && (isset($cacheArray['admins']['password'][$admin])) && (!empty($admin))) {
-               // Count cache hits
-               incrementConfigEntry('cache_hits');
-
                // Login data is valid or not?
                $valPass = generatePassString($cacheArray['admins']['password'][$admin]);
 
                // Cache it away
                $cacheArray['admin_hash'] = $valPass;
+
+               // Count cache hits
+               incrementConfigEntry('cache_hits');
        } elseif ((!empty($admin)) && ((!EXT_IS_ACTIVE("cache"))) || (!isset($cacheArray['admins']['password'][$admin]))) {
                // Search for admin
                $result = SQL_QUERY_ESC("SELECT HIGH_PRIORITY password FROM `{!_MYSQL_PREFIX!}_admins` WHERE login='%s' LIMIT 1",
@@ -498,21 +504,18 @@ function IS_ADMIN ($admin="") {
        //* DEBUG: */ if (!$ret) echo __LINE__."OK!<br />";
        return $ret;
 }
-//
-function ADD_MAX_RECEIVE_LIST($MODE, $default="", $return=false)
-{
+
+// Generates a list of "max receiveable emails per day"
+function ADD_MAX_RECEIVE_LIST ($MODE, $default = "", $return = false) {
        global $_POST;
        $OUT = "";
-       switch ($MODE)
-       {
+       switch ($MODE) {
        case "guest":
                // Guests (in the registration form) are not allowed to select 0 mails per day.
                $result = SQL_QUERY("SELECT value, comment FROM `{!_MYSQL_PREFIX!}_max_receive` WHERE value > 0 ORDER BY value", __FILE__, __LINE__);
-               if (SQL_NUMROWS($result) > 0)
-               {
+               if (SQL_NUMROWS($result) > 0) {
                        $OUT = "";
-                       while (list($value, $comment) = SQL_FETCHROW($result))
-                       {
+                       while (list($value, $comment) = SQL_FETCHROW($result)) {
                                $OUT .= "      <option value=\"".$value."\"";
                                if ($_POST['max_mails'] == $value) $OUT .= " selected=\"selected\"";
                                $OUT .= ">".$value." ".PER_DAY;
@@ -524,21 +527,18 @@ function ADD_MAX_RECEIVE_LIST($MODE, $default="", $return=false)
                        // Free memory
                        SQL_FREERESULT($result);
                        $OUT = LOAD_TEMPLATE("guest_receive_table", true);
-               }
-                else
-               {
+               } else {
                        // Maybe the admin has to setup some maximum values?
+                       debug_report_bug("Nothing is being done here?");
                }
                break;
 
        case "member":
                // Members are allowed to set to zero mails per day (we will change this soon!)
                $result = SQL_QUERY("SELECT value, comment FROM `{!_MYSQL_PREFIX!}_max_receive` ORDER BY value", __FILE__, __LINE__);
-               if (SQL_NUMROWS($result) > 0)
-               {
+               if (SQL_NUMROWS($result) > 0) {
                        $OUT = "";
-                       while (list($value, $comment) = SQL_FETCHROW($result))
-                       {
+                       while (list($value, $comment) = SQL_FETCHROW($result)) {
                                $OUT .= "      <option value=\"".$value."\"";
                                if ($default == $value) $OUT .= " selected=\"selected\"";
                                $OUT .= ">".$value." ".PER_DAY;
@@ -548,53 +548,75 @@ function ADD_MAX_RECEIVE_LIST($MODE, $default="", $return=false)
                        define('__MAX_RECEIVE_OPTIONS', $OUT);
                        SQL_FREERESULT($result);
                        $OUT = LOAD_TEMPLATE("member_receive_table", true);
-               }
-                else
-               {
+               } else {
                        // Maybe the admin has to setup some maximum values?
                        $OUT = LOAD_TEMPLATE("admin_settings_saved", true, NO_MAX_VALUES);
                }
                break;
        }
-       if ($return)
-       {
+
+       if ($return) {
                // Return generated HTML code
                return $OUT;
-       }
-        else
-       {
+       } else {
                // Output directly (default)
                OUTPUT_HTML($OUT);
        }
 }
-//
-function SEARCH_EMAIL_USERTAB($email)
-{
-       $ret = false;
-       $result = SQL_QUERY_ESC("SELECT userid FROM `{!_MYSQL_PREFIX!}_user_data` WHERE email LIKE '{PER}%s{PER}' LIMIT 1", array($email), __FILE__, __LINE__);
-       if (SQL_NUMROWS($result) == 1) $ret = true;
+
+// Checks wether the given email address is used.
+function SEARCH_EMAIL_USERTAB ($email) {
+       // Query the database
+       $result = SQL_QUERY_ESC("SELECT userid FROM `{!_MYSQL_PREFIX!}_user_data` WHERE email LIKE '{PER}%s{PER}' LIMIT 1",
+               array($email), __FILE__, __LINE__);
+
+       // Is the email there?
+       $ret = (SQL_NUMROWS($result) == 1);
+
+       // Free the result
        SQL_FREERESULT($result);
+
+       // Return result
        return $ret;
 }
-//
-function WHAT_IS_VALID($act, $wht, $type="guest")
-{
-       if (IS_ADMIN())
-       {
+
+// Checks wether 'what' is valid
+// @TODO This cache can be flushed to hard drive
+function WHAT_IS_VALID ($act, $wht, $type="guest") {
+       global $cacheArray;
+
+       // Is admin or cache entry is there?
+       if (IS_ADMIN()) {
                // Everything is valid to the admin :-)
-               return true;
-       }
-        else
-       {
+               $ret = true;
+       } elseif (isset($cacheArray['what_valid'][$type][$act][$what])) {
+               // Use the cache
+               $ret = $cacheArray['what_valid'][$type][$act][$what];
+
+               // Count the cache hit
+               incrementConfigEntry('cache_hits');
+       } else {
+               // By default is nothing valid
                $ret = false;
+
+               // Look in database
                $result = SQL_QUERY_ESC("SELECT id FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE action='%s' AND what='%s' AND `locked`='N' LIMIT 1", array($type, $act, $wht), __FILE__, __LINE__);
+
                // Is "what" valid?
-               if (SQL_NUMROWS($result) == 1) $ret = true;
+               $ret = (SQL_NUMROWS($result) == 1);
+
+               // Free the result
                SQL_FREERESULT($result);
-               return $ret;
        }
+
+       // Cache the entry
+       $cacheArray['what_valid'][$type][$act][$what] = $ret;
+
+       // Return the result
+       return $ret;
 }
-//
+
+// Checks wether the current user is a member
 function IS_MEMBER () {
        global $status, $LAST, $cacheArray;
        if (!is_array($LAST)) $LAST = array();
@@ -657,12 +679,16 @@ function IS_MEMBER () {
        // Return status
        return $ret;
 }
-//
-function VALIDATE_MENU_ACTION ($MODE, $act, $wht, $UPDATE=false)
-{
+
+// Validate the given menu action
+function VALIDATE_MENU_ACTION ($MODE, $act, $wht, $UPDATE=false) {
+       // By default nothing is valid
        $ret = false;
+
+       // Look in all menus or only unlocked
        $ADD = "";
        if ((!IS_ADMIN()) && ($MODE != "admin")) $ADD = " AND `locked`='N'";
+
        //* DEBUG: */ echo __LINE__.":".$MODE."/".$act."/".$wht."*<br />\n";
        if (($MODE != "admin") && ($UPDATE)) {
                // Update guest or member menu
index 05ce24c..67ff3d4 100644 (file)
@@ -48,19 +48,16 @@ WHERE
 OR
        (last_update=0 AND last_profile_sent=0 AND joined < (UNIX_TIMESTAMP() - %s))
 ORDER BY userid", array(getConfig('profile_update'), getConfig('resend_profile_update'), getConfig('profile_update')), __FILE__, __LINE__);
-       if (SQL_NUMROWS($result) > 0)
-       {
+
+       // Do we have some notifications to sent?
+       if (SQL_NUMROWS($result) > 0) {
                // We need to send-out notifications...
-               while ($DATA = SQL_FETCHROW($result))
-               {
+               while ($DATA = SQL_FETCHROW($result)) {
                        $DATA[6] = MAKE_DATETIME($DATA[6], "0");
-                       if (round($DATA[5]) == "0")
-                       {
+                       if (round($DATA[5]) == "0") {
                                // Has never changed his accont
                                $DATA[5] = PROFILE_NEVER_CHANGED;
-                       }
-                        else
-                       {
+                       } else {
                                // Has changed his account
                                $DATA[5] = MAKE_DATETIME($DATA[5], "0");
                        }
@@ -72,11 +69,12 @@ ORDER BY userid", array(getConfig('profile_update'), getConfig('resend_profile_u
                        // Update profile data
                        SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET last_profile_sent=UNIX_TIMESTAMP(), notified='Y' WHERE userid=%s LIMIT 1",
                                array(bigintval($DATA[0])), __FILE__, __LINE__);
-               }
-       }
+               } // END - while
+       } // END - if
 
        // Free result
        SQL_FREERESULT($result);
 }
+
 //
 ?>
index a84660c..8c62a7b 100644 (file)
@@ -60,31 +60,31 @@ if (($CSS == "1") || (getConfig('css_php') == "DIRECT")) {
        foreach ($STYLES as $value) {
                // Only include found CSS files (to reduce 404 requests)
                $BASE = sprintf("%stheme/%s/css/", constant('PATH'), GET_CURR_THEME());
-               $file = $BASE.$value;
+               $FQFN = $BASE.$value;
 
                // Do include only existing files and whose are not empty
-               if ((FILE_READABLE($file)) && (filesize($file) > 0)) {
+               if ((FILE_READABLE($FQFN)) && (filesize($FQFN) > 0)) {
                        switch (getConfig('css_php')) {
                                case "DIRECT":
                                        OUTPUT_HTML("<link rel=\"stylesheet\" type=\"text/css\" href=\"{!URL!}/theme/".GET_CURR_THEME()."/".$value."\" />");
                                        break;
 
                                case "FILE":
-                                       OUTPUT_HTML(READ_FILE($file));
+                                       OUTPUT_HTML(READ_FILE($FQFN));
                                        break;
-                       }
-               }
-       }
+                       } // END - switch
+               } // END - if
+       } // END - foreach
 } else {
        // Now we load all CSS files from css.php!
        OUTPUT_HTML("<link rel=\"stylesheet\" type=\"text/css\" href=\"{!URL!}/css.php", false);
        if (isBooleanConstantAndTrue('mxchange_installing')) {
                // Default theme first
                $NEW_THEME = "default";
-               if (!empty($_GET['theme'])) $NEW_THEME = SQL_ESCAPE($_GET['theme']);
+               if (!empty($_GET['theme']))  $NEW_THEME = SQL_ESCAPE($_GET['theme']);
                if (!empty($_POST['theme'])) $NEW_THEME = SQL_ESCAPE($_POST['theme']);
                OUTPUT_HTML("?theme=".$NEW_THEME."&amp;installing=1", false);
-       }
+       } // END - if
        OUTPUT_HTML("\" />");
 }
 
index cf0ec43..b52f974 100644 (file)
@@ -225,7 +225,7 @@ if (isBooleanConstantAndTrue('mxchange_installed')) {
                                                                                $template = "mailid_points_done";
 
                                                                                // Right code entered add points and remove entry
-                                                                               if (($ref_pay > 0) && (getConfig('allow_direct_pay') == "N")) {
+                                                                               if (($ref_pay > 0) && (getConfig('allow_direct_pay') != "Y")) {
                                                                                        // Don't add points over the referal system
                                                                                        $locked = true;
                                                                                        $template = "mailid_points_locked";
index bee4bea..17271bc 100644 (file)
@@ -49,7 +49,7 @@ if (empty($_GET['module'])) $_GET['module'] = "index";
 
 // Secure action/what if present
 if (!empty($_GET['action'])) $GLOBALS['action'] = secureString($_GET['action']);
-if (!empty($_GET['what'])) $GLOBALS['what'] = secureString($_GET['what']);
+if (!empty($_GET['what']))   $GLOBALS['what']   = secureString($_GET['what']);
 
 // Secure the module name (very important line!)
 $GLOBALS['module'] = secureString($_GET['module']);
@@ -87,10 +87,10 @@ if (IS_MEMBER()) {
        SQL_FREERESULT($result);
 } elseif (IS_ADMIN()) {
        // Admin is there
-       $username = _ADMIN;
+       $username = getMessage('_ADMIN');
 } else {
        // He's a guest, hello there... ;-)
-       $username = _GUEST;
+       $username = getMessage('_GUEST');
 }
 
 // The header file