Some rewrites

[mailer.git] / inc / modules / admin / what-list_beg.php

diff --git a/inc/modules/admin/what-list_beg.php b/inc/modules/admin/what-list_beg.php
index f665ea2209063ff8f8077d26f2d0b82c92f1118b..b3d19ef974405dd29b3e2aa5198f859c9e5a89a5 100644 (file)
--- a/inc/modules/admin/what-list_beg.php
+++ b/inc/modules/admin/what-list_beg.php
@@ -49,7 +49,7 @@ if (getConfig('beg_rallye') == 'Y') {
        if (isPostRequestElementSet(('withdraw'))) {
                // Okay, let's prepare...
                $curr = date('m', time()) - 1;
-               if (strlen($curr) == 1) $curr = 0 . $curr;
+               if (strlen($curr) == 1) $curr = '0' . $curr;
                updateConfiguration('last_month', $curr);
                loadTemplate('admin_settings_saved', false, getMessage('ADMIN_BEG_WITHDRAW_PREPARED'));
        } // END - if
@@ -71,31 +71,31 @@ ORDER BY beg_points DESC, last_online DESC, userid",
 
        if (SQL_NUMROWS($result) > 0) {
                // List users
-               $OUT = '';$SW = 2; $cnt = 1; $total = 0;
+               $OUT = '';$SW = 2; $cnt = 1; $total = '0';
                while ($content = SQL_FETCHARRAY($result)) {
                        // Init variables
                        $WIN1 = ''; $WIN2 = '';
+
+                       // Maybe he can win his active beg?
                        if ($cnt <= getConfig('beg_ranks')) {
-                               // Maybe he can win his active beg?
-                               $WIN1 = "<strong>";
-                               $WIN2 = "</strong>";
+                               // Mark him
+                               $WIN1 = '<strong>';
+                               $WIN2 = '</strong>';
                        } // END - if
 
                        // Prepare content
-                       // @TODO Rewrite templates so s/family will be replaced by surname/family
-                       // @TODO to shorten this block. Please also do so with userid->userid
                        $content = array(
-                               'userid'    => $content['userid'],
-                               'email'  => generateEmailLink($content['email'], 'user_data'),
-                               'gender' => translateGender($content['gender']),
-                               'surname'  => $content['surname'],
-                               'family'  => $content['family'],
-                               'turbo'  => translateComma($content['beg_points']),
-                               'last_online'   => generateDateTime($content['last_online'], 2),
-                               'sw'     => $SW,
-                               'win1'   => $WIN1,
-                               'win2'   => $WIN2,
-                               'cnt'    => $cnt,
+                               'userid'      => $content['userid'],
+                               'email'       => generateEmailLink($content['email'], 'user_data'),
+                               'gender'      => translateGender($content['gender']),
+                               'surname'     => $content['surname'],
+                               'family'      => $content['family'],
+                               'turbo'       => translateComma($content['beg_points']),
+                               'last_online' => generateDateTime($content['last_online'], 2),
+                               'sw'          => $SW,
+                               'win1'        => $WIN1,
+                               'win2'        => $WIN2,
+                               'cnt'         => $cnt,
                        );
 
                        // Load template and add it