imp: Hide the organization submenu if user can only see tickets

Probesys · Dec 10, 2024 · 7699a93 · 7699a93
1 parent bf1b2c5
commit 7699a93
Showing 1 changed file with 75 additions and 67 deletions.
diff --git a/templates/organization.html.twig b/templates/organization.html.twig
@@ -9,87 +9,95 @@
 {% set currentPage = 'organizations' %}
 
 {% block submenu %}
-    <nav class="layout__submenu submenu" data-controller="submenu" aria-labelledby="submenu-title">
-        <div class="cols cols--always cols--center">
-            <div class="submenu__title col--extend" id="submenu-title">
-                {{ organization.name }}
-            </div>
-
-            <div class="submenu__scroller only-mobile">
-                <button
-                    class="button--icon button--discreet"
-                    data-submenu-target="buttonLeft"
-                    data-action="submenu#scrollLeft"
-                >
-                    {{ icon('angle-down', 'icon--rotate90') }}
+    {% set canSeeContracts = is_granted('orga:see:contracts', organization) %}
+    {% set canSeeUsers = is_granted('orga:see:users', organization) %}
+    {% set canManage = is_granted('orga:manage', organization) %}
 
-                    <span class="sr-only">
-                        {{ 'submenu.scroller.scroll_to_left' | trans }}
-                    </span>
-                </button>
+    {% set canSeeSubMenu = canSeeContracts or canSeeUsers or canManage %}
 
-                <button
-                    class="button--icon button--discreet"
-                    data-submenu-target="buttonRight"
-                    data-action="submenu#scrollRight"
-                >
-                    {{ icon('angle-down', 'icon--rotate270') }}
+    {% if canSeeSubMenu %}
+        <nav class="layout__submenu submenu" data-controller="submenu" aria-labelledby="submenu-title">
+            <div class="cols cols--always cols--center">
+                <div class="submenu__title col--extend" id="submenu-title">
+                    {{ organization.name }}
+                </div>
 
-                    <span class="sr-only">
-                        {{ 'submenu.scroller.scroll_to_right' | trans }}
-                    </span>
-                </button>
-            </div>
-        </div>
+                <div class="submenu__scroller only-mobile">
+                    <button
+                        class="button--icon button--discreet"
+                        data-submenu-target="buttonLeft"
+                        data-action="submenu#scrollLeft"
+                    >
+                        {{ icon('angle-down', 'icon--rotate90') }}
 
-        <ul class="submenu__menu list--nostyle" data-submenu-target="menu">
-            <li class="submenu__item">
-                <a
-                    class="submenu__anchor"
-                    href="{{ path('organization tickets', { uid: organization.uid }) }}"
-                    {{ currentMenu == 'tickets' ? 'aria-current="page"' }}
-                >
-                    {{ 'tickets.index.title' | trans }}
-                </a>
-            </li>
+                        <span class="sr-only">
+                            {{ 'submenu.scroller.scroll_to_left' | trans }}
+                        </span>
+                    </button>
 
-            {% if is_granted('orga:see:contracts', organization) %}
-                <li class="submenu__item">
-                    <a
-                        class="submenu__anchor"
-                        href="{{ path('organization contracts', { uid: organization.uid }) }}"
-                        {{ currentMenu == 'contracts' ? 'aria-current="page"' }}
+                    <button
+                        class="button--icon button--discreet"
+                        data-submenu-target="buttonRight"
+                        data-action="submenu#scrollRight"
                     >
-                        {{ 'contracts.index.title' | trans }}
-                    </a>
-                </li>
-            {% endif %}
+                        {{ icon('angle-down', 'icon--rotate270') }}
 
-            {% if is_granted('orga:see:users', organization) %}
-                <li class="submenu__item">
-                    <a
-                        class="submenu__anchor"
-                        href="{{ path('organization users', { uid: organization.uid }) }}"
-                        {{ currentMenu == 'users' ? 'aria-current="page"' }}
-                    >
-                        {{ 'users.index.title' | trans }}
-                    </a>
-                </li>
-            {% endif %}
+                        <span class="sr-only">
+                            {{ 'submenu.scroller.scroll_to_right' | trans }}
+                        </span>
+                    </button>
+                </div>
+            </div>
 
-            {% if is_granted('orga:manage', organization) %}
+            <ul class="submenu__menu list--nostyle" data-submenu-target="menu">
                 <li class="submenu__item">
                     <a
                         class="submenu__anchor"
-                        href="{{ path('organization settings', { uid: organization.uid }) }}"
-                        {{ currentMenu == 'settings' ? 'aria-current="page"' }}
+                        href="{{ path('organization tickets', { uid: organization.uid }) }}"
+                        {{ currentMenu == 'tickets' ? 'aria-current="page"' }}
                     >
-                        {{ 'organizations.settings.title' | trans }}
+                        {{ 'tickets.index.title' | trans }}
                     </a>
                 </li>
-            {% endif %}
-        </ul>
-    </nav>
+
+                {% if canSeeContracts %}
+                    <li class="submenu__item">
+                        <a
+                            class="submenu__anchor"
+                            href="{{ path('organization contracts', { uid: organization.uid }) }}"
+                            {{ currentMenu == 'contracts' ? 'aria-current="page"' }}
+                        >
+                            {{ 'contracts.index.title' | trans }}
+                        </a>
+                    </li>
+                {% endif %}
+
+                {% if canSeeUsers %}
+                    <li class="submenu__item">
+                        <a
+                            class="submenu__anchor"
+                            href="{{ path('organization users', { uid: organization.uid }) }}"
+                            {{ currentMenu == 'users' ? 'aria-current="page"' }}
+                        >
+                            {{ 'users.index.title' | trans }}
+                        </a>
+                    </li>
+                {% endif %}
+
+                {% if canManage %}
+                    <li class="submenu__item">
+                        <a
+                            class="submenu__anchor"
+                            href="{{ path('organization settings', { uid: organization.uid }) }}"
+                            {{ currentMenu == 'settings' ? 'aria-current="page"' }}
+                        >
+                            {{ 'organizations.settings.title' | trans }}
+                        </a>
+                    </li>
+                {% endif %}
+            </ul>
+        </nav>
+    {% endif %}
 
     {% if currentMenu == 'tickets' and view is defined and is_agent(organization) %}
         {{ include('tickets/_menu.html.twig', {