Page MenuHomePhabricator

Bring "fixes x as y" parser forward and use new parsers instead of old ones
ClosedPublic

Authored by epriestley on Feb 17 2014, 8:10 PM.
Tags
None
Referenced Files
F13108040: D8263.id19652.diff
Sun, Apr 28, 12:05 AM
F13105285: D8263.diff
Sat, Apr 27, 2:55 PM
Unknown Object (File)
Wed, Apr 24, 10:28 PM
Unknown Object (File)
Tue, Apr 23, 9:40 PM
Unknown Object (File)
Tue, Apr 23, 9:40 PM
Unknown Object (File)
Tue, Apr 23, 9:40 PM
Unknown Object (File)
Tue, Apr 23, 9:40 PM
Unknown Object (File)
Mon, Apr 22, 8:21 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T1812: Allow for arbitrary values for task status
Restricted Maniphest Task
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP8a725ece0f8b: Bring "fixes x as y" parser forward and use new parsers instead of old ones
Summary

Fixes T3872. Ref T1812. Ref T3886. Modernize the "closes x as y" string parser, and use all the new parsers instead of the old ones.

Test Plan

Made a commit full of a pile of these trigger strings, then used scripts/repository/reparse.php --message to reparse it. Verified that parses came back as expected using a bunch of var_dump().

Diff Detail

Repository
rP Phabricator
Branch
diff3
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

src/applications/differential/field/specification/DifferentialFreeformFieldSpecification.php
12–45

This duplicated stuff is going away soon.

src/applications/maniphest/field/parser/__tests__/ManiphestCustomFieldStatusParserTestCase.php
3

Oops, this didn't get updated properly. I'll update.

epriestley updated this revision to Unknown Object (????).Feb 17 2014, 8:17 PM
  • Include appropriate unit tests.