Page MenuHomePhabricator

Don't allow forms which can't create objects to be added to profile menus
ClosedPublic

Authored by epriestley on Feb 16 2017, 11:17 PM.
Tags
None
Referenced Files
F19733154: D17372.id41774.diff
Sat, Feb 14, 9:32 PM
F19724071: D17372.diff
Fri, Feb 13, 12:49 PM
F19662463: D17372.id.diff
Feb 7 2026, 9:13 AM
F19607171: D17372.diff
Feb 3 2026, 10:37 AM
F19555251: D17372.id41774.diff
Jan 30 2026, 11:19 AM
F19303192: D17372.id.diff
Dec 23 2025, 8:14 PM
F19255515: D17372.diff
Dec 22 2025, 12:37 PM
F18826752: D17372.id41773.diff
Oct 24 2025, 5:33 AM
Subscribers
None

Details

Summary

Fixes T12281. Some forms (like Settings) can't actually create new objects. Currently, though, you can select them and add them to profile menus; if you do, they fail when building an item.

Kick them out of the typeahead, and decline to render them in menus.

Test Plan

Added "Create Settings" to a menu, no longer fatals after patch (item vanished from menu, still editable normally to get rid of it).

Tried to add another "Create Settings", no longer available in typehaead.

Added some normal stuff.

Viewed a choose-among-forms dropdown in Maniphest, which still worked normally.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable