diff --git a/1.2-requests-templates/pagination/pagination/urls.py b/1.2-requests-templates/pagination/pagination/urls.py index e13ffaf2c..c87bdef6c 100644 --- a/1.2-requests-templates/pagination/pagination/urls.py +++ b/1.2-requests-templates/pagination/pagination/urls.py @@ -17,5 +17,5 @@ from django.urls import path, include urlpatterns = [ - path('', include('stations.urls')), + path('', include('stations.urls'), name='stations'), ] diff --git a/1.2-requests-templates/pagination/requirements.txt b/1.2-requests-templates/pagination/requirements.txt index d3e4ba564..57fb7d602 100644 --- a/1.2-requests-templates/pagination/requirements.txt +++ b/1.2-requests-templates/pagination/requirements.txt @@ -1 +1,3 @@ -django +asgiref==3.5.2 +Django==4.0.6 +sqlparse==0.4.2 diff --git a/1.2-requests-templates/pagination/stations/templates/stations/index.html b/1.2-requests-templates/pagination/stations/templates/stations/index.html index 775da3370..448e9be5e 100644 --- a/1.2-requests-templates/pagination/stations/templates/stations/index.html +++ b/1.2-requests-templates/pagination/stations/templates/stations/index.html @@ -1,4 +1,4 @@ -{% load static %} + {% load static %} @@ -31,7 +31,7 @@ - {% for station in bus_stations %} + {% for station in page %} {{ station.Name }} {{ station.Street }} diff --git a/1.2-requests-templates/pagination/stations/views.py b/1.2-requests-templates/pagination/stations/views.py index ac3f83a9a..d3d3e187b 100644 --- a/1.2-requests-templates/pagination/stations/views.py +++ b/1.2-requests-templates/pagination/stations/views.py @@ -1,5 +1,9 @@ +from django.core.paginator import Paginator from django.shortcuts import render, redirect from django.urls import reverse +import csv + +import pagination.settings def index(request): @@ -7,11 +11,11 @@ def index(request): def bus_stations(request): - # получите текущую страницу и передайте ее в контекст - # также передайте в контекст список станций на странице - - context = { - # 'bus_stations': ..., - # 'page': ..., - } + page_number = int(request.GET.get('page', 1)) + with open(pagination.settings.BUS_STATION_CSV, encoding='utf8') as csvfile: + reader = list(csv.DictReader(csvfile)) + paginator = Paginator(reader, 10) + page = paginator.get_page(page_number) + context = {'page': page, + } return render(request, 'stations/index.html', context) diff --git a/1.2-requests-templates/recipes/calculator/views.py b/1.2-requests-templates/recipes/calculator/views.py index 0337130fb..fe534f470 100644 --- a/1.2-requests-templates/recipes/calculator/views.py +++ b/1.2-requests-templates/recipes/calculator/views.py @@ -19,8 +19,13 @@ # можете добавить свои рецепты ;) } -# Напишите ваш обработчик. Используйте DATA как источник данных -# Результат - render(request, 'calculator/index.html', context) + +def recipes(request, dish): + servings = int(request.GET.get('servings', 1)) + context = {'recipe': {key: value * servings for key, value in DATA[dish].items()}} + return render(request, 'calculator/index.html', context) + + # В качестве контекста должен быть передан словарь с рецептом: # context = { # 'recipe': { diff --git a/1.2-requests-templates/recipes/recipes/urls.py b/1.2-requests-templates/recipes/recipes/urls.py index 2d6f1024f..14290ea7a 100644 --- a/1.2-requests-templates/recipes/recipes/urls.py +++ b/1.2-requests-templates/recipes/recipes/urls.py @@ -15,7 +15,8 @@ """ from django.urls import path +from calculator.views import recipes urlpatterns = [ - # здесь зарегистрируйте вашу view-функцию + path('/', recipes, name='recipe') ] diff --git a/1.2-requests-templates/recipes/requirements.txt b/1.2-requests-templates/recipes/requirements.txt index d3e4ba564..57fb7d602 100644 --- a/1.2-requests-templates/recipes/requirements.txt +++ b/1.2-requests-templates/recipes/requirements.txt @@ -1 +1,3 @@ -django +asgiref==3.5.2 +Django==4.0.6 +sqlparse==0.4.2 diff --git a/2.1-databases/models_list_displaying/books/views.py b/2.1-databases/models_list_displaying/books/views.py index 03e0a4bbe..39ee9762a 100644 --- a/2.1-databases/models_list_displaying/books/views.py +++ b/2.1-databases/models_list_displaying/books/views.py @@ -1,7 +1,32 @@ +from django.core.paginator import Paginator from django.shortcuts import render +from books.models import Book def books_view(request): template = 'books/books_list.html' - context = {} + context = {'books': list(Book.objects.all())} + return render(request, template, context) + + +def books_paginator(request, pub_date): + template = 'books/books_pagi.html' + books_object = Book.objects.all().order_by('pub_date') + dates = [str(book_date) for book_dates in books_object.values('pub_date') for book_date in book_dates.values()] + pub_date_index = dates.index(pub_date) + paginator = Paginator(dates, 1) + page = paginator.get_page(dates.index(pub_date) + 1) + if pub_date_index < len(dates) - 1: + next_page = dates[pub_date_index + 1] + else: + next_page = None + if pub_date_index > 0: + previous_page = dates[pub_date_index - 1] + else: + previous_page = None + context = {'page': page, + 'previous_page': previous_page, + 'next_page': next_page, + 'books': list(Book.objects.filter(pub_date=pub_date)), + 'pub_date': pub_date} return render(request, template, context) diff --git a/2.1-databases/models_list_displaying/fixtures/books.json b/2.1-databases/models_list_displaying/fixtures/books.json index 04f5bf5d7..d1cf21676 100644 --- a/2.1-databases/models_list_displaying/fixtures/books.json +++ b/2.1-databases/models_list_displaying/fixtures/books.json @@ -4,7 +4,7 @@ "pk": 1, "fields": { "name": "Война и мир", - "author": "Ф.Д. Достоевский", + "author": "Лев Толстой", "pub_date": "2018-02-27" } }, @@ -16,5 +16,23 @@ "author": "Джордж Оруэл", "pub_date": "2016-12-06" } + }, + { + "model": "books.book", + "pk": 3, + "fields": { + "name": "Колыбельная", + "author": "Чак Паланик", + "pub_date": "2017-02-15" + } + }, + { + "model": "books.book", + "pk": 4, + "fields": { + "name": "Пикник на обочине", + "author": "Аркадий и Борис Стругацкие", + "pub_date": "2021-08-09" + } } ] \ No newline at end of file diff --git a/2.1-databases/models_list_displaying/main/urls.py b/2.1-databases/models_list_displaying/main/urls.py index b71c2c89b..268cf0da7 100644 --- a/2.1-databases/models_list_displaying/main/urls.py +++ b/2.1-databases/models_list_displaying/main/urls.py @@ -16,9 +16,11 @@ from django.contrib import admin from django.urls import path -from books.views import books_view +from books.views import books_view, books_paginator urlpatterns = [ path('', books_view, name='books'), + path('books//', books_paginator, name='books_paginator'), + path('books/', books_view, name='books'), path('admin/', admin.site.urls), ] diff --git a/2.1-databases/models_list_displaying/requirements.txt b/2.1-databases/models_list_displaying/requirements.txt index f00395277..bf49b73e1 100644 --- a/2.1-databases/models_list_displaying/requirements.txt +++ b/2.1-databases/models_list_displaying/requirements.txt @@ -1,2 +1,4 @@ -django -psycopg2-binary +asgiref==3.5.2 +Django==4.1 +psycopg2-binary==2.9.3 +sqlparse==0.4.2 diff --git a/2.1-databases/models_list_displaying/templates/books/books_list.html b/2.1-databases/models_list_displaying/templates/books/books_list.html index e36e0654e..6e29a694f 100644 --- a/2.1-databases/models_list_displaying/templates/books/books_list.html +++ b/2.1-databases/models_list_displaying/templates/books/books_list.html @@ -7,17 +7,15 @@ -
-
-

Сияние

-

Автор: Стивен Кинг

-

Дата публикации: 2018-09-10

-
-
-

1984

-

Автор: Джордж Оруэл

-

Дата публикации: 2015-03-11

+ +
+ {% for book in books %} +
+

{{ book.name }}

+

Автор: {{ book.author }}

+

Дата публикации: {{ book.pub_date }}

+ {% endfor %}
{% endblock %} \ No newline at end of file diff --git a/2.1-databases/models_list_displaying/templates/books/books_pagi.html b/2.1-databases/models_list_displaying/templates/books/books_pagi.html new file mode 100644 index 000000000..fa9ea2d7d --- /dev/null +++ b/2.1-databases/models_list_displaying/templates/books/books_pagi.html @@ -0,0 +1,31 @@ + + + + + Title +
+
+

Книги выпущенные {{ pub_date }}

+
+
+ + + {% for book in books %} +
+

{{ book.name }}

+

Автор: {{ book.author }}

+

Дата публикации: {{ book.pub_date }}

+
+ {% endfor %} + +
+ + {% if page.has_previous %} + < {{ previous_page }} + {% endif %} + + {% if page.has_next %} + {{ next_page }} > + {% endif %} +
+
\ No newline at end of file diff --git a/2.1-databases/work_with_database/main/settings.py b/2.1-databases/work_with_database/main/settings.py index 7a57a4ceb..6374cd683 100644 --- a/2.1-databases/work_with_database/main/settings.py +++ b/2.1-databases/work_with_database/main/settings.py @@ -55,7 +55,7 @@ 'HOST': '127.0.0.1', 'PORT': '5432', 'USER': 'postgres', - 'PASSWORD': 'ваш пароль', + 'PASSWORD': '1', } } diff --git a/2.1-databases/work_with_database/phones/admin.py b/2.1-databases/work_with_database/phones/admin.py index 8c38f3f3d..dcab2a211 100644 --- a/2.1-databases/work_with_database/phones/admin.py +++ b/2.1-databases/work_with_database/phones/admin.py @@ -1,3 +1,7 @@ from django.contrib import admin +from .models import Phone -# Register your models here. + +@admin.register(Phone) +class PhoneAdmin(admin.ModelAdmin): + pass diff --git a/2.1-databases/work_with_database/phones/management/commands/import_phones.py b/2.1-databases/work_with_database/phones/management/commands/import_phones.py index be777c517..b17b4cdd5 100644 --- a/2.1-databases/work_with_database/phones/management/commands/import_phones.py +++ b/2.1-databases/work_with_database/phones/management/commands/import_phones.py @@ -14,4 +14,10 @@ def handle(self, *args, **options): for phone in phones: # TODO: Добавьте сохранение модели - pass + Phone.objects.create(name=phone['name'], + price=phone['price'], + image=phone['image'], + release_date=phone['release_date'], + lte_exists=phone['lte_exists'], + slug='-'.join(phone['name'].split()) + ) diff --git a/2.1-databases/work_with_database/phones/migrations/0001_initial.py b/2.1-databases/work_with_database/phones/migrations/0001_initial.py new file mode 100644 index 000000000..1f48d02e0 --- /dev/null +++ b/2.1-databases/work_with_database/phones/migrations/0001_initial.py @@ -0,0 +1,26 @@ +# Generated by Django 4.0.6 on 2022-08-02 18:00 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + initial = True + + dependencies = [ + ] + + operations = [ + migrations.CreateModel( + name='Phone', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=50)), + ('price', models.DecimalField(decimal_places=2, max_digits=9)), + ('image', models.ImageField(upload_to='')), + ('releaze_date', models.DateField()), + ('lte_exists', models.BooleanField()), + ('slug', models.SlugField(allow_unicode=True)), + ], + ), + ] diff --git a/2.1-databases/work_with_database/phones/migrations/0002_rename_releaze_date_phone_release_date.py b/2.1-databases/work_with_database/phones/migrations/0002_rename_releaze_date_phone_release_date.py new file mode 100644 index 000000000..6b66b6b7b --- /dev/null +++ b/2.1-databases/work_with_database/phones/migrations/0002_rename_releaze_date_phone_release_date.py @@ -0,0 +1,18 @@ +# Generated by Django 4.0.6 on 2022-08-04 19:26 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('phones', '0001_initial'), + ] + + operations = [ + migrations.RenameField( + model_name='phone', + old_name='releaze_date', + new_name='release_date', + ), + ] diff --git a/2.1-databases/work_with_database/phones/models.py b/2.1-databases/work_with_database/phones/models.py index e79b3ba32..4fde983ed 100644 --- a/2.1-databases/work_with_database/phones/models.py +++ b/2.1-databases/work_with_database/phones/models.py @@ -2,5 +2,9 @@ class Phone(models.Model): - # TODO: Добавьте требуемые поля - pass + name = models.CharField(max_length=50) + price = models.DecimalField(max_digits=9, decimal_places=2) + image = models.ImageField() + release_date = models.DateField() + lte_exists = models.BooleanField() + slug = models.SlugField(allow_unicode=True) diff --git a/2.1-databases/work_with_database/phones/views.py b/2.1-databases/work_with_database/phones/views.py index afec3b5ec..bacd61aca 100644 --- a/2.1-databases/work_with_database/phones/views.py +++ b/2.1-databases/work_with_database/phones/views.py @@ -1,17 +1,30 @@ from django.shortcuts import render, redirect +from phones.models import Phone + def index(request): return redirect('catalog') def show_catalog(request): + sort = request.GET.get('sort', None) + if sort == 'name': + phones = list(Phone.objects.all().order_by(sort)) + elif sort == 'min_price': + phones = list(Phone.objects.all().order_by('price')) + elif sort == 'max_price': + phones = list(Phone.objects.all().order_by('-price')) + else: + phones = list(Phone.objects.all()) + print(phones) template = 'catalog.html' - context = {} + context = {'phones': phones} return render(request, template, context) def show_product(request, slug): template = 'product.html' - context = {} + context = {'phone': Phone.objects.get(slug=slug)} + print(context['phone']) return render(request, template, context) diff --git a/2.1-databases/work_with_database/requirements.txt b/2.1-databases/work_with_database/requirements.txt index f00395277..4106edc58 100644 --- a/2.1-databases/work_with_database/requirements.txt +++ b/2.1-databases/work_with_database/requirements.txt @@ -1,2 +1,5 @@ -django -psycopg2-binary +asgiref==3.5.2 +Django==4.0.6 +Pillow==9.2.0 +psycopg2-binary==2.9.3 +sqlparse==0.4.2 diff --git a/2.1-databases/work_with_database/templates/product.html b/2.1-databases/work_with_database/templates/product.html index 4fe945507..616b51df1 100644 --- a/2.1-databases/work_with_database/templates/product.html +++ b/2.1-databases/work_with_database/templates/product.html @@ -19,4 +19,5 @@

{{ phone.name }}

Дата выпуска: {{ phone.release_date }}
LTE: {{ phone.lte_exists|yesno:'есть,нет' }}
+ {% endblock %} diff --git a/2.2-databases-2/m2m-relations/articles.json b/2.2-databases-2/m2m-relations/articles.json index e122dabdb..cee98ef78 100644 --- a/2.2-databases-2/m2m-relations/articles.json +++ b/2.2-databases-2/m2m-relations/articles.json @@ -28,5 +28,47 @@ "published_at": "2018-07-30T20:08:20Z", "image": "th_1521804489_cfyJCMV.jpg" } + }, + { + "model": "articles.teg", + "pk": 1, + "fields": { + "name": "Культура" + } + }, + { + "model": "articles.teg", + "pk": 2, + "fields": { + "name": "Город" + } + }, + { + "model": "articles.teg", + "pk": 3, + "fields": { + "name": "Здоровье" + } + }, + { + "model": "articles.teg", + "pk": 4, + "fields": { + "name": "Наука" + } + }, + { + "model": "articles.teg", + "pk": 5, + "fields": { + "name": "Космос" + } + }, + { + "model": "articles.teg", + "pk": 6, + "fields": { + "name": "Международные отношения" + } } ] \ No newline at end of file diff --git a/2.2-databases-2/m2m-relations/articles/admin.py b/2.2-databases-2/m2m-relations/articles/admin.py index 4799babf4..bb0f457ec 100644 --- a/2.2-databases-2/m2m-relations/articles/admin.py +++ b/2.2-databases-2/m2m-relations/articles/admin.py @@ -1,8 +1,42 @@ from django.contrib import admin +from django.core.exceptions import ValidationError +from django.forms import BaseInlineFormSet -from .models import Article +from .models import Article, Teg, ArticleTeg + + +class ArticleTegInlineFormset(BaseInlineFormSet): + def clean(self): + main_teg_list = [] + for form in self.forms: + # В form.cleaned_data будет словарь с данными + # каждой отдельной формы, которые вы можете проверить + if form.cleaned_data: + main_teg_list.append(form.cleaned_data['main_teg']) + # вызовом исключения ValidationError можно указать админке о наличие ошибки + # таким образом объект не будет сохранен, + # а пользователю выведется соответствующее сообщение об ошибке + if not any(main_teg_list): + raise ValidationError('Укажите основной раздел') + if sum(main_teg_list) > 1: + raise ValidationError('Основным может быть только один раздел') + return super().clean() # вызываем базовый код переопределяемого метода + + +class ArticleTegInline(admin.TabularInline): + model = ArticleTeg + extra = 1 + formset = ArticleTegInlineFormset @admin.register(Article) class ArticleAdmin(admin.ModelAdmin): - pass + list_display = ['id', 'title', 'text', 'published_at'] + list_filter = ['published_at'] + inlines = [ArticleTegInline, ] + + +@admin.register(Teg) +class ArticleAdmin(admin.ModelAdmin): + list_display = ['id', 'name'] + list_filter = ['name'] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0002_teg_tegofarticle.py b/2.2-databases-2/m2m-relations/articles/migrations/0002_teg_tegofarticle.py new file mode 100644 index 000000000..6899c998e --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0002_teg_tegofarticle.py @@ -0,0 +1,31 @@ +# Generated by Django 4.0.6 on 2022-08-11 05:01 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0001_initial'), + ] + + operations = [ + migrations.CreateModel( + name='Teg', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=20)), + ('articles', models.ManyToManyField(related_name='tegs', to='articles.article')), + ], + ), + migrations.CreateModel( + name='TegOfArticle', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('main_teg', models.BooleanField()), + ('article', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='positions', to='articles.article')), + ('teg', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='positions', to='articles.teg')), + ], + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0003_alter_teg_options_alter_teg_name.py b/2.2-databases-2/m2m-relations/articles/migrations/0003_alter_teg_options_alter_teg_name.py new file mode 100644 index 000000000..5c395686f --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0003_alter_teg_options_alter_teg_name.py @@ -0,0 +1,22 @@ +# Generated by Django 4.0.6 on 2022-08-11 05:23 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0002_teg_tegofarticle'), + ] + + operations = [ + migrations.AlterModelOptions( + name='teg', + options={'verbose_name': 'Тег', 'verbose_name_plural': 'Теги'}, + ), + migrations.AlterField( + model_name='teg', + name='name', + field=models.CharField(max_length=30), + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0004_alter_article_options_delete_tegofarticle.py b/2.2-databases-2/m2m-relations/articles/migrations/0004_alter_article_options_delete_tegofarticle.py new file mode 100644 index 000000000..3deaff06e --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0004_alter_article_options_delete_tegofarticle.py @@ -0,0 +1,20 @@ +# Generated by Django 4.0.6 on 2022-08-11 05:44 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0003_alter_teg_options_alter_teg_name'), + ] + + operations = [ + migrations.AlterModelOptions( + name='article', + options={'ordering': ['-published_at'], 'verbose_name': 'Статья', 'verbose_name_plural': 'Статьи'}, + ), + migrations.DeleteModel( + name='TegOfArticle', + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0005_tegofarticle.py b/2.2-databases-2/m2m-relations/articles/migrations/0005_tegofarticle.py new file mode 100644 index 000000000..d1239c630 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0005_tegofarticle.py @@ -0,0 +1,23 @@ +# Generated by Django 4.0.6 on 2022-08-11 06:21 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0004_alter_article_options_delete_tegofarticle'), + ] + + operations = [ + migrations.CreateModel( + name='TegOfArticle', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('main_teg', models.BooleanField()), + ('article', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='positions', to='articles.article')), + ('teg', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='positions', to='articles.teg')), + ], + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0006_alter_teg_options_alter_tegofarticle_article_and_more.py b/2.2-databases-2/m2m-relations/articles/migrations/0006_alter_teg_options_alter_tegofarticle_article_and_more.py new file mode 100644 index 000000000..8cb047c20 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0006_alter_teg_options_alter_tegofarticle_article_and_more.py @@ -0,0 +1,28 @@ +# Generated by Django 4.0.6 on 2022-08-11 10:12 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0005_tegofarticle'), + ] + + operations = [ + migrations.AlterModelOptions( + name='teg', + options={'verbose_name': 'Раздел', 'verbose_name_plural': 'Разделы'}, + ), + migrations.AlterField( + model_name='tegofarticle', + name='article', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='scopes', to='articles.article'), + ), + migrations.AlterField( + model_name='tegofarticle', + name='teg', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='scopes', to='articles.teg'), + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0007_alter_teg_articles_alter_tegofarticle_article_and_more.py b/2.2-databases-2/m2m-relations/articles/migrations/0007_alter_teg_articles_alter_tegofarticle_article_and_more.py new file mode 100644 index 000000000..dbbd00231 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0007_alter_teg_articles_alter_tegofarticle_article_and_more.py @@ -0,0 +1,29 @@ +# Generated by Django 4.0.6 on 2022-08-11 10:17 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0006_alter_teg_options_alter_tegofarticle_article_and_more'), + ] + + operations = [ + migrations.AlterField( + model_name='teg', + name='articles', + field=models.ManyToManyField(related_name='scopes', to='articles.article'), + ), + migrations.AlterField( + model_name='tegofarticle', + name='article', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='counts', to='articles.article'), + ), + migrations.AlterField( + model_name='tegofarticle', + name='teg', + field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='counts', to='articles.teg'), + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0008_remove_teg_articles_remove_tegofarticle_article_and_more.py b/2.2-databases-2/m2m-relations/articles/migrations/0008_remove_teg_articles_remove_tegofarticle_article_and_more.py new file mode 100644 index 000000000..cead80207 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0008_remove_teg_articles_remove_tegofarticle_article_and_more.py @@ -0,0 +1,34 @@ +# Generated by Django 4.0.6 on 2022-08-11 10:57 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0007_alter_teg_articles_alter_tegofarticle_article_and_more'), + ] + + operations = [ + migrations.RemoveField( + model_name='teg', + name='articles', + ), + migrations.RemoveField( + model_name='tegofarticle', + name='article', + ), + migrations.RemoveField( + model_name='tegofarticle', + name='teg', + ), + migrations.DeleteModel( + name='Article', + ), + migrations.DeleteModel( + name='Teg', + ), + migrations.DeleteModel( + name='TegOfArticle', + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0009_initial.py b/2.2-databases-2/m2m-relations/articles/migrations/0009_initial.py new file mode 100644 index 000000000..383d5b03a --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0009_initial.py @@ -0,0 +1,52 @@ +# Generated by Django 4.0.6 on 2022-08-11 11:02 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + initial = True + + dependencies = [ + ('articles', '0008_remove_teg_articles_remove_tegofarticle_article_and_more'), + ] + + operations = [ + migrations.CreateModel( + name='Article', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('title', models.CharField(max_length=256, verbose_name='Название')), + ('text', models.TextField(verbose_name='Текст')), + ('published_at', models.DateTimeField(verbose_name='Дата публикации')), + ('image', models.ImageField(blank=True, null=True, upload_to='', verbose_name='Изображение')), + ], + options={ + 'verbose_name': 'Статья', + 'verbose_name_plural': 'Статьи', + 'ordering': ['-published_at'], + }, + ), + migrations.CreateModel( + name='Teg', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=30)), + ('articles', models.ManyToManyField(related_name='scopes', to='articles.article')), + ], + options={ + 'verbose_name': 'Раздел', + 'verbose_name_plural': 'Разделы', + }, + ), + migrations.CreateModel( + name='TegOfArticle', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('main_teg', models.BooleanField()), + ('article', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='counts', to='articles.article')), + ('teg', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='counts', to='articles.teg')), + ], + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0010_remove_tegofarticle_article_remove_tegofarticle_teg_and_more.py b/2.2-databases-2/m2m-relations/articles/migrations/0010_remove_tegofarticle_article_remove_tegofarticle_teg_and_more.py new file mode 100644 index 000000000..f12ac91eb --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0010_remove_tegofarticle_article_remove_tegofarticle_teg_and_more.py @@ -0,0 +1,31 @@ +# Generated by Django 4.0.6 on 2022-08-11 11:18 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0009_initial'), + ] + + operations = [ + migrations.RemoveField( + model_name='tegofarticle', + name='article', + ), + migrations.RemoveField( + model_name='tegofarticle', + name='teg', + ), + migrations.AlterModelOptions( + name='article', + options={'verbose_name': 'Статья', 'verbose_name_plural': 'Статьи'}, + ), + migrations.DeleteModel( + name='Teg', + ), + migrations.DeleteModel( + name='TegOfArticle', + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0011_teg.py b/2.2-databases-2/m2m-relations/articles/migrations/0011_teg.py new file mode 100644 index 000000000..20119c6dd --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0011_teg.py @@ -0,0 +1,26 @@ +# Generated by Django 4.0.6 on 2022-08-11 11:48 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0010_remove_tegofarticle_article_remove_tegofarticle_teg_and_more'), + ] + + operations = [ + migrations.CreateModel( + name='Teg', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name', models.CharField(max_length=30)), + ('main_teg', models.BooleanField(verbose_name='Основной')), + ('teg', models.ManyToManyField(related_name='scopes', to='articles.article')), + ], + options={ + 'verbose_name': 'Раздел', + 'verbose_name_plural': 'Разделы', + }, + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0012_remove_teg_main_teg.py b/2.2-databases-2/m2m-relations/articles/migrations/0012_remove_teg_main_teg.py new file mode 100644 index 000000000..27655fb19 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0012_remove_teg_main_teg.py @@ -0,0 +1,17 @@ +# Generated by Django 4.0.6 on 2022-08-11 11:50 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0011_teg'), + ] + + operations = [ + migrations.RemoveField( + model_name='teg', + name='main_teg', + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/migrations/0013_remove_teg_teg_articleteg.py b/2.2-databases-2/m2m-relations/articles/migrations/0013_remove_teg_teg_articleteg.py new file mode 100644 index 000000000..1e63a31a0 --- /dev/null +++ b/2.2-databases-2/m2m-relations/articles/migrations/0013_remove_teg_teg_articleteg.py @@ -0,0 +1,27 @@ +# Generated by Django 4.0.6 on 2022-08-11 19:18 + +from django.db import migrations, models +import django.db.models.deletion + + +class Migration(migrations.Migration): + + dependencies = [ + ('articles', '0012_remove_teg_main_teg'), + ] + + operations = [ + migrations.RemoveField( + model_name='teg', + name='teg', + ), + migrations.CreateModel( + name='ArticleTeg', + fields=[ + ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('main_teg', models.BooleanField(verbose_name='Основной')), + ('article', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='scopes', to='articles.article')), + ('teg', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='scopes', to='articles.teg')), + ], + ), + ] diff --git a/2.2-databases-2/m2m-relations/articles/models.py b/2.2-databases-2/m2m-relations/articles/models.py index 0486c9fe8..0648613b9 100644 --- a/2.2-databases-2/m2m-relations/articles/models.py +++ b/2.2-databases-2/m2m-relations/articles/models.py @@ -11,6 +11,25 @@ class Article(models.Model): class Meta: verbose_name = 'Статья' verbose_name_plural = 'Статьи' + ordering = ['-published_at'] def __str__(self): return self.title + + +class Teg(models.Model): + name = models.CharField(max_length=30) + articles = models.ManyToManyField(Article, related_name='articles', through='ArticleTeg') + + class Meta: + verbose_name = 'Раздел' + verbose_name_plural = 'Разделы' + + def __str__(self): + return self.name + + +class ArticleTeg(models.Model): + article = models.ForeignKey(Article, on_delete=models.CASCADE, related_name='scopes') + teg = models.ForeignKey(Teg, on_delete=models.CASCADE, related_name='scopes') + main_teg = models.BooleanField(verbose_name='Основной') \ No newline at end of file diff --git a/2.2-databases-2/m2m-relations/articles/views.py b/2.2-databases-2/m2m-relations/articles/views.py index e1d022b45..9118ba003 100644 --- a/2.2-databases-2/m2m-relations/articles/views.py +++ b/2.2-databases-2/m2m-relations/articles/views.py @@ -5,10 +5,9 @@ def articles_list(request): template = 'articles/news.html' - context = {} - + articles = Article.objects.all() + context = {'object_list': articles} # используйте этот параметр для упорядочивания результатов # https://docs.djangoproject.com/en/3.1/ref/models/querysets/#django.db.models.query.QuerySet.order_by - ordering = '-published_at' return render(request, template, context) diff --git a/2.2-databases-2/m2m-relations/templates/articles/news.html b/2.2-databases-2/m2m-relations/templates/articles/news.html index 17d5a5ddc..da5decaba 100644 --- a/2.2-databases-2/m2m-relations/templates/articles/news.html +++ b/2.2-databases-2/m2m-relations/templates/articles/news.html @@ -16,7 +16,7 @@

{{ article.text }}

{% for scope in article.scopes.all %} - {{ scope.tag.name }} + {{ scope.teg }} {% endfor %} diff --git a/2.2-databases-2/orm_migrations/requirements.txt b/2.2-databases-2/orm_migrations/requirements.txt index f00395277..e283f78dc 100644 --- a/2.2-databases-2/orm_migrations/requirements.txt +++ b/2.2-databases-2/orm_migrations/requirements.txt @@ -1,2 +1,5 @@ -django -psycopg2-binary +asgiref==3.5.2 +Django==4.1 +django-debug-toolbar==3.5.0 +psycopg2-binary==2.9.3 +sqlparse==0.4.2 diff --git a/2.2-databases-2/orm_migrations/school/admin.py b/2.2-databases-2/orm_migrations/school/admin.py index eb856a6af..c9c6af275 100644 --- a/2.2-databases-2/orm_migrations/school/admin.py +++ b/2.2-databases-2/orm_migrations/school/admin.py @@ -5,9 +5,10 @@ @admin.register(Student) class StudentAdmin(admin.ModelAdmin): - pass + list_display = ['id', 'name'] + list_filter = ['name', 'group'] @admin.register(Teacher) class TeacherAdmin(admin.ModelAdmin): - pass + list_display = ['id', 'name', 'subject'] diff --git a/2.2-databases-2/orm_migrations/school/migrations/0002_alter_student_group_alter_student_name_and_more.py b/2.2-databases-2/orm_migrations/school/migrations/0002_alter_student_group_alter_student_name_and_more.py new file mode 100644 index 000000000..00e0f2432 --- /dev/null +++ b/2.2-databases-2/orm_migrations/school/migrations/0002_alter_student_group_alter_student_name_and_more.py @@ -0,0 +1,42 @@ +# Generated by Django 4.1 on 2022-08-10 07:11 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('school', '0001_initial'), + ] + + operations = [ + migrations.AlterField( + model_name='student', + name='group', + field=models.CharField(max_length=10, verbose_name='Класс'), + ), + migrations.AlterField( + model_name='student', + name='name', + field=models.CharField(max_length=30, verbose_name='Имя'), + ), + migrations.RemoveField( + model_name='student', + name='teacher', + ), + migrations.AlterField( + model_name='teacher', + name='name', + field=models.CharField(max_length=30, verbose_name='Имя'), + ), + migrations.AlterField( + model_name='teacher', + name='subject', + field=models.CharField(max_length=10, verbose_name='Предмет'), + ), + migrations.AddField( + model_name='student', + name='teacher', + field=models.ManyToManyField(related_name='students', to='school.teacher'), + ), + ] diff --git a/2.2-databases-2/orm_migrations/school/migrations/0003_alter_student_teacher.py b/2.2-databases-2/orm_migrations/school/migrations/0003_alter_student_teacher.py new file mode 100644 index 000000000..a75ba57b4 --- /dev/null +++ b/2.2-databases-2/orm_migrations/school/migrations/0003_alter_student_teacher.py @@ -0,0 +1,18 @@ +# Generated by Django 4.1 on 2022-08-10 15:42 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('school', '0002_alter_student_group_alter_student_name_and_more'), + ] + + operations = [ + migrations.AlterField( + model_name='student', + name='teacher', + field=models.ManyToManyField(related_name='teachers', to='school.teacher'), + ), + ] diff --git a/2.2-databases-2/orm_migrations/school/migrations/0004_alter_student_teacher.py b/2.2-databases-2/orm_migrations/school/migrations/0004_alter_student_teacher.py new file mode 100644 index 000000000..768a60475 --- /dev/null +++ b/2.2-databases-2/orm_migrations/school/migrations/0004_alter_student_teacher.py @@ -0,0 +1,18 @@ +# Generated by Django 4.1 on 2022-08-10 15:44 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('school', '0003_alter_student_teacher'), + ] + + operations = [ + migrations.AlterField( + model_name='student', + name='teacher', + field=models.ManyToManyField(related_name='students', to='school.teacher'), + ), + ] diff --git a/2.2-databases-2/orm_migrations/school/models.py b/2.2-databases-2/orm_migrations/school/models.py index abab28192..46d32c8b7 100644 --- a/2.2-databases-2/orm_migrations/school/models.py +++ b/2.2-databases-2/orm_migrations/school/models.py @@ -15,7 +15,7 @@ def __str__(self): class Student(models.Model): name = models.CharField(max_length=30, verbose_name='Имя') - teacher = models.ForeignKey(Teacher, on_delete=models.CASCADE) + teacher = models.ManyToManyField(Teacher, related_name='students') group = models.CharField(max_length=10, verbose_name='Класс') class Meta: diff --git a/2.2-databases-2/orm_migrations/school/urls.py b/2.2-databases-2/orm_migrations/school/urls.py index 68ac9dd12..df266224f 100644 --- a/2.2-databases-2/orm_migrations/school/urls.py +++ b/2.2-databases-2/orm_migrations/school/urls.py @@ -1,7 +1,12 @@ -from django.urls import path +import debug_toolbar +from django.contrib import admin +from django.urls import include, path +from django.conf import settings from school.views import students_list urlpatterns = [ path('', students_list, name='students'), + path('__debug__', include(debug_toolbar.urls)), + path('admin/', admin.site.urls) ] diff --git a/2.2-databases-2/orm_migrations/school/views.py b/2.2-databases-2/orm_migrations/school/views.py index e3cb4d625..51facae03 100644 --- a/2.2-databases-2/orm_migrations/school/views.py +++ b/2.2-databases-2/orm_migrations/school/views.py @@ -6,8 +6,8 @@ def students_list(request): template = 'school/students_list.html' - context = {} - + students = Student.objects.all().prefetch_related('teacher') + context = {'students': students} # используйте этот параметр для упорядочивания результатов # https://docs.djangoproject.com/en/2.2/ref/models/querysets/#django.db.models.query.QuerySet.order_by ordering = 'group' diff --git a/2.2-databases-2/orm_migrations/templates/school/students_list.html b/2.2-databases-2/orm_migrations/templates/school/students_list.html index d7d9b2d21..0ab756548 100644 --- a/2.2-databases-2/orm_migrations/templates/school/students_list.html +++ b/2.2-databases-2/orm_migrations/templates/school/students_list.html @@ -5,8 +5,12 @@ {% block content %}
    - {% for student in object_list %} -
  • {{ student.name }} {{ student.group }}
    Преподаватель: {{ student.teacher.name }} {{ student.teacher.subject }}
  • + {% for student in students %} +
  • {{ student.name }} {{ student.group }}
    Преподаватель(и): + {% for teacher in student.teacher.all %} +

    {{ teacher.name }}: {{ teacher.subject }}

    + {% endfor %} +
  • {% endfor %}
diff --git a/2.2-databases-2/orm_migrations/website/settings.py b/2.2-databases-2/orm_migrations/website/settings.py index 32bc56c38..76702fa4a 100644 --- a/2.2-databases-2/orm_migrations/website/settings.py +++ b/2.2-databases-2/orm_migrations/website/settings.py @@ -36,10 +36,12 @@ 'django.contrib.sessions', 'django.contrib.messages', 'django.contrib.staticfiles', + 'debug_toolbar', 'school', ] MIDDLEWARE = [ + 'debug_toolbar.middleware.DebugToolbarMiddleware', 'django.middleware.security.SecurityMiddleware', 'django.contrib.sessions.middleware.SessionMiddleware', 'django.middleware.common.CommonMiddleware', @@ -126,6 +128,10 @@ MEDIA_ROOT = os.path.join(BASE_DIR, 'media') +INTERNAL_IPS = [ + '127.0.0.1', +] + try: from .settings_local import * except ImportError: diff --git a/3.1-drf-intro/smart_home/README.md b/3.1-drf-intro/smart_home/README.md index 8f370abdc..5ded488ba 100644 --- a/3.1-drf-intro/smart_home/README.md +++ b/3.1-drf-intro/smart_home/README.md @@ -2,13 +2,17 @@ ## Техническая задача: реализовать некоторые действия из CRUD, используя Django Rest Framework. -**CRUD** – аббревиатура для Create-Read-Update-Delete. Ей обозначают логику для операций создания-чтения-обновления-удаления сущностей. Подробнее: https://ru.wikipedia.org/wiki/CRUD +**CRUD** – аббревиатура для Create-Read-Update-Delete. Ей обозначают логику для операций создания-чтения- +обновления-удаления сущностей. Подробнее: https://ru.wikipedia.org/wiki/CRUD ## Описание -У нас есть программируемые датчики, измеряющие температуру. Раз в некоторый интервал времени датчики делают запрос по API и записывают свои показания. В показания датчики передают свой ID и текущую температуру в градусах Цельсия. +У нас есть программируемые датчики, измеряющие температуру. Раз в некоторый интервал времени датчики делают +запрос по API и записывают свои показания. В показания датчики передают свой ID и текущую температуру в +градусах Цельсия. -Необходимо реализовать REST API для добавления/изменения датчиков, их просмотра и добавления новых измерений температуры. +Необходимо реализовать REST API для добавления/изменения датчиков, их просмотра и добавления новых +измерений температуры. Требуется задать 2 модели (они уже описаны в models.py): @@ -49,7 +53,8 @@ ] ``` -5. Получить информацию по конкретному датчику. Выдается полная информация по датчику: ID, название, описание и список всех измерений с температурой и временем. +5. Получить информацию по конкретному датчику. Выдается полная информация по датчику: ID, название, + описание и список всех измерений с температурой и временем. ```json { @@ -73,11 +78,14 @@ ## Подсказки -1. Вам необходимо будет изменить файлы `models.py`, `serializers.py`, `views.py` и `urls.py`. В места, где нужно добавлять код, включены `TODO`-комментарии. После того как вы добавите код, комментарии можно удалить. +1. Вам необходимо будет изменить файлы `models.py`, `serializers.py`, `views.py` и `urls.py`. В места, где нужно + добавлять код, включены `TODO`-комментарии. После того как вы добавите код, комментарии можно удалить. -2. Для автоматического проставления времени используйте аргументы: `auto_now` (при обновлении) и `auto_now_add` (при создании). Подробнее: https://docs.djangoproject.com/en/3.1/ref/models/fields/#django.db.models.DateField. +2. Для автоматического проставления времени используйте аргументы: `auto_now` (при обновлении) и + `auto_now_add` (при создании). Подробнее: https://docs.djangoproject.com/en/3.1/ref/models/fields/#django.db.models.DateField. -3. Для сериализатора с подробной информацией по датчику для отображения списка измерений необходимо использовать [вложенный сериализатор](https://www.django-rest-framework.org/api-guide/serializers/#dealing-with-nested-objects). Должен получиться примерно такой код: +3. Для сериализатора с подробной информацией по датчику для отображения списка измерений необходимо + использовать [вложенный сериализатор](https://www.django-rest-framework.org/api-guide/serializers/#dealing-with-nested-objects). Должен получиться примерно такой код: ```python class MeasurementSerializer(serializers.ModelSerializer): @@ -98,9 +106,11 @@ class SensorDetailSerializer(serializers.ModelSerializer): ### Прикрепление картинки к измерению -Датчики стали более продвинутыми и могут также прикреплять снимки. Добавьте nullable-поле к модели `Measurement` для сохранения изображений https://www.django-rest-framework.org/api-guide/fields/#imagefield +Датчики стали более продвинутыми и могут также прикреплять снимки. Добавьте nullable-поле к модели +`Measurement` для сохранения изображений https://www.django-rest-framework.org/api-guide/fields/#imagefield -Обратите внимание, что поле должно быть опциональным – некоторые датчики прикладывают фото, а некоторые нет. Для старых датчиков ничего не должно сломаться. +Обратите внимание, что поле должно быть опциональным – некоторые датчики прикладывают фото, +а некоторые нет. Для старых датчиков ничего не должно сломаться. ## Документация по проекту diff --git a/3.1-drf-intro/smart_home/measurement/admin.py b/3.1-drf-intro/smart_home/measurement/admin.py index 8c38f3f3d..238488398 100644 --- a/3.1-drf-intro/smart_home/measurement/admin.py +++ b/3.1-drf-intro/smart_home/measurement/admin.py @@ -1,3 +1,21 @@ from django.contrib import admin -# Register your models here. +from measurement.models import Sensor, Measurement + + +class MeasurementInline(admin.TabularInline): + model = Measurement + extra = 0 + + +@admin.register(Sensor) +class SensorAdmin(admin.ModelAdmin): + list_display = ['id', 'title', 'description'] + list_filter = ['title', 'description'] + inlines = [MeasurementInline, ] + + +@admin.register(Measurement) +class MeasurementAdmin(admin.ModelAdmin): + list_display = ['id', 'sensor_id', 'temperature', 'image', 'measurement_date'] + list_filter = ['sensor_id', 'measurement_date'] diff --git a/3.1-drf-intro/smart_home/measurement/models.py b/3.1-drf-intro/smart_home/measurement/models.py index 54f0ea2e4..bc6603467 100644 --- a/3.1-drf-intro/smart_home/measurement/models.py +++ b/3.1-drf-intro/smart_home/measurement/models.py @@ -1,3 +1,31 @@ from django.db import models # TODO: опишите модели датчика (Sensor) и измерения (Measurement) + + +class Sensor(models.Model): + title = models.CharField(max_length=50, verbose_name='Название датчика') + description = models.CharField(max_length=200, verbose_name='Описание датчика') + + class Meta: + verbose_name = 'Датчик' + verbose_name_plural = 'Датчики' + ordering = ['id'] + + def __str__(self): + return self.title + + +class Measurement(models.Model): + sensor_id = models.ForeignKey(Sensor, on_delete=models.CASCADE, related_name='measurements') + temperature = models.FloatField(verbose_name='Температура при измерении') + image = models.ImageField(upload_to='sensor_images/%Y%m%d/', null=True, verbose_name='Фото с датчика') + measurement_date = models.DateTimeField(auto_now=True, verbose_name='Дата измерения') + + class Meta: + verbose_name = 'Измерение' + verbose_name_plural = 'Измерения' + ordering = ['measurement_date'] + + def __str__(self): + return f'Дата измерения: {self.measurement_date}, Температура: {self.temperature}' diff --git a/3.1-drf-intro/smart_home/measurement/serializers.py b/3.1-drf-intro/smart_home/measurement/serializers.py index 5f2dcc713..48c00acff 100644 --- a/3.1-drf-intro/smart_home/measurement/serializers.py +++ b/3.1-drf-intro/smart_home/measurement/serializers.py @@ -1,3 +1,28 @@ from rest_framework import serializers +from measurement.models import Sensor, Measurement -# TODO: опишите необходимые сериализаторы + +class SensorSerializer(serializers.ModelSerializer): + class Meta: + model = Sensor + fields = ['id', 'title', 'description'] + + +class MeasurementSerializer(serializers.ModelSerializer): + class Meta: + model = Measurement + fields = ['id', 'sensor_id', 'temperature', 'image', 'measurement_date'] + + +class MeasurementForSensorSerializer(serializers.ModelSerializer): + class Meta: + model = Measurement + fields = ['temperature', 'image', 'measurement_date'] + + +class SensorDetailsSerializer(serializers.ModelSerializer): + measurements = MeasurementForSensorSerializer(read_only=True, many=True) + + class Meta: + model = Sensor + fields = ['id', 'title', 'description', 'measurements'] \ No newline at end of file diff --git a/3.1-drf-intro/smart_home/measurement/urls.py b/3.1-drf-intro/smart_home/measurement/urls.py index f3e565996..8537039db 100644 --- a/3.1-drf-intro/smart_home/measurement/urls.py +++ b/3.1-drf-intro/smart_home/measurement/urls.py @@ -1,5 +1,10 @@ from django.urls import path +from measurement.views import SensorsView, SensorView, MeasurementsView + urlpatterns = [ + path('sensors', SensorsView.as_view(), name='sensors'), + path('sensors/', SensorView.as_view(), name='sensor'), + path('measurements', MeasurementsView.as_view(), name='measurements') # TODO: зарегистрируйте необходимые маршруты ] diff --git a/3.1-drf-intro/smart_home/measurement/views.py b/3.1-drf-intro/smart_home/measurement/views.py index 257bca6a1..b3f5e3d35 100644 --- a/3.1-drf-intro/smart_home/measurement/views.py +++ b/3.1-drf-intro/smart_home/measurement/views.py @@ -1,2 +1,44 @@ # TODO: опишите необходимые обработчики, рекомендуется использовать generics APIView классы: # TODO: ListCreateAPIView, RetrieveUpdateAPIView, CreateAPIView +from rest_framework.generics import ListAPIView, RetrieveAPIView +from rest_framework.response import Response + +from measurement.models import Sensor, Measurement +from measurement.serializers import SensorSerializer, MeasurementSerializer, SensorDetailsSerializer + + +class SensorsView(ListAPIView): + queryset = Sensor.objects.all() + serializer_class = SensorSerializer + + def post(self, request): + title = request.data['name'] + description = request.data['description'] + sensor = Sensor.objects.create(title=title, description=description) + return Response(f'Датчик {sensor} добавлен') + + +class SensorView(RetrieveAPIView): + queryset = Sensor.objects.all() + serializer_class = SensorDetailsSerializer + + def patch(self, request, pk): + sensor = Sensor.objects.get(pk=pk) + if request.data.get('description'): + sensor.description = request.data['description'] + if request.data.get('title'): + sensor.title = request.data['title'] + sensor.save() + return Response(f'Датчик {sensor} изменён') + + +class MeasurementsView(ListAPIView): + queryset = Measurement.objects.all() + serializer_class = MeasurementSerializer + + def post(self, request): + sensor = Sensor.objects.get(pk=request.data['sensor']) + temperature = request.data['temperature'] + image = request.data.get('image') + mes = Measurement.objects.create(sensor_id=sensor, image=image, temperature=temperature) + return Response(f'{mes}. Измерение для датчика {sensor} добавлено') diff --git a/3.1-drf-intro/smart_home/media/sensor_images/20220824/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg b/3.1-drf-intro/smart_home/media/sensor_images/20220824/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg new file mode 100644 index 000000000..da4c4714c Binary files /dev/null and b/3.1-drf-intro/smart_home/media/sensor_images/20220824/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg differ diff --git a/3.1-drf-intro/smart_home/media/sensor_images/20220824/sensor_lcd_04.png b/3.1-drf-intro/smart_home/media/sensor_images/20220824/sensor_lcd_04.png new file mode 100644 index 000000000..45ae675ea Binary files /dev/null and b/3.1-drf-intro/smart_home/media/sensor_images/20220824/sensor_lcd_04.png differ diff --git a/3.1-drf-intro/smart_home/media/sensor_images/20220825/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg b/3.1-drf-intro/smart_home/media/sensor_images/20220825/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg new file mode 100644 index 000000000..da4c4714c Binary files /dev/null and b/3.1-drf-intro/smart_home/media/sensor_images/20220825/datchik-temperatury-i-vlazhnosti-xiaomi-mi-2.jpg differ diff --git a/3.1-drf-intro/smart_home/media/sensor_images/20220825/sensor_lcd_04.png b/3.1-drf-intro/smart_home/media/sensor_images/20220825/sensor_lcd_04.png new file mode 100644 index 000000000..45ae675ea Binary files /dev/null and b/3.1-drf-intro/smart_home/media/sensor_images/20220825/sensor_lcd_04.png differ diff --git a/3.1-drf-intro/smart_home/requirements.txt b/3.1-drf-intro/smart_home/requirements.txt index fd625f9e8..809accc43 100644 --- a/3.1-drf-intro/smart_home/requirements.txt +++ b/3.1-drf-intro/smart_home/requirements.txt @@ -1,3 +1,7 @@ -django -djangorestframework -psycopg2-binary +asgiref==3.5.2 +Django==4.1 +djangorestframework==3.13.1 +Pillow==9.2.0 +psycopg2-binary==2.9.3 +pytz==2022.2.1 +sqlparse==0.4.2 diff --git a/3.1-drf-intro/smart_home/smart_home/settings.py b/3.1-drf-intro/smart_home/smart_home/settings.py index b166a601f..e166a0267 100644 --- a/3.1-drf-intro/smart_home/smart_home/settings.py +++ b/3.1-drf-intro/smart_home/smart_home/settings.py @@ -9,7 +9,7 @@ For the full list of settings and their values, see https://docs.djangoproject.com/en/3.2/ref/settings/ """ - +import os from pathlib import Path # Build paths inside the project like this: BASE_DIR / 'subdir'. @@ -129,3 +129,6 @@ # https://docs.djangoproject.com/en/3.2/ref/settings/#default-auto-field DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField' + +MEDIA_ROOT = os.path.join(BASE_DIR, 'media') +MEDIA_URL = '/media/' \ No newline at end of file diff --git a/3.1-drf-intro/smart_home/smart_home/urls.py b/3.1-drf-intro/smart_home/smart_home/urls.py index 5c5c18d97..19a1bad6f 100644 --- a/3.1-drf-intro/smart_home/smart_home/urls.py +++ b/3.1-drf-intro/smart_home/smart_home/urls.py @@ -13,10 +13,16 @@ 1. Import the include() function: from django.urls import include, path 2. Add a URL to urlpatterns: path('blog/', include('blog.urls')) """ +from django.conf.urls.static import static from django.contrib import admin from django.urls import path, include +from smart_home import settings + urlpatterns = [ path('admin/', admin.site.urls), path('api/', include('measurement.urls')), # подключаем маршруты из приложения measurement ] + +if settings.DEBUG: + urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT) \ No newline at end of file diff --git a/3.2-crud/stocks_products/README.md b/3.2-crud/stocks_products/README.md index 3e686137e..c2895ab74 100644 --- a/3.2-crud/stocks_products/README.md +++ b/3.2-crud/stocks_products/README.md @@ -2,15 +2,21 @@ ## Техническая задача: реализовать CRUD-логику для продуктов и складов, используя Django Rest Framework. -**CRUD** – аббревиатура для Create-Read-Update-Delete. Ей обозначают логику для операций создания-чтения-обновления-удаления сущностей. Подробнее: https://ru.wikipedia.org/wiki/CRUD +**CRUD** – аббревиатура для Create-Read-Update-Delete. Ей обозначают логику для операций создания-чтения-обновления- +удаления сущностей. Подробнее: https://ru.wikipedia.org/wiki/CRUD ## Описание -У нас есть продукты, которыми торгует компания. Продукты описываются названием и необязательным описанием (см. `models.py`). Также компания имеет ряд складов, на которых эти продукты хранятся. У продукта на складе есть стоимость хранения, поэтому один и тот же продукт может иметь разные стоимости на разных складах. +У нас есть продукты, которыми торгует компания. Продукты описываются названием и необязательным описанием +(см. `models.py`). Также компания имеет ряд складов, на которых эти продукты хранятся. У продукта на складе есть +стоимость хранения, поэтому один и тот же продукт может иметь разные стоимости на разных складах. -Необходимо реализовать REST API для создания/получения/обновления/удаления продуктов и складов. Так как склады имеют информацию о своих продуктах (через связанную таблицу) - необходимо переопределить методы создания и обновления объектов в сериализаторе (см. `serializers.py`). +Необходимо реализовать REST API для создания/получения/обновления/удаления продуктов и складов. Так как склады +имеют информацию о своих продуктах (через связанную таблицу) - необходимо переопределить методы создания и +обновления объектов в сериализаторе (см. `serializers.py`). -Помимо CRUD-операций необходимо реализовать поиск продуктов по названиям и описанию. И поиск складов, в которых есть определенный продукт (по идентификатору). Подробности в файле `requests-examples.http`. +Помимо CRUD-операций необходимо реализовать поиск продуктов по названиям и описанию. И поиск складов, в которых +есть определенный продукт (по идентификатору). Подробности в файле `requests-examples.http`. Так как продуктов и складов может быть много, то необходимо реализовать пагинацию для вывода списков. @@ -18,15 +24,18 @@ ## Подсказки -1. Вам необходимо будет задать логику во views и serializers. В места, где нужно добавлять код, включены комментарии. После того как вы добавите код, комментарии можно удалить. +1. Вам необходимо будет задать логику во views и serializers. В места, где нужно добавлять код, включены комментарии. + После того как вы добавите код, комментарии можно удалить. -2. Для обновления объектов удобно использовать метод `update_or_create`: https://docs.djangoproject.com/en/3.2/ref/models/querysets/#update-or-create +2. Для обновления объектов удобно использовать метод `update_or_create`: + ## Дополнительное задание ### Поиск складов с продуктами -Реализуйте поиск складов, в которых есть определенный продукт, но при этом указывать хочется не идентификатор продукта, а название (или его часть) или часть описания. +Реализуйте поиск складов, в которых есть определенный продукт, но при этом указывать хочется не идентификатор +продукта, а название (или его часть) или часть описания. Пример запроса: diff --git a/3.2-crud/stocks_products/logistic/admin.py b/3.2-crud/stocks_products/logistic/admin.py index 8c38f3f3d..6af73bbfb 100644 --- a/3.2-crud/stocks_products/logistic/admin.py +++ b/3.2-crud/stocks_products/logistic/admin.py @@ -1,3 +1,26 @@ from django.contrib import admin -# Register your models here. +from logistic.models import Product, Stock, StockProduct + + +class StockProductInline(admin.TabularInline): + model = StockProduct + extra = 0 + + +@admin.register(Product) +class ProductAdmin(admin.ModelAdmin): + list_display = ['id', 'title', 'description', 'stocks'] + list_filter = ['title'] + + +@admin.register(Stock) +class StockAdmin(admin.ModelAdmin): + list_display = ['id', 'address', 'positions'] + list_filter = ['id', 'address'] + inlines = [StockProductInline, ] + + +@admin.register(StockProduct) +class StockProductAdmin(admin.ModelAdmin): + list_display = ['stock', 'product', 'quantity', 'price'] diff --git a/3.2-crud/stocks_products/logistic/models.py b/3.2-crud/stocks_products/logistic/models.py index effc7dd2f..4d08393fc 100644 --- a/3.2-crud/stocks_products/logistic/models.py +++ b/3.2-crud/stocks_products/logistic/models.py @@ -6,6 +6,13 @@ class Product(models.Model): title = models.CharField(max_length=60, unique=True) description = models.TextField(null=True, blank=True) + class Meta: + verbose_name = 'Продукт' + verbose_name_plural = 'Продукты' + + def __str__(self): + return self.title + class Stock(models.Model): address = models.CharField(max_length=200, unique=True) @@ -15,19 +22,26 @@ class Stock(models.Model): related_name='stocks', ) + class Meta: + verbose_name = 'Склад' + verbose_name_plural = 'Склады' + + def __str__(self): + return self.address + class StockProduct(models.Model): stock = models.ForeignKey( Stock, on_delete=models.CASCADE, - related_name='positions', + related_name='positions' ) product = models.ForeignKey( Product, on_delete=models.CASCADE, related_name='positions', ) - quantity = models.PositiveIntegerField(default=1) + quantity = models.PositiveIntegerField(default=1, verbose_name='Количество') price = models.DecimalField( max_digits=18, decimal_places=2, diff --git a/3.2-crud/stocks_products/logistic/serializers.py b/3.2-crud/stocks_products/logistic/serializers.py index af8b3fd4e..a179610be 100644 --- a/3.2-crud/stocks_products/logistic/serializers.py +++ b/3.2-crud/stocks_products/logistic/serializers.py @@ -1,20 +1,26 @@ from rest_framework import serializers +from logistic.models import Product, Stock, StockProduct + class ProductSerializer(serializers.ModelSerializer): - # настройте сериализатор для продукта - pass + class Meta: + model = Product + fields = ['id', 'title', 'description', 'stocks'] class ProductPositionSerializer(serializers.ModelSerializer): - # настройте сериализатор для позиции продукта на складе - pass + class Meta: + model = StockProduct + fields = ['product', 'quantity', 'price'] class StockSerializer(serializers.ModelSerializer): positions = ProductPositionSerializer(many=True) - # настройте сериализатор для склада + class Meta: + model = Stock + fields = ['id', 'address', 'positions'] def create(self, validated_data): # достаем связанные данные для других таблиц @@ -23,9 +29,8 @@ def create(self, validated_data): # создаем склад по его параметрам stock = super().create(validated_data) - # здесь вам надо заполнить связанные таблицы - # в нашем случае: таблицу StockProduct - # с помощью списка positions + for position in positions: + StockProduct.objects.create(stock=stock, **position) return stock @@ -36,8 +41,9 @@ def update(self, instance, validated_data): # обновляем склад по его параметрам stock = super().update(instance, validated_data) - # здесь вам надо обновить связанные таблицы - # в нашем случае: таблицу StockProduct - # с помощью списка positions + for position in positions: + StockProduct.objects.update_or_create(stock=stock, + product=position.pop('product'), + defaults=position) return stock diff --git a/3.2-crud/stocks_products/logistic/views.py b/3.2-crud/stocks_products/logistic/views.py index 101131aac..bdecf8825 100644 --- a/3.2-crud/stocks_products/logistic/views.py +++ b/3.2-crud/stocks_products/logistic/views.py @@ -1,3 +1,5 @@ +from django_filters.rest_framework import DjangoFilterBackend +from rest_framework.pagination import LimitOffsetPagination from rest_framework.viewsets import ModelViewSet from logistic.models import Product, Stock @@ -7,10 +9,12 @@ class ProductViewSet(ModelViewSet): queryset = Product.objects.all() serializer_class = ProductSerializer - # при необходимости добавьте параметры фильтрации + search_fields = ['title', 'description'] class StockViewSet(ModelViewSet): queryset = Stock.objects.all() serializer_class = StockSerializer - # при необходимости добавьте параметры фильтрации + filter_backends = (DjangoFilterBackend, ) + filterset_fields = ['products'] + search_fields = ['products__title', 'products__description'] diff --git a/3.2-crud/stocks_products/requests-examples.http b/3.2-crud/stocks_products/requests-examples.http index 2d15f24a5..a19d95a13 100644 --- a/3.2-crud/stocks_products/requests-examples.http +++ b/3.2-crud/stocks_products/requests-examples.http @@ -45,6 +45,9 @@ Content-Type: application/json POST {{baseUrl}}/stocks/ Content-Type: application/json + + +### { "address": "мой адрес не дом и не улица, мой адрес сегодня такой: www.ленинград-спб.ru3", "positions": [ @@ -60,9 +63,6 @@ Content-Type: application/json } ] } - -### - # обновляем записи на складе PATCH {{baseUrl}}/stocks/4/ Content-Type: application/json diff --git a/3.2-crud/stocks_products/requirements.txt b/3.2-crud/stocks_products/requirements.txt index fd625f9e8..909ce49ae 100644 --- a/3.2-crud/stocks_products/requirements.txt +++ b/3.2-crud/stocks_products/requirements.txt @@ -1,3 +1,7 @@ -django -djangorestframework -psycopg2-binary +asgiref==3.5.2 +Django==4.1 +django-filter==22.1 +djangorestframework==3.13.1 +psycopg2-binary==2.9.3 +pytz==2022.2.1 +sqlparse==0.4.2 diff --git a/3.2-crud/stocks_products/stocks_products/settings.py b/3.2-crud/stocks_products/stocks_products/settings.py index 40a0dbc56..bd5b28223 100644 --- a/3.2-crud/stocks_products/stocks_products/settings.py +++ b/3.2-crud/stocks_products/stocks_products/settings.py @@ -39,6 +39,7 @@ 'django.contrib.staticfiles', 'rest_framework', + 'django_filters', 'logistic', ] @@ -129,3 +130,9 @@ # https://docs.djangoproject.com/en/3.2/ref/settings/#default-auto-field DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField' + +REST_FRAMEWORK = { + 'DEFAULT_FILTER_BACKENDS': ['rest_framework.filters.SearchFilter'], + 'DEFAULT_PAGINATION_CLASS': 'rest_framework.pagination.PageNumberPagination', + 'PAGE_SIZE': 3 +} \ No newline at end of file diff --git a/3.3-permissions/api_with_restrictions/README.md b/3.3-permissions/api_with_restrictions/README.md index 1c04ca48d..695476877 100644 --- a/3.3-permissions/api_with_restrictions/README.md +++ b/3.3-permissions/api_with_restrictions/README.md @@ -2,11 +2,13 @@ ## Описание -Необходимо реализовать бекенд для мобильного приложения с объявлениями. Объявления можно создавать и просматривать. Есть возможность фильтровать объявления по дате и статусу. +Необходимо реализовать бекенд для мобильного приложения с объявлениями. Объявления можно создавать и +просматривать. Есть возможность фильтровать объявления по дате и статусу. Создавать могут только авторизованные пользователи. Для просмотра объявлений авторизация не нужна. -У объявления есть статусы: `OPEN`, `CLOSED`. Необходимо валидировать, что у пользователя не больше 10 открытых объявлений. +У объявления есть статусы: `OPEN`, `CLOSED`. Необходимо валидировать, что у пользователя не больше 10 +открытых объявлений. Обновлять и удалять объявление может только автор этого объявления. @@ -17,18 +19,21 @@ ## Реализация -- используйте `DateFromToRangeFilter` для фильтрации по дате https://django-filter.readthedocs.io/en/stable/ref/filters.html#datefromtorangefilter +- используйте `DateFromToRangeFilter` для фильтрации по дате +- https://django-filter.readthedocs.io/en/stable/ref/filters.html#datefromtorangefilter Пример работы: ![Фильтрация по дате](./screenshots/date_filter.png) -- В настройках подключено приложение `rest_framework.authtoken` и сконфигурирован `DEFAULT_AUTHENTICATION_CLASSES`. Для того, чтобы завести токен для пользователя, проделайте следующие шаги: +- В настройках подключено приложение `rest_framework.authtoken` и сконфигурирован `DEFAULT_AUTHENTICATION_CLASSES`. +- Для того, чтобы завести токен для пользователя, проделайте следующие шаги: - создайте пользователя через админку - также через админку, заведите ему токен - этот токен используйте в запросах, передавая его в заголовках -- Так как интерфейс BrowserableAPI в DRF не позволяет передавать заголовки с токеном, используйте Postman или HTTP-клиент VSCode. +- Так как интерфейс BrowserableAPI в DRF не позволяет передавать заголовки с токеном, используйте Postman или +- HTTP-клиент VSCode. Примеры: @@ -38,12 +43,15 @@ Неправильный токен: ![Неправильный токен](./screenshots/bad_token.png) -- Для переопределения доступов для отдельных методов `ViewSet` используется метод `get_permissions` (добавлен в заготовку, следует с ним ознакомиться и посмотреть с помощью breakpoint'ов в какой момент DRF его вызывает) +- Для переопределения доступов для отдельных методов `ViewSet` используется метод `get_permissions` + (добавлен в заготовку, следует с ним ознакомиться и посмотреть с помощью breakpoint'ов в какой момент DRF его вызывает) - валидацию удаления чужого объявления следует делать: - либо внутри метода `destroy` https://www.django-rest-framework.org/api-guide/viewsets/#viewset-actions (чуть проще) - - либо определяя дополнительный класс-наследник `BasePermission`, дополнительно добавляя его в список `get_permissions` https://www.django-rest-framework.org/api-guide/permissions/#examples (правильнее и этот класс можно переиспользовать для других методов) + - либо определяя дополнительный класс-наследник `BasePermission`, дополнительно добавляя его в список + `get_permissions` https://www.django-rest-framework.org/api-guide/permissions/#examples (правильнее и этот класс можно + переиспользовать для других методов) Любой вариант допустим в рамках данного задания. @@ -51,17 +59,21 @@ # ВАЖНО -Приложение называется `advertisements`, такие слова являются триггерами для блокировщиков рекламы (например, uBlock Origin). Рекомендуется отключить блокировщик, если вы пользуетесь таковым, либо переименовать приложение. +Приложение называется `advertisements`, такие слова являются триггерами для блокировщиков рекламы (например, +uBlock Origin). Рекомендуется отключить блокировщик, если вы пользуетесь таковым, либо переименовать приложение. ## Подсказки 1. В места, где нужно добавлять код, включены `TODO`-комментарии. -2. Ознакомьтесь целиком со структурой проекта. Разберите, как работает код в заготовке, например, проставляется поле создателя объявления таким образом, чтобы злоумышленник не мог создавать объявления от чужого лица. +2. Ознакомьтесь целиком со структурой проекта. Разберите, как работает код в заготовке, например, проставляется + поле создателя объявления таким образом, чтобы злоумышленник не мог создавать объявления от чужого лица. -3. Используйте возможность указывать `fields` в `Meta` внутри `FilterSet` класса, чтобы не задавать фильтры, которые могут сгенерироваться автоматически. +3. Используйте возможность указывать `fields` в `Meta` внутри `FilterSet` класса, чтобы не задавать фильтры, которые + могут сгенерироваться автоматически. -4. Админка Джанго по-умолчанию даст возможность создания и редактирования пользователей и токенов. Этим удобно пользоваться для локального создания сущностей. +4. Админка Джанго по-умолчанию даст возможность создания и редактирования пользователей и токенов. Этим удобно + пользоваться для локального создания сущностей. ## Доп-задания (не обязательные к выполнению) @@ -71,13 +83,17 @@ ### Избранные объявления -- Добавить возможность добавлять объявления в избранное. Автор объявления не может добавить свое объявление в избранное. Должна быть возможность фильтрации по избранным объявлениям. Например, пользователь хочет посмотреть все объявления, которые он добавил в избранное. +- Добавить возможность добавлять объявления в избранное. Автор объявления не может добавить свое объявление в + избранное. Должна быть возможность фильтрации по избранным объявлениям. Например, пользователь хочет + посмотреть все объявления, которые он добавил в избранное. -- для того, чтобы добавить дополнительный метод с урлом во ViewSet, вам может пригодиться декоратор `action` из `DRF`. +- для того, чтобы добавить дополнительный метод с урлом во ViewSet, вам может пригодиться декоратор + `action` из `DRF`. ### Добавить статус `DRAFT` -- добавьте статус `DRAFT` - черновик. Пока объявление в черновике, оно показывается только автору объявления, другим пользователям оно недоступно. +- добавьте статус `DRAFT` - черновик. Пока объявление в черновике, оно показывается только автору объявления, + другим пользователям оно недоступно. ## Документация по проекту diff --git a/3.3-permissions/api_with_restrictions/advertisements/admin.py b/3.3-permissions/api_with_restrictions/advertisements/admin.py index 8c38f3f3d..925adeec9 100644 --- a/3.3-permissions/api_with_restrictions/advertisements/admin.py +++ b/3.3-permissions/api_with_restrictions/advertisements/admin.py @@ -1,3 +1,4 @@ from django.contrib import admin -# Register your models here. + + diff --git a/3.3-permissions/api_with_restrictions/advertisements/filters.py b/3.3-permissions/api_with_restrictions/advertisements/filters.py index 455b636e8..2d1e7424e 100644 --- a/3.3-permissions/api_with_restrictions/advertisements/filters.py +++ b/3.3-permissions/api_with_restrictions/advertisements/filters.py @@ -1,4 +1,4 @@ -from django_filters import rest_framework as filters +from django_filters import rest_framework as filters, DateFromToRangeFilter from advertisements.models import Advertisement @@ -6,7 +6,9 @@ class AdvertisementFilter(filters.FilterSet): """Фильтры для объявлений.""" - # TODO: задайте требуемые фильтры + created_at = DateFromToRangeFilter() + updated_at = DateFromToRangeFilter() class Meta: model = Advertisement + fields = ('id', 'created_at', 'updated_at', 'status') diff --git a/3.3-permissions/api_with_restrictions/advertisements/models.py b/3.3-permissions/api_with_restrictions/advertisements/models.py index 2376e499c..e29860651 100644 --- a/3.3-permissions/api_with_restrictions/advertisements/models.py +++ b/3.3-permissions/api_with_restrictions/advertisements/models.py @@ -1,5 +1,6 @@ from django.conf import settings from django.db import models +from django.forms import BaseInlineFormSet class AdvertisementStatusChoices(models.TextChoices): @@ -28,3 +29,10 @@ class Advertisement(models.Model): updated_at = models.DateTimeField( auto_now=True ) + draft = models.BooleanField(default=False) + + +class Favorite(models.Model): + owner = models.ForeignKey(settings.AUTH_USER_MODEL, on_delete=models.CASCADE) + ad = models.ForeignKey(Advertisement, on_delete=models.CASCADE, related_name='favorites') + diff --git a/3.3-permissions/api_with_restrictions/advertisements/permissions.py b/3.3-permissions/api_with_restrictions/advertisements/permissions.py new file mode 100644 index 000000000..b951f6942 --- /dev/null +++ b/3.3-permissions/api_with_restrictions/advertisements/permissions.py @@ -0,0 +1,14 @@ +from rest_framework import permissions +from rest_framework.permissions import BasePermission + + +class IsOwner(BasePermission): + def has_object_permission(self, request, view, obj): + return request.user.is_staff or request.user == obj.creator + + +class IsOwnerDraft(BasePermission): + def has_object_permission(self, request, view, obj): + if request.method in [*permissions.SAFE_METHODS, 'POST'] and not obj.draft: + return True + return obj.draft and obj.creator == request.user diff --git a/3.3-permissions/api_with_restrictions/advertisements/serializers.py b/3.3-permissions/api_with_restrictions/advertisements/serializers.py index be8ccadfa..df7abe846 100644 --- a/3.3-permissions/api_with_restrictions/advertisements/serializers.py +++ b/3.3-permissions/api_with_restrictions/advertisements/serializers.py @@ -1,5 +1,6 @@ from django.contrib.auth.models import User from rest_framework import serializers +from rest_framework.exceptions import ValidationError from advertisements.models import Advertisement @@ -16,14 +17,12 @@ class Meta: class AdvertisementSerializer(serializers.ModelSerializer): """Serializer для объявления.""" - creator = UserSerializer( - read_only=True, - ) + creator = UserSerializer(read_only=True, ) class Meta: model = Advertisement fields = ('id', 'title', 'description', 'creator', - 'status', 'created_at', ) + 'status', 'created_at', 'draft') def create(self, validated_data): """Метод для создания""" @@ -31,15 +30,22 @@ def create(self, validated_data): # Простановка значения поля создатель по-умолчанию. # Текущий пользователь является создателем объявления # изменить или переопределить его через API нельзя. - # обратите внимание на `context` – он выставляется автоматически + # Обратите внимание на `context` – он выставляется автоматически # через методы ViewSet. - # само поле при этом объявляется как `read_only=True` + # Само поле при этом объявляется как `read_only=True` validated_data["creator"] = self.context["request"].user return super().create(validated_data) def validate(self, data): """Метод для валидации. Вызывается при создании и обновлении.""" + request = self.context['request'].method + user = self.context['request'].user - # TODO: добавьте требуемую валидацию + if request == 'POST' and Advertisement.objects.filter(creator=user, status='OPEN').count() >= 10: + raise ValidationError('You cannot create more than 10 open ads') + + if request in ('PATCH', 'PUT') and data.get('status') == 'OPEN' \ + and Advertisement.objects.filter(creator=user, status='OPEN').count() >= 10: + raise ValidationError('You cannot create more than 10 open ads') return data diff --git a/3.3-permissions/api_with_restrictions/advertisements/views.py b/3.3-permissions/api_with_restrictions/advertisements/views.py index 996b6c345..7b328b5a8 100644 --- a/3.3-permissions/api_with_restrictions/advertisements/views.py +++ b/3.3-permissions/api_with_restrictions/advertisements/views.py @@ -1,15 +1,58 @@ +from django.db.models import Q +from django_filters.rest_framework import DjangoFilterBackend +from rest_framework.decorators import action +from rest_framework.exceptions import ValidationError from rest_framework.permissions import IsAuthenticated +from rest_framework.response import Response from rest_framework.viewsets import ModelViewSet +from advertisements.filters import AdvertisementFilter +from advertisements.models import Advertisement, Favorite +from advertisements.permissions import IsOwner, IsOwnerDraft +from advertisements.serializers import AdvertisementSerializer + class AdvertisementViewSet(ModelViewSet): """ViewSet для объявлений.""" - # TODO: настройте ViewSet, укажите атрибуты для кверисета, - # сериализаторов и фильтров + queryset = Advertisement.objects.all() + serializer_class = AdvertisementSerializer + + filter_backends = [DjangoFilterBackend] + filterset_class = AdvertisementFilter + + @action(detail=False, permission_classes=[IsAuthenticated]) + def favorites(self, request): + queryset = Advertisement.objects.filter(favorites__owner=request.user) + serializer = self.get_serializer(queryset, many=True) + return Response(serializer.data) + + @action(detail=True, methods=['post'], permission_classes=[IsAuthenticated]) + def favorite(self, request, pk): + ad = self.get_object() + user = request.user + + if user == ad.creator: + raise ValidationError('You cannot add your listings to favorites') + + if Favorite.objects.filter(owner=user, ad=ad).ordered: + raise ValidationError('This ad is already in your favorites') + + Favorite.objects.create(owner=user, ad=ad) + return Response('Ad added to favorites') + + def get_queryset(self): + if self.request.user.pk is None: + self.queryset = Advertisement.objects.filter(draft=False) + else: + self.queryset = Advertisement.objects.filter(Q(creator=self.request.user) | Q(draft=False)) + return self.queryset def get_permissions(self): """Получение прав для действий.""" - if self.action in ["create", "update", "partial_update"]: + if self.action in ["create"]: return [IsAuthenticated()] - return [] + elif self.action in ["update", "partial_update", "destroy"]: + return [IsOwner()] + else: + return [IsOwnerDraft()] diff --git a/3.3-permissions/api_with_restrictions/api_with_restrictions/settings.py b/3.3-permissions/api_with_restrictions/api_with_restrictions/settings.py index f5e94d9fc..839977640 100644 --- a/3.3-permissions/api_with_restrictions/api_with_restrictions/settings.py +++ b/3.3-permissions/api_with_restrictions/api_with_restrictions/settings.py @@ -76,7 +76,14 @@ REST_FRAMEWORK = { 'DEFAULT_AUTHENTICATION_CLASSES': [ 'rest_framework.authentication.TokenAuthentication', - ] + ], + 'DEFAULT_THROTTLE_CLASSES': [ + 'rest_framework.throttling.UserRateThrottle' + ], + 'DEFAULT_THROTTLE_RATES': { + 'user': '20/min', + 'anon': '10/min' + } } WSGI_APPLICATION = 'api_with_restrictions.wsgi.application' diff --git a/3.3-permissions/api_with_restrictions/api_with_restrictions/urls.py b/3.3-permissions/api_with_restrictions/api_with_restrictions/urls.py index b13dcf24b..0317701fe 100644 --- a/3.3-permissions/api_with_restrictions/api_with_restrictions/urls.py +++ b/3.3-permissions/api_with_restrictions/api_with_restrictions/urls.py @@ -18,8 +18,10 @@ from rest_framework.routers import DefaultRouter +from advertisements.views import AdvertisementViewSet + router = DefaultRouter() -# TODO: подключите `AdvertisementViewSet` +router.register('advertisements', AdvertisementViewSet) urlpatterns = [ diff --git a/3.3-permissions/api_with_restrictions/requirements.txt b/3.3-permissions/api_with_restrictions/requirements.txt index 17df7655f..909ce49ae 100644 --- a/3.3-permissions/api_with_restrictions/requirements.txt +++ b/3.3-permissions/api_with_restrictions/requirements.txt @@ -1,4 +1,7 @@ -django -djangorestframework -psycopg2-binary -django-filter +asgiref==3.5.2 +Django==4.1 +django-filter==22.1 +djangorestframework==3.13.1 +psycopg2-binary==2.9.3 +pytz==2022.2.1 +sqlparse==0.4.2 diff --git a/3.4-django-testing/django_testing/.coveragerc b/3.4-django-testing/django_testing/.coveragerc new file mode 100644 index 000000000..ead3ad667 --- /dev/null +++ b/3.4-django-testing/django_testing/.coveragerc @@ -0,0 +1,2 @@ +[run] +omit = tests/*,.venv/*,manage.py,tests/*,django_testing/asgi.py,django_testing/wsgi.py \ No newline at end of file diff --git a/3.4-django-testing/django_testing/django_testing/settings.py b/3.4-django-testing/django_testing/django_testing/settings.py index 6afa0057e..78670fe21 100644 --- a/3.4-django-testing/django_testing/django_testing/settings.py +++ b/3.4-django-testing/django_testing/django_testing/settings.py @@ -81,6 +81,8 @@ 'default': { 'ENGINE': 'django.db.backends.postgresql', 'NAME': 'netology_django_testing', + 'USER': 'postgres', + 'PASSWORD': '1', 'HOST': '127.0.0.1', 'PORT': '5432', } @@ -124,3 +126,9 @@ # https://docs.djangoproject.com/en/3.1/howto/static-files/ STATIC_URL = '/static/' + +REST_FRAMEWORK = { + 'TEST_REQUEST_DEFAULT_FORMAT': 'json' +} + +MAX_STUDENTS_PER_COURSE = 20 \ No newline at end of file diff --git a/3.4-django-testing/django_testing/django_testing/urls.py b/3.4-django-testing/django_testing/django_testing/urls.py index 00430cedb..61c519e89 100644 --- a/3.4-django-testing/django_testing/django_testing/urls.py +++ b/3.4-django-testing/django_testing/django_testing/urls.py @@ -17,10 +17,11 @@ from django.urls import path, include from rest_framework.routers import DefaultRouter -from students.views import CoursesViewSet +from students.views import CoursesViewSet, StudentsViewSet router = DefaultRouter() router.register("courses", CoursesViewSet, basename="courses") +router.register("students", StudentsViewSet, basename="students") urlpatterns = [ path('admin/', admin.site.urls), diff --git a/3.4-django-testing/django_testing/requirements-dev.txt b/3.4-django-testing/django_testing/requirements-dev.txt index 10121c07a..4e49e369b 100644 --- a/3.4-django-testing/django_testing/requirements-dev.txt +++ b/3.4-django-testing/django_testing/requirements-dev.txt @@ -1,4 +1,5 @@ --r requirements.txt +model-r requirements.txt pytest pytest-django model_bakery +pytest-cov \ No newline at end of file diff --git a/3.4-django-testing/django_testing/students/serializers.py b/3.4-django-testing/django_testing/students/serializers.py index 7b6a8ca48..834bd7c28 100644 --- a/3.4-django-testing/django_testing/students/serializers.py +++ b/3.4-django-testing/django_testing/students/serializers.py @@ -1,6 +1,8 @@ from rest_framework import serializers +from rest_framework.exceptions import ValidationError -from students.models import Course +from django_testing.settings import MAX_STUDENTS_PER_COURSE +from students.models import Course, Student class CourseSerializer(serializers.ModelSerializer): @@ -8,3 +10,19 @@ class CourseSerializer(serializers.ModelSerializer): class Meta: model = Course fields = ("id", "name", "students") + + def validate(self, data): + """Метод для валидации. Вызывается при создании и обновлении.""" + students = data.get('students') + print(students) + if len(data.get('students')) > MAX_STUDENTS_PER_COURSE: + raise ValidationError('You cannot add more than 20 students') + + return data + + +class StudentSerializer(serializers.ModelSerializer): + + class Meta: + model = Student + fields = ("id", "name", "birth_date") \ No newline at end of file diff --git a/3.4-django-testing/django_testing/students/views.py b/3.4-django-testing/django_testing/students/views.py index f5db3b729..77caaa1c8 100644 --- a/3.4-django-testing/django_testing/students/views.py +++ b/3.4-django-testing/django_testing/students/views.py @@ -2,8 +2,8 @@ from rest_framework.viewsets import ModelViewSet from students.filters import CourseFilter -from students.models import Course -from students.serializers import CourseSerializer +from students.models import Course, Student +from students.serializers import CourseSerializer, StudentSerializer class CoursesViewSet(ModelViewSet): @@ -12,3 +12,11 @@ class CoursesViewSet(ModelViewSet): serializer_class = CourseSerializer filter_backends = (DjangoFilterBackend, ) filterset_class = CourseFilter + + +class StudentsViewSet(ModelViewSet): + + queryset = Student.objects.all() + serializer_class = StudentSerializer + filter_backends = (DjangoFilterBackend, ) + # filterset_class = CourseFilter \ No newline at end of file diff --git a/3.4-django-testing/django_testing/tests/students/test_courses_api.py b/3.4-django-testing/django_testing/tests/students/test_courses_api.py index ada4669ff..acbee78f5 100644 --- a/3.4-django-testing/django_testing/tests/students/test_courses_api.py +++ b/3.4-django-testing/django_testing/tests/students/test_courses_api.py @@ -1,2 +1,136 @@ -def test_example(): - assert False, "Just test example" +import random + +from model_bakery import baker +from rest_framework.status import HTTP_201_CREATED, HTTP_400_BAD_REQUEST, HTTP_200_OK +from rest_framework.test import APIClient +import pytest + +from students.models import Student, Course +from django_testing import settings + +settings.MAX_STUDENTS_PER_COURSE = 3 + + +@pytest.fixture +def client(): + return APIClient() + + +@pytest.fixture +def students_factory(): + def factory(*args, **kwargs): + return baker.make(Student, *args, **kwargs) + return factory + + +@pytest.fixture +def courses_factory(): + def factory(*args, **kwargs): + return baker.make(Course, *args, **kwargs) + return factory + + +@pytest.mark.django_db +def test_get_course(client, courses_factory): + # Arrange + course = courses_factory(_quantity=1) + + # Act + response = client.get(f'/api/v1/courses/{course[0].id}/') + + # Assert + assert response.status_code == 200 + data = response.json() + assert data['name'] == course[0].name + + +@pytest.mark.django_db +def test_get_courses(client, courses_factory): + # Arrange + courses = courses_factory(_quantity=10) + + # Act + response = client.get('/api/v1/courses/') + + # Assert + assert response.status_code == 200 + data = response.json() + assert len(data) == len(courses) + for i, m in enumerate(data): + assert m['name'] == courses[i].name + + +@pytest.mark.django_db +def test_get_filter_courses(client, courses_factory): + # Arrange + course = courses_factory(_quantity=10) + id = course[random.randint(0, 10)].id + name = course[random.randint(0, 10)].name + + # Act + response_id = client.get(f'/api/v1/courses/?id={id}') + response_name = client.get(f'/api/v1/courses/?name={name}') + + # Assert + assert response_id.status_code == 200 + assert response_name.status_code == 200 + data_id = response_id.json() + data_name = response_name.json() + assert data_id[0]['id'] == id + assert data_name[0]['name'] == name + + +@pytest.mark.parametrize( + ["students_count", "expected_status"], + ((1, HTTP_201_CREATED), + (5, HTTP_400_BAD_REQUEST), + (0, HTTP_201_CREATED), + ) + ) +@pytest.mark.django_db +def test_create_course(client, students_factory, students_count, expected_status): + count = Course.objects.count() + if students_count > 0: + students = students_factory(_quantity=students_count) + students_id = [student.id for student in students] + else: + students_id = [] + count += 1 + + response = client.post('/api/v1/courses/', data={'name': 'Any_course', 'students': students_id}) + + assert response.status_code == expected_status + if response.status_code == HTTP_201_CREATED: + assert Course.objects.count() == count + students_count + + +@pytest.mark.parametrize( + ["students_count", "expected_status"], + ((1, HTTP_200_OK), + (5, HTTP_400_BAD_REQUEST), + ) + ) +@pytest.mark.django_db +def test_update_course(client, students_factory, courses_factory, students_count, expected_status): + course = courses_factory(_quantity=1) + students = students_factory(_quantity=students_count) + students_id = [student.id for student in students] + + response = client.patch(f'/api/v1/courses/{course[0].id}/', data={'students': students_id}) + + assert response.status_code == expected_status + data = response.json() + if response.status_code == 200: + assert data['students'] == students_id + + +@pytest.mark.django_db +def test_delete_course(client, courses_factory): + course = courses_factory(_quantity=1) + id = course[0].id + count = Course.objects.count() + + response = client.delete(f'/api/v1/courses/{id}/') + + assert response.status_code == 204 + assert Course.objects.count() == count - 1