Page MenuHomePhabricator

Move many task status hardcodes into ManiphestTaskStatus
ClosedPublic

Authored by epriestley on Feb 17 2014, 9:00 PM.
Tags
None
Referenced Files
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
F15397928: D8264.id19654.diff
Sun, Mar 16, 11:00 PM
F15396008: D8264.id.diff
Sun, Mar 16, 11:10 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

Lint
Lint Skipped
Unit
Tests Skipped