X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Flibs%2Ftask_functions.php;h=f0e1ed2cd71bb0ba0924ce67874f49b8c88629f2;hp=6497d3c91706ef7c64c83bc8719754d8a4d5d1b5;hb=56156f6c4392510cdbe0eb4f2ccefc23b43e2672;hpb=963e55ca1ea79e255f235e359cde9f7862191dc5

diff --git a/inc/libs/task_functions.php b/inc/libs/task_functions.php
index 6497d3c917..f0e1ed2cd7 100644
--- a/inc/libs/task_functions.php
+++ b/inc/libs/task_functions.php
@@ -50,21 +50,22 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	global $_CONFIG;
 
 	// Init variables/arrays
-	$EXTRAS = ""; $OUT = ""; $SQLs = array(); $WHATs = array(); $DESCRs = array(); $TITLEs = array();
+	$EXTRAS = "";
+	$OUT = "";
+	$SQLs = array();
+	$WHATs = array();
+	$DESCRs = array();
+	$TITLEs = array();
 
 	// Chheck for new extensions and updates
 	$JOBS_DONE = OUTPUT_STANDARD_OVERVIEW($result_main);
 
 	if (!$JOBS_DONE) {
 		// New extensions or updates found
-		$result = SQL_QUERY_ESC("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE assigned_admin='%s' AND status='NEW' AND task_type='EXTENSION_UPDATE'",
-		 array(GET_ADMIN_ID(get_session('admin_login'))), __FILE__, __LINE__);
-
-		$value = SQL_NUMROWS($result);
-		SQL_FREERESULT($result);
+		$value = GET_TOTAL_DATA(GET_CURRENT_ADMIN_ID(), "task_system", "id", "assigned_admin", true, " AND status='NEW' AND task_type='EXTENSION_UPDATE'");
 
 		if ($value > 0) {
-			define('__TASK_UPDATE_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=updates\">".$value."</A>");
+			define('__TASK_UPDATE_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=updates\">".$value."</a>");
 		} else {
 			define('__TASK_UPDATE_VALUE', "0");
 		}
@@ -76,34 +77,28 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	// First check for all account status seperately
 	//
 	// Confirmed accounts
-	$result = SQL_QUERY("SELECT userid FROM "._MYSQL_PREFIX."_user_data WHERE status='CONFIRMED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("CONFIRMED", "user_data", "userid", "status", true);
 
 	if ($value > 0) {
-		define('__CONFIRMED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_user&amp;mode=confirmed\">".$value."</A>");
+		define('__CONFIRMED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_user&amp;status=confirmed\">".$value."</a>");
 	} else {
 		define('__CONFIRMED_VALUE', "0");
 	}
 
 	// Unconfirmed accounts
-	$result = SQL_QUERY("SELECT userid FROM "._MYSQL_PREFIX."_user_data WHERE status='UNCONFIRMED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("UNCONFIRMED", "user_data", "userid", "status", true);
 
 	if ($value > 0) {
-		define('__UNCONFIRMED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=chk_regs\">".$value."</A>");
+		define('__UNCONFIRMED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=chk_regs\">".$value."</a>");
 	} else {
 		define('__UNCONFIRMED_VALUE', "0");
 	}
 
 	// And locked accounts
-	$result = SQL_QUERY("SELECT userid FROM "._MYSQL_PREFIX."_user_data WHERE status='LOCKED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("LOCKED", "user_data", "userid", "status", true);
 
 	if ($value > 0) {
-		define('__LOCKED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_user&amp;mode=locked\">".$value."</A>");
+		define('__LOCKED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_user&amp;status=locked\">".$value."</a>");
 	} else {
 		define('__LOCKED_VALUE', "0");
 	}
@@ -111,12 +106,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Unassigned tasks
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE assigned_admin='0' AND status != 'DELETED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("0", "task_system", "id", "assigned_admin", true, " AND `status` != 'DELETED'");
 
 	if ($value > 0) {
-		define('__TASK_NEW_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=unassigned\">".$value."</A>");
+		define('__TASK_NEW_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=unassigned\">".$value."</a>");
 	} else {
 		define('__TASK_NEW_VALUE', "0");
 	}
@@ -124,12 +117,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Closed tasks
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE status = 'CLOSED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("CLOSED", "task_system", "id", "status", true);
 
 	if ($value > 0) {
-		define('__TASK_CLOSED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=closed\">".$value."</A>");
+		define('__TASK_CLOSED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=closed\">".$value."</a>");
 	} else {
 		define('__TASK_CLOSED_VALUE', "0");
 	}
@@ -137,12 +128,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Deleted tasks
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE status = 'DELETED'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("DELETED", "task_system", "id", "status", true);
 
 	if ($value > 0) {
-		define('__TASK_DELETED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=deleted\">".$value."</A>");
+		define('__TASK_DELETED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=deleted\">".$value."</a>");
 	} else {
 		define('__TASK_DELETED_VALUE', "0");
 	}
@@ -150,13 +139,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Solved tasks
 	//
-	$result = SQL_QUERY_ESC("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE status = 'SOLVED' AND assigned_admin='%s'",
-	 array(GET_ADMIN_ID(get_session('admin_login'))), __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("SOLVED", "task_system", "id", "status", true, sprintf(" AND assigned_admin=%s", GET_CURRENT_ADMIN_ID()));
 
 	if ($value > 0) {
-		define('__TASK_SOLVED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=solved\">".$value."</A>");
+		define('__TASK_SOLVED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task&amp;type=solved\">".$value."</a>");
 	} else {
 		define('__TASK_SOLVED_VALUE', "0");
 	}
@@ -164,13 +150,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Your tasks
 	//
-	$result = SQL_QUERY_ESC("SELECT id FROM "._MYSQL_PREFIX."_task_system WHERE assigned_admin='%s' AND status = 'NEW' AND task_type != 'EXTENSION_UPDATE'",
-	 array(GET_ADMIN_ID(get_session('admin_login'))), __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA(GET_CURRENT_ADMIN_ID(), "task_system", "id", "assigned_admin", true, " AND status = 'NEW' AND task_type != 'EXTENSION_UPDATE'");
 
 	if ($value > 0) {
-		define('__TASK_YOUR_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=list_task\">".$value."</A>");
+		define('__TASK_YOUR_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=list_task\">".$value."</a>");
 	} else {
 		define('__TASK_YOUR_VALUE', "0");
 	}
@@ -178,12 +161,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Mails waiting to be approved
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_pool WHERE data_type='ADMIN'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("ADMIN", "pool", "id", "data_type", true);
 
 	if ($value > 0) {
-		define('__MAIL_ADMIN_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=unlock_emails\">".$value."</A>");
+		define('__MAIL_ADMIN_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=unlock_emails\">".$value."</a>");
 	} else {
 		define('__MAIL_ADMIN_VALUE', "0");
 	}
@@ -191,12 +172,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Unfinished mail orders
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_pool WHERE data_type='TEMP'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("TEMP", "pool", "id", "data_type", true);
 
 	if ($value > 0) {
-		define('__MAIL_TEMP_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</A>");
+		define('__MAIL_TEMP_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</a>");
 	} else {
 		define('__MAIL_TEMP_VALUE', "0");
 	}
@@ -204,12 +183,10 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	//
 	// Sent mail orders
 	//
-	$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_pool WHERE data_type='SEND'", __FILE__, __LINE__);
-	$value = SQL_NUMROWS($result);
-	SQL_FREERESULT($result);
+	$value = GET_TOTAL_DATA("SEND", "pool", "id", "data_type", true);
 
 	if ($value > 0) {
-		define('__MAIL_SEND_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=email_archiv\">".$value."</A>");
+		define('__MAIL_SEND_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=email_archiv\">".$value."</a>");
 	} else {
 		define('__MAIL_SEND_VALUE', "0");
 	}
@@ -218,12 +195,11 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 	// Autopurged mails
 	//
 	if (EXT_IS_ACTIVE("autopurge")) {
-		$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_pool WHERE data_type='DELETED'", __FILE__, __LINE__);
-		$value = SQL_NUMROWS($result);
-		SQL_FREERESULT($result);
+		// Get auto-purged mails
+		$value = GET_TOTAL_DATA("DELETED", "pool", "id", "data_type", true);
 
 		if ($value > 0) {
-			define('__MAIL_DELETED_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</A>");
+			define('__MAIL_DELETED_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</a>");
 		} else {
 			define('__MAIL_DELETED_VALUE', "0");
 		}
@@ -232,35 +208,33 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 		// Autopurged bonus mails
 		//
 		if (GET_EXT_VERSION("bonus") >= "0.1.8") {
-			$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_bonus WHERE data_type='DELETED'", __FILE__, __LINE__);
-			$value = SQL_NUMROWS($result);
-			SQL_FREERESULT($result);
+			// Get auto-purged bonus mails
+			$value = GET_TOTAL_DATA("DELETED", "bonus", "id", "data_type", true);
 
 			if ($value > 0) {
-				define('__MAIL_BONUS_AP_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</A>");
+				define('__MAIL_BONUS_AP_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</a>");
 			} else {
 				define('__MAIL_BONUS_AP_VALUE', "0");
 			}
 		} elseif (EXT_IS_ACTIVE("bonus")) {
 			define('__MAIL_BONUS_AP_VALUE', ADMIN_EXT_BONUS_OUTDATED_1."0.1.8".ADMIN_EXT_BONUS_OUTDATED_2);
 		} else {
-			define('__MAIL_BONUS_AP_VALUE', "<FONT class=\"admin_note\">".ADMIN_EXT_BONUS_404."</FONT>");
+			define('__MAIL_BONUS_AP_VALUE', "<div class=\"admin_note\">".ADMIN_EXT_BONUS_404."</div>");
 		}
 	} else {
-		define('__MAIL_BONUS_AP_VALUE', "<FONT class=\"admin_note\">".ADMIN_EXT_AUTOPURGE_404."</FONT>");
-		define('__MAIL_DELETED_VALUE', "<FONT class=\"admin_note\">".ADMIN_EXT_AUTOPURGE_404."</FONT>");
+		define('__MAIL_BONUS_AP_VALUE', "<div class=\"admin_note\">".ADMIN_EXT_AUTOPURGE_404."</div>");
+		define('__MAIL_DELETED_VALUE', "<div class=\"admin_note\">".ADMIN_EXT_AUTOPURGE_404."</div>");
 	}
 
 	//
 	// Sent bonus mails
 	//
 	if (GET_EXT_VERSION("bonus") >= "0.1.8") {
-		$result = SQL_QUERY("SELECT id FROM "._MYSQL_PREFIX."_bonus WHERE data_type='SEND'", __FILE__, __LINE__);
-		$value = SQL_NUMROWS($result);
-		SQL_FREERESULT($result);
+		// Get sent bonus mails (but not notifications)
+		$value = GET_TOTAL_DATA("SEND", "bonus", "id", "data_type", true, " AND is_notify='N'");
 
 		if ($value > 0) {
-			define('__MAIL_BONUS_SEND_VALUE', "<A href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</A>");
+			define('__MAIL_BONUS_SEND_VALUE', "<a href=\"".URL."/modules.php?module=admin&amp;what=email_details\">".$value."</a>");
 		} else {
 			define('__MAIL_BONUS_SEND_VALUE', "0");
 		}
@@ -272,23 +246,22 @@ function OUTPUT_ADVANCED_OVERVIEW (&$result_main)
 
 	if (EXT_IS_ACTIVE("autopurge")) {
 		// Start finding them...
-		$since = (time() - $_CONFIG['ap_in_since']);
-		$EXCLUDE_LIST = " AND d.userid != c.def_refid";
+		$since = getConfig('ap_inactive_since');
+		$EXCLUDE_LIST = "";
+		if (getConfig('def_refid') > 0) {
+			$EXCLUDE_LIST = " AND d.userid != ".getConfig('def_refid')."";
+		} // END - if
 
 		// Check for more extensions
-		if (EXT_IS_ACTIVE("beg"))     $EXCLUDE_LIST .= " AND d.userid != c.beg_uid";
-		if (EXT_IS_ACTIVE("bonus"))   $EXCLUDE_LIST .= " AND d.userid != c.bonus_uid";
-		if (EXT_IS_ACTIVE("doubler")) $EXCLUDE_LIST .= " AND d.userid != c.doubler_uid";
-
-		// Check for new holiday system
-		if (GET_EXT_VERSION("holiday") >= "0.1.3") {
-			$EXCLUDE_LIST .= " AND d.holiday_active = 'N'";
-		}
+		if (EXT_IS_ACTIVE("beg"))                  $EXCLUDE_LIST .= " AND d.userid != ".getConfig('beg_uid')."";
+		if (EXT_IS_ACTIVE("bonus"))                $EXCLUDE_LIST .= " AND d.userid != ".getConfig('bonus_uid')."";
+		if (EXT_IS_ACTIVE("doubler"))              $EXCLUDE_LIST .= " AND d.userid != ".getConfig('doubler_uid')."";
+		if (GET_EXT_VERSION("holiday") >= "0.1.3") $EXCLUDE_LIST .= " AND d.holiday_active='N'";
 
 		// Check for all accounts
 		$SQLs[] = "SELECT DISTINCT d.userid, d.email, d.last_online
-FROM "._MYSQL_PREFIX."_user_data AS d, "._MYSQL_PREFIX."_config AS c
-WHERE d.status='CONFIRMED' AND d.joined < ".$since." AND d.last_online < ".$since." AND d.ap_notified < ".$since."
+FROM `"._MYSQL_PREFIX."_user_data` AS d
+WHERE d.status='CONFIRMED' AND d.joined < (UNIX_TIMESTAMP() - ".$since.") AND d.last_online < (UNIX_TIMESTAMP() - ".$since.") AND d.ap_notified < (UNIX_TIMESTAMP() - ".$since.")
 ".$EXCLUDE_LIST."
 ORDER BY d.userid";
 		$WHATs[]  = "list_autopurge";
@@ -297,11 +270,11 @@ ORDER BY d.userid";
 	}
 
 	if (GET_EXT_VERSION("sql_patches") >= "0.3.4") {
-		// Check for accounts without referral
-		$SQLs[]   = "SELECT userid FROM "._MYSQL_PREFIX."_user_data WHERE refid='0' ORDER BY userid";
+		// Check for accounts without referal
+		$SQLs[]   = "SELECT userid FROM `"._MYSQL_PREFIX."_user_data` WHERE refid='0' ORDER BY userid";
 		$DESCRs[] = TASK_ADMIN_LIST_ACCOUNT_NOREF;
 		$TITLEs[] = TASK_ADMIN_LIST_ACCOUNT_NOREF_TITLE;
-		$WHATs[]  = "list_norefs";
+		$WHATs[]  = "list_user&amp;mode=norefs";
 	}
 
 	if (EXT_IS_ACTIVE("payout")) {
@@ -340,16 +313,24 @@ ORDER BY d.userid";
 		$TITLEs[] = TASK_ADMIN_LIST_HOLIDAYS_TITLE;
 	}
 
+	if (GET_EXT_VERSION("bonus") >= "0.8.7") {
+		// List all notifications
+		$SQLs[]   = "SELECT id FROM "._MYSQL_PREFIX."_bonus WHERE is_notify='Y' ORDER BY timestamp DESC";
+		$WHATs[]  = "list_notifications";
+		$DESCRs[] = TASK_ADMIN_LIST_NOTIFICATIONS;
+		$TITLEs[] = TASK_ADMIN_LIST_NOTIFICATIONS_TITLE;
+	} // END - if
+
 	if (GET_EXT_VERSION("bonus") >= "0.2.3") {
 		// Active rallye
 		if (GET_EXT_VERSION("bonus") >= "0.6.9") {
 			// Add more bonus points here
 			$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 {
 			// Old version ???
@@ -357,16 +338,16 @@ ORDER BY d.userid";
 		}
 
 		// Autopurge installed?
-		$LAST = ""; $ONLINE = "";
-		if ((EXT_IS_ACTIVE("autopurge")) && ($_CONFIG['ap_inactive'] == "Y") && ($_CONFIG['ap_in_since'] > 0)) {
+		$LAST = "";
+		if ((EXT_IS_ACTIVE("autopurge")) && (getConfig('autopurge_inactive') == "Y") && (getConfig('ap_inactive_since') > 0)) {
 			// Use last online timestamp to keep inactive members away from here
-			$LAST   = " AND last_online >= ";
-			$ONLINE = bigintval(time() - $_CONFIG['ap_in_since']);
+			$LAST   = sprintf(" AND last_online >= (UNIX_TIMESTAMP() - %s)", getConfig('ap_inactive_since'));
 		}
 
-		$SQLs[]   = "SELECT ".$USE." AS active_bonus FROM "._MYSQL_PREFIX."_user_data
-WHERE status='CONFIRMED' AND ".$USE.">0".$LAST."".$ONLINE."
-ORDER BY active_bonus DESC, userid";
+		$SQLs[]   = "SELECT ".$USE." AS points
+FROM "._MYSQL_PREFIX."_user_data
+WHERE status='CONFIRMED' AND ".$USE.">0".$LAST."
+ORDER BY points DESC, userid";
 		$WHATs[]  = "list_bonus";
 		$DESCRs[] = TASK_ADMIN_LIST_BONUS;
 		$TITLEs[] = TASK_ADMIN_LIST_BONUS_TITLE;
@@ -376,15 +357,14 @@ ORDER BY active_bonus DESC, userid";
 		// Begging rallye
 
 		// Autopurge installed?
-		$LAST = ""; $ONLINE = "";
-		if ((EXT_IS_ACTIVE("autopurge")) && ($_CONFIG['ap_inactive'] == "Y") && ($_CONFIG['ap_in_since'] > 0)) {
+		$LAST = "";
+		if ((EXT_IS_ACTIVE("autopurge")) && (getConfig('autopurge_inactive') == "Y") && (getConfig('ap_inactive_since') > 0)) {
 			// Use last online timestamp to keep inactive members away from here
-			$LAST   = " AND last_online >= ";
-			$ONLINE = bigintval(time() - $_CONFIG['ap_in_since']);
+			$LAST   = sprintf(" AND last_online >= (UNIX_TIMESTAMP() - %s)", getConfig('ap_inactive_since'));
 		}
 
 		$SQLs[]   = "SELECT userid FROM "._MYSQL_PREFIX."_user_data
-WHERE status='CONFIRMED' AND beg_points>0".$LAST."".$ONLINE."
+WHERE status='CONFIRMED' AND beg_points>0".$LAST."
 ORDER BY beg_points DESC, userid";
 		$WHATs[]  = "list_beg";
 		$DESCRs[] = TASK_ADMIN_LIST_BEG;
@@ -404,7 +384,7 @@ ORDER BY beg_points DESC, userid";
 	}
 
 	//
-	// All referral banner
+	// All referal banner
 	//
 	$SQLs[] = "SELECT id FROM "._MYSQL_PREFIX."_refbanner ORDER BY id";
 	$WHATs[] = "refbanner";
@@ -412,7 +392,7 @@ ORDER BY beg_points DESC, userid";
 	$TITLEs[] = TASK_ADMIN_LIST_REFBANNER_ALL_TITLE;
 
 	//
-	// All activated referral banner
+	// All activated referal banner
 	//
 	$SQLs[] = "SELECT id FROM "._MYSQL_PREFIX."_refbanner WHERE visible='Y' ORDER BY id";
 	$WHATs[] = "refbanner";
@@ -505,13 +485,13 @@ ORDER BY beg_points DESC, userid";
 
 	if (EXT_IS_ACTIVE("theme")) {
 		// List all themes
-		$SQLs[]   = "SELECT id FROM "._MYSQL_PREFIX."_themes ORDER BY id";
+		$SQLs[]   = "SELECT id FROM `"._MYSQL_PREFIX."_themes` ORDER BY id";
 		$WHATs[]  = "theme_edit";
 		$DESCRs[] = TASK_ADMIN_LIST_THEME_ALL;
 		$TITLEs[] = TASK_ADMIN_LIST_THEME_ALL_TITLE;
 
 		// List active themes
-		$SQLs[]   = "SELECT id FROM "._MYSQL_PREFIX."_themes WHERE theme_active='Y' ORDER BY id";
+		$SQLs[]   = "SELECT id FROM `"._MYSQL_PREFIX."_themes` WHERE theme_active='Y' ORDER BY id";
 		$WHATs[]  = "theme_edit";
 		$DESCRs[] = TASK_ADMIN_LIST_THEME_ACTIVE;
 		$TITLEs[] = TASK_ADMIN_LIST_THEME_ACTIVE_TITLE;
@@ -559,7 +539,11 @@ function TASK_CREATE_EXTRA_ROWS($SQLs, $WHATs, $DESCRs, $TITLEs){
 	foreach ($SQLs as $key => $sql) {
 		// Run SQL command, get line numbers and free memory
 		$result = SQL_QUERY($sql, __FILE__, __LINE__);
+
+		// Get rows
 		$value = SQL_NUMROWS($result);
+
+		// Free result
 		SQL_FREERESULT($result);
 
 		$content = array(
@@ -571,7 +555,7 @@ function TASK_CREATE_EXTRA_ROWS($SQLs, $WHATs, $DESCRs, $TITLEs){
 		if ($key < $MAX) $content['bottom'] = "bottom2";
 		if (($value > 0) && (!empty($WHATs[$key]))) {
 			if (empty($TITLEs[$key])) $TITLEs[$key] = TASK_ADMIN_UNKNOWN_LIST_TITLE;
-			$content['row_link'] = "<A href=\"".URL."/modules.php?module=admin&amp;what=".$WHATs[$key]."\" title=\"".$TITLEs[$key]."\">".$value."</A>";
+			$content['row_link'] = "<a href=\"".URL."/modules.php?module=admin&amp;what=".$WHATs[$key]."\" title=\"".$TITLEs[$key]."\">".$value."</a>";
 		} else {
 			$content['row_link'] = 0;
 		}
@@ -581,7 +565,7 @@ function TASK_CREATE_EXTRA_ROWS($SQLs, $WHATs, $DESCRs, $TITLEs){
 
 		// And insert the final string into extras template
 		$OUT .= LOAD_TEMPLATE("admin_overview_task_rows", true, $content);
-	}
+	} // END - if
 
 	// Return output
 	return LOAD_TEMPLATE("admin_overview_task_extras", true, $OUT);