Dockery/Pinry
1
0
Fork 0
mirror of https://github.com/pinry/pinry.git synced 2025-11-13 08:35:41 +01:00
Code Issues Packages Projects Releases Activity

Fix: Add unique-together validation for board-rename

Browse Source
This commit is contained in:
winkidney
2019-12-07 15:33:06 +08:00
committed by Isaac Bythewood
parent 3323d8d3c6
commit d32a66a44d
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
core/serializers.py
View File

@@ -1,6 +1,7 @@
from django.conf import settings from django.conf import settings
from rest_framework import serializers from rest_framework import serializers
from rest_framework.exceptions import ValidationError from rest_framework.exceptions import ValidationError
from rest_framework.validators import UniqueTogetherValidator
from taggit.models import Tag from taggit.models import Tag
from core.models import Image, Board from core.models import Image, Board
@@ -194,6 +195,13 @@ class BoardSerializer(serializers.HyperlinkedModelSerializer):
def update(self, instance: Board, validated_data): def update(self, instance: Board, validated_data):
pins_to_add = validated_data.pop("pins_to_add", []) pins_to_add = validated_data.pop("pins_to_add", [])
pins_to_remove = validated_data.pop("pins_to_remove", []) pins_to_remove = validated_data.pop("pins_to_remove", [])
if Board.objects.filter(
submitter=instance.submitter,
name=validated_data.get('name', None)
).exists():
raise ValidationError(
detail={'name': "Board with this name already exists"}
)
instance = super(BoardSerializer, self).update(instance, validated_data) instance = super(BoardSerializer, self).update(instance, validated_data)
changed = False changed = False
if pins_to_add: if pins_to_add: