More SQLs rewritten fix_menu.php is now 'pool-ed'

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

diff --git a/inc/monthly/monthly_beg.php b/inc/monthly/monthly_beg.php
index f9949262771bccc73fb1287a3d7c25b34c081927..857e65636e696eb93998eddbb6d12826e8a08548 100644 (file)
--- a/inc/monthly/monthly_beg.php
+++ b/inc/monthly/monthly_beg.php
@@ -58,7 +58,7 @@ if (($curr != getConfig('last_month')) && (getConfig('last_month') > 0) && (getC
        // Shall we exclude webmaster's own userid?
        if ((getConfig('beg_include_own') != 'Y') && (getConfig('beg_userid') > 0)) {
                // Exclude it
-               $whereStatement1 = sprintf(" AND `userid` != %s", getConfig('beg_userid'));
+               $whereStatement1 = " AND `userid` != {?beg_userid?}";
        } // END - if
 
        // Let's check if there are some points left we can 'pay'...
@@ -66,7 +66,7 @@ if (($curr != getConfig('last_month')) && (getConfig('last_month') > 0) && (getC
                // Use last online stamp only when autopurge for inactive members is activated
                if ((getConfig('ap_inactive_since') > 0) && (getConfig('beg_active') == 'Y')) {
                        // Okay, include last online timestamp
-                       $whereStatement1 .= sprintf(" AND `last_online` >= (UNIX_TIMESTAMP() - %s)", getConfig('ap_inactive_since'));
+                       $whereStatement1 .= " AND `last_online` >= (UNIX_TIMESTAMP() - {?ap_inactive_since?})";
                } // END - if
        } // END - if
 
@@ -76,9 +76,11 @@ if (($curr != getConfig('last_month')) && (getConfig('last_month') > 0) && (getC
 FROM
        `{?_MYSQL_PREFIX?}_user_data`
 WHERE
-       `status`='CONFIRMED'" . $whereStatement1 . " AND `beg_points` > 0
+       `status`='CONFIRMED'" . $whereStatement1 . " AND
+       `beg_points` > 0
 ORDER BY
-       `beg_points` DESC, `userid` ASC
+       `beg_points` DESC,
+       `userid` ASC
 LIMIT {?beg_ranks?}", __FILE__, __LINE__);
 
        if (SQL_NUMROWS($result_main) > 0) {