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
F18809839: D14689.id.diff
Oct 19 2025, 5:15 PM
F18762154: D14689.id.diff
Oct 6 2025, 5:46 PM
F18758463: D14689.diff
Oct 5 2025, 10:26 PM
F18745104: D14689.diff
Oct 3 2025, 5:20 AM
F18744718: D14689.id35526.diff
Oct 3 2025, 3:56 AM
F18506750: D14689.id.diff
Sep 5 2025, 1:45 AM
F18504633: D14689.diff
Sep 4 2025, 11:56 PM
F18499449: D14689.diff
Sep 4 2025, 7:46 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