X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fautopurge%2Fpurge-inact.php;h=3f98e7023ac2fda561b0bf5bb7886dc7487507a8;hb=19b197c32c96ce8eea26561fb7b40e87c57716ae;hp=93dcb5b2b65a0a62ba27171f8e3e1653ed612ea5;hpb=64c8349613addc3da2242c5cd6b99d64e3fb5f8e;p=mailer.git

diff --git a/inc/autopurge/purge-inact.php b/inc/autopurge/purge-inact.php
index 93dcb5b2b6..3f98e7023a 100644
--- a/inc/autopurge/purge-inact.php
+++ b/inc/autopurge/purge-inact.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 09/14/2008 *
- * ===============                              Last change: 09/14/2008 *
+ * Mailer v0.2.1-FINAL                                Start: 09/14/2008 *
+ * ===================                          Last change: 09/14/2008 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : purge-inactive.php                               *
@@ -18,6 +18,7 @@
  * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2009 by Roland Haeder                           *
+ * Copyright (c) 2009, 2010 by Mailer Developer Team                    *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -53,18 +54,19 @@ if (getConfig('autopurge_inactive') == 'Y') {
 	initSqls();
 
 	// Init exclusion list
+	// @TODO Rewrite these if() blocks to a filter
 	$EXCLUDE_LIST = '';
 	if (getConfig('def_refid') > 0) {
-		$EXCLUDE_LIST = " AND d.userid != ".getConfig('def_refid');
+		$EXCLUDE_LIST = ' AND d.`userid` != {?def_refid?}';
 	} // END - if
 
 	// Check for more extensions
-	if (isExtensionActive('beg'))     $EXCLUDE_LIST .= " AND d.userid != ".getConfig('beg_userid')."";
-	if (isExtensionActive('bonus'))   $EXCLUDE_LIST .= " AND d.userid != ".getConfig('bonus_userid')."";
-	if (isExtensionActive('doubler')) $EXCLUDE_LIST .= " AND d.userid != ".getConfig('doubler_userid')."";
+	if (isExtensionActive('beg'))     $EXCLUDE_LIST .= ' AND d.`userid` != {?beg_userid?}';
+	if (isExtensionActive('bonus'))   $EXCLUDE_LIST .= ' AND d.`userid` != {?bonus_userid?}';
+	if (isExtensionActive('doubler')) $EXCLUDE_LIST .= ' AND d.`userid` != {?doubler_userid?}';
 
 	// Check for new holiday extension
-	if (getExtensionVersion('holiday') >= '0.1.3') {
+	if (isExtensionInstalledAndNewer('holiday', '0.1.3')) {
 		// Include only users with no active holiday
 		$EXCLUDE_LIST .= " AND d.`holiday_active`='N'";
 	} // END - if
@@ -89,17 +91,20 @@ ORDER BY
 
 	if (SQL_NUMROWS($result_inactive) > 0) {
 		// Prepare variables and constants...
-		$UIDs = '';
+		$useridsContent = '';
 		$content['since'] = (getConfig('ap_inactive_since') / 60 / 60);
 		$content['time']  = (getConfig('ap_inactive_time')  / 60 / 60);
 
 		// Mark found accounts as inactive and send an email
-		while ($content = SQL_FETCHARRAY($result_inactive)) {
+		while ($row = SQL_FETCHARRAY($result_inactive)) {
+			// Merge both arrays
+			$content = merge_array($content, $row);
+
 			// Remember userids for the admin
-			$UIDs .= $content['userid'].", ";
+			$useridsContent .= $content['userid'].", ";
 
 			// Get date/time from timestamp
-			$content['last_online'] = generateDateTime($content['last_online'], '0');
+			$content['last_online'] = generateDateTime($content['last_online'], 0);
 
 			// Load mail template
 			$message = loadEmailTemplate('member_autopurge_inactive', $content, bigintval($content['userid']));
@@ -111,10 +116,10 @@ ORDER BY
 		} // END - while
 
 		// Remove last comma
-		$UIDs = str_replace(', ', "\n", substr($UIDs, 0, -2));
+		$useridsContent = str_replace(', ', "\n", substr($useridsContent, 0, -2));
 
 		// Send mail notification to admin
-		sendAdminNotification(getMessage('AUTOPURGE_ADMIN_INACTIVE_SUBJECT'), 'admin_autopurge_inactive', $UIDs, '');
+		sendAdminNotification(getMessage('AUTOPURGE_ADMIN_INACTIVE_SUBJECT'), 'admin_autopurge_inactive', $useridsContent);
 	} // END - if
 
 	// Free memory
@@ -122,39 +127,41 @@ ORDER BY
 
 	// Now let's have a look for inactive accounts we want to delete we newly use the same exclude list
 	// here for e.g. excluding holiday users
-	$result_inactive = SQL_QUERY_ESC("SELECT d.userid, d.email, d.last_online
-FROM `{?_MYSQL_PREFIX?}_user_data` AS d
-WHERE `status`='CONFIRMED' AND `joined` < (UNIX_TIMESTAMP() - %s) AND `last_online` < (UNIX_TIMESTAMP() - %s) AND `ap_notified` < (UNIX_TIMESTAMP() - %s)
+	$result_inactive = SQL_QUERY("SELECT
+	d.userid, d.email, d.last_online
+FROM
+	`{?_MYSQL_PREFIX?}_user_data` AS d
+WHERE
+	`status`='CONFIRMED' AND
+	`joined` < (UNIX_TIMESTAMP() - {?ap_inactive_since?}) AND
+	`last_online` < (UNIX_TIMESTAMP() - {?ap_inactive_since?}) AND
+	`ap_notified` < (UNIX_TIMESTAMP() - {?ap_inactive_time?})
 ".$EXCLUDE_LIST."
-ORDER BY `userid` ASC",
-		array(
-			getConfig('ap_inactive_since'),
-			getConfig('ap_inactive_since'),
-			getConfig('ap_inactive_time')
-		), __FILE__, __LINE__);
+ORDER BY
+	`userid` ASC", __FILE__, __LINE__);
 
 	if (SQL_NUMROWS($result_inactive) > 0) {
 		// Prepare variable...
-		$UIDs = '';
+		$useridsContent = '';
 
 		// Delete inactive accounts
 		while ($content = SQL_FETCHARRAY($result_inactive)) {
 			// Remember userids for the admin
-			$UIDs .= $content['userid'] . ', ';
+			$useridsContent .= $content['userid'] . ', ';
 
 			// Get date/time from timestamp
-			$content['last_online'] = generateDateTime($content['last_online'], '0');
+			$content['last_online'] = generateDateTime($content['last_online'], 0);
 
 			// Finnaly delete this inactive account
 			deleteUserAccount($content['userid'], loadEmailTemplate('member_autopurge_delete', $content['last_online'], ''));
 		} // END - while
 
 		// Remove last comma
-		$UIDs = str_replace(", ", "\n", substr($UIDs, 0, -2));
+		$useridsContent = str_replace(', ', "\n", substr($useridsContent, 0, -2));
 
 		// Send mail notification to admin
 		if (getConfig('ap_in_notify') == 'Y') {
-			sendAdminNotification(getMessage('AUTOPURGE_ADMIN_DELETE_SUBJECT'), 'admin_autopurge_delete', $UIDs, '');
+			sendAdminNotification(getMessage('AUTOPURGE_ADMIN_DELETE_SUBJECT'), 'admin_autopurge_delete', $useridsContent);
 		} // END - if
 	} // END - if