Django : Ошибка: 'дублирующее значение ключа нарушает ограничение уникальности'

Мне нужно сохранить изображения с помощью s3 при создании нового объекта для моего сайта.

У меня есть код для сохранения моих изображений, который я использую в методе сохранения моей модели.

Когда я использую административную панель django, я могу легко создать свой объект без какой-либо ошибки.

Но когда я пытаюсь создать свой объект через представление, я получаю эту ошибку : Error: 'duplicate key value violates unique constraint « store_shop_pkey »'

Я думаю, что в методе create моего сериализатора я пытаюсь сохранить объект дважды: один раз для изображения и один раз для остальных ключей объекта.

Я не знаю, как решить эту проблему.

Хорошо работает с методом PUT.

Вот мой код :

models.py :

class Shop(models.Model):

    name = models.CharField(max_length=255)
    category = models.ForeignKey(ShopCategory, on_delete=models.SET_NULL, null=True, blank=True)
    description = models.TextField(blank=True, null=True)
    path = models.CharField(max_length=255, unique=True, null=True, blank=True) # Set a null and blank = True for serializer
    mustBeLogged = models.BooleanField(default=False)
    deliveries = models.FloatField(validators=[MinValueValidator(0),], default=7)
    message = models.TextField(null=True, blank=True)
    banner = models.ImageField(null=True, blank=True)

    def save(self, *args, **kwargs):
        try:
            """If we want to update"""
            this = Shop.objects.get(id=self.id)
            if self.banner:
                image_resize(self.banner, 300, 300)
                if this.banner != self.banner:
                    this.banner.delete(save=False)
            else:
                this.banner.delete(save=False)
        except:
            """If we want to create a shop"""
            if self.banner:
                image_resize(self.banner, 300, 300)
        super().save(*args, **kwargs)

    def delete(self):
        self.banner.delete(save=False)
        super().delete()
        
    def __str__(self):
        return self.name

utils.py :

def image_resize(image, width, height):
    # Open the image using Pillow
    img = Image.open(image)
    # check if either the width or height is greater than the max
    if img.width > width or img.height > height:
        output_size = (width, height)
        # Create a new resized “thumbnail” version of the image with Pillow
        img.thumbnail(output_size)
        # Find the file name of the image
        img_filename = Path(image.file.name).name
        # Spilt the filename on “.” to get the file extension only
        img_suffix = Path(image.file.name).name.split(".")[-1]
        # Use the file extension to determine the file type from the image_types dictionary
        img_format = image_types[img_suffix]
        # Save the resized image into the buffer, noting the correct file type
        buffer = BytesIO()
        img.save(buffer, format=img_format)
        # Wrap the buffer in File object
        file_object = File(buffer)
        # Save the new resized file as usual, which will save to S3 using django-storages
        image.save(img_filename, file_object)

views.py :

class ShopList(ShopListView):
    """Create shop"""
    def post(self, request):
        """For admin to create shop"""
        serializer = MakeShopSerializer(data=request.data)
        if serializer.is_valid():
            serializer.save()
            return Response(serializer.data)
        return Response(serializer.errors)

serializers.py :

class MakeShopSerializer(serializers.ModelSerializer):
    class Meta:
        model = Shop
        fields = '__all__'

    def create(self, validated_data):
        # validated_data.pop('path')
        path = validated_data["name"].replace(" ", "-").lower()
        path = unidecode.unidecode(path)
        unique = False
        while unique == False:
            if len(Shop.objects.filter(path=path)) == 0:
                unique = True
            else:
                # Generate a random string
                char = "abcdefghijklmnopqrstuvwxyz"
                path += "-{}".format("".join(random.sample(char, 5)))
        shop = Shop.objects.create(**validated_data, path=path)
        shop.save()
        return shop
    
    def update(self, instance, validated_data):
        #You will have path in validated_data
        #And you may have to check if the values are null
        return super(MakeShopSerializer, self).update(instance, validated_data)

Наконец, вот данные, которые я отправляю :

Data send

Заранее благодарю за помощь

В следующей строке возможна проблема с первичным ключом:

shop = Shop.objects.create(**validated_data, path=path)

вы можете попытаться разместить все атрибуты по одному, например, так:

shop = Shop.objects.create(
name=validated_data['name'],
category=validated_data['category'],
...
path=path)

Если это не решило вашу проблему, пожалуйста, опишите подробнее вашу ошибку из терминала

Уникальный ключ вашей модели - path, поэтому внутри функции create вашего сериализатора,

shop = Shop.objects.create(**validated_data, path=path), пытается создать новую модель. Если у второго экземпляра, который вы пытаетесь добавить, тот же путь, что и у предыдущего, то в этом случае вы получите эту ошибку, так как path должен быть уникальным, а вы пытаетесь добавить еще одну модель с тем же значением, что отвергается СУБД. Вы можете попробовать следующее, создать или обновить экземпляр. Если вас устраивает обновление экземпляра, когда ваш второй экземпляр имеет тот же путь, что и предыдущий, то используйте

Shop.objects.update_or_create(path=path,
                        defaults=validated_data)

else

попробуйте добавить уникальные ограничения, если ваша модель не может поддерживать уникальность только с помощью поля path. документация django

Вернуться на верх