Add text description to the PermissionDenied exception raised by

the acls managers.

Signed-off-by: Roberto Rosario <roberto.rosario.gonzalez@gmail.com>
This commit is contained in:
Roberto Rosario
2017-08-26 02:54:43 -04:00
parent 5e2d66d201
commit 4dcaa5f883

View File

@@ -6,7 +6,7 @@ from django.contrib.contenttypes.models import ContentType
from django.core.exceptions import PermissionDenied
from django.db import models
from django.db.models import Q
from django.utils.translation import ugettext
from django.utils.translation import ugettext, ugettext_lazy as _
from common.utils import return_attrib
from permissions import Permission
@@ -54,7 +54,7 @@ class AccessControlListManager(models.Manager):
except AttributeError:
# AttributeError means non model objects: ie Statistics
# These can't have ACLs so we raise PermissionDenied
raise PermissionDenied
raise PermissionDenied(_('Insufficient access for: %s') % obj)
except KeyError:
pass
else:
@@ -83,7 +83,7 @@ class AccessControlListManager(models.Manager):
'Permissions "%s" on "%s" denied for user "%s"',
permissions, obj, user
)
raise PermissionDenied(ugettext('Insufficient access.'))
raise PermissionDenied(ugettext('Insufficient access for: %s') % obj)
logger.debug(
'Permissions "%s" on "%s" granted to user "%s" through roles "%s" by direct ACL',