Gestion de la page des logs avec has_perm

logs/acl.py

@@ -35,5 +35,5 @@ def can_view(user):
         A couple (allowed, msg) where allowed is a boolean which is True if
         viewing is granted and msg is a message (can be None).
     """
-    can = user.has_module_perms('logs')
+    can = user.has_module_perms('admin')
     return can, None if can else "Vous ne pouvez pas voir cette application."

logs/views.py

@@ -408,7 +408,7 @@ def stats_users(request):
                 num=Count('whitelist')
             ).order_by('-num')[:10],
             'Droits': User.objects.annotate(
-                num=Count('right')
+                num=Count('groups')
             ).order_by('-num')[:10],
         },
         'Etablissement': {

re2o/acl.py

@@ -222,7 +222,7 @@ def can_view_app(app_name):
 def can_edit_history(view):
     """Decorator to check if an user can edit history."""
     def wrapper(request, *args, **kwargs):
-        if request.user.has_perms(('admin',)):
+        if request.user.has_perm('admin.change_logentry'):
             return view(request, *args, **kwargs)
         messages.error(
            request,

re2o/templatetags/acl.py

@@ -183,9 +183,9 @@ def get_callback(tag_name, obj=None):
     if tag_name == 'cannot_view_app':
         return acl_fct(sys.modules[obj].can_view, True)
     if tag_name == 'can_edit_history':
-        return acl_fct(lambda user:(user.has_perms(('admin',)),None),False)
+        return acl_fct(lambda user:(user.has_perm('admin.change_logentry'),None),False)
     if tag_name == 'cannot_edit_history':
-        return acl_fct(lambda user:(user.has_perms(('admin',)),None),True)
+        return acl_fct(lambda user:(user.has_perm('admin.change_logentry'),None),True)
 
     raise template.TemplateSyntaxError(
         "%r tag is not a valid can_xxx tag" % tag_name

users/migrations/0066_grouppermissions.py

@@ -68,7 +68,8 @@ class Migration(migrations.Migration):
           'change_whitelist',
           'delete_whitelist',
           'view_whitelist'],
-         'cableur': ['view_article',
+         'cableur': ['add_logentry',
+          'view_article',
           'add_banque',
           'change_banque',
           'delete_banque',
@@ -185,7 +186,10 @@ class Migration(migrations.Migration):
           'change_vente',
           'delete_vente',
           'view_vente'],
-        'admin': ['add_assooption',
+        'admin': ['add_logentry',
+          'change_logentry',
+          'delete_logentry',
+          'add_assooption',
           'change_assooption',
           'delete_assooption',
           'view_assooption',