Ref link fixed, nickname fixed, several rewrites, TODOs.txt updated:

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

diff --git a/inc/modules/member/what-stats.php b/inc/modules/member/what-stats.php
index 739b83a55b0f262415df34f32587e20933921bc6..110701133dc3abbf37be41670019474f0fec20c2 100644 (file)
--- a/inc/modules/member/what-stats.php
+++ b/inc/modules/member/what-stats.php
@@ -97,38 +97,41 @@ if (SQL_NUMROWS($result) > 0) {
 SQL_FREERESULT($result);
 
 // Load sent orders
-//                               0     1         2         3      4            5            6            7           8
-$result = SQL_QUERY_ESC("SELECT id, cat_id, payment_id, subject, url, timestamp_ordered, max_rec, timestamp_send, clicks
-FROM `{!_MYSQL_PREFIX!}_user_stats`
-WHERE userid=%s
-ORDER BY timestamp_ordered DESC",
-array(getUserId()), __FILE__, __LINE__);
+$result = SQL_QUERY_ESC("SELECT
+       `id`, `cat_id`, `payment_id`, `subject`, `url`, `timestamp_ordered`, `max_rec`, `timestamp_send`, `clicks`
+FROM
+       `{!_MYSQL_PREFIX!}_user_stats`
+WHERE
+       `userid`=%s
+ORDER BY
+       `timestamp_ordered` DESC",
+       array(getUserId()), __FILE__, __LINE__);
 
 if (SQL_NUMROWS($result) > 0) {
        // Mail orders are in pool so we can display them
        $OUT = ''; $SW = 2;
-       while ($data = SQL_FETCHROW($result)) {
+       while ($content = SQL_FETCHARRAY($result)) {
                // Prepare data for the template
                $content = array(
                        'sw'    => $SW,
-                       'cat'   => getCategory($data[1]),
-                       'pay'   => getPaymentTitlePrice($data[2]),
-                       'subj'  => COMPILE_CODE($data[3]),
-                       'url'   => DEREFERER($data[4]),
-                       'stamp' => generateDateTime($data[5], '2'),
-                       'recs'  => $data[6],
-                       'sent'  => generateDateTime($data[7], '2'),
-                       'clix'  => $data[8],
-                       'perc'  => COMPILE_CODE($data[8] / $data[6] * 100)."%",
+                       'cat'   => getCategory($content['cat_id']),
+                       'pay'   => getPaymentTitlePrice($content['payment_id']),
+                       'subj'  => COMPILE_CODE($content['subject']),
+                       'url'   => DEREFERER($content['url']),
+                       'stamp' => generateDateTime($content['timestamp_ordered'], '2'),
+                       'recs'  => $content['max_rec'],
+                       'sent'  => generateDateTime($content['timestamp_send'], '2'),
+                       'clix'  => $content['clicks'],
+                       'perc'  => COMPILE_CODE($content['clicks'] / $content['max_rec'] * 100)."%",
                );
 
                // Load row template and switch colors
-               $OUT .= LOAD_TEMPLATE("member_stats_row", true, $content);
+               $OUT .= LOAD_TEMPLATE('member_stats_row', true, $content);
                $SW = 3 - $SW;
        }
 
        // Load main template
-       $main_content['stats'] = LOAD_TEMPLATE("member_stats_table", true, $OUT);
+       $main_content['stats'] = LOAD_TEMPLATE('member_stats_table', true, $OUT);
 } else {
        // No mail orders fond
        $main_content['stats'] = LOAD_TEMPLATE('admin_settings_saved', true, getMessage('MEMBER_NO_MAILS_IN_STATS'));
@@ -138,7 +141,7 @@ if (SQL_NUMROWS($result) > 0) {
 SQL_FREERESULT($result);
 
 // Load main template
-LOAD_TEMPLATE("member_stats_pool", false, $main_content);
+LOAD_TEMPLATE('member_stats_pool', false, $main_content);
 
 //
 ?>