Rewrote some parts:
[mailer.git] / inc / modules / guest / what-stats.php
index ca20611192b8a1b766402a26b5d71c8bef11a9b9..24193fda39ceff6acfcf80e74d9c62009b5b9cad 100644 (file)
@@ -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 - 2013 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();
 } // END - if
 
 // Add description as navigation point
@@ -45,46 +45,47 @@ addYouAreHereLink('guest', __FILE__);
 
 // This part only works if ext-user is active
 if ((!isExtensionActive('user')) && (!isAdmin())) {
-       displayMessage(generateExtensionInactiveNotInstalledMessage('user'));
+       displayMessage('{%pipe,generateExtensionInactiveNotInstalledMessage=user%}');
        return;
 } // END - if
 
-// Determine which stats we want and set mode and title for the link below stats block
-if (!isGetRequestParameterSet('mode')) {
-       setGetRequestParameter('mode', strtolower(getConfig('guest_stats')));
-} // END - if
+// Default mode is from config
+$do = getConfig('guest_stats');
 
-// Set config temporarily
-setConfigEntry('guest_stats', strtoupper(getRequestParameter('mode')));
+// Determine which stats are wanted and set mode and title for the link below stats block
+if (isGetRequestElementSet('do')) {
+       // Use it from get value
+       $do = strtoupper(getRequestElement('do'));
+} // END - if
 
-switch (getRequestParameter('mode')) {
-       case 'members' :
-               $lmode = 'modules';
+switch ($do) {
+       case 'MEMBERS' :
+               $lmode = 'MODULES';
                break;
 
-       case 'modules' :
-               $lmode = 'members';
+       case 'MODULES' :
+               $lmode = 'MEMBERS';
                break;
 
-       case 'inactive':
-               $lmode = 'inactive';
+       case 'INACTIVE':
+               $lmode = 'INACTIVE';
                break;
 
        default:
                // Unsupported mode
-               debug_report_bug(__FILE__, __LINE__, sprintf("Unsupported mode <span class=\"data\">%s</span> detected.", secureString(getRequestParameter('mode'))));
+               reportBug(__FILE__, __LINE__, sprintf('Unsupported mode <span class="data">%s</span> detected.', sqlEscapeString($do)));
                break;
-}
+} // END - switch
 
 // Set link title
-$ltitle = '{--GUEST_STATS_' . strtoupper($lmode) . '--}';
+$ltitle = '{--GUEST_STATS_' . $lmode . '--}';
 
 // @TODO This can be rewritten in a dynamic include
-switch (getConfig('guest_stats')) {
+switch (strtoupper($do)) {
        case 'MEMBERS': // Statistics about your members
                // Only males / females
-               $male   = countSumTotalData('M', 'user_data', 'userid', 'gender', true, " AND `status`='CONFIRMED'");
-               $female = countSumTotalData('F', 'user_data', 'userid', 'gender', true, " AND `status`='CONFIRMED'");
+               $male   = countSumTotalData('M', 'user_data', 'userid', 'gender', TRUE, runFilterChain('user_exclusion_sql', " AND `status`='CONFIRMED'"));
+               $female = countSumTotalData('F', 'user_data', 'userid', 'gender', TRUE, runFilterChain('user_exclusion_sql', " AND `status`='CONFIRMED'"));
 
                // List every month
                $months = array();
@@ -93,39 +94,42 @@ switch (getConfig('guest_stats')) {
                        $month = $idx;
 
                        // Append leading zero
-                       if ($idx < 10) $month = '0' . $idx;
+                       if ($idx < 10) {
+                               $month = '0' . $idx;
+                       } // END - if
 
                        // Count months
-                       $months[$month] = countSumTotalData(bigintval($month), 'user_data', 'userid', 'birth_month', true, " AND `status`='CONFIRMED'");
+                       $months[$month] = countSumTotalData(bigintval($month), 'user_data', 'userid', 'birth_month', TRUE, runFilterChain('user_exclusion_sql', " AND `status`='CONFIRMED'"));
                } // END - for
 
                // Members in categories
-               $result = SQL_QUERY("SELECT `id`, `cat` FROM `{?_MYSQL_PREFIX?}_cats` WHERE `visible`='Y' ORDER BY `id` ASC", __FILE__, __LINE__);
+               $result = sqlQuery("SELECT `id`, `cat` FROM `{?_MYSQL_PREFIX?}_cats` WHERE `visible`='Y' ORDER BY `id` ASC", __FILE__, __LINE__);
 
                // Load categories first
                $cats = array(); $cat_cnt = array();
 
                // @TODO This can be somehow rewritten
-               while ($content = SQL_FETCHARRAY($result)) {
+               while ($content = sqlFetchArray($result)) {
                        // Simple...
                        $cats[$content['id']] = $content['cat'];
                } // END - while
 
-               // Now we have all categories loaded, count members
+               // Now all categories have been loaded, count members
                foreach ($cats as $id => $dummy) {
-                       // We only need id and nothing more to count...
-                       $cat_cnt[$id] = countSumTotalData(bigintval($id), 'user_cats', 'id', 'cat_id', true);
+                       // Only the id needs to be counted
+                       // @TODO This doesn't exclude tester accounts
+                       $cat_cnt[$id] = countSumTotalData(bigintval($id), 'user_cats', 'id', 'cat_id', TRUE);
                } // END - foreach
 
                // Prepare data for the template
                $content['total_users']   = ($male + $female);
-               $content['unconfirmed']   = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `status` != 'CONFIRMED'", __FILE__, __LINE__));
+               $content['unconfirmed']   = sqlNumRows(sqlQuery("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `status` != 'CONFIRMED'" . runFilterChain('user_exclusion_sql', ' '), __FILE__, __LINE__));
                $content['total_males']   = $male;
                $content['total_females'] = $female;
-               $content['tmem_count']    = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= {?START_TDAY?} AND `status`='CONFIRMED'", __FILE__, __LINE__));
-               $content['ymem_count']    = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= {?START_YDAY?} AND `last_online` < {?START_TDAY?} AND `status`='CONFIRMED'", __FILE__, __LINE__));
-               $content['treg_count']    = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= {?START_TDAY?}", __FILE__, __LINE__));
-               $content['yreg_count']    = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= {?START_YDAY?} AND `joined` < {?START_TDAY?}", __FILE__, __LINE__));
+               $content['tmem_count']    = sqlNumRows(sqlQuery("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= {?START_TDAY?}" . runFilterChain('user_exclusion_sql', " AND `status`='CONFIRMED'"), __FILE__, __LINE__));
+               $content['ymem_count']    = sqlNumRows(sqlQuery("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= {?START_YDAY?} AND `last_online` < {?START_TDAY?}" . runFilterChain('user_exclusion_sql', " AND `status`='CONFIRMED'"), __FILE__, __LINE__));
+               $content['treg_count']    = sqlNumRows(sqlQuery("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= {?START_TDAY?}" . runFilterChain('user_exclusion_sql', ' '), __FILE__, __LINE__));
+               $content['yreg_count']    = sqlNumRows(sqlQuery("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= {?START_YDAY?} AND `joined` < {?START_TDAY?}" . runFilterChain('user_exclusion_sql', ' '), __FILE__, __LINE__));
                $content['lmode']         = $lmode;
                $content['ltitle']        = $ltitle;
 
@@ -135,16 +139,16 @@ switch (getConfig('guest_stats')) {
                        if ($SW == 2) $OUT .= '<tr>';
 
                        // Prepare data for template
-                       $data = array(
+                       $row = array(
                                'l_class'  => $l,
                                'm_descr'  => $GLOBALS['month_descr'][$month],
                                'r_class'  => $r,
                                'r2_class' => $r2,
-                               'cnt'      => $count
+                               'count'    => $count
                        );
 
                        // Load row template
-                       $OUT .= loadTemplate('guest_stats_month_row', true, $data);
+                       $OUT .= loadTemplate('guest_stats_month_row', TRUE, $row);
 
                        if ($SW == 2) {
                                $r2 = '';
@@ -162,18 +166,18 @@ switch (getConfig('guest_stats')) {
                $OUT = '';
                foreach ($cat_cnt as $id => $count) {
                        // Prepare data for the template
-                       $data = array(
-                               'cat' => $cats[$id],
-                               'cnt' => $count,
+                       $row = array(
+                               'cat'   => $cats[$id],
+                               'count' => $count,
                        );
 
                        // Load row template and switch colors
-                       $OUT .= loadTemplate('guest_stats_cats_row', true, $data);
+                       $OUT .= loadTemplate('guest_stats_cats_row', TRUE, $row);
                } // END - foreach
                $content['cats_rows'] = $OUT;
 
                // Load final template
-               loadTemplate('guest_stats_member', false, $content);
+               loadTemplate('guest_stats_member', FALSE, $content);
                break;
 
        case 'MODULES': // TOP10 module clicks
@@ -182,29 +186,30 @@ switch (getConfig('guest_stats')) {
                if (isAdmin()) $AND = '';
 
                // Query for guest and member menus
-               $guest_t10 = SQL_QUERY('SELECT `counter`, `title` FROM `{?_MYSQL_PREFIX?}_guest_menu` WHERE `counter` > 0' . $AND . ' ORDER BY `counter` DESC LIMIT 0,10', __FILE__, __LINE__);
-               $mem_t10   = SQL_QUERY('SELECT `counter`, `title` FROM `{?_MYSQL_PREFIX?}_member_menu` WHERE `counter` > 0' . $AND . ' ORDER BY `counter` DESC LIMIT 0,10', __FILE__, __LINE__);
-               $OUT = ''; 
+               $guest_t10 = sqlQuery('SELECT `counter`, `title` FROM `{?_MYSQL_PREFIX?}_guest_menu` WHERE `counter` > 0' . $AND . ' ORDER BY `counter` DESC LIMIT 0,10', __FILE__, __LINE__);
+               $mem_t10   = sqlQuery('SELECT `counter`, `title` FROM `{?_MYSQL_PREFIX?}_member_menu` WHERE `counter` > 0' . $AND . ' ORDER BY `counter` DESC LIMIT 0,10', __FILE__, __LINE__);
+               $OUT = '';
 
-               if (!SQL_HASZERONUMS($guest_t10)) {
+               if (!ifSqlHasZeroNums($guest_t10)) {
                        // Guest clicks
-                       $OUT .= loadTemplate('guest_stats_header', true, '{--GUEST_TOP_GUEST_STATS--}');
-                       while ($content = SQL_FETCHARRAY($guest_t10)) {
+                       $OUT .= loadTemplate('guest_stats_header', TRUE, '{--GUEST_TOP_GUEST_STATS--}');
+                       while ($content = sqlFetchArray($guest_t10)) {
                                // Load row template
-                               $OUT .= loadTemplate('guest_stats_row', true, $content);
+                               $OUT .= loadTemplate('guest_stats_row', TRUE, $content);
                        } // END - while
                } // END - if
 
-               if (!SQL_HASZERONUMS($mem_t10)) {
+               if (!ifSqlHasZeroNums($mem_t10)) {
                        // Member clicks
-                       $OUT .= loadTemplate('guest_stats_header', true, '{--GUEST_TOP_MEMBER_STATS--}');
-                       while ($content = SQL_FETCHARRAY($mem_t10)) {
+                       $OUT .= loadTemplate('guest_stats_header', TRUE, '{--GUEST_TOP_MEMBER_STATS--}');
+                       while ($content = sqlFetchArray($mem_t10)) {
                                // Load row template
-                               $OUT .= loadTemplate('guest_stats_row', true, $content);
+                               $OUT .= loadTemplate('guest_stats_row', TRUE, $content);
                        } // END - while
                } // END - if
 
-               if ((!SQL_HASZERONUMS($guest_t10)) || (!SQL_HASZERONUMS($mem_t10))) {
+               // Is something generated?
+               if (!empty($OUT)) {
                        // Prepare content
                        $content = array(
                                'rows'   => $OUT,
@@ -213,7 +218,7 @@ switch (getConfig('guest_stats')) {
                        );
 
                        // Load final template
-                       loadTemplate('guest_stats_table', false, $content);
+                       loadTemplate('guest_stats_table', FALSE, $content);
                } else {
                        // No clicks detected
                        displayMessage('{--GUEST_STATS_NO_CLICKS--}');