Page MenuHomePhabricator
Differential D8579

Despecialize most task status handling
ClosedPublic
Actions

Authored by epriestley on Mar 21 2014, 11:42 AM.
Tags
None
Referenced Files
F15553703: D8579.id20412.diff
Mon, Apr 28, 3:20 AM
F15548020: D8579.id20356.diff
Sat, Apr 26, 9:28 PM
F15539500: D8579.diff
Fri, Apr 25, 1:54 AM
F15518315: D8579.id20356.diff
Sat, Apr 19, 12:38 PM
F15516806: D8579.id.diff
Fri, Apr 18, 9:30 PM
F15514448: D8579.id20412.diff
Fri, Apr 18, 2:46 AM
F15512762: D8579.diff
Thu, Apr 17, 1:35 PM
F15478822: D8579.diff
Tue, Apr 8, 3:03 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T1812: Allow for arbitrary values for task status
Commits
Restricted Diffusion Commit
rP33bda2d5903b: Despecialize most task status handling
Summary

Ref T1812. Moves most specialized status handling into ManiphestTaskStatus. The only real missing case is reports.

Test Plan

Browsed most of the affected interfaces. Changed task status:

Screen_Shot_2014-03-21_at_4.37.50_AM.png (390×970 px, 44 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20356.Mar 21 2014, 11:42 AM
epriestley retitled this revision from to Despecialize most task status handling.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T1812: Allow for arbitrary values for task status.
Herald added a subscriber: epriestley.Mar 21 2014, 11:42 AM
btrahan accepted this revision.Mar 25 2014, 7:02 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 25 2014, 7:02 PM
epriestley closed this revision.Mar 25 2014, 8:47 PM
epriestley updated this revision to Diff 20412.

Closed by commit rP33bda2d5903b (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
constants/
59 lines
controller/
33 lines
4 lines
mail/
2 lines
phid/
2 lines
query/
10 lines
search/
6 lines
storage/
4 lines
169 lines
view/
2 lines
3 lines
search/
controller/
2 lines
webroot/
rsrc/
js/
application/
maniphest/
2 lines

Diff 20412

View Options

src/applications/maniphest/constants/ManiphestTaskStatus.php

Loading...
View Options

src/applications/maniphest/controller/ManiphestTaskDetailController.php

Loading...
View Options

src/applications/maniphest/controller/ManiphestTransactionSaveController.php

Loading...
View Options

src/applications/maniphest/mail/ManiphestReplyHandler.php

Loading...
View Options

src/applications/maniphest/phid/ManiphestPHIDTypeTask.php

Loading...
View Options

src/applications/maniphest/query/ManiphestTaskQuery.php

Loading...
View Options

src/applications/maniphest/search/ManiphestSearchIndexer.php

Loading...
View Options

src/applications/maniphest/storage/ManiphestTask.php

Loading...
View Options

src/applications/maniphest/storage/ManiphestTransaction.php

Loading...
View Options

src/applications/maniphest/view/ManiphestTaskListView.php

Loading...
View Options

src/applications/maniphest/view/ManiphestView.php

Loading...
View Options

src/applications/search/controller/PhabricatorSearchAttachController.php

Loading...
View Options

webroot/rsrc/js/application/maniphest/behavior-batch-editor.js

Loading...
Phabricator · Built by Phacility