Code style changed, ext-user continued:

[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 3411c12dff23b800d53172b8564e1a601f59cdec..46afe0b1ef409f01b730eb8ba3ba827a42c4886c 100644 (file)
--- a/inc/modules/member/what-stats.php
+++ b/inc/modules/member/what-stats.php
@@ -16,8 +16,8 @@
  * $Author::                                                          $ *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2009 by Roland Haeder                           *
- * Copyright (c) 2009 - 2011 by Mailer Developer Team                   *
- * For more information visit: http://www.mxchange.org                  *
+ * Copyright (c) 2009 - 2012 by Mailer Developer Team                   *
+ * For more information visit: http://mxchange.org                      *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
  * it under the terms of the GNU General Public License as published by *
@@ -37,7 +37,7 @@
 
 // Some security stuff...
 if (!defined('__SECURITY')) {
-       die();
+       exit();
 } elseif (!isMember()) {
        redirectToIndexMemberOnlyModule();
 }
@@ -50,7 +50,15 @@ $main_content = array();
 
 // Load waiting/pending orders
 $result = SQL_QUERY_ESC("SELECT
-       `id`, `cat_id`, `payment_id`, `subject`, `url`, `timestamp`, `target_send`, `data_type`, `zip`
+       `id`,
+       `cat_id`,
+       `payment_id`,
+       `subject`,
+       `url`,
+       `timestamp`,
+       `target_send`,
+       `data_type`,
+       `zip`
 FROM
        `{?_MYSQL_PREFIX?}_pool`
 WHERE
@@ -66,31 +74,39 @@ if (!SQL_HASZERONUMS($result)) {
        $OUT = '';
        while ($content = SQL_FETCHARRAY($result)) {
                // Prepare content for output
-               $content['timestamp'] = generateDateTime($content['timestamp'], 2);
+               $content['timestamp'] = generateDateTime($content['timestamp'], '2');
 
                // Load template
-               $OUT .= loadTemplate('member_pool_row', true, $content);
+               $OUT .= loadTemplate('member_pool_row', TRUE, $content);
        } // END - if
 
        // Load main template
-       $main_content['pool'] = loadTemplate('member_pool_table', true, $OUT);
+       $main_content['pool'] = loadTemplate('member_pool_table', TRUE, $OUT);
 } else {
        // No mails in pool!
-       $main_content['pool'] = displayMessage('{--MEMBER_NO_MAILS_IN_POOL--}', true);
+       $main_content['pool'] = displayMessage('{--MEMBER_NO_MAILS_IN_POOL--}', TRUE);
 }
 
 // Free result
 SQL_FREERESULT($result);
 
 // Load sent orders
-$result = SQL_QUERY_ESC("SELECT
-       `id`, `cat_id`, `payment_id`, `subject`, `url`, `timestamp_ordered`, `max_rec`, `timestamp_send`, `clicks`
+$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",
+       `timestamp_ordered` DESC',
        array(getMemberId()), __FILE__, __LINE__);
 
 if (!SQL_HASZERONUMS($result)) {
@@ -98,26 +114,33 @@ if (!SQL_HASZERONUMS($result)) {
        $OUT = '';
        while ($content = SQL_FETCHARRAY($result)) {
                // Prepare data for the template
-               $content['timestamp_ordered'] = generateDateTime($content['timestamp_ordered'], 2);
-               $content['timestamp_sent']    = generateDateTime($content['timestamp_send'], 2);
-               $content['percents']          = ($content['clicks'] / $content['max_rec'] * 100);
+               $content['timestamp_ordered'] = generateDateTime($content['timestamp_ordered'], '2');
+               $content['timestamp_sent']    = generateDateTime($content['timestamp_send'], '2');
+
+               // Click rate
+               $content['click_rate'] = '0';
+
+               // Better protection against 'divison-by-zero'
+               if ($content['max_rec'] > 0) {
+                       $content['click_rate'] = ($content['clicks'] / $content['max_rec'] * 100);
+               } // END - if
 
                // Load row template and switch colors
-               $OUT .= loadTemplate('member_stats_row', true, $content);
+               $OUT .= loadTemplate('member_stats_row', TRUE, $content);
        } // END - while
 
        // Load main template
-       $main_content['stats'] = loadTemplate('member_stats_table', true, $OUT);
+       $main_content['stats'] = loadTemplate('member_stats_table', TRUE, $OUT);
 } else {
        // No mail orders fond
-       $main_content['stats'] = displayMessage('{--MEMBER_NO_MAILS_IN_STATS--}', true);
+       $main_content['stats'] = displayMessage('{--MEMBER_NO_MAILS_IN_STATS--}', TRUE);
 }
 
 // Free result
 SQL_FREERESULT($result);
 
 // Load main template
-loadTemplate('member_stats_pool', false, $main_content);
+loadTemplate('member_stats_pool', FALSE, $main_content);
 
 // [EOF]
 ?>