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

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

diff --git a/inc/modules/admin/what-list_transfer.php b/inc/modules/admin/what-list_transfer.php
index c2a1d56268ed9af912082ecbccfc7828e2dbf698..960970328922f119a0c98a09e98acd3a08278382 100644 (file)
--- a/inc/modules/admin/what-list_transfer.php
+++ b/inc/modules/admin/what-list_transfer.php
@@ -47,22 +47,21 @@ if (SQL_NUMROWS($result) > 0)
 {
        // Output rows
        $OUT = ""; $SW = 2;
-       while (list($uid1, $uid2, $idx, $points, $reason, $stamp) = SQL_FETCHROW($result))
-       {
+       while ($content = SQL_FETCHARRAY($result)) {
                // Prepare data for the row template
                $content = array(
                        'sw'        => $SW,
-                       'idx'       => $idx,
-                       'stamp'     => MAKE_DATETIME($stamp, "3"),
-                       'from_link' => ADMIN_USER_PROFILE_LINK($uid1),
-                       'to_link'   => ADMIN_USER_PROFILE_LINK($uid2),
-                       'reason'    => wordwrap($reason, 15),
-                       'points'    => TRANSLATE_COMMA($points),
+                       'idx'       => $content['trans_id'],
+                       'stamp'     => MAKE_DATETIME($content['time_trans'], "3"),
+                       'from_link' => ADMIN_USER_PROFILE_LINK($content['userid']),
+                       'to_link'   => ADMIN_USER_PROFILE_LINK($content['to_uid']),
+                       'reason'    => wordwrap($content['reason'], 15),
+                       'points'    => TRANSLATE_COMMA($content['points']),
                );
 
                // Load row template add points and switch color
                $OUT .= LOAD_TEMPLATE("admin_list_transfer_row", true, $content);
-               $total += $points; $SW = 3 - $SW;
+               $total += $content['points']; $SW = 3 - $SW;
        }
 
        // Free memory
@@ -73,22 +72,23 @@ if (SQL_NUMROWS($result) > 0)
        // Nothing for in and out
        $OUT = "<tr>
   <td colspan=\"6\" align=\"center\" class=\"bottom2\" height=\"70\">
-    ".LOAD_TEMPLATE("admin_settings_saved", true, TRANSFER_NO_INOUT_TRANSFERS)."
+    ".LOAD_TEMPLATE("admin_settings_saved", true, getMessage('TRANSFER_NO_INOUT_TRANSFERS'))."
   </td>
 </tr>";
 }
 
 // ... and add them to a constant for the template
+// @TODO Rewrite these constants
 define('__TRANSFER_ROWS', $OUT);
 
 // Remeber total amount
 define('__TRANSFER_TOTAL_VALUE', TRANSLATE_COMMA($total));
 
 // Set title
-define('__TRANSFER_TITLE', TRANSFER_LIST_ALL);
+define('__TRANSFER_TITLE', getMessage('TRANSFER_LIST_ALL'));
 
 // Set "balance" word
-define('__TRANSFER_SUM', TRANSFER_TOTAL_BALANCE);
+define('__TRANSFER_SUM', getMessage('TRANSFER_TOTAL_BALANCE'));
 
 // Don't show a delete button
 define('__TRANSFER_DELETE', "&nbsp;");