Page MenuHomePhabricator

Reduce code duplication in Herald fields
ClosedPublic

Authored by epriestley on Jul 9 2015, 3:42 PM.
Tags
None
Referenced Files
F14017154: D13612.id32970.diff
Mon, Nov 4, 2:27 PM
F14017115: D13612.id32894.diff
Mon, Nov 4, 2:20 PM
F13994940: D13612.id32894.diff
Wed, Oct 23, 9:33 AM
F13992274: D13612.id.diff
Tue, Oct 22, 3:34 PM
F13989937: D13612.id.diff
Tue, Oct 22, 12:02 AM
F13989934: D13612.id.diff
Tue, Oct 22, 12:00 AM
F13989645: D13612.id32970.diff
Mon, Oct 21, 10:03 PM
F13989629: D13612.id32894.diff
Mon, Oct 21, 9:51 PM
Subscribers
Tokens
"Doubloon" token, awarded by btrahan.

Details

Summary

Ref T8726. I want to modularize values and reduce how hard-coded / copypasta'd they are.

  • Rename get...StandardCondition() to get...StandardType(), since we can drive both conditions and values from it.
  • Rename STANDARD_LIST to STANDARD_PHID_LIST for consistency: all "lists" are lists of PHIDs.
  • For all standard types which don't require typehaeads, lift their logic into the base class.
  • I'll lift typeaheads soon, but need to generalize them first.
Test Plan

Edited various Herald rules, saw value UI generate correctly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Reduce code duplication in Herald fields.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 16 2015, 8:36 PM
This revision was automatically updated to reflect the committed changes.