fix: ingredients mess on recipe editing

Author: ClaireMaud

backend/foodgram/foodapp/serializers.py

@@ -269,20 +269,34 @@ def create(self, validated_data):
     def update(self, instance, validated_data):
         if 'tags' not in validated_data:
             raise serializers.ValidationError({'tags': 'Обязательное поле.'})
-        else:
-            instance.tags.set(validated_data.pop('tags'))
         if 'recipe_ingredients' not in validated_data:
-            raise serializers.ValidationError({'tags': 'Обязательное поле.'})
-        else:
-            ingredients = validated_data.pop('recipe_ingredients')
-            instance.recipe_ingredients.all().delete()
-            for ingredient in ingredients:
-                ingredient_instance = Ingredient.objects.get(
-                    id=ingredient['id'])
-                RecipeIngredient.objects.create(recipe=instance,
-                                                ingredient=ingredient_instance,
-                                                amount=ingredient['amount'])
-        return super().update(instance, validated_data)
+            raise serializers.ValidationError({'ingredients': 'Обязательное поле.'})
+
+        # Обновляем базовые поля
+        instance.name = validated_data.get('name', instance.name)
+        instance.image = validated_data.get('image', instance.image)
+        instance.text = validated_data.get('text', instance.text)
+        instance.cooking_time = validated_data.get('cooking_time', instance.cooking_time)
+
+        # Обновляем теги
+        instance.tags.clear()  # Сначала очищаем
+        instance.tags.set(validated_data.pop('tags'))  # Потом устанавливаем новые
+
+        # Обновляем ингредиенты
+        ingredients_data = validated_data.pop('recipe_ingredients')
+        instance.recipe_ingredients.all().delete()  # Удаляем старые ингредиенты
+
+        # Создаем новые ингредиенты
+        RecipeIngredient.objects.bulk_create([
+            RecipeIngredient(
+                recipe=instance,
+                ingredient_id=ingredient_data['id'],
+                amount=ingredient_data['amount']
+            ) for ingredient_data in ingredients_data
+        ])
+
+        instance.save()
+        return instance
 
 
 class RecipeShortSerializer(serializers.ModelSerializer):

backend/foodgram/foodgram/asgi.py

@@ -1,12 +1,3 @@
-"""
-ASGI config for foodgram project.
-
-It exposes the ASGI callable as a module-level variable named ``application``.
-
-For more information on this file, see
-https://docs.djangoproject.com/en/4.2/howto/deployment/asgi/
-"""
-
 import os
 
 from django.core.asgi import get_asgi_application