From 597ad0f6fce41fef571b64d9f3c557f41d4b04dd Mon Sep 17 00:00:00 2001 From: Pasha Date: Mon, 2 Sep 2024 22:10:49 +0300 Subject: [PATCH] Solution --- ...ovie_actors_alter_movie_genres_and_more.py | 95 +++++++++++++++++++ db/models.py | 70 +++++++++++++- services/movie.py | 26 +++-- services/movie_session.py | 13 ++- services/order.py | 42 ++++++++ services/user.py | 65 +++++++++++++ settings.py | 4 + 7 files changed, 303 insertions(+), 12 deletions(-) create mode 100644 db/migrations/0002_user_order_alter_movie_actors_alter_movie_genres_and_more.py create mode 100644 services/order.py create mode 100644 services/user.py diff --git a/db/migrations/0002_user_order_alter_movie_actors_alter_movie_genres_and_more.py b/db/migrations/0002_user_order_alter_movie_actors_alter_movie_genres_and_more.py new file mode 100644 index 000000000..99cadf597 --- /dev/null +++ b/db/migrations/0002_user_order_alter_movie_actors_alter_movie_genres_and_more.py @@ -0,0 +1,95 @@ +# Generated by Django 4.0.2 on 2024-09-02 20:19 + +from django.conf import settings +import django.contrib.auth.models +import django.contrib.auth.validators +from django.db import migrations, models +import django.db.models.deletion +import django.utils.timezone + + +class Migration(migrations.Migration): + + dependencies = [ + ('auth', '0012_alter_user_first_name_max_length'), + ('db', '0001_initial'), + ] + + operations = [ + migrations.CreateModel( + name='User', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('password', models.CharField(max_length=128, verbose_name='password')), + ('last_login', models.DateTimeField(blank=True, null=True, verbose_name='last login')), + ('is_superuser', models.BooleanField(default=False, help_text='Designates that this user has all permissions without explicitly assigning them.', verbose_name='superuser status')), + ('username', models.CharField(error_messages={'unique': 'A user with that username already exists.'}, help_text='Required. 150 characters or fewer. Letters, digits and @/./+/-/_ only.', max_length=150, unique=True, validators=[django.contrib.auth.validators.UnicodeUsernameValidator()], verbose_name='username')), + ('first_name', models.CharField(blank=True, max_length=150, verbose_name='first name')), + ('last_name', models.CharField(blank=True, max_length=150, verbose_name='last name')), + ('email', models.EmailField(blank=True, max_length=254, verbose_name='email address')), + ('is_staff', models.BooleanField(default=False, help_text='Designates whether the user can log into this admin site.', verbose_name='staff status')), + ('is_active', models.BooleanField(default=True, help_text='Designates whether this user should be treated as active. Unselect this instead of deleting accounts.', verbose_name='active')), + ('date_joined', models.DateTimeField(default=django.utils.timezone.now, verbose_name='date joined')), + ('groups', models.ManyToManyField(blank=True, help_text='The groups this user belongs to. A user will get all permissions granted to each of their groups.', related_name='user_set', related_query_name='user', to='auth.Group', verbose_name='groups')), + ('user_permissions', models.ManyToManyField(blank=True, help_text='Specific permissions for this user.', related_name='user_set', related_query_name='user', to='auth.Permission', verbose_name='user permissions')), + ], + options={ + 'verbose_name': 'user', + 'verbose_name_plural': 'users', + 'abstract': False, + }, + managers=[ + ('objects', django.contrib.auth.models.UserManager()), + ], + ), + migrations.CreateModel( + name='Order', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('created_at', models.DateTimeField(auto_now_add=True)), + ('user', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='orders', to=settings.AUTH_USER_MODEL)), + ], + options={ + 'ordering': ['-created_at'], + }, + ), + migrations.AlterField( + model_name='movie', + name='actors', + field=models.ManyToManyField(related_name='movies', to='db.Actor'), + ), + migrations.AlterField( + model_name='movie', + name='genres', + field=models.ManyToManyField(related_name='movies', to='db.Genre'), + ), + migrations.AlterField( + model_name='movie', + name='title', + field=models.CharField(db_index=True, max_length=255), + ), + migrations.AlterField( + model_name='moviesession', + name='cinema_hall', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='movie_sessions', to='db.cinemahall'), + ), + migrations.AlterField( + model_name='moviesession', + name='movie', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='movie_sessions', to='db.movie'), + ), + migrations.CreateModel( + name='Ticket', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('row', models.IntegerField()), + ('seat', models.IntegerField()), + ('movie_session', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='tickets', to='db.moviesession')), + ('order', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='tickets', to='db.order')), + ], + ), + migrations.AddConstraint( + model_name='ticket', + constraint=models.UniqueConstraint(fields=('movie_session', 'seat', 'row'), name='unique_row_seat_in_movie_session'), + ), + ] diff --git a/db/models.py b/db/models.py index 2a3359a63..988dcc0e2 100644 --- a/db/models.py +++ b/db/models.py @@ -1,4 +1,8 @@ +from django.conf import settings +from django.contrib.auth.models import AbstractUser +from django.core.exceptions import ValidationError from django.db import models +from typing_extensions import Callable class Genre(models.Model): @@ -17,7 +21,7 @@ def __str__(self) -> str: class Movie(models.Model): - title = models.CharField(max_length=255) + title = models.CharField(max_length=255, db_index=True) description = models.TextField() actors = models.ManyToManyField(to=Actor, related_name="movies") genres = models.ManyToManyField(to=Genre, related_name="movies") @@ -50,3 +54,67 @@ class MovieSession(models.Model): def __str__(self) -> str: return f"{self.movie.title} {str(self.show_time)}" + + +class User(AbstractUser): + pass + + +class Order(models.Model): + created_at = models.DateTimeField(auto_now_add=True) + user = models.ForeignKey( + to=settings.AUTH_USER_MODEL, + on_delete=models.CASCADE, + related_name="orders" + ) + + class Meta: + ordering = ["-created_at"] + + def __str__(self) -> str: + return f"{self.created_at}" + + +class Ticket(models.Model): + movie_session = models.ForeignKey( + to=MovieSession, on_delete=models.CASCADE, related_name="tickets" + ) + order = models.ForeignKey( + to=Order, on_delete=models.CASCADE, related_name="tickets" + ) + row = models.IntegerField() + seat = models.IntegerField() + + class Meta: + constraints = [ + models.UniqueConstraint( + fields=["movie_session", "seat", "row"], + name="unique_row_seat_in_movie_session", + ) + ] + + def __str__(self) -> str: + return ( + f"{self.movie_session.movie} " + f"{self.movie_session.show_time} " + f"(row: {self.row}, seat: {self.seat})" + ) + + def clean(self) -> None: + if not (1 <= self.row <= self.movie_session.cinema_hall.rows): + raise ValidationError({ + "row": f"row number must be in available range: " + f"(1, rows): " + f"(1, {self.movie_session.cinema_hall.rows})" + }) + + if not (1 <= self.seat <= self.movie_session.cinema_hall.seats_in_row): + raise ValidationError({ + "seat": f"seat number must be in available range: " + f"(1, seats_in_row): " + f"(1, {self.movie_session.cinema_hall.seats_in_row})" + }) + + def save(self, *args, **kwargs) -> Callable: + self.full_clean() + return super().save(*args, **kwargs) diff --git a/services/movie.py b/services/movie.py index 4d0f63237..a96f8b352 100644 --- a/services/movie.py +++ b/services/movie.py @@ -1,3 +1,4 @@ +from django.db import transaction from django.db.models import QuerySet from db.models import Movie @@ -6,6 +7,7 @@ def get_movies( genres_ids: list[int] = None, actors_ids: list[int] = None, + title: str = None ) -> QuerySet: queryset = Movie.objects.all() @@ -15,6 +17,9 @@ def get_movies( if actors_ids: queryset = queryset.filter(actors__id__in=actors_ids) + if title: + queryset = queryset.filter(title__icontains=title) + return queryset @@ -28,13 +33,14 @@ def create_movie( genres_ids: list = None, actors_ids: list = None, ) -> Movie: - movie = Movie.objects.create( - title=movie_title, - description=movie_description, - ) - if genres_ids: - movie.genres.set(genres_ids) - if actors_ids: - movie.actors.set(actors_ids) - - return movie + with transaction.atomic(): + movie = Movie.objects.create( + title=movie_title, + description=movie_description, + ) + if genres_ids: + movie.genres.set(genres_ids) + if actors_ids: + movie.actors.set(actors_ids) + + return movie diff --git a/services/movie_session.py b/services/movie_session.py index f326a082e..9701fa2f8 100644 --- a/services/movie_session.py +++ b/services/movie_session.py @@ -1,6 +1,6 @@ from django.db.models import QuerySet -from db.models import MovieSession +from db.models import MovieSession, Ticket def create_movie_session( @@ -30,6 +30,9 @@ def update_movie_session( movie_id: int = None, cinema_hall_id: int = None, ) -> None: + if not any([show_time, movie_id, cinema_hall_id]): + return None + movie_session = MovieSession.objects.get(id=session_id) if show_time: movie_session.show_time = show_time @@ -42,3 +45,11 @@ def update_movie_session( def delete_movie_session_by_id(session_id: int) -> None: MovieSession.objects.get(id=session_id).delete() + + +def get_taken_seats(movie_session_id: int) -> list: + return list( + Ticket.objects + .filter(movie_session_id=movie_session_id) + .values("row", "seat") + ) diff --git a/services/order.py b/services/order.py new file mode 100644 index 000000000..a1909d7da --- /dev/null +++ b/services/order.py @@ -0,0 +1,42 @@ +from datetime import datetime + +from django.db import transaction +from django.db.models import QuerySet + +from db.models import Order, Ticket, User, MovieSession + + +def create_order( + tickets: list[dict], + username: str, + date: datetime = None +) -> Order: + with transaction.atomic(): + order = Order.objects.create(user=User.objects.get(username=username)) + if date: + order.created_at = date + order.save() + + new_tickets = [ + Ticket( + movie_session=MovieSession.objects.get( + pk=ticket["movie_session"] + ), + order=order, + row=ticket["row"], + seat=ticket["seat"] + ) + for ticket in tickets + ] + + Ticket.objects.bulk_create(new_tickets) + return order + + +def get_orders(username: str = None) -> QuerySet: + orders = Order.objects.all() + + if username: + orders = orders.filter(user__username=username) + + return orders diff --git a/services/user.py b/services/user.py new file mode 100644 index 000000000..d9aa01d1e --- /dev/null +++ b/services/user.py @@ -0,0 +1,65 @@ +from __future__ import annotations +from django.contrib.auth import get_user_model + +from db.models import User + + +def create_user( + username: str, + password: str, + email: str = None, + first_name: str = None, + last_name: str = None +) -> User: + user = get_user_model().objects.create_user( + username=username, + password=password + ) + + if email: + user.email = email + + if first_name: + user.first_name = first_name + + if last_name: + user.last_name = last_name + + user.save() + return user + + +def get_user(user_id: int) -> User: + return get_user_model().objects.get(pk=user_id) + + +def update_user( + user_id: int, + username: str = None, + password: str = None, + email: str = None, + first_name: str = None, + last_name: str = None, +) -> User | None: + if not any([username, password, email, first_name, last_name]): + return None + + user = get_user(user_id) + + if username: + user.username = username + + if password: + user.set_password(password) + + if email: + user.email = email + + if first_name: + user.first_name = first_name + + if last_name: + user.last_name = last_name + + user.save() + return user diff --git a/settings.py b/settings.py index f25910b30..f9f0f92fb 100644 --- a/settings.py +++ b/settings.py @@ -25,4 +25,8 @@ INSTALLED_APPS = [ "db", + "django.contrib.auth", + "django.contrib.contenttypes" ] + +AUTH_USER_MODEL = "db.User"