Page MenuHomePhabricator
Differential D18041

Detect and prevent invalid configuation of "ui.footer-items"
ClosedPublic
Actions

Authored by epriestley on May 30 2017, 4:32 PM.
Tags
None
Referenced Files
F18708612: D18041.diff
Sun, Sep 28, 10:17 PM
F18699741: D18041.id.diff
Sat, Sep 27, 1:18 PM
F18684785: D18041.diff
Fri, Sep 26, 9:58 AM
F18645056: D18041.id43394.diff
Fri, Sep 19, 6:54 AM
F18638781: D18041.diff
Wed, Sep 17, 11:28 AM
F18638778: D18041.diff
Wed, Sep 17, 11:27 AM
F18619586: D18041.diff
Mon, Sep 15, 2:50 AM
F18596962: D18041.diff
Sat, Sep 13, 1:52 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12775: InvalidArgumentException in PhabricatorStandardPageView after setting unexpected JSON in ui.footer-items
Commits
rPc4e45c6c8c2e: Detect and prevent invalid configuation of "ui.footer-items"
Summary

Fixes T12775. Currently, we do not validate this option and it's possible to configure it in an invalid way.

Test Plan

Tried to misconfigure things, was helpfully pointed toward errors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 30 2017, 4:32 PM
Harbormaster completed remote builds in B17307: Diff 43394.May 30 2017, 4:33 PM
chad accepted this revision.May 30 2017, 4:38 PM
This revision is now accepted and ready to land.May 30 2017, 4:38 PM
Closed by commit rPc4e45c6c8c2e: Detect and prevent invalid configuation of "ui.footer-items" (authored by epriestley, committed by epriestley). · Explain WhyMay 30 2017, 5:15 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
config/
custom/
41 lines
option/
3 lines

Diff 43395

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/config/custom/PhabricatorCustomUIFooterConfigType.php

Loading...
View Options

src/applications/config/option/PhabricatorUIConfigOptions.php

Loading...
Phabricator · Built by Phacility