Homec4science

Clean up "reorder" permissions in MenuEngine for personal favorites

Authored by epriestley <git@epriestley.com> on Jan 27 2017, 04:42.

Description

Clean up "reorder" permissions in MenuEngine for personal favorites

Summary:
Fixes T12159. This is similar to D17228, which fixed this for the main configuration operation.

Most other edit operations only test for edit capability on the MenuItem itself, which we already do correctly. However, because reordering affects all items, we test for capability on the object.

Weaken this when reordering custom items.

Test Plan: Reordered custom items in Favorites as a non-administrator.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12159

Differential Revision: https://secure.phabricator.com/D17257

Details

Committed
epriestley <git@epriestley.com>Jan 27 2017, 04:47
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbee043b16362: Display paste line count alongside snippets
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH9829ecddd695: Clean up "reorder" permissions in MenuEngine for personal favorites (authored by epriestley <git@epriestley.com>).Jan 27 2017, 04:47