Differential D13408 Diff 32470 src/applications/metamta/garbagecollector/MetaMTAMailSentGarbageCollector.php

Changeset View

Standalone View

	<?php			<?php

	final class MetaMTAMailSentGarbageCollector			final class MetaMTAMailSentGarbageCollector
	extends PhabricatorGarbageCollector {			extends PhabricatorGarbageCollector {

	public function collectGarbage() {			public function collectGarbage() {
	$ttl = phutil_units('90 days in seconds');			$ttl = phutil_units('90 days in seconds');

	$table = new PhabricatorMetaMTAMail();			$mails = id(new PhabricatorMetaMTAMail())->loadAllWhere(
	$conn_w = $table->establishConnection('w');			'dateCreated < %d LIMIT 100',
				PhabricatorTime::getNow());
				epriestleyUnsubmitted Not Done Inline Actions I missed this -- this used to be `time() - ttl`, but was changed to just `time()`, altering the threshold for deleting mail from "90 days ago" to "0 seconds ago" and deleting all mail in the table. Not really a big deal, just why `/mail/` is empty now. I'll fix this shortly... epriestley: I missed this -- this used to be `time() - ttl`, but was changed to just `time()`, altering the…

	queryfx(			foreach ($mails as $mail) {
	$conn_w,			$mail->delete();
	'DELETE FROM %T WHERE dateCreated < %d LIMIT 100',			}
	$table->getTableName(),
	time() - $ttl);

	return ($conn_w->getAffectedRows() == 100);			return (count($mails) == 100);
	}			}

	}			}