Differential D13245 Diff 32019 src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
| Show First 20 Lines • Show All 930 Lines • ▼ Show 20 Lines | if ($this->getIsHeraldEditor()) { | ||||
| ->setActor($herald_actor) | ->setActor($herald_actor) | ||||
| ->setActingAsPHID($herald_phid) | ->setActingAsPHID($herald_phid) | ||||
| ->setContentSource($herald_source); | ->setContentSource($herald_source); | ||||
| $herald_xactions = $herald_editor->applyTransactions( | $herald_xactions = $herald_editor->applyTransactions( | ||||
| $object, | $object, | ||||
| $herald_xactions); | $herald_xactions); | ||||
| $adapter = $this->getHeraldAdapter(); | |||||
| $this->heraldEmailPHIDs = $adapter->getEmailPHIDs(); | |||||
| $this->heraldForcedEmailPHIDs = $adapter->getForcedEmailPHIDs(); | |||||
| // Merge the new transactions into the transaction list: we want to | // Merge the new transactions into the transaction list: we want to | ||||
| // send email and publish feed stories about them, too. | // send email and publish feed stories about them, too. | ||||
| $xactions = array_merge($xactions, $herald_xactions); | $xactions = array_merge($xactions, $herald_xactions); | ||||
| } | } | ||||
| // If Herald did not generate transactions, we may still need to handle | |||||
| // "Send an Email" rules. | |||||
| $adapter = $this->getHeraldAdapter(); | |||||
| $this->heraldEmailPHIDs = $adapter->getEmailPHIDs(); | |||||
| $this->heraldForcedEmailPHIDs = $adapter->getForcedEmailPHIDs(); | |||||
| } | } | ||||
| $this->didApplyTransactions($xactions); | $this->didApplyTransactions($xactions); | ||||
| if ($object instanceof PhabricatorCustomFieldInterface) { | if ($object instanceof PhabricatorCustomFieldInterface) { | ||||
| // Maybe this makes more sense to move into the search index itself? For | // Maybe this makes more sense to move into the search index itself? For | ||||
| // now I'm putting it here since I think we might end up with things that | // now I'm putting it here since I think we might end up with things that | ||||
| // need it to be up to date once the next page loads, but if we don't go | // need it to be up to date once the next page loads, but if we don't go | ||||
| ▲ Show 20 Lines • Show All 2,124 Lines • Show Last 20 Lines | |||||