X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-bonus.php;h=e99e0c64f0047ef81d5ddb5942358efeb647c77a;hp=1c20685a78f851da8d566469045efb3d6351b3f1;hb=3f51c40f4fede87228216f9285b745a339e3891d;hpb=ee715f8d4c903b61f943df392b93acb6ad9c2602

diff --git a/inc/modules/member/what-bonus.php b/inc/modules/member/what-bonus.php
index 1c20685a78..e99e0c64f0 100644
--- a/inc/modules/member/what-bonus.php
+++ b/inc/modules/member/what-bonus.php
@@ -10,7 +10,12 @@
  * -------------------------------------------------------------------- *
  * Kurzbeschreibung  : Bonusseiten fuer ein paar Extrapunkte            *
  * -------------------------------------------------------------------- *
- *                                                                      *
+ * $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,73 +38,82 @@
 
 // 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 (!IS_MEMBER()) {
-	LOAD_URL("modules.php[5~?module=index");
-} elseif (!EXT_IS_ACTIVE("bonus"))
-	ADD_FATAL(EXTENSION_PROBLEM_EXT_INACTIVE, "bonus");
+	redirectToUrl('modules.php?module=index');
+} elseif ((!EXT_IS_ACTIVE('bonus')) && (!IS_ADMIN())) {
+	addFatalMessage(__FILE__, __LINE__, generateExtensionInactiveNotInstalledMessage('bonus'));
 	return;
 }
 
 // Add description as navigation point
-ADD_DESCR("member", __FILE__);
+ADD_DESCR('member', __FILE__);
 
-if (GET_EXT_VERSION("bonus") >= "0.6.9")
-{
+if (GET_EXT_VERSION('bonus') >= '0.6.9') {
 	// Add more bonus points here
+	// @TODO Rewrite this to a filter
 	$USE = "(0";
-	if ($_CONFIG['bonus_click_yn'] == "Y") $USE .= " + turbo_bonus";
-	if ($_CONFIG['bonus_login_yn'] == "Y") $USE .= " + login_bonus";
-	if ($_CONFIG['bonus_order_yn'] == "Y") $USE .= " + bonus_order";
-	if ($_CONFIG['bonus_stats_yn'] == "Y") $USE .= " + bonus_stats";
-	if ($_CONFIG['bonus_ref_yn']   == "Y") $USE .= " + bonus_ref";
+	if (getConfig('bonus_click_yn') == 'Y') $USE .= " + turbo_bonus";
+	if (getConfig('bonus_login_yn') == 'Y') $USE .= " + login_bonus";
+	if (getConfig('bonus_order_yn') == 'Y') $USE .= " + bonus_order";
+	if (getConfig('bonus_stats_yn') == 'Y') $USE .= " + bonus_stats";
+	if (getConfig('bonus_ref_yn')   == 'Y') $USE .= " + bonus_ref";
 	$USE .= ")";
-}
- else
-{
+} else {
 	// Old version ???
 	$USE = "turbo_bonus";
 }
 
 // Autopurge installed?
-$LAST = "%s"; $ONLINE = "";
-if (EXT_IS_ACTIVE("autopurge")) {
+$lastOnline = "%s"; $ONLINE = '';
+if (EXT_IS_ACTIVE('autopurge')) {
 	// Use last online timestamp to keep inactive members away from here
-	$LAST   = " AND last_online >= (UNIX_TIMESTAMP() - %s)";
-	$ONLINE = $_CONFIG['ap_inactive_since'];
+	$lastOnline   = " AND `last_online` >= (UNIX_TIMESTAMP() - %s)";
+	$ONLINE = getConfig('ap_inactive_since');
 }
 
-// Let's check if there are some points left we can "pay"...
-$result = SQL_QUERY_ESC("SELECT userid, ".$USE." AS points, last_online FROM "._MYSQL_PREFIX."_user_data
-WHERE ".$USE." > 0 AND status='CONFIRMED'".$LAST."
-ORDER BY points DESC, last_online DESC, userid
+// Let's check if there are some points left we can 'pay'...
+$result = SQL_QUERY_ESC("SELECT
+	`userid`, `".$USE."` AS points, `last_online`
+FROM
+	`{!_MYSQL_PREFIX!}_user_data`
+WHERE
+	`".$USE."` > 0 AND `status`='CONFIRMED'".$lastOnline."
+ORDER BY
+	`points` DESC,
+	last_online DESC,
+	userid ASC
 LIMIT %s",
- array($ONLINE, $_CONFIG['bonus_ranks']), __FILE__, __LINE__);
+	array(
+		$ONLINE,
+		getConfig('bonus_ranks')
+	), __FILE__, __LINE__);
 
 // Reset temporary variable and check for users
-$OUT = "";
+$OUT = '';
 if (SQL_NUMROWS($result) > 0) {
 	// Load our winners...
 	$SW = 2; $cnt = 1;
 	while ($content = SQL_FETCHARRAY($result)) {
 		// Prepare data for the template
 		$content = array(
-		$content['sw']     = $SW;
-		$content['cnt']    = $cnt;
-		$content['uid']    = bigintval($content['uid']);
-		$content['points'] = TRANSLATE_COMMA($content['points']);
-		$content['last']   = MAKE_DATETIME($content['last'], "2");
+			'sw'     => $SW,
+			'cnt'    => $cnt,
+			'uid'    => bigintval($content['uid']),
+			'points' => translateComma($content['points']),
+			'last'   => generateDateTime($content['last'], '2')
+		);
 
 		// Load row template
-		$OUT .= LOAD_TEMPLATE("member_bonus_row", true, $content);
+		$OUT .= LOAD_TEMPLATE('member_bonus_row', true, $content);
 
 		// Count one up and switch colors
-		$cnt++;$SW = 3 - $SW;
+		$cnt++; $SW = 3 - $SW;
 	}
 } else {
 	// No one is interested in our "active rallye" ! :-(
-	$OUT = LOAD_TEMPLATE("member_bonus_404", true);
+	$OUT = LOAD_TEMPLATE('member_bonus_404', true);
 }
 
 // Free memory
@@ -109,7 +123,7 @@ SQL_FREERESULT($result);
 define('__BONUS_ROWS', $OUT);
 
 // Load final template
-LOAD_TEMPLATE("member_bonus");
+LOAD_TEMPLATE('member_bonus');
 
 //
 ?>