From a95cbe22750fc62f4fda3dd1ba523c25f8a1306e Mon Sep 17 00:00:00 2001 From: Sigurd <sigurd1994@hotmail.com> Date: Mon, 28 Mar 2022 16:37:43 +0200 Subject: [PATCH] users/serializers - Grouped imports / Fixed variables / long line fix Todo: Update coverage test for serializer --- backend/secfit/users/serializers.py | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/backend/secfit/users/serializers.py b/backend/secfit/users/serializers.py index e46a4c5..2355921 100644 --- a/backend/secfit/users/serializers.py +++ b/backend/secfit/users/serializers.py @@ -1,7 +1,7 @@ from rest_framework import serializers +from django import forms from django.contrib.auth import get_user_model, password_validation from users.models import Offer, AthleteFile -from django import forms class UserSerializer(serializers.HyperlinkedModelSerializer): @@ -36,6 +36,9 @@ class UserSerializer(serializers.HyperlinkedModelSerializer): password = data.get("password") password1 = data.get("password1") + if password != password1(): + raise serializers.ValidationError("The passwords must match") + try: password_validation.validate_password(password) except forms.ValidationError as error: @@ -53,7 +56,16 @@ class UserSerializer(serializers.HyperlinkedModelSerializer): street_address = validated_data["street_address"] favourite_exercise = validated_data["favourite_exercise"] main_gym = validated_data["main_gym"] - user_obj = get_user_model()(username=username, email=email, phone_number=phone_number, country=country, city=city, street_address=street_address, favourite_exercise=favourite_exercise, main_gym=main_gym) + user_obj = get_user_model()( + username=username, + email=email, + phone_number=phone_number, + country=country, + city=city, + street_address=street_address, + favourite_exercise=favourite_exercise, + main_gym=main_gym + ) user_obj.set_password(password) user_obj.save() -- GitLab