Page MenuHomePhabricator
Differential D14689

Validate configuration of `maniphest.priorities`
ClosedPublic
Actions

Authored by epriestley on Dec 6 2015, 12:52 AM.
Tags
None
Referenced Files
F19872772: D14689.diff
Mar 15 2026, 1:54 PM
F19832592: D14689.id35526.diff
Mar 10 2026, 11:36 PM
F19813702: D14689.diff
Mar 4 2026, 10:36 AM
F19767443: D14689.diff
Feb 19 2026, 4:10 PM
F19759304: D14689.id35527.diff
Feb 18 2026, 3:31 AM
F19725717: D14689.id35526.diff
Feb 13 2026, 4:32 PM
F19612348: D14689.id35526.diff
Feb 3 2026, 7:06 PM
F18809839: D14689.id.diff
Oct 19 2025, 5:15 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T6132: Negative Maniphest priorities can be created, but not used
Commits
Restricted Diffusion Commit
rP20e6a4200dbc: Validate configuration of `maniphest.priorities`
Summary

Fixes T6132. We currently allow invalid configuration here; validate it.

Test Plan

Tried to save invalid config (negative priorities, string priorities, etc).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 35526.Dec 6 2015, 12:52 AM
epriestley retitled this revision from to Validate configuration of `maniphest.priorities`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T6132: Negative Maniphest priorities can be created, but not used.
chad accepted this revision.Dec 6 2015, 1:01 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 6 2015, 1:01 AM
Closed by commit rP20e6a4200dbc: Validate configuration of `maniphest.priorities` (authored by epriestley, committed by epriestley). · Explain WhyDec 6 2015, 1:11 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 35527

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/maniphest/config/ManiphestPriorityConfigOptionType.php

Loading...
View Options

src/applications/maniphest/config/PhabricatorManiphestConfigOptions.php

Loading...
View Options

src/applications/maniphest/constants/ManiphestTaskPriority.php

Loading...
Phabricator · Built by Phacility