misc: use is_authenticated as a straight boolean value (#41633)

This commit is contained in:
Frédéric Péters 2020-04-13 10:52:41 +02:00
parent 20499dc46e
commit 9236efe243
2 changed files with 3 additions and 3 deletions

View File

@ -41,7 +41,7 @@ from .visualization.utils import Visualization as VisuUtil
class AuthorizationMixin(object):
def authorize(self, request):
if request.user.is_authenticated():
if request.user.is_authenticated:
if not request.user.is_superuser:
raise PermissionDenied(_('You must be superuser'))
return True

View File

@ -184,7 +184,7 @@ class VisualizationView(views.AuthorizationMixin, CubeDisplayMixin, DetailView):
class SignatureAuthorizationMixin(views.AuthorizationMixin):
def authorize(self, request):
if request.user.is_authenticated() and request.user.is_superuser:
if request.user.is_authenticated and request.user.is_superuser:
return True
if 'signature' in request.GET:
signature = request.path + settings.SECRET_KEY
@ -230,7 +230,7 @@ class VisualizationsJSONView(MultipleObjectMixin, View):
model = models.Visualization
def get(self, request, *args, **kwargs):
if not request.user.is_authenticated() or not request.user.is_superuser:
if not request.user.is_authenticated or not request.user.is_superuser:
known_services = getattr(settings, 'KNOWN_SERVICES', [])
if known_services:
key = None