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
F18754252: D8579.id.diff
Sat, Oct 4, 10:06 PM
F18746938: D8579.diff
Fri, Oct 3, 1:59 PM
F18734704: D8579.id.diff
Tue, Sep 30, 10:59 PM
F18684434: D8579.diff
Fri, Sep 26, 9:31 AM
F18611227: D8579.diff
Sep 14 2025, 6:24 AM
F18611224: D8579.diff
Sep 14 2025, 6:24 AM
F18609522: D8579.id20356.diff
Sep 14 2025, 3:13 AM
F18593673: D8579.diff
Sep 12 2025, 3:35 PM
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