Paths

Table of Contentst

Differential D8263

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

Authored by epriestley on Feb 17 2014, 8:10 PM.

Tags

None

Referenced Files

	F15462042: D8263.id19662.diff
	Tue, Apr 1, 12:46 PM

	F15462041: D8263.id19653.diff
	Tue, Apr 1, 12:45 PM

	F15462040: D8263.id.diff
	Tue, Apr 1, 12:45 PM

	F15462039: D8263.diff
	Tue, Apr 1, 12:45 PM

	F15427191: D8263.id19652.diff
	Sun, Mar 23, 1:16 PM

	F15416355: D8263.id.diff
	Thu, Mar 20, 10:38 AM

	F15392975: D8263.diff
	Sat, Mar 15, 6:38 PM

	F15391199: D8263.diff
	Sat, Mar 15, 8:40 AM

Subscribers

Details

Reviewers

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

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley added inline comments.Feb 17 2014, 8:11 PM

src/applications/differential/field/specification/DifferentialFreeformFieldSpecification.php
12–45	This duplicated stuff is going away soon.
src/applications/maniphest/field/parser/__tests__/ManiphestCustomFieldStatusParserTestCase.php
4	Oops, this didn't get updated properly. I'll update.

Include appropriate unit tests.

btrahan accepted this revision.Feb 17 2014, 10:22 PM

Closed by commit rP8a725ece0f8b (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

4 lines

applications/

differential/

field/

specification/

DifferentialFreeformFieldSpecification.php

94 lines

maniphest/

field/

parser/

ManiphestCustomFieldStatusParser.php

59 lines

__tests__/

ManiphestCustomFieldStatusParserTestCase.php

63 lines

infrastructure/

customfield/

parser/

PhabricatorCustomFieldMonogramParser.php

2 lines

Diff 19662

src/__phutil_library_map__.php

Loading...

src/applications/differential/field/specification/DifferentialFreeformFieldSpecification.php

Loading...

src/applications/maniphest/field/parser/ManiphestCustomFieldStatusParser.php

Loading...

src/applications/maniphest/field/parser/tests/ManiphestCustomFieldStatusParserTestCase.php

Loading...

src/infrastructure/customfield/parser/PhabricatorCustomFieldMonogramParser.php

Loading...