diff --git a/cinema/migrations/0001_initial.py b/cinema/migrations/0001_initial.py index 514947a64..bbd827568 100644 --- a/cinema/migrations/0001_initial.py +++ b/cinema/migrations/0001_initial.py @@ -1,4 +1,4 @@ -# Generated by Django 4.0.2 on 2022-05-03 13:41 +# Generated by Django 4.1 on 2024-09-27 14:18 from django.db import migrations, models @@ -7,24 +7,43 @@ class Migration(migrations.Migration): initial = True - dependencies = [] + dependencies = [ + ] operations = [ migrations.CreateModel( - name="Movie", + name='Actor', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('first_name', models.CharField(max_length=255)), + ('last_name', models.CharField(max_length=255)), + ], + ), + migrations.CreateModel( + name='CinemaHall', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=255)), + ('rows', models.IntegerField()), + ('seats_in_row', models.IntegerField()), + ], + ), + migrations.CreateModel( + name='Genre', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=255, unique=True)), + ], + ), + migrations.CreateModel( + name='Movie', fields=[ - ( - "id", - models.BigAutoField( - auto_created=True, - primary_key=True, - serialize=False, - verbose_name="ID", - ), - ), - ("title", models.CharField(max_length=255)), - ("description", models.TextField()), - ("duration", models.IntegerField()), + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('title', models.CharField(max_length=255)), + ('description', models.TextField()), + ('duration', models.IntegerField()), + ('actors', models.ManyToManyField(related_name='movies', to='cinema.actor')), + ('genres', models.ManyToManyField(related_name='movies', to='cinema.genre')), ], ), ] diff --git a/cinema/models.py b/cinema/models.py index cc477513f..0236176e1 100644 --- a/cinema/models.py +++ b/cinema/models.py @@ -1,10 +1,27 @@ from django.db import models +class Actor(models.Model): + first_name = models.CharField(max_length=255) + last_name = models.CharField(max_length=255) + + +class Genre(models.Model): + name = models.CharField(max_length=255, unique=True) + + +class CinemaHall(models.Model): + name = models.CharField(max_length=255) + rows = models.IntegerField() + seats_in_row = models.IntegerField() + + class Movie(models.Model): title = models.CharField(max_length=255) description = models.TextField() duration = models.IntegerField() + actors = models.ManyToManyField(Actor, related_name="movies") + genres = models.ManyToManyField(Genre, related_name="movies") def __str__(self): return self.title diff --git a/cinema/serializers.py b/cinema/serializers.py index 050db5771..31cccccb9 100644 --- a/cinema/serializers.py +++ b/cinema/serializers.py @@ -1,24 +1,39 @@ from rest_framework import serializers -from cinema.models import Movie +from cinema.models import Movie, Genre, CinemaHall, Actor -class MovieSerializer(serializers.Serializer): - id = serializers.IntegerField(read_only=True) - title = serializers.CharField(max_length=255) - description = serializers.CharField() - duration = serializers.IntegerField() +class MovieSerializer(serializers.ModelSerializer): + genres = serializers.SerializerMethodField() + actors = serializers.SerializerMethodField() - def create(self, validated_data): - return Movie.objects.create(**validated_data) + class Meta: + model = Movie + fields = "__all__" - def update(self, instance, validated_data): - instance.title = validated_data.get("title", instance.title) - instance.description = validated_data.get( - "description", instance.description - ) - instance.duration = validated_data.get("duration", instance.duration) + def get_genres(self, movie: Movie) -> list[str]: + return [genre.name for genre in movie.genres.all()] - instance.save() + def get_actors(self, movie: Movie) -> list[str]: + return [ + f"{actor.first_name} {actor.last_name}" + for actor in movie.actors.all() + ] - return instance + +class GenreSerializer(serializers.ModelSerializer): + class Meta: + model = Genre + fields = "__all__" + + +class ActorSerializer(serializers.ModelSerializer): + class Meta: + model = Actor + fields = "__all__" + + +class CinemaHallSerializer(serializers.ModelSerializer): + class Meta: + model = CinemaHall + fields = "__all__" diff --git a/cinema/urls.py b/cinema/urls.py index 1ae7d5cb0..4e34a0def 100644 --- a/cinema/urls.py +++ b/cinema/urls.py @@ -1,10 +1,24 @@ -from django.urls import path +from django.urls import path, include +from rest_framework import routers -from cinema.views import movie_list, movie_detail +from cinema.views import ( + ActorList, + ActorDetail, + GenreDetail, + GenreList, + CinemaHallViewSet, + MovieViewSet, +) +router = routers.DefaultRouter() +router.register("movies", MovieViewSet) +router.register("cinema_halls", CinemaHallViewSet) urlpatterns = [ - path("movies/", movie_list, name="movie-list"), - path("movies//", movie_detail, name="movie-detail"), + path("", include(router.urls)), + path("actors/", ActorList.as_view(), name="actor-list"), + path("actors//", ActorDetail.as_view(), name="actor-detail"), + path("genres/", GenreList.as_view(), name="genre-list"), + path("genres//", GenreDetail.as_view(), name="genre-detail"), ] app_name = "cinema" diff --git a/cinema/views.py b/cinema/views.py index 78ba8a79c..ccbf54c2e 100644 --- a/cinema/views.py +++ b/cinema/views.py @@ -1,45 +1,109 @@ -from rest_framework.decorators import api_view +from rest_framework import status, generics, mixins, viewsets +from rest_framework.generics import get_object_or_404 +from rest_framework.views import APIView from rest_framework.response import Response -from rest_framework import status +from django.http import HttpRequest -from django.shortcuts import get_object_or_404 +from cinema.models import Movie, Actor, Genre, CinemaHall +from cinema.serializers import ( + MovieSerializer, + ActorSerializer, + GenreSerializer, + CinemaHallSerializer +) -from cinema.models import Movie -from cinema.serializers import MovieSerializer +class MovieViewSet(viewsets.ModelViewSet): + queryset = Movie.objects.prefetch_related("genres", "actors") + serializer_class = MovieSerializer -@api_view(["GET", "POST"]) -def movie_list(request): - if request.method == "GET": - movies = Movie.objects.all() - serializer = MovieSerializer(movies, many=True) - return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "POST": - serializer = MovieSerializer(data=request.data) - if serializer.is_valid(): - serializer.save() - return Response(serializer.data, status=status.HTTP_201_CREATED) +class ActorList( + mixins.ListModelMixin, + mixins.CreateModelMixin, + generics.GenericAPIView +): + queryset = Actor.objects.all() + serializer_class = ActorSerializer + + def get(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.list(request, *args, **kwargs) + + def post(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.create(request, *args, **kwargs) + + +class ActorDetail( + mixins.RetrieveModelMixin, + mixins.UpdateModelMixin, + mixins.DestroyModelMixin, + generics.GenericAPIView +): + queryset = Actor.objects.all() + serializer_class = ActorSerializer + + def get(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.retrieve(request, *args, **kwargs) + + def put(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.update(request, *args, **kwargs) + + def patch(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.partial_update(request, *args, **kwargs) - return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) + def delete(self, request: HttpRequest, *args, **kwargs) -> Response: + return self.destroy(request, *args, **kwargs) -@api_view(["GET", "PUT", "DELETE"]) -def movie_detail(request, pk): - movie = get_object_or_404(Movie, pk=pk) +class CinemaHallViewSet( + mixins.ListModelMixin, + mixins.CreateModelMixin, + mixins.RetrieveModelMixin, + mixins.UpdateModelMixin, + mixins.DestroyModelMixin, + viewsets.GenericViewSet +): + queryset = CinemaHall.objects.all() + serializer_class = CinemaHallSerializer - if request.method == "GET": - serializer = MovieSerializer(movie) + +class GenreList(APIView): + def get(self, request: HttpRequest) -> Response: + genres = Genre.objects.all() + serializer = GenreSerializer(genres, many=True) + return Response(serializer.data, status=status.HTTP_200_OK) + + def post(self, request: HttpRequest) -> Response: + serializer = GenreSerializer(data=request.data) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data, status=status.HTTP_201_CREATED) + + +class GenreDetail(APIView): + def get_object(self, pk: int) -> Genre: + return get_object_or_404(Genre, pk=pk) + + def get(self, request: HttpRequest, pk: int) -> Response: + genre = self.get_object(pk) + serializer = GenreSerializer(genre) return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "PUT": - serializer = MovieSerializer(movie, data=request.data) - if serializer.is_valid(): - serializer.save() - return Response(serializer.data, status=status.HTTP_200_OK) + def put(self, request: HttpRequest, pk: int) -> Response: + genre = self.get_object(pk) + serializer = GenreSerializer(genre, data=request.data) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data, status=status.HTTP_200_OK) - return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) + def patch(self, request : HttpRequest, pk: int) -> Response: + genre = self.get_object(pk) + serializer = GenreSerializer(genre, data=request.data, partial=True) + serializer.is_valid(raise_exception=True) + serializer.save() + return Response(serializer.data, status=status.HTTP_200_OK) - if request.method == "DELETE": - movie.delete() + def delete(self, request: HttpRequest, pk: int) -> Response: + genre = self.get_object(pk) + genre.delete() return Response(status=status.HTTP_204_NO_CONTENT)