Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UI] Displaying elements based on user permissions in Twig #212

Open
wants to merge 10 commits into
base: 5.x
Choose a base branch
from
Open
101 changes: 101 additions & 0 deletions docs/design/displaying_elements_based_on_user_permissions.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,101 @@
Displaying elements based on User permissions
=============================================

In Mautic, it's possible to control the visibility of elements on the user interface based on the User's permissions. This allows for showing or hiding certain features, links, or sections depending on the User's Role and the permissions associated with that Role.

Check warning on line 4 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Mautic.FeatureList] Is this referring to a Mautic feature? If so, use 'User' instead of 'user'. Raw Output: {"message": "[Mautic.FeatureList] Is this referring to a Mautic feature? If so, use 'User' instead of 'user'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 4, "column": 71}}}, "severity": "INFO"}

This approach enhances security and provides a tailored experience for each User based on their role and access level.

Check warning on line 6 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Mautic.FeatureList] Is this referring to a Mautic feature? If so, use 'Role' instead of 'role'. Raw Output: {"message": "[Mautic.FeatureList] Is this referring to a Mautic feature? If so, use 'Role' instead of 'role'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 6, "column": 97}}}, "severity": "INFO"}

Using the ``securityIsGranted`` function
------------------------------------

To display elements conditionally based on User permissions, use the ``securityIsGranted`` function in Twig templates. The ``securityIsGranted`` function checks if the current User has the specified permission and returns a boolean value indicating whether the permission is granted or not.

Check warning on line 11 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.Passive] In general, use active voice instead of passive voice ('is granted'). Raw Output: {"message": "[Google.Passive] In general, use active voice instead of passive voice ('is granted').", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 11, "column": 273}}}, "severity": "INFO"}

Here's the basic syntax:

.. code-block:: twig

{% if securityIsGranted('permission:string') %}
<!-- Content to display if the user has the specified permission -->
{% endif %}

In this structure, ``permission:string`` represents the specific permission being checked for. Mautic uses a hierarchical permission system, in the format of ``bundle:level:permission``.

Check warning on line 21 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.Passive] In general, use active voice instead of passive voice ('being checked'). Raw Output: {"message": "[Google.Passive] In general, use active voice instead of passive voice ('being checked').", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 21, "column": 77}}}, "severity": "INFO"}

Displaying a User invitation link as example
--------------------------------------------

Let's examine a practical example of how to use this function to display a link for inviting new Users to the platform. This link should only be visible to Users who have the permission to create new User accounts.

Check warning on line 26 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.We] Try to avoid using first-person plural like 'Let's'. Raw Output: {"message": "[Google.We] Try to avoid using first-person plural like 'Let's'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 26, "column": 3}}}, "severity": "WARNING"}

In this example, we're using the ``securityIsGranted`` function to check if the current User has the ``user:users:create`` permission. This permission string is structured to indicate that we're checking for the ability to create new Users within the User management system.

Check warning on line 28 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.We] Try to avoid using first-person plural like 'we'. Raw Output: {"message": "[Google.We] Try to avoid using first-person plural like 'we'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 28, "column": 18}}}, "severity": "WARNING"}

Check warning on line 28 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.WordList] Use 'select' instead of 'check'. Raw Output: {"message": "[Google.WordList] Use 'select' instead of 'check'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 28, "column": 68}}}, "severity": "WARNING"}

Check warning on line 28 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.Passive] In general, use active voice instead of passive voice ('is structured'). Raw Output: {"message": "[Google.Passive] In general, use active voice instead of passive voice ('is structured').", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 28, "column": 159}}}, "severity": "INFO"}

Check warning on line 28 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.We] Try to avoid using first-person plural like 'we'. Raw Output: {"message": "[Google.We] Try to avoid using first-person plural like 'we'.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 28, "column": 190}}}, "severity": "WARNING"}

.. code-block:: twig

{% if securityIsGranted('user:users:create') %}
<li>
<a href="{{ path('mautic_user_action', {objectAction: 'new'}) }}">
<i class="ri-team-line"></i>
<span>{{ 'mautic.user.profile.invite'|trans }}</span>
</a>
</li>
{% endif %}

If the current User has the ``user:users:create`` permission, the code inside the if block is rendered, displaying the link to invite new users. The path function creates the link, which generates a URL based on the specified route (``mautic_user_action```) and any additional parameters (``{objectAction: 'new'}``).

Check warning on line 41 in docs/design/displaying_elements_based_on_user_permissions.rst

View workflow job for this annotation

GitHub Actions / prose

[vale] reported by reviewdog 🐶 [Google.Parens] Use parentheses judiciously. Raw Output: {"message": "[Google.Parens] Use parentheses judiciously.", "location": {"path": "docs/design/displaying_elements_based_on_user_permissions.rst", "range": {"start": {"line": 41, "column": 1}}}, "severity": "INFO"}

The ``'mautic.user.profile.invite'|trans`` expression is used to translate the text 'Invite your team' using Mautic's translation system. This ensures that the text is displayed in the appropriate language based on the user's locale settings.

This not only prevents unauthorized access but also keeps the interface clean and relevant for each user's role.

When implementing permission-based displays, it's essential to also secure the backend routes and actions that these interface elements might trigger. The frontend permission check should be considered an additional layer of security and user experience enhancement, not the sole method of access control.

Locating defined permissions
----------------------------

Mautic organizes its permissions on a per-bundle basis. Each bundle typically defines its own set of permissions in a dedicated PHP file. The standard location for these permission definitions is:

``[BundleName]/Security/[BundleName]Permissions.php``

For example:

- User permissions: ``UserBundle/Security/UserPermissions.php``
- Email permissions: ``EmailBundle/Security/EmailPermissions.php``
- SMS permissions: ``SmsBundle/Security/SmsPermissions.php``

These PHP files contain classes that extend ``AbstractPermissions`` and define the specific permissions available for that bundle. They usually include methods for building the permission matrix and checking individual permissions.

Examining permission files
--------------------------

When opening one of these permission files, they'll typically find:

- A ``definePermissions`` method that outlines all available permissions for the bundle.
- Constants defining permission levels (for example, ``LEVEL_VIEW, LEVEL_EDIT, LEVEL_FULL``).
- Methods for checking specific permissions (for example, ``canViewUsers``, ``canEditEmails``).

For example, in the ``UserPermissions.php`` file, the ``UserPermissions`` class defines the available permissions for the ``UserBundle`` using a more structured approach. Let's go through the important parts:

.. code-block:: php

$this->permissions = [
'profile' => [
'editusername' => 1,
'editemail' => 2,
'editposition' => 4,
'editname' => 8,
'full' => 1024,
],
];

In this example, the profile key represents the permission category, and the nested array defines the specific permission levels for actions like editing the username, email, position, name, and having full access to the user profile.

To use these permission keys with the ``securityIsGranted`` function in Twig templates, construct the appropriate permission string. The permission string follows the format: ``[bundle]:[level]:[permission]``.

Map the permission keys from the UserPermissions class to the corresponding permission strings:

- ``editusername`` => ``user:profile:editusername``
- ``editemail`` => ``user:profile:editemail``
- ``editposition`` => ``user:profile:editposition``
- ``editname`` => ``user:profile:editname``
- ``full`` => ``user:profile:full``

In each if statement, the ``securityIsGranted`` function is used with the corresponding permission string. If the current user has the specified permission, the code inside the if block will be executed, displaying the relevant form fields for editing the user profile information.

For more information, refer to the Security documentation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we link to the relevant page/resource here?

Loading