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

Author: ClaireMaud

backend/foodgram/foodapp/admin.py

@@ -1,13 +1,66 @@
+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
+        exclude = ('hashcode',)
+
+    def clean(self):
+        cleaned_data = super().clean()
+
+        cooking_time = cleaned_data.get('cooking_time')
+        if cooking_time is None or cooking_time < MIN_COOKING_TIME:
+            raise ValidationError({
+                'cooking_time': f'Минимальное время '
+                                f'приготовления: {MIN_COOKING_TIME}'})
+
+        tags = cleaned_data.get('tags')
+        if not tags or not tags.exists():
+            raise ValidationError(
+                {'tags': 'Необходимо указать хотя бы один тег.'})
+
+        ingredients_list = self.data.getlist('recipe_ingredients')
+        if not ingredients_list:
+            raise ValidationError(
+                {'recipe_ingredients': 'Список ингредиентов '
+                                       'не может быть пустым.'})
+
+        ingredients_ids_set = set()
+        for ingredient_data in ingredients_list:
+            ingredient_id, amount = map(int, ingredient_data.split(","))
+            if ingredient_id in ingredients_ids_set:
+                raise ValidationError({
+                    'recipe_ingredients': 'Ингредиенты не должны повторяться.'
+                })
+            if amount < INGREDIENT_MIN_AMOUNT:
+                raise ValidationError({
+                    'recipe_ingredients': f'Минимальное кол-во ингредиента: '
+                                          f'{INGREDIENT_MIN_AMOUNT}'})
+            ingredients_ids_set.add(ingredient_id)
+
+        return cleaned_data
+
+
 @admin.register(User)
 class FoodgramUserAdmin(UserAdmin):
     """Админка для пользователей."""
@@ -39,6 +92,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,12 +3,10 @@
 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,
+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,
@@ -100,39 +98,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