ListUser: Remove "Détails" column if the doesn't have retrieve permis…

…sion
borgia-app · Mar 5, 2018 · 8ce546e · 8ce546e
1 parent e16fd8d
commit 8ce546e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/users/templates/users/user_list.html b/users/templates/users/user_list.html
@@ -45,7 +45,9 @@
           </a></th>
         {% endfor %}
           <th>Etat</th>
-          <th>Détail</th>
+          {% if has_perm_retrieve_user %}
+            <th>Détail</th>
+          {% endif %}
       </tr>
     </thead>
     <tbody>
@@ -59,9 +61,11 @@
         <td>{{ user.year }}</td>
         <td>{{ user.balance }}€</td>
         <td>{% if user.is_active %}Activé{% else %}Désactivé{% endif %}</td>
-        <td>
-          <a href="{% url 'url_user_retrieve' group_name=group_name pk=user.pk %}">Détail</a>
-        </td>
+        {% if has_perm_retrieve_user %}
+          <td>
+            <a href="{% url 'url_user_retrieve' group_name=group_name pk=user.pk %}">Détail</a>
+          </td>
+        {% endif %}
       </tr>
       {% endfor %}
     </tbody>

diff --git a/users/views.py b/users/views.py
@@ -371,7 +371,9 @@ def get_context_data(self, **kwargs):
             context['user_list'] = self.form_query(
               User.objects.all().exclude(groups=1))
 
-
+        # Permission Retrieveuser
+        if Permission.objects.get(codename='retrieve_user') in self.group.permissions.all():
+            context['has_perm_retrieve_user'] = True
 
         return context