X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=inc%2Fmodules%2Fguest%2Fwhat-top10.php;h=b784135a098b63a4bbda012909148c1269319e5b;hb=c22d26c52883c3468645a3d63f61e6d4ab965c95;hp=b2c8104d52bf64bdd2938014561aaccf01330d1b;hpb=9b794f45ed937ac4476d972bb2f88fbe27aa8bc8;p=mailer.git

diff --git a/inc/modules/guest/what-top10.php b/inc/modules/guest/what-top10.php
index b2c8104d52..b784135a09 100644
--- a/inc/modules/guest/what-top10.php
+++ b/inc/modules/guest/what-top10.php
@@ -10,7 +10,12 @@
  * -------------------------------------------------------------------- *
  * Kurzbeschreibung  : TOP-Logins / Bestverdiener usw.                  *
  * -------------------------------------------------------------------- *
- *                                                                      *
+ * $Revision::                                                        $ *
+ * $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 - 2008 by Roland Haeder                           *
  * For more information visit: http://www.mxchange.org                  *
@@ -33,15 +38,15 @@
 
 // Some security stuff...
 if (!defined('__SECURITY')) {
-	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
+	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
 	require($INC);
-} elseif ((!EXT_IS_ACTIVE("top10")) && (!IS_ADMIN())) {
-	addFatalMessage(EXTENSION_PROBLEM_EXT_INACTIVE, "top10");
+} elseif ((!EXT_IS_ACTIVE('top10')) && (!IS_ADMIN())) {
+	addFatalMessage(__FILE__, __LINE__, generateExtensionInactiveNotInstalledMessage('top10'));
 	return;
 }
 
 // Add description as navigation point
-ADD_DESCR("guest", __FILE__);
+ADD_DESCR('guest', __FILE__);
 
 //// TOP logins
 $result = SQL_QUERY_ESC("
@@ -55,24 +60,26 @@ WHERE
 ORDER BY
 	total_logins DESC
 LIMIT %s",
-	array(getConfig('top10_max')), __FILE__, __LINE__);
+array(getConfig('top10_max')), __FILE__, __LINE__);
 
-$OUT = ""; $SW = 2; $cnt = 1;
-while (list($uid, $logins, $last) = SQL_FETCHROW($result)) {
-	$nick2 = "---";
+$OUT = ''; $SW = 2; $cnt = 1;
+while ($content = SQL_FETCHARRAY($result)) {
+	// Init nickname
+	$content['nickname'] = '---';
 
 	// Get nickname
-	if (EXT_IS_ACTIVE("nickname")) $nick2 = NICKNAME_GET_NICK($uid);
+	if (EXT_IS_ACTIVE('nickname')) $content['nickname'] = NICKNAME_GET_NICK($content['userid']);
 
 	// Prepare data for template
+	// @TODO Rewritings: uid->userid,nick->nickname,logins->total_logins in template
 	$content = array(
 		'sw'     => $SW,
 		'cnt'    => $cnt,
-		'uid'    => $uid,
-		'nick'   => $nick2,
-		'logins' => $logins,
-		'points' => TRANSLATE_COMMA(GET_TOTAL_DATA($uid, "user_points", "points")),
-		'last'   => MAKE_DATETIME($last, "3"),
+		'uid'    => $content['userid'],
+		'nick'   => $content['nickname'],
+		'logins' => $content['total_logins'],
+		'points' => translateComma(GET_TOTAL_DATA($content['userid'], "user_points", "points")),
+		'last'   => generateDateTime($content['last_online'], '3'),
 	);
 
 	// Load row template
@@ -80,7 +87,7 @@ while (list($uid, $logins, $last) = SQL_FETCHROW($result)) {
 
 	// Switch colors and count one up
 	$SW = 3 - $SW; $cnt++;
-}
+} // END - while
 
 if ($cnt < getConfig('top10_max')) {
 	// Add more "blank" rows
@@ -94,8 +101,10 @@ if ($cnt < getConfig('top10_max')) {
 		// Load row template
 		$OUT .= LOAD_TEMPLATE("guest_top10_empty5", true, $content);
 		$SW = 3 - $SW;
-	}
-}
+	} // END - for
+} // END - if
+
+// @TODO Rewrite this constant
 define('__TOP_LOGINS_ROWS', $OUT);
 
 // Free result
@@ -110,7 +119,7 @@ SELECT DISTINCT
 FROM
 	`{!_MYSQL_PREFIX!}_user_data` AS d
 LEFT JOIN
-	{!MYSQL_PREFIX!}_user_points AS p
+	{!_MYSQL_PREFIX!}_user_points AS p
 ON
 	p.userid=d.userid
 WHERE
@@ -122,23 +131,25 @@ ORDER BY
 	tpoints DESC,
 	d.last_online DESC
 LIMIT %s",
-	array(getConfig('top10_max')), __FILE__, __LINE__);
+array(getConfig('top10_max')), __FILE__, __LINE__);
 
-$OUT = ""; $SW = 2; $cnt = 1;
-while (list($uid, $points, $last) = SQL_FETCHROW($result)) {
-	$nick2 = "---";
+$OUT = ''; $SW = 2; $cnt = 1;
+while ($content = SQL_FETCHARRAY($result)) {
+	// Init nickname
+	$content['nickname'] = '---';
 
 	// Get nickname
-	if (EXT_IS_ACTIVE("nickname")) $nick2 = NICKNAME_GET_NICK($uid);
+	if (EXT_IS_ACTIVE('nickname')) $content['nickname'] = NICKNAME_GET_NICK($content['userid']);
 
 	// Prepare data for template
+	// @TODO Rewritings: nick->nickname,uid->userid in template
 	$content = array(
 		'sw'     => $SW,
 		'cnt'    => $cnt,
-		'uid'    => bigintval($uid),
-		'nick'   => $nick2,
-		'points' => TRANSLATE_COMMA($points),
-		'last'   => MAKE_DATETIME($last, "3")
+		'uid'    => $content['userid'],
+		'nick'   => $content['nickname'],
+		'points' => translateComma($content['points']),
+		'last'   => generateDateTime($content['last_online'], '3')
 	);
 
 	// Load row template
@@ -146,7 +157,7 @@ while (list($uid, $points, $last) = SQL_FETCHROW($result)) {
 
 	// Switch colors and count one up
 	$SW = 3 - $SW; $cnt++;
-}
+} // END - while
 
 if ($cnt < getConfig('top10_max')) {
 	// Add more "blank" rows
@@ -160,8 +171,10 @@ if ($cnt < getConfig('top10_max')) {
 		// Load row template
 		$OUT .= LOAD_TEMPLATE("guest_top10_empty4", true, $content);
 		$SW = 3 - $SW;
-	}
-}
+	} // END - for
+} // END - if
+
+// @TODO Rewrite this constant
 define('__TOP_POINTS_ROWS', $OUT);
 
 // Free result
@@ -176,7 +189,7 @@ SELECT DISTINCT
 FROM
 	`{!_MYSQL_PREFIX!}_user_data` AS d
 LEFT JOIN
-	{!MYSQL_PREFIX!}_refsystem AS r
+	{!_MYSQL_PREFIX!}_refsystem AS r
 ON
 	r.userid=d.userid
 WHERE
@@ -188,24 +201,26 @@ ORDER BY
 	refs DESC,
 	d.last_online DESC
 LIMIT %s",
-	array(getConfig('top10_max')), __FILE__, __LINE__);
+array(getConfig('top10_max')), __FILE__, __LINE__);
 
-$OUT = ""; $SW = 2; $cnt = 1;
-while (list($uid, $refs, $last) = SQL_FETCHROW($result)) {
-	$nick2 = "---";
+$OUT = ''; $SW = 2; $cnt = 1;
+while ($content = SQL_FETCHARRAY($result)) {
+	// Init nickname
+	$content['nickname'] = '---';
 
 	// Get nickname
-	if (EXT_IS_ACTIVE("nickname")) $nick2 = NICKNAME_GET_NICK($uid);
+	if (EXT_IS_ACTIVE('nickname')) $content['nickname'] = NICKNAME_GET_NICK($content['userid']);
 
 	// Prepare data for template
+	// @TODO Rewritings: nick->nickname,uid->userid in template
 	$content = array(
 		'sw'     => $SW,
 		'cnt'    => $cnt,
-		'uid'    => bigintval($uid),
-		'refs'   => $refs,
-		'nick'   => $nick2,
-		'points' => TRANSLATE_COMMA(GET_TOTAL_DATA($uid, "user_points", "points")),
-		'last'   => MAKE_DATETIME($last, "3")
+		'uid'    => $content['userid'],
+		'refs'   => $content['refs'],
+		'nick'   => $content['nickname'],
+		'points' => translateComma(GET_TOTAL_DATA($content['userid'], "user_points", "points")),
+		'last'   => generateDateTime($content['last_online'], '3')
 	);
 
 	// Load row template
@@ -213,7 +228,7 @@ while (list($uid, $refs, $last) = SQL_FETCHROW($result)) {
 
 	// Switch colors and count one up
 	$SW = 3 - $SW; $cnt++;
-}
+} // END - while
 
 if ($cnt < getConfig('top10_max')) {
 	// Add more "blank" rows
@@ -227,14 +242,17 @@ if ($cnt < getConfig('top10_max')) {
 		// Load row template
 		$OUT .= LOAD_TEMPLATE("guest_top10_empty5", true, $content);
 		$SW = 3 - $SW;
-	}
-}
+	} // END - for
+} // END - if
+
+// @TODO Rewrite this constant
 define('__TOP_REFERRAL_ROWS', $OUT);
 
 // Free result
 SQL_FREERESULT($result);
 
 // Remember other values in constants
+// @TODO Rewrite this constant
 define('__TOP10_MAX', getConfig('top10_max'));
 
 // Load final template