X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FWorker%2FExpirePosts.php;h=34e43ec5b4bb16a7e183f84cffefa7cd4516561b;hb=bf8fb215a9cc554b5ec5b774168a52fb56fa43e6;hp=22136467b7eff1d510b0ceb8d8550ac0cddeeb16;hpb=84b6c18b94551e2d21c89a5c44867dbdc5b6c15f;p=friendica.git

diff --git a/src/Worker/ExpirePosts.php b/src/Worker/ExpirePosts.php
index 22136467b7..34e43ec5b4 100644
--- a/src/Worker/ExpirePosts.php
+++ b/src/Worker/ExpirePosts.php
@@ -22,9 +22,9 @@
 namespace Friendica\Worker;
 
 use Friendica\Core\Logger;
+use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\DI;
-use Friendica\Util\DateTimeFormat;
 
 class ExpirePosts
 {
@@ -39,35 +39,59 @@ class ExpirePosts
 			$expire_days_unclaimed = $expire_days;
 		}
 
+		$limit = DI::config()->get('system', 'dbclean-expire-limit');
+		if (empty($limit)) {
+			return;
+		}
+
 		if (!empty($expire_days)) {
-			Logger::notice('Start deleting expired threads', ['expiry_days' => $expire_days, 'count' => DBA::count('item')]);
-			$ret = DBA::e("DELETE FROM `item-uri` WHERE `id` IN
-				(SELECT `uri-id` FROM `thread`
-				INNER JOIN `contact` ON `id` = `contact-id` AND NOT `notify_new_posts`
-				WHERE `received` < UTC_TIMESTAMP() - INTERVAL ? DAY
-					AND NOT `mention` AND NOT `starred` AND NOT `wall` AND NOT `origin`
-					AND `thread`.`uid` != 0 AND NOT `iid` IN (SELECT `parent` FROM `item`
-						WHERE (`item`.`starred` OR (`item`.`resource-id` != '')
-							OR (`item`.`event-id` != '') OR (`item`.`attach` != '')
-							OR `item`.`wall` OR `item`.`origin`
-							OR `uri-id` IN (SELECT `uri-id` FROM `post-category`
-								WHERE `uri-id` = `item`.`uri-id`))
-							AND `item`.`parent` = `thread`.`iid`))", $expire_days);
+			do {
+				Logger::notice('Start deleting expired threads', ['expiry_days' => $expire_days]);
+				$ret = DBA::e("DELETE FROM `item-uri` WHERE `id` IN
+					(SELECT `uri-id` FROM `thread`
+					INNER JOIN `contact` ON `id` = `contact-id` AND NOT `notify_new_posts`
+					WHERE `thread`.`received` < UTC_TIMESTAMP() - INTERVAL ? DAY
+						AND NOT `thread`.`mention` AND NOT `thread`.`starred`
+						AND NOT `thread`.`wall` AND NOT `thread`.`origin`
+						AND `thread`.`uid` != 0 AND NOT `iid` IN (SELECT `parent` FROM `item`
+							WHERE (`item`.`starred` OR (`item`.`resource-id` != '')
+								OR (`item`.`event-id` != '') OR (`item`.`attach` != '')
+								OR `item`.`wall` OR `item`.`origin`
+								OR `item`.`uri-id` IN (SELECT `uri-id` FROM `post-category`
+									WHERE `post-category`.`uri-id` = `item`.`uri-id`))
+								AND `item`.`parent` = `thread`.`iid`))
+					ORDER BY `id` LIMIT ?", $expire_days, $limit);
+
+				$rows = DBA::affectedRows();
+				Logger::notice('Deleted expired threads', ['result' => $ret, 'rows' => $rows]);
 
-			Logger::notice('Deleted expired threads', ['result' => $ret, 'rows' => DBA::affectedRows(), 'count' => DBA::count('item')]);
+				if (!Worker::isInMaintenanceWindow()) {
+					Logger::notice('We are outside of the maintenance window, quitting');
+					return;
+				}
+			} while ($rows >= $limit);
 		}
 
 		if (!empty($expire_days_unclaimed)) {
-			$expiry_date = DateTimeFormat::utc('now - ' . $expire_days_unclaimed . ' days', DateTimeFormat::MYSQL);
+			do {
+				Logger::notice('Start deleting unclaimed public items', ['expiry_days' => $expire_days_unclaimed]);
+				$ret = DBA::e("DELETE FROM `item-uri` WHERE `id` IN
+					(SELECT `uri-id` FROM `item` WHERE `gravity` = ? AND `uid` = ? AND `received` < UTC_TIMESTAMP() - INTERVAL ? DAY
+						AND NOT `uri-id` IN (SELECT `parent-uri-id` FROM `item` AS `i` WHERE `i`.`uid` != ?
+							AND `i`.`parent-uri-id` = `item`.`uri-id`)
+						AND NOT `uri-id` IN (SELECT `parent-uri-id` FROM `item` AS `i` WHERE `i`.`uid` = ?
+							AND `i`.`parent-uri-id` = `item`.`uri-id` AND `i`.`received` > UTC_TIMESTAMP() - INTERVAL ? DAY))
+					ORDER BY `id` LIMIT ?",
+					GRAVITY_PARENT, 0, $expire_days_unclaimed, 0, 0, $expire_days_unclaimed, $limit);
 
-			Logger::notice('Start deleting unclaimed public items', ['expiry_days' => $expire_days_unclaimed, 'expired' => $expiry_date, 'count' => DBA::count('item')]);
-			$ret = DBA::e("DELETE FROM `item-uri` WHERE `id` IN
-				(SELECT `uri-id` FROM `item` WHERE `gravity` = ? AND `uid` = ? AND `received` < ?
-					AND NOT `uri-id` IN (SELECT `parent-uri-id` FROM `item` WHERE `uid` != ?)
-					AND NOT `uri-id` IN (SELECT `parent-uri-id` FROM `item` WHERE `uid` = ? AND `received` > ?))",
-				GRAVITY_PARENT, 0, $expiry_date, 0, 0, $expiry_date);
+				$rows = DBA::affectedRows();
+				Logger::notice('Deleted unclaimed public items', ['result' => $ret, 'rows' => $rows]);
 
-			Logger::notice('Deleted unclaimed public items', ['result' => $ret, 'rows' => DBA::affectedRows(), 'count' => DBA::count('item')]);
+				if (!Worker::isInMaintenanceWindow()) {
+					Logger::notice('We are outside of the maintenance window, quitting');
+					return;
+				}
+			} while ($rows >= $limit);
 		}
 	}
 }