From cff60057abb4af31fbaedbcd0e01d8cd3dd78e93 Mon Sep 17 00:00:00 2001
From: Tore Brede <Tore.Brede@uib.no>
Date: Tue, 16 Nov 2021 11:39:21 +0100
Subject: [PATCH] Fixing issue where mobile phone number was not updated, only
 added if it was not set already

---
 gregui/api/serializers/guest.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/gregui/api/serializers/guest.py b/gregui/api/serializers/guest.py
index 4836baf9..a7fe9a16 100644
--- a/gregui/api/serializers/guest.py
+++ b/gregui/api/serializers/guest.py
@@ -22,15 +22,14 @@ class GuestRegisterSerializer(serializers.ModelSerializer):
     passport = serializers.CharField(required=False)
 
     def update(self, instance, validated_data):
-        mobile_phone = validated_data.pop("mobile_phone")
-
         if "email" in validated_data:
             email = validated_data.pop("email")
             create_identity_or_update(
                 Identity.IdentityType.PRIVATE_EMAIL, email, instance
             )
 
-        if not instance.private_mobile:
+        if "mobile_phone" in validated_data:
+            mobile_phone = validated_data.pop("mobile_phone")
             create_identity_or_update(
                 Identity.IdentityType.PRIVATE_MOBILE_NUMBER, mobile_phone, instance
             )
-- 
GitLab