X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fguest%2Fwhat-stats.php;h=24193fda39ceff6acfcf80e74d9c62009b5b9cad;hp=9e12b77d1345d5706534fc267400a9716a582198;hb=e5527fd38a6585c8466dc28d013f12d21eb7c07a;hpb=76b1b077bda73310c536f658d3a9bb5e12232f39

diff --git a/inc/modules/guest/what-stats.php b/inc/modules/guest/what-stats.php
index 9e12b77d13..24193fda39 100644
--- a/inc/modules/guest/what-stats.php
+++ b/inc/modules/guest/what-stats.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
  * Mailer v0.2.1-FINAL                                Start: 12/14/2003 *
- * ================                             Last change: 08/22/2004 *
+ * ===================                          Last change: 08/22/2004 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : what-stats.php                                   *
@@ -14,11 +14,10 @@
  * $Date::                                                            $ *
  * $Tag:: 0.2.1-FINAL                                                 $ *
  * $Author::                                                          $ *
- * Needs to be in all Files and every File needs "svn propset           *
- * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2009 by Roland Haeder                           *
- * 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 *
@@ -38,60 +37,55 @@
 
 // Some security stuff...
 if (!defined('__SECURITY')) {
-	die();
+	exit();
 } // END - if
 
 // Add description as navigation point
-addMenuDescription('guest', __FILE__);
+addYouAreHereLink('guest', __FILE__);
 
-// Derterminate which stats we want and set mode and title for the link below stats block
-if (!isGetRequestElementSet('mode')) setRequestGetElement('mode', strtolower(getConfig('guest_stats')));
+// This part only works if ext-user is active
+if ((!isExtensionActive('user')) && (!isAdmin())) {
+	displayMessage('{%pipe,generateExtensionInactiveNotInstalledMessage=user%}');
+	return;
+} // END - if
+
+// Default mode is from config
+$do = getConfig('guest_stats');
 
-// Set config temporarily
-setConfigEntry('guest_stats', strtoupper(getRequestElement('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 (getRequestElement('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(sprintf("Unsupported mode <strong>%s</strong> detected.", secureString(getRequestElement('mode'))));
+		reportBug(__FILE__, __LINE__, sprintf('Unsupported mode <span class="data">%s</span> detected.', sqlEscapeString($do)));
 		break;
-}
+} // END - switch
 
 // Set link title
-$ltitle = getMessage('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
-		// Members yesterday / today online
-		$ymem = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= ".getConfig('START_YDAY')." AND `last_online` < ".getConfig('START_TDAY')." AND `status`='CONFIRMED'", __FILE__, __LINE__));
-		$tmem = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `last_online` >= ".getConfig('START_TDAY')." AND `status`='CONFIRMED'", __FILE__, __LINE__));
-
-		// Yesterday / today registered
-		$yreg = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= ".getConfig('START_YDAY')." AND `joined` < ".getConfig('START_TDAY'), __FILE__, __LINE__));
-		$treg = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `joined` >= ".getConfig('START_TDAY'), __FILE__, __LINE__));
-
 		// 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'");
-
-		// Unconfirmed accounts
-		$unconfirmed = SQL_NUMROWS(SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `status` != 'CONFIRMED'", __FILE__, __LINE__));
-
-		// Total members
-		$total = $male + $female;
+		$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();
@@ -100,89 +94,90 @@ 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']   = $total;
-		$content['unconfirmed']   = $unconfirmed;
+		$content['total_users']   = ($male + $female);
+		$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']    = $tmem;
-		$content['ymem_count']    = $ymem;
-		$content['treg_count']    = $treg;
-		$content['yreg_count']    = $yreg;
+		$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;
 
 		// Generate monthly stats
 		$SW = 2; $r2 = ' right'; $l = 'll'; $r = 'lr'; $OUT = '';
-		foreach ($months as $month => $cnt) {
-			if ($SW == 2) $OUT .= "<tr>\n";
+		foreach ($months as $month => $count) {
+			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'      => $cnt
+				'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 = '';
 				$l = 'rl'; $r = 'rr';
 			} else {
-				$OUT .= "</tr>\n";
+				$OUT .= '</tr>';
 				$r2 = ' right';
 				$l = 'll'; $r = 'lr';
 			}
 			$SW = 3 - $SW;
-		}
+		} // END - foreach
 		$content['month_rows'] = $OUT;
 
 		// Generate category stats
-		$OUT = ''; $SW = 2;
-		foreach ($cat_cnt as $id => $cnt) {
+		$OUT = '';
+		foreach ($cat_cnt as $id => $count) {
 			// Prepare data for the template
-			$data = array(
-				'sw'  => $SW,
-				'cat' => $cats[$id],
-				'cnt' => $cnt,
+			$row = array(
+				'cat'   => $cats[$id],
+				'count' => $count,
 			);
 
 			// Load row template and switch colors
-			$OUT .= loadTemplate('guest_stats_cats_row', true, $data);
-			$SW = 3 - $SW;
-		}
+			$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
@@ -191,37 +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_NUMROWS($guest_t10) > 0) {
+		if (!ifSqlHasZeroNums($guest_t10)) {
 			// Guest clicks
-			$OUT .= loadTemplate('guest_stats_header', true, getMessage('GUEST_TOP_GUEST_STATS'));
-			$SW = 2;
-			while ($content = SQL_FETCHARRAY($guest_t10)) {
-				$content['sw'] = $SW;
-
+			$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);
-				$SW = 3 - $SW;
+				$OUT .= loadTemplate('guest_stats_row', TRUE, $content);
 			} // END - while
 		} // END - if
 
-		if (SQL_NUMROWS($mem_t10) > 0) {
+		if (!ifSqlHasZeroNums($mem_t10)) {
 			// Member clicks
-			$OUT .= loadTemplate('guest_stats_header', true, getMessage('GUEST_TOP_MEMBER_STATS'));
-			$SW = 2;
-			while ($content = SQL_FETCHARRAY($mem_t10)) {
-				$content['sw'] = $SW;
-
+			$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);
-				$SW = 3 - $SW;
+				$OUT .= loadTemplate('guest_stats_row', TRUE, $content);
 			} // END - while
 		} // END - if
 
-		if ((SQL_NUMROWS($guest_t10) > 0) || (SQL_NUMROWS($mem_t10) > 0)) {
+		// Is something generated?
+		if (!empty($OUT)) {
 			// Prepare content
 			$content = array(
 				'rows'   => $OUT,
@@ -230,17 +218,17 @@ switch (getConfig('guest_stats')) {
 			);
 
 			// Load final template
-			loadTemplate('guest_stats_table', false, $content);
+			loadTemplate('guest_stats_table', FALSE, $content);
 		} else {
 			// No clicks detected
-			loadTemplate('admin_settings_saved', false, getMessage('GUEST_STATS_NO_CLICKS'));
+			displayMessage('{--GUEST_STATS_NO_CLICKS--}');
 		}
 		break;
 
 	case 'INACTIVE': // Deactivated stats
-		loadTemplate('admin_settings_saved', false, getMessage('GUEST_STATS_DEACTIVATED'));
+		displayMessage('{--GUEST_STATS_DEACTIVATED--}');
 		break;
-}
+} // END - switch
 
 // [EOF]
 ?>