Page MenuHomePhabricator
Differential D20795

Index Herald fields, not just actions, when identifying objects related to a particular Herald rule
ClosedPublic
Actions

Authored by epriestley on Sep 9 2019, 7:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 10, 8:15 PM
Unknown Object (File)
Sun, Feb 9, 8:06 AM
Unknown Object (File)
Sun, Feb 9, 8:06 AM
Unknown Object (File)
Sun, Feb 9, 8:06 AM
Unknown Object (File)
Sun, Feb 9, 8:06 AM
Unknown Object (File)
Sat, Jan 25, 5:43 AM
Unknown Object (File)
Sat, Jan 25, 5:43 AM
Unknown Object (File)
Sat, Jan 25, 5:43 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13408: Examine conditions, not just actions, when indexing Herald rules for links to other objects
Commits
rPd965d9a669b5: Index Herald fields, not just actions, when identifying objects related to a…
Summary

Fixes T13408. Currently, when a package (or other object) appears in a field (rather than an action), it is not indexed.

Instead: index fields too, not just actions.

Test Plan
  • Wrote a rule like "[ Affected packages include ] ...".
  • Updated the search index.
  • Saw rule appear on "Affected By Herald Rules" on the package detail page.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 9 2019, 7:44 PM
Owners added a subscriber: Restricted Owners Package.Sep 9 2019, 7:44 PM
Harbormaster completed remote builds in B23409: Diff 49584.Sep 9 2019, 7:45 PM
epriestley requested review of this revision.Sep 9 2019, 7:45 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 9 2019, 7:50 PM
Closed by commit rPd965d9a669b5: Index Herald fields, not just actions, when identifying objects related to a… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPd965d9a669b5: Index Herald fields, not just actions, when identifying objects related to a….

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
3 lines
src/
applications/
herald/
engineextension/
14 lines
field/
22 lines

Diff 49586

View Options

resources/sql/autopatches/20190909.herald.01.rebuild.php

Loading...
View Options

src/applications/herald/engineextension/HeraldRuleIndexEngineExtension.php

Loading...
View Options

src/applications/herald/field/HeraldField.php

Loading...
Phabricator · Built by Phacility