Page MenuHomePhabricator

Move many task status hardcodes into ManiphestTaskStatus
ClosedPublic

Authored by epriestley on Feb 17 2014, 9:00 PM.
Tags
None
Referenced Files
F15503597: D8264.id19654.diff
Mon, Apr 14, 11:11 AM
F15471931: D8264.id19663.diff
Sat, Apr 5, 9:45 AM
F15462045: D8264.id19654.diff
Tue, Apr 1, 12:46 PM
F15462044: D8264.id.diff
Tue, Apr 1, 12:46 PM
F15462043: D8264.diff
Tue, Apr 1, 12:46 PM
F15415286: D8264.id.diff
Thu, Mar 20, 5:12 AM
F15415285: D8264.id19654.diff
Thu, Mar 20, 5:12 AM
F15411416: D8264.id.diff
Wed, Mar 19, 9:17 AM
Subscribers

Details

Summary

Ref T1812. This cleans up most of the easy hard-coded references to specific statuses:

  • The "fixes" language moves into ManiphestTaskStatus.
  • Add a method to list open statuses.
  • Add a method to test if a status is open.
  • Add a method to get default status for new tasks.
Test Plan

Browsed around, lint, grep, created, filtered and updated tasks.

Diff Detail

Repository
rP Phabricator
Branch
diff4
Lint
Lint Passed
Unit
Tests Passed