A lot while() conditions rewritten to SQL_FETCHARRAY(), see bug #107, @TODO tags...

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

diff --git a/inc/modules/member/what-beg2.php b/inc/modules/member/what-beg2.php
index 116fd821d0722256917bc46af2789922e82c3d28..78573eef5cecaebbd8f61a1aa1828446019d30c2 100644 (file)
--- a/inc/modules/member/what-beg2.php
+++ b/inc/modules/member/what-beg2.php
@@ -38,7 +38,7 @@ if (!defined('__SECURITY')) {
 } elseif (!IS_MEMBER()) {
        LOAD_URL("modules.php[5~?module=index");
 } elseif ((!EXT_IS_ACTIVE("beg")) && (!IS_ADMIN())) {
-       addFatalMessage(EXTENSION_PROBLEM_EXT_INACTIVE, "beg");
+       addFatalMessage(__FILE__, __LINE__, getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), "beg");
        return;
 }
 
@@ -46,16 +46,16 @@ if (!defined('__SECURITY')) {
 ADD_DESCR("member", __FILE__);
 
 // Autopurge installed?
-$LAST = "%s"; $ONLINE = "";
+$lastOnline = "%s"; $ONLINE = "";
 if (EXT_IS_ACTIVE("autopurge")) {
        // Use last online timestamp to keep inactive members away from here
-       $LAST   = " AND last_online >= (UNIX_TIMESTAMP() - %s)";
+       $lastOnline   = " AND last_online >= (UNIX_TIMESTAMP() - %s)";
        $ONLINE = getConfig('ap_inactive_since');
 }
 
 // Let's check if there are some points left we can "pay"...
-$result = SQL_QUERY_ESC("SELECT userid, beg_points AS beg, last_online FROM "._MYSQL_PREFIX."_user_data
-WHERE beg_points > 0 AND status='CONFIRMED'".$LAST."
+$result = SQL_QUERY_ESC("SELECT userid, beg_points AS beg, last_online FROM `{!_MYSQL_PREFIX!}_user_data`
+WHERE beg_points > 0 AND `status`='CONFIRMED'".$lastOnline."
 ORDER BY beg_points DESC, last_online DESC, userid LIMIT %s",
        array($ONLINE, getConfig('beg_ranks')), __FILE__, __LINE__);
 
@@ -64,21 +64,21 @@ $OUT = "";
 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"),
+                       'uid'    => bigintval($content['userid']),
+                       'points' => TRANSLATE_COMMA($content['beg']),
+                       'last'   => MAKE_DATETIME($content['last_online'], "2"),
                );
 
                // Load row template
                $OUT .= LOAD_TEMPLATE("member_list_beg_row", true, $content);
 
                // Count one up and switch colors
-               $cnt++;$SW = 3 - $SW;
+               $cnt++; $SW = 3 - $SW;
        }
 } else {
        // No one is interested in our "active rallye" ! :-(