Fix bug where previous_usernames or previous_discriminators is empty

This commit is contained in:
Dustin Pianalto 2019-12-14 01:58:26 -09:00
parent 151a705b8d
commit 6cc0aabb5a

View File

@ -38,11 +38,11 @@ class UserSerializer(serializers.HyperlinkedModelSerializer):
def update(self, instance, validated_data): def update(self, instance, validated_data):
if 'username' in validated_data and instance.username != validated_data['username']: if 'username' in validated_data and instance.username != validated_data['username']:
prev_usernames = set(instance.previous_usernames) prev_usernames = set(instance.previous_usernames or [])
prev_usernames.add(instance.username) prev_usernames.add(instance.username)
validated_data['previous_usernames'] = list(prev_usernames) validated_data['previous_usernames'] = list(prev_usernames)
if 'discriminator' in validated_data and instance.discriminator != validated_data['discriminator']: if 'discriminator' in validated_data and instance.discriminator != validated_data['discriminator']:
prev_discrims = set(instance.previous_discriminators) prev_discrims = set(instance.previous_discriminators or [])
prev_discrims.add(instance.discriminator) prev_discrims.add(instance.discriminator)
validated_data['previous_discriminators'] = list(prev_discrims) validated_data['previous_discriminators'] = list(prev_discrims)
super(UserSerializer, self).update(instance, validated_data) super(UserSerializer, self).update(instance, validated_data)