Missing column 'active_bonus' fixed, some while() loops rewritten

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

diff --git a/inc/modules/member/what-bonus.php b/inc/modules/member/what-bonus.php
index 9a223e9a2023f2f15f22d1e4e9fc0d4e0c2a92c0..1c20685a78f851da8d566469045efb3d6351b3f1 100644 (file)
--- a/inc/modules/member/what-bonus.php
+++ b/inc/modules/member/what-bonus.php
@@ -71,27 +71,25 @@ if (EXT_IS_ACTIVE("autopurge")) {
 }
 
 // Let's check if there are some points left we can "pay"...
-$result = SQL_QUERY_ESC("SELECT userid, ".$USE." AS active_bonus, last_online FROM "._MYSQL_PREFIX."_user_data
+$result = SQL_QUERY_ESC("SELECT userid, ".$USE." AS points, last_online FROM "._MYSQL_PREFIX."_user_data
 WHERE ".$USE." > 0 AND status='CONFIRMED'".$LAST."
-ORDER BY active_bonus DESC, last_online DESC, userid LIMIT %s",
+ORDER BY points DESC, last_online DESC, userid
+LIMIT %s",
  array($ONLINE, $_CONFIG['bonus_ranks']), __FILE__, __LINE__);
 
 // Reset temporary variable and check for users
 $OUT = "";
-if (SQL_NUMROWS($result) > 0)
-{
+if (SQL_NUMROWS($result) > 0) {
        // Load our winners...
        $SW = 2; $cnt = 1;
-       while(list($uid, $turbo, $last) = SQL_FETCHROW($result))
-       {
+       while ($content = SQL_FETCHARRAY($result)) {
                // Prepare data for the template
                $content = array(
-                       'sw'     => $SW,
-                       'cnt'    => $cnt,
-                       'uid'    => bigintval($uid),
-                       'points' => TRANSLATE_COMMA($turbo),
-                       'last'   => MAKE_DATETIME($last, "2"),
-               );
+               $content['sw']     = $SW;
+               $content['cnt']    = $cnt;
+               $content['uid']    = bigintval($content['uid']);
+               $content['points'] = TRANSLATE_COMMA($content['points']);
+               $content['last']   = MAKE_DATETIME($content['last'], "2");
 
                // Load row template
                $OUT .= LOAD_TEMPLATE("member_bonus_row", true, $content);
@@ -99,9 +97,7 @@ if (SQL_NUMROWS($result) > 0)
                // Count one up and switch colors
                $cnt++;$SW = 3 - $SW;
        }
-}
- else
-{
+} else {
        // No one is interested in our "active rallye" ! :-(
        $OUT = LOAD_TEMPLATE("member_bonus_404", true);
 }