[SELC-6087] fix findMailUuidAndUpdateUserRegistry method to retrieve …

…workContact id
pagopa · Nov 29, 2024 · 993de8e · 993de8e
1 parent ad4e2db
commit 993de8e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/apps/user-ms/src/main/docs/openapi.json b/apps/user-ms/src/main/docs/openapi.json
@@ -1632,7 +1632,7 @@
             "type" : "string"
           },
           "mobilePhone" : {
-            "pattern" : "^\\+?[0-9]{9,15}$",
+            "pattern" : "^\\+?[0-9]{7,15}$",
             "type" : "string"
           }
         }

diff --git a/apps/user-ms/src/main/docs/openapi.yaml b/apps/user-ms/src/main/docs/openapi.yaml
@@ -1191,7 +1191,7 @@ components:
         email:
           type: string
         mobilePhone:
-          pattern: "^\\+?[0-9]{9,15}$"
+          pattern: "^\\+?[0-9]{7,15}$"
           type: string
     User:
       required:

diff --git a/apps/user-ms/src/main/java/it/pagopa/selfcare/user/model/UpdateUserRequest.java b/apps/user-ms/src/main/java/it/pagopa/selfcare/user/model/UpdateUserRequest.java
@@ -9,6 +9,6 @@ public class UpdateUserRequest {
     private String name;
     private String familyName;
     private String email;
-    @Pattern(regexp = "^\\+?[0-9]{9,15}$", message = "Il numero di telefono non è valido")
+    @Pattern(regexp = "^\\+?[0-9]{7,15}$", message = "Il numero di telefono non è valido")
     private String mobilePhone;
 }
diff --git a/apps/user-ms/src/main/java/it/pagopa/selfcare/user/service/UserRegistryServiceImpl.java b/apps/user-ms/src/main/java/it/pagopa/selfcare/user/service/UserRegistryServiceImpl.java
@@ -139,11 +139,12 @@ private Uni<String> findMailUuidAndUpdateUserRegistry(UserResource userResource,
                         .filter(stringWorkContactResourceEntry -> existsWorkContactResourceForPhoneAndMail(stringWorkContactResourceEntry, emailToCompare, mobilePhoneToCompare))
                         .findFirst()
                         .map(Map.Entry::getKey))
-                .orElse(null);
+                .orElse(idContacts);
+
 
         return updateUsingPATCH(userResource.getId().toString(),
-                userMapper.toMutableUserFieldsDto(userDto, userResource, idContacts))
-                .replaceWith(StringUtils.isBlank(existedUserMailUuid) ? idContacts : existedUserMailUuid);
+                userMapper.toMutableUserFieldsDto(userDto, userResource, existedUserMailUuid))
+                .replaceWith(existedUserMailUuid);
     }
 
     private static boolean existsWorkContactResourceForPhoneAndMail(Map.Entry<String, WorkContactResource> stringWorkContactResourceEntry, String emailToCompare, String mobilePhoneToCompare) {