From d2d47d122c8b853c7fb2bba62ca1fb33a929e9b3 Mon Sep 17 00:00:00 2001 From: Alexandr Politov Date: Sun, 1 Sep 2024 13:34:55 +0300 Subject: [PATCH] Solution Politov --- ...rder_ticket_alter_movie_actors_and_more.py | 114 ++++++++++++++++++ ...ticket_movie_session_alter_ticket_order.py | 24 ++++ db/models.py | 79 ++++++++++++ services/movie.py | 26 ++-- services/movie_session.py | 9 +- services/order.py | 36 ++++++ services/user.py | 50 ++++++++ settings.py | 4 + 8 files changed, 331 insertions(+), 11 deletions(-) create mode 100644 db/migrations/0002_user_order_ticket_alter_movie_actors_and_more.py create mode 100644 db/migrations/0003_alter_ticket_movie_session_alter_ticket_order.py create mode 100644 services/order.py create mode 100644 services/user.py diff --git a/db/migrations/0002_user_order_ticket_alter_movie_actors_and_more.py b/db/migrations/0002_user_order_ticket_alter_movie_actors_and_more.py new file mode 100644 index 000000000..8a554bb7a --- /dev/null +++ b/db/migrations/0002_user_order_ticket_alter_movie_actors_and_more.py @@ -0,0 +1,114 @@ +# Generated by Django 4.0.2 on 2024-08-31 16:53 + +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')), + ], + 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)), + ], + options={ + 'ordering': ['-created_at'], + }, + ), + 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()), + ], + ), + 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='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.AddIndex( + model_name='movie', + index=models.Index(fields=['title'], name='db_movie_title_5d0841_idx'), + ), + migrations.AddField( + model_name='ticket', + name='movie_session', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='db.moviesession'), + ), + migrations.AddField( + model_name='ticket', + name='order', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='db.order'), + ), + migrations.AddField( + model_name='order', + name='user', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='orders', to=settings.AUTH_USER_MODEL), + ), + migrations.AddField( + model_name='user', + name='groups', + field=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'), + ), + migrations.AddField( + model_name='user', + name='user_permissions', + field=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'), + ), + migrations.AddConstraint( + model_name='ticket', + constraint=models.UniqueConstraint(fields=('movie_session', 'row', 'seat'), name='unique_ticket'), + ), + ] diff --git a/db/migrations/0003_alter_ticket_movie_session_alter_ticket_order.py b/db/migrations/0003_alter_ticket_movie_session_alter_ticket_order.py new file mode 100644 index 000000000..36b5e04de --- /dev/null +++ b/db/migrations/0003_alter_ticket_movie_session_alter_ticket_order.py @@ -0,0 +1,24 @@ +# Generated by Django 4.0.2 on 2024-08-31 17:26 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('db', '0002_user_order_ticket_alter_movie_actors_and_more'), + ] + + operations = [ + migrations.AlterField( + model_name='ticket', + name='movie_session', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='tickets', to='db.moviesession'), + ), + migrations.AlterField( + model_name='ticket', + name='order', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='tickets', to='db.order'), + ), + ] diff --git a/db/models.py b/db/models.py index 2a3359a63..d992761a3 100644 --- a/db/models.py +++ b/db/models.py @@ -1,4 +1,13 @@ +from django.contrib.auth.models import AbstractUser +from django.core.exceptions import ValidationError from django.db import models +from django.db.models import UniqueConstraint + +import settings + + +class User(AbstractUser): + pass class Genre(models.Model): @@ -22,6 +31,9 @@ class Movie(models.Model): actors = models.ManyToManyField(to=Actor, related_name="movies") genres = models.ManyToManyField(to=Genre, related_name="movies") + class Meta: + indexes = [models.Index(fields=["title"])] + def __str__(self) -> str: return self.title @@ -50,3 +62,70 @@ class MovieSession(models.Model): def __str__(self) -> str: return f"{self.movie.title} {str(self.show_time)}" + + +class Order(models.Model): + created_at = models.DateTimeField(auto_now_add=True) + user = models.ForeignKey( + settings.AUTH_USER_MODEL, + on_delete=models.CASCADE, + related_name="orders" + ) + + class Meta: + ordering = ["-created_at"] + + def __str__(self) -> str: + return self.created_at.strftime("%Y-%m-%d %H:%M:%S") + + +class Ticket(models.Model): + movie_session = models.ForeignKey( + MovieSession, + on_delete=models.CASCADE, + related_name="tickets") + order = models.ForeignKey( + Order, + on_delete=models.CASCADE, + related_name="tickets") + row = models.IntegerField() + seat = models.IntegerField() + + class Meta: + constraints = [ + UniqueConstraint( + fields=["movie_session", "row", "seat"], + name="unique_ticket" + ) + ] + + def __str__(self) -> str: + return ( + f"{self.movie_session.movie.title} " + 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: + rows_limit = self.movie_session.cinema_hall.rows + raise ValidationError( + { + "row": f"row number must be in available range: " + f"(1, rows): (1, {rows_limit})" + } + ) + + if not 1 <= self.seat <= self.movie_session.cinema_hall.seats_in_row: + seats_limit = self.movie_session.cinema_hall.seats_in_row + raise ValidationError( + { + "seat": f"seat number must be in available range: " + f"(1, seats_in_row): (1, {seats_limit})" + } + ) + + def save(self, *args, **kwargs) -> models.Model: + self.full_clean() + return super().save(*args, **kwargs) diff --git a/services/movie.py b/services/movie.py index 4d0f63237..b067efbb1 100644 --- a/services/movie.py +++ b/services/movie.py @@ -1,14 +1,19 @@ +from django.db import transaction from django.db.models import QuerySet from db.models import Movie def get_movies( + title: str | None = None, genres_ids: list[int] = None, actors_ids: list[int] = None, ) -> QuerySet: queryset = Movie.objects.all() + if title: + queryset = queryset.filter(title__icontains=title) + if genres_ids: queryset = queryset.filter(genres__id__in=genres_ids) @@ -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..ab155e0b1 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( @@ -42,3 +42,10 @@ 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[dict]: + tickets = Ticket.objects.filter( + movie_session_id=movie_session_id + ).values("row", "seat") + return list(tickets) diff --git a/services/order.py b/services/order.py new file mode 100644 index 000000000..ff120f970 --- /dev/null +++ b/services/order.py @@ -0,0 +1,36 @@ +from django.contrib.auth import get_user_model +from django.db import transaction +from django.db.models import QuerySet +from django.utils.datetime_safe import datetime + +from db.models import Order, Ticket + + +def create_order( + tickets: list[dict], + username: str, + date: datetime | None = None +) -> None: + with transaction.atomic(): + user = get_user_model().objects.get(username=username) + order = Order.objects.create(user=user) + + if date: + order.created_at = date + order.save() + + for ticket in tickets: + Ticket.objects.create( + movie_session_id=ticket["movie_session"], + row=ticket["row"], + seat=ticket["seat"], + order=order, + ) + + +def get_orders(username: str | None = None) -> QuerySet: + orders = Order.objects.all() + if username: + user = get_user_model().objects.get(username=username) + orders = orders.filter(user=user) + return orders diff --git a/services/user.py b/services/user.py new file mode 100644 index 000000000..209a954b6 --- /dev/null +++ b/services/user.py @@ -0,0 +1,50 @@ +from django.contrib.auth import get_user_model + +from db.models import User + + +def create_user( + username: str, + password: str, + email: str | None = None, + first_name: str | None = None, + last_name: str | None = None, +) -> None: + new_user = get_user_model().objects.create_user( + username=username, + password=password + ) + + if email: + new_user.email = email + if first_name: + new_user.first_name = first_name + if last_name: + new_user.last_name = last_name + new_user.save() + + +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 = None, + password: str | None = None, + email: str | None = None, + first_name: str | None = None, + last_name: str | None = None, +) -> None: + user = get_user_model().objects.get(pk=user_id) + if password: + user.set_password(password) + if username: + user.username = username + if email: + user.email = email + if first_name: + user.first_name = first_name + if last_name: + user.last_name = last_name + user.save() 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"