wip: bs-194 - migrating book APIs

backend/book_api/urls/v1.py

@@ -1,18 +1,19 @@
-from django.urls import path
+# from django.urls import path
 from rest_framework.routers import DefaultRouter
 
 from book_api.views import v1
 
 
 router = DefaultRouter()
 
+router.register('', v1.BookViewSet, basename='book')
 router.register('genre', v1.GenreViewSet, basename='genre')
 router.register('topic', v1.TopicViewSet, basename='topic')
 
 urlpatterns = [
-    path('', v1.BookView.as_view()),
-    path('<int:book_code>/', v1.BookView.as_view()),
-    path('search', v1.BookSearchView.as_view()),
+    # path('', v1.BookView.as_view()),
+    # path('<int:book_code>/', v1.BookView.as_view()),
+    # path('search', v1.BookSearchView.as_view()),
 ]
 
 urlpatterns += router.urls

backend/book_api/views/v1/__init__.py

@@ -1,12 +1,14 @@
-from book_api.views.v1.book import BookView
+# from book_api.views.v1.book import BookView
+from book_api.views.v1.book import BookViewSet
 from book_api.views.v1.genre import GenreViewSet
 from book_api.views.v1.topic import TopicViewSet
-from book_api.views.v1.search import BookSearchView
+# from book_api.views.v1.search import BookSearchView
 
 
 __all__ = [
-    BookView,
+    # BookView,
+    BookViewSet,
     GenreViewSet,
     TopicViewSet,
-    BookSearchView,
+    # BookSearchView,
 ]

backend/book_api/views/v1/book.py

@@ -1,4 +1,5 @@
 from rest_framework.views import APIView
+from rest_framework.viewsets import ModelViewSet
 from rest_framework.response import Response
 from rest_framework.permissions import AllowAny
 from rest_framework.exceptions import NotFound
@@ -8,76 +9,89 @@
 
 from BookShelf.utilities.pagination import Pagination
 from BookShelf.utilities.permissions import IsAdminOrModerator
+from BookShelf.utilities.filters import SearchFilter
 from activity_api.utilities.event import event_logger
 
 from book_api.models import Book
 
 from book_api.serializers.v1 import BookSerializer
 
 
-@method_decorator(cache_page(60*1), name='get')
-class BookView(APIView):
-    permission_classes = [AllowAny]
-
-    def get(self, request, *args, **kwargs):
-        book_code = kwargs.get('book_code', None)
-        genre = request.query_params.get('genre', None)
-        topic = request.query_params.get('topic', None)
-
-        if book_code:
-            try:
-                book = Book.objects.get(book_code=book_code, is_deleted=False)
-            except Book.DoesNotExist:
-                raise NotFound(detail="Book not found.")
-
-            event_logger(
-                event='retrieve',
-                object='book',
-                user=request.user,
-                device=request.device,
-                ip_address=request.ip_address,
-                data={
-                    'model': 'Book',
-                    'id': book.id
-                }
-            )
-
-            return Response(BookSerializer(book).data)
-
-        books = Book.objects.filter(
-            is_deleted=False
-        )
-
-        if genre:
-            books = books.filter(genres__id=genre)
-
-        if topic:
-            books = books.filter(topics__id=topic)
-
-        books = books.order_by('-id')
-        paginator = Pagination()
-        page = paginator.paginate_queryset(books, request)
-
-        event_logger(
-            event='retrieve',
-            object='book',
-            user=request.user,
-            device=request.device,
-            ip_address=request.ip_address,
-        )
-
-        return paginator.get_paginated_response(
-            BookSerializer(page, many=True).data
-        )
-
-    def post(self, request, *args, **kwargs):
-        self.permission_classes = [IsAdminOrModerator]
-        self.check_permissions(request)
-        request.data['added_by'] = request.user.id
-        serializer = BookSerializer(
-            data=request.data,
-        )
-        serializer.is_valid(raise_exception=True)
-        serializer.save()
-
-        return Response(serializer.data, status=201)
+class BookViewSet(ModelViewSet):
+    serializer_class = BookSerializer
+    queryset = Book.objects.filter(is_deleted=False)
+    permission_classes = [
+        IsAdminOrModerator,
+    ]
+    filter_backends = [SearchFilter]
+    search_fields = ['title']
+
+
+
+
+# @method_decorator(cache_page(60*1), name='get')
+# class BookView(APIView):
+#     permission_classes = [AllowAny]
+
+#     def get(self, request, *args, **kwargs):
+#         book_code = kwargs.get('book_code', None)
+#         genre = request.query_params.get('genre', None)
+#         topic = request.query_params.get('topic', None)
+
+#         if book_code:
+#             try:
+#                 book = Book.objects.get(book_code=book_code, is_deleted=False)
+#             except Book.DoesNotExist:
+#                 raise NotFound(detail="Book not found.")
+
+#             event_logger(
+#                 event='retrieve',
+#                 object='book',
+#                 user=request.user,
+#                 device=request.device,
+#                 ip_address=request.ip_address,
+#                 data={
+#                     'model': 'Book',
+#                     'id': book.id
+#                 }
+#             )
+
+#             return Response(BookSerializer(book).data)
+
+#         books = Book.objects.filter(
+#             is_deleted=False
+#         )
+
+#         if genre:
+#             books = books.filter(genres__id=genre)
+
+#         if topic:
+#             books = books.filter(topics__id=topic)
+
+#         books = books.order_by('-id')
+#         paginator = Pagination()
+#         page = paginator.paginate_queryset(books, request)
+
+#         event_logger(
+#             event='retrieve',
+#             object='book',
+#             user=request.user,
+#             device=request.device,
+#             ip_address=request.ip_address,
+#         )
+
+#         return paginator.get_paginated_response(
+#             BookSerializer(page, many=True).data
+#         )
+
+#     def post(self, request, *args, **kwargs):
+#         self.permission_classes = [IsAdminOrModerator]
+#         self.check_permissions(request)
+#         request.data['added_by'] = request.user.id
+#         serializer = BookSerializer(
+#             data=request.data,
+#         )
+#         serializer.is_valid(raise_exception=True)
+#         serializer.save()
+
+#         return Response(serializer.data, status=201)