D10087.id24263.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D10087.id24263.diff
View Options

	diff --git a/src/applications/differential/mail/DifferentialReplyHandler.php b/src/applications/differential/mail/DifferentialReplyHandler.php
	--- a/src/applications/differential/mail/DifferentialReplyHandler.php
	+++ b/src/applications/differential/mail/DifferentialReplyHandler.php
	@@ -146,15 +146,21 @@
	->setContent($body));
	}

	+ // NOTE: We have to be careful about this because Facebook's
	+ // implementation jumps straight into handleAction() and will not have
	+ // a PhabricatorMetaMTAReceivedMail object.
	+ if ($this->receivedMail) {
	+ $exclude = $this->receivedMail->getExcludeMailRecipientPHIDs();
	+ } else {
	+ $exclude = $this->getExcludeMailRecipientPHIDs();
	+ }
	+
	$editor = id(new DifferentialTransactionEditor())
	->setActor($actor)
	- ->setExcludeMailRecipientPHIDs($this->getExcludeMailRecipientPHIDs())
	+ ->setExcludeMailRecipientPHIDs($exclude)
	->setContinueOnMissingFields(true)
	->setContinueOnNoEffect(true);

	- // NOTE: We have to be careful about this because Facebook's
	- // implementation jumps straight into handleAction() and will not have
	- // a PhabricatorMetaMTAReceivedMail object.
	if ($this->receivedMail) {
	$content_source = PhabricatorContentSource::newForSource(
	PhabricatorContentSource::SOURCE_EMAIL,