Skip to content
Snippets Groups Projects
Verified Commit 0559b1b1 authored by Eliot Berriot's avatar Eliot Berriot
Browse files

Switched to is_authenticated (no parenthesis)

parent e9c3eb59
No related branches found
No related tags found
No related merge requests found
...@@ -7,5 +7,5 @@ class ConditionalAuthentication(BasePermission): ...@@ -7,5 +7,5 @@ class ConditionalAuthentication(BasePermission):
def has_permission(self, request, view): def has_permission(self, request, view):
if settings.API_AUTHENTICATION_REQUIRED: if settings.API_AUTHENTICATION_REQUIRED:
return request.user and request.user.is_authenticated() return request.user and request.user.is_authenticated
return True return True
...@@ -22,14 +22,14 @@ class ListeningViewSet(mixins.CreateModelMixin, ...@@ -22,14 +22,14 @@ class ListeningViewSet(mixins.CreateModelMixin,
def get_queryset(self): def get_queryset(self):
queryset = super().get_queryset() queryset = super().get_queryset()
if self.request.user.is_authenticated(): if self.request.user.is_authenticated:
return queryset.filter(user=self.request.user) return queryset.filter(user=self.request.user)
else: else:
return queryset.filter(session_key=self.request.session.session_key) return queryset.filter(session_key=self.request.session.session_key)
def get_serializer_context(self): def get_serializer_context(self):
context = super().get_serializer_context() context = super().get_serializer_context()
if self.request.user.is_authenticated(): if self.request.user.is_authenticated:
context['user'] = self.request.user context['user'] = self.request.user
else: else:
context['session_key'] = self.request.session.session_key context['session_key'] = self.request.session.session_key
......
...@@ -102,7 +102,7 @@ class TrackViewSet(TagViewSetMixin, SearchMixin, viewsets.ReadOnlyModelViewSet): ...@@ -102,7 +102,7 @@ class TrackViewSet(TagViewSetMixin, SearchMixin, viewsets.ReadOnlyModelViewSet):
queryset = super().get_queryset() queryset = super().get_queryset()
filter_favorites = self.request.GET.get('favorites', None) filter_favorites = self.request.GET.get('favorites', None)
user = self.request.user user = self.request.user
if user.is_authenticated() and filter_favorites == 'true': if user.is_authenticated and filter_favorites == 'true':
queryset = queryset.filter(track_favorites__user=user) queryset = queryset.filter(track_favorites__user=user)
return queryset return queryset
......
...@@ -19,14 +19,14 @@ class RadioSessionViewSet(mixins.CreateModelMixin, ...@@ -19,14 +19,14 @@ class RadioSessionViewSet(mixins.CreateModelMixin,
def get_queryset(self): def get_queryset(self):
queryset = super().get_queryset() queryset = super().get_queryset()
if self.request.user.is_authenticated(): if self.request.user.is_authenticated:
return queryset.filter(user=self.request.user) return queryset.filter(user=self.request.user)
else: else:
return queryset.filter(session_key=self.request.session.session_key) return queryset.filter(session_key=self.request.session.session_key)
def get_serializer_context(self): def get_serializer_context(self):
context = super().get_serializer_context() context = super().get_serializer_context()
if self.request.user.is_authenticated(): if self.request.user.is_authenticated:
context['user'] = self.request.user context['user'] = self.request.user
else: else:
context['session_key'] = self.request.session.session_key context['session_key'] = self.request.session.session_key
...@@ -44,7 +44,7 @@ class RadioSessionTrackViewSet(mixins.CreateModelMixin, ...@@ -44,7 +44,7 @@ class RadioSessionTrackViewSet(mixins.CreateModelMixin,
serializer.is_valid(raise_exception=True) serializer.is_valid(raise_exception=True)
session = serializer.validated_data['session'] session = serializer.validated_data['session']
try: try:
if request.user.is_authenticated(): if request.user.is_authenticated:
assert request.user == session.user assert request.user == session.user
else: else:
assert request.session.session_key == session.session_key assert request.session.session_key == session.session_key
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment