Differential D8584

Add config validation for task status config
ClosedPublic
Actions

Authored by epriestley on Mar 21 2014, 1:38 PM.

Tags

None

Referenced Files

	F15535151: D8584.diff
	Thu, Apr 24, 3:45 AM

	F15525671: D8584.id20362.diff
	Mon, Apr 21, 4:02 PM

	F15518641: D8584.id20421.diff
	Sat, Apr 19, 3:23 PM

	F15515083: D8584.id.diff
	Fri, Apr 18, 7:32 AM

	F15513521: D8584.diff
	Thu, Apr 17, 7:30 PM

	F15505567: D8584.id20421.diff
	Tue, Apr 15, 4:04 AM

	F15489558: D8584.id20362.diff
	Fri, Apr 11, 10:09 AM

	F15471731: D8584.id20421.diff
	Apr 5 2025, 7:49 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T1812: Allow for arbitrary values for task status

Commits

Restricted Diffusion Commit
rP7713fb5d99a3: Add config validation for task status config

Summary

Ref T1812. This still doesn't expose configuration to the user, but adds validation for it.

Test Plan

Added a pile of unit tests.

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20362.Mar 21 2014, 1:38 PM

epriestley retitled this revision from to Add config validation for task status config.

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, 1:38 PM

epriestley added inline comments.Mar 21 2014, 1:40 PM

src/applications/maniphest/constants/ManiphestTaskStatus.php
157	This relaxes things a little bit: if you configure "silly" statuses as special statuses, they'll still work even in serious business mode. This saves us from some awkward situations and more complex validation, and seems fundamentally reasonable.

btrahan accepted this revision.Mar 25 2014, 7:21 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 25 2014, 7:21 PM

Closed by commit rP7713fb5d99a3 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

maniphest/

constants/

ManiphestTaskStatus.php

137 lines

__tests__/

ManiphestTaskStatusTestCase.php

114 lines

Diff 20421

src/__phutil_library_map__.php

Loading...

src/applications/maniphest/constants/ManiphestTaskStatus.php

Loading...

src/applications/maniphest/constants/tests/ManiphestTaskStatusTestCase.php

Loading...