General A/B testing.

[wolnelektury.git] / src / catalogue / api / views.py

diff --git a/src/catalogue/api/views.py b/src/catalogue/api/views.py
index 5a064c1..34971c4 100644 (file)
--- a/src/catalogue/api/views.py
+++ b/src/catalogue/api/views.py
@@ -3,12 +3,15 @@
 #
 import json
 from django.http import Http404, HttpResponse
 #
 import json
 from django.http import Http404, HttpResponse

+from django.utils.decorators import method_decorator
+from django.views.decorators.cache import never_cache

 from rest_framework.generics import ListAPIView, RetrieveAPIView, get_object_or_404
 from rest_framework.permissions import DjangoModelPermissionsOrAnonReadOnly
 from rest_framework.response import Response
 from rest_framework import status
 from api.handlers import read_tags
 from api.utils import vary_on_auth
 from rest_framework.generics import ListAPIView, RetrieveAPIView, get_object_or_404
 from rest_framework.permissions import DjangoModelPermissionsOrAnonReadOnly
 from rest_framework.response import Response
 from rest_framework import status
 from api.handlers import read_tags
 from api.utils import vary_on_auth

+from club.models import Membership

 from .helpers import books_after, order_books
 from . import serializers
 from catalogue.forms import BookImportForm
 from .helpers import books_after, order_books
 from . import serializers
 from catalogue.forms import BookImportForm


@@ -69,6 +72,9 @@ class BookList(ListAPIView):
             books = Book.objects.all()
         books = order_books(books, new_api)
 
             books = Book.objects.all()
         books = order_books(books, new_api)
 

+        if not Membership.is_active_for(self.request.user):
+            books = books.exclude(preview=True)
+

         if self.kwargs.get('top_level'):
             books = books.filter(parent=None)
         if self.kwargs.get('audiobooks'):
         if self.kwargs.get('top_level'):
             books = books.filter(parent=None)
         if self.kwargs.get('audiobooks'):


@@ -140,15 +146,23 @@ class BookDetail(RetrieveAPIView):
     serializer_class = serializers.BookDetailSerializer
 
 
     serializer_class = serializers.BookDetailSerializer
 
 

+@vary_on_auth  # Because of embargo links.

 class EbookList(BookList):
     serializer_class = serializers.EbookSerializer
 
 
 class EbookList(BookList):
     serializer_class = serializers.EbookSerializer
 
 

-@vary_on_auth  # Because of 'liked'.
+@method_decorator(never_cache, name='dispatch')

 class Preview(ListAPIView):
 class Preview(ListAPIView):

-    queryset = Book.objects.filter(preview=True)
+    #queryset = Book.objects.filter(preview=True)

     serializer_class = serializers.BookPreviewSerializer
 
     serializer_class = serializers.BookPreviewSerializer
 

+    def get_queryset(self):
+        qs = Book.objects.filter(preview=True)
+        # FIXME: temporary workaround for a problem with iOS app; see #3954.
+        if 'Darwin' in self.request.META.get('HTTP_USER_AGENT', '') and 'debug' not in self.request.GET:
+            qs = qs.none()
+        return qs
+

 
 @vary_on_auth  # Because of 'liked'.
 class FilterBookList(ListAPIView):
 
 @vary_on_auth  # Because of 'liked'.
 class FilterBookList(ListAPIView):


@@ -166,6 +180,8 @@ class FilterBookList(ListAPIView):
         is_lektura = self.parse_bool(self.request.query_params.get('lektura'))
         is_audiobook = self.parse_bool(self.request.query_params.get('audiobook'))
         preview = self.parse_bool(self.request.query_params.get('preview'))
         is_lektura = self.parse_bool(self.request.query_params.get('lektura'))
         is_audiobook = self.parse_bool(self.request.query_params.get('audiobook'))
         preview = self.parse_bool(self.request.query_params.get('preview'))

+        if not Membership.is_active_for(self.request.user):
+            preview = False

 
         new_api = self.request.query_params.get('new_api')
         after = self.request.query_params.get('after')
 
         new_api = self.request.query_params.get('new_api')
         after = self.request.query_params.get('after')


@@ -227,6 +243,7 @@ class EpubView(RetrieveAPIView):
     lookup_field = 'slug'
     permission_classes = [IsClubMember]
 
     lookup_field = 'slug'
     permission_classes = [IsClubMember]
 

+    @method_decorator(never_cache)

     def get(self, *args, **kwargs):
         return HttpResponse(self.get_object().get_media('epub'))
 
     def get(self, *args, **kwargs):
         return HttpResponse(self.get_object().get_media('epub'))