More queries now depends on UNIX_TIMESTAMP() SQL function, wrong index in autopurge...

[mailer.git] / inc / monthly / monthly_bonus.php

diff --git a/inc/monthly/monthly_bonus.php b/inc/monthly/monthly_bonus.php
index d877e5e6f9f50bf9818b108ea9f6f6f14d551047..c00a486e606ad53b5e7c1ca22f3d4d5de110a25d 100644 (file)
--- a/inc/monthly/monthly_bonus.php
+++ b/inc/monthly/monthly_bonus.php
@@ -32,58 +32,55 @@
  ************************************************************************/
 
 // Some security stuff...
-if (ereg(basename(__FILE__), $_SERVER['PHP_SELF']))
-{
+if (!defined('__SECURITY')) {
        $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
        require($INC);
-}
- elseif ((!EXT_IS_ACTIVE("bonus")) && (!IS_ADMIN()))
-{
+} elseif ((!EXT_IS_ACTIVE("bonus")) && (!IS_ADMIN())) {
        ADD_FATAL(EXTENSION_PROBLEM_EXT_INACTIVE, "bonus");
        return;
 }
 
 // Do not execute when script is in CSS mode or no daily reset
-if (($CSS == 1) || (!defined('__DAILY_RESET'))) return;
+if (($CSS == 1) || (!isBooleanConstantAndTrue('__DAILY_RESET'))) return;
 
 // Get current month (2 digits)
 $curr = date("m", time());
 
-if (($curr != $CONFIG['bonus_month']) && ($CONFIG['bonus_ranks'] > 0) && ($CSS != 1))
+if (($curr != $_CONFIG['last_month']) && ($_CONFIG['bonus_ranks'] > 0) && ($CSS != 1))
 {
        // Extension "autopurge" is inactive or purging of inactive accounts is deactivated
-       $WHERE1 = "WHERE status='CONFIRMED'";
-       $WHERE2 = 0;
-       $WHERE3 = bigintval($CONFIG['bonus_ranks']);
+       $whereStatement1 = "WHERE status='CONFIRMED'";
+       $whereStatement2 = 0;
+       $whereStatement3 = bigintval($_CONFIG['bonus_ranks']);
 
        // Shall I keep inactive members away from here? (mostly wanted in an "active-rallye" ...)
        if (EXT_IS_ACTIVE("autopurge"))
        {
                // Use last online stamp only when autopurge for inactive members is activated
-               if ($CONFIG['ap_in_since'] > 0)
+               if ($_CONFIG['ap_inactive_since'] > 0)
                {
                        // Okay, include last online timestamp
-                       $WHERE1 = "WHERE status='CONFIRMED' AND last_online >=";
-                       $WHERE2 = bigintval(time() - $CONFIG['ap_in_since']);
-                       $WHERE3 = bigintval($CONFIG['bonus_ranks']);
+                       $whereStatement1 = "WHERE status='CONFIRMED' AND last_online >=";
+                       $whereStatement2 = bigintval(time() - $_CONFIG['ap_inactive_since']);
+                       $whereStatement3 = bigintval($_CONFIG['bonus_ranks']);
                }
        }
 
        // Add more bonus points here
        $ADD = " AND (0";
-       if ($CONFIG['bonus_click_yn'] == 'Y') $ADD .= " + turbo_bonus";
-       if ($CONFIG['bonus_login_yn'] == 'Y') $ADD .= " + login_bonus";
-       if ($CONFIG['bonus_order_yn'] == 'Y') $ADD .= " + bonus_order";
-       if ($CONFIG['bonus_stats_yn'] == 'Y') $ADD .= " + bonus_stats";
-       if ($CONFIG['bonus_ref_yn']   == 'Y') $ADD .= " + bonus_ref";
+       if ($_CONFIG['bonus_click_yn'] == "Y") $ADD .= " + turbo_bonus";
+       if ($_CONFIG['bonus_login_yn'] == "Y") $ADD .= " + login_bonus";
+       if ($_CONFIG['bonus_order_yn'] == "Y") $ADD .= " + bonus_order";
+       if ($_CONFIG['bonus_stats_yn'] == "Y") $ADD .= " + bonus_stats";
+       if ($_CONFIG['bonus_ref_yn']   == "Y") $ADD .= " + bonus_ref";
        $ADD .= ") > 0";
 
        // SQL string to check for accounts
        $result_main = SQL_QUERY_ESC("SELECT userid, email, (turbo_bonus + login_bonus + bonus_order + bonus_stats + bonus_ref) AS active_bonus
 FROM "._MYSQL_PREFIX."_user_data
-".$WHERE1." %s".$ADD."
+".$whereStatement1." %s".$ADD."
 ORDER BY active_bonus DESC, userid LIMIT %s",
- array($WHERE2, $WHERE3), __FILE__, __LINE__);
+ array($whereStatement2, $whereStatement3), __FILE__, __LINE__);
 
        if (SQL_NUMROWS($result_main) > 0)
        {
@@ -96,7 +93,7 @@ ORDER BY active_bonus DESC, userid LIMIT %s",
 
                        // Add points to user's account directly
                        $result_data = SQL_QUERY_ESC("UPDATE "._MYSQL_PREFIX."_user_points
-SET points=points+%s WHERE ref_depth=0 AND userid=%d LIMIT 1",
+SET points=points+%s WHERE ref_depth=0 AND userid=%s LIMIT 1",
  array($points, bigintval($uid)), __FILE__, __LINE__);
 
                        // Update mediadata as well
@@ -126,13 +123,7 @@ SET turbo_bonus=0, login_bonus=0, bonus_order=0, bonus_stats=0, bonus_ref=0", __
 
        // Free memory
        SQL_FREERESULT($result_main);
-
-       // Finally update database and config array
-       $CONFIG['bonus_month'] = $curr;
-       $result_update = SQL_QUERY_ESC("UPDATE "._MYSQL_PREFIX."_config SET last_bonus_month='%s' WHERE config='0' LIMIT 1",
-        array($curr), __FILE__, __LINE__);
-
-       // Destroy cache
 }
+
 //
 ?>