HomePhabricator

Move Differential commit message rendering to dedicated classes

Description

Move Differential commit message rendering to dedicated classes

Summary:
Ref T11114. This probably still has some bugs, but survives basic sanity checks.

Continue pulling commit message logic out of CustomField so we can reduce the amount of responsibility/bloat in the classtree and send more code through EditEngine.

Test Plan:

  • Called differential.getcommitmessage via API console for various revisions/parameters (edit and create mode, with and without fields, with and without revisions).
  • Used --create, --edit and --update modes of arc diff from the CLI.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11114

Differential Revision: https://secure.phabricator.com/D17066

Details

Provenance
epriestleyAuthored on Dec 15 2016, 6:11 PM
epriestleyPushed on Dec 16 2016, 6:08 PM
Reviewer
chad
Differential Revision
D17066: Move Differential commit message rendering to dedicated classes
Parents
rP0387d626325c: Add Dashboard typeaheads
Branches
Unknown
Tags
Unknown
Tasks
T11114: Move Differential to EditEngine
Build Status
Buildable 14971
Build 19637: Run Core Tests