Skip to content

Commit

Permalink
minor #4015 [Doc] clarify the purpose of the menu item permission (ja…
Browse files Browse the repository at this point in the history
…viereguiluz)

This PR was merged into the 3.0.x-dev branch.

Discussion
----------

[Doc] clarify the purpose of the menu item permission

Fixes #4013.

Commits
-------

463ae1b [Doc] clarify the purpose of the menu item permission
  • Loading branch information
javiereguiluz committed Dec 5, 2020
2 parents 81fee76 + 463ae1b commit 82f49bc
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions doc/security.rst
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,13 @@ user must have in order to see the menu item::
];
}

.. note::

This permission only shows/hides menu items. The actions associated to those
menu items are still executable, even if the user can't see the menu items.
Use the :ref:`actions permissions <security-permissions-actions>` to also
restrict the access to those actions.

If your needs are more advanced, remember that the dashboard class is a regular
Symfony controller, so you can use any service related to security to evaluate
complex expressions. In those cases, it's more convenient to use the alternative
Expand All @@ -96,6 +103,8 @@ menu item definition to not have to deal with array merges::
// ...
}

.. _security-permissions-actions:

Restrict Access to Actions
--------------------------

Expand Down

0 comments on commit 82f49bc

Please sign in to comment.