Remove clean() from Recipe model as validation now is handled in forms

Author: ClaireMaud

backend/foodgram/foodapp/admin.py

@@ -1,13 +1,68 @@
+from django import forms
 from django.contrib import admin
 from django.contrib.auth import get_user_model
 from django.contrib.auth.admin import UserAdmin
+from django.core.exceptions import ValidationError
 
+from .constants import INGREDIENT_MIN_AMOUNT, MIN_COOKING_TIME
 from .models import (Favorite, Ingredient, Recipe, RecipeIngredient,
                      ShoppingCart, Tag)
 
 User = get_user_model()
 
 
+class RecipeAdminForm(forms.ModelForm):
+    """Проверка на:
+        - минимальное допустимое время приготовления рецепта
+        - наличие тегов
+        - наличие ингредиентов
+        - отсутствие повторяющихся ингредиентов
+        - минимальное количество ингредиента
+    при добавлении рецепта через админ панель.
+    """
+
+    class Meta:
+        model = Recipe
+        fields = '__all__'
+
+    def clean(self):
+        cleaned_data = super().clean()
+
+        cooking_time = cleaned_data.get("cooking_time")
+        if cooking_time and cooking_time < MIN_COOKING_TIME:
+            raise ValidationError({
+                'cooking_time': f'Минимальное время '
+                                f'приготовления: {MIN_COOKING_TIME}'})
+
+        if not cleaned_data.get("tags").exists():
+            raise ValidationError(
+                {'tags': 'Необходимо указать хотя бы один тег.'})
+
+        instance = self.instance
+        if instance.pk:
+            if not RecipeIngredient.objects.filter(recipe=instance).exists():
+                raise ValidationError({
+                    'recipe_ingredients': 'Список '
+                                          'ингредиентов не может быть '
+                                          'пустым.'})
+
+            ingredients_ids_set = set()
+            for ri in RecipeIngredient.objects.filter(recipe=instance):
+                ingredient_id = ri.ingredient.id
+                if ingredient_id in ingredients_ids_set:
+                    raise ValidationError({
+                        'recipe_ingredients': 'Ингредиенты не должны '
+                                              'повторяться.'})
+                if ri.amount < INGREDIENT_MIN_AMOUNT:
+                    raise ValidationError({
+                        'recipe_ingredients': f'Минимальное количество '
+                                              f'ингредиента: '
+                                              f'{INGREDIENT_MIN_AMOUNT}'})
+                ingredients_ids_set.add(ingredient_id)
+
+        return cleaned_data
+
+
 @admin.register(User)
 class FoodgramUserAdmin(UserAdmin):
     """Админка для пользователей."""
@@ -39,6 +94,7 @@ class RecipeIngredientInline(admin.TabularInline):
 @admin.register(Recipe)
 class RecipeAdmin(admin.ModelAdmin):
     """Админка для рецептов."""
+    form = RecipeAdminForm
     list_display = ('id', 'name', 'author',
                     'get_tags', 'favorite_count')
     search_fields = ('name', 'author__username')

backend/foodgram/foodapp/models.py

@@ -3,13 +3,12 @@
 from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import AbstractUser
-from django.core.exceptions import ValidationError
 from django.core.validators import RegexValidator
 from django.db import models
 
-from .constants import (INGREDIENT_MIN_AMOUNT, INGREDIENT_NAME_MAX_LEN,
-                        MEASUREMENT_UNIT_MAX_LEN, MIN_COOKING_TIME,
-                        RECIPE_HASHCODE_MAX_LEN, RECIPE_NAME_MAX_LEN,
+from .constants import (INGREDIENT_NAME_MAX_LEN,
+                        MEASUREMENT_UNIT_MAX_LEN, RECIPE_HASHCODE_MAX_LEN,
+                        RECIPE_NAME_MAX_LEN,
                         TAG_NAME_MAX_LEN, TAG_SLUG_MAX_LEN,
                         USER_FIRST_NAME_MAX_LEN, USER_LAST_NAME_MAX_LEN,
                         USER_USERNAME_MAX_LEN)
@@ -100,39 +99,6 @@ class Meta:
         verbose_name_plural = 'Рецепты'
         ordering = ('-created_at',)
 
-    def clean(self):
-        """Проверка на:
-            - наличие тегов
-            - минимальное допустимое время приготовления рецепта
-            - отсутствие повторяющихся ингредиентов
-            - минимальное количество ингредиента
-        при добавлении рецепта через админ панель.
-        """
-        if self.cooking_time < MIN_COOKING_TIME:
-            raise ValidationError({
-                'cooking_time': f'Минимальное время '
-                                f'приготовления: {MIN_COOKING_TIME}'})
-        if not self.pk and not self.tags.count():
-            raise ValidationError(
-                {'tags': 'Необходимо указать хотя бы один тег.'})
-        if not self.recipe_ingredients.exists():
-            raise ValidationError(
-                {'recipe_ingredients': 'Список ингредиентов '
-                                       'не может быть пустым.'})
-
-        ingredients_ids_set = set()
-        for ingredient in self.recipe_ingredients.all():
-            ingredient_id = ingredient.ingredient.id
-            if ingredient_id in ingredients_ids_set:
-                raise ValidationError(
-                    {'recipe_ingredients': 'Ингредиенты не должны '
-                                           'повторяться.'})
-            if ingredient.amount < INGREDIENT_MIN_AMOUNT:
-                raise ValidationError(
-                    {'recipe_ingredients': f'Минимальное кол-во ингредиента: '
-                                           f'{INGREDIENT_MIN_AMOUNT}'})
-            ingredients_ids_set.add(ingredient_id)
-
     def __str__(self):
         return self.name