Differential D8185 Diff 18526 src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Changeset 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