From fe43d53aa234552b1614c83d5cadafcd734fa735 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Tue, 16 Jun 2015 04:13:01 -0400 Subject: [PATCH] Use tuples instead of lists to represent single permission collections --- mayan/apps/common/mixins.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/mayan/apps/common/mixins.py b/mayan/apps/common/mixins.py index 58559ddf05..3d33520837 100644 --- a/mayan/apps/common/mixins.py +++ b/mayan/apps/common/mixins.py @@ -29,7 +29,7 @@ class ObjectListPermissionFilterMixin(object): if self.object_permission: try: # Check to see if the user has the permissions globally - Permission.objects.check_permissions(self.request.user, [self.object_permission]) + Permission.objects.check_permissions(self.request.user, (self.object_permission,)) except PermissionDenied: # No global permission, filter ther queryset per object + permission return AccessEntry.objects.filter_objects_by_access(self.object_permission, self.request.user, queryset) @@ -47,7 +47,7 @@ class ObjectPermissionCheckMixin(object): if self.object_permission: try: - Permission.objects.check_permissions(request.user, [self.object_permission]) + Permission.objects.check_permissions(request.user, (self.object_permission,)) except PermissionDenied: AccessEntry.objects.check_access(self.object_permission, request.user, self.get_object()) @@ -80,6 +80,6 @@ class ViewPermissionCheckMixin(object): def dispatch(self, request, *args, **kwargs): if self.view_permission: - Permission.objects.check_permissions(self.request.user, [self.view_permission]) + Permission.objects.check_permissions(self.request.user, (self.view_permission,)) return super(ViewPermissionCheckMixin, self).dispatch(request, *args, **kwargs)