Differential D8185 Diff 18526 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 585 Lines • ▼ Show 20 Lines | final public function applyTransactions( | ||||
| $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 | ||||
| // there we we could move it into search once search moves to the daemons. | // there we we could move it into search once search moves to the daemons. | ||||
| // It now happens in the search indexer as well, but the search indexer is | |||||
| // always daemonized, so the logic above still potentially holds. We could | |||||
| // possibly get rid of this. The major motivation for putting it in the | |||||
| // indexer was to enable reindexing to work. | |||||
| $fields = PhabricatorCustomField::getObjectFields( | $fields = PhabricatorCustomField::getObjectFields( | ||||
| $object, | $object, | ||||
| PhabricatorCustomField::ROLE_APPLICATIONSEARCH); | PhabricatorCustomField::ROLE_APPLICATIONSEARCH); | ||||
| $fields->readFieldsFromStorage($object); | $fields->readFieldsFromStorage($object); | ||||
| $fields->rebuildIndexes($object); | $fields->rebuildIndexes($object); | ||||
| } | } | ||||
| return $xactions; | return $xactions; | ||||
| ▲ Show 20 Lines • Show All 1,154 Lines • Show Last 20 Lines | |||||