Page MenuHomePhabricator
Differential D18159

Convert 'class' config options to new validation
ClosedPublic
Actions

Authored by epriestley on Jun 26 2017, 5:15 PM.
Tags
None
Referenced Files
F18903222: D18159.diff
Sat, Nov 8, 4:28 AM
F18817553: D18159.id.diff
Oct 21 2025, 4:28 PM
F18809653: D18159.id.diff
Oct 19 2025, 3:51 PM
F18805834: D18159.diff
Oct 18 2025, 4:43 PM
F18772166: D18159.id43687.diff
Oct 9 2025, 12:45 AM
F18758769: D18159.id43706.diff
Oct 5 2025, 11:32 PM
F18659267: D18159.diff
Sep 23 2025, 10:46 AM
F18647773: D18159.diff
Sep 19 2025, 2:53 PM
View All Files
Subscribers
None

Details

Reviewers
chad
amckinley
Maniphest Tasks
T12845: Convert remaining ancient configuration type classes to modern modular type classes
Commits
rP0afdabff00ae: Convert 'class' config options to new validation
Summary

Ref T12845. These options prompt the user to select from among concrete subclasses of some base class.

Test Plan

Set, deleted and mangled these values from the web UI and CLI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 26 2017, 5:15 PM
Harbormaster completed remote builds in B17578: Diff 43687.Jun 26 2017, 5:16 PM
amckinley accepted this revision.Jun 26 2017, 6:14 PM
This revision is now accepted and ready to land.Jun 26 2017, 6:14 PM
Closed by commit rP0afdabff00ae: Convert 'class' config options to new validation (authored by epriestley, committed by epriestley). · Explain WhyJun 27 2017, 7:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 43706

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/config/controller/PhabricatorConfigEditController.php

Loading...
View Options

src/applications/config/management/PhabricatorConfigManagementSetWorkflow.php

Loading...
View Options

src/applications/config/option/PhabricatorApplicationConfigOptions.php

Loading...
View Options

src/applications/config/type/PhabricatorClassConfigType.php

Loading...
Phabricator · Built by Phacility