fix: change order of updates to make it idempotent

virtualidentityag · May 14, 2024 · 17bf384 · 17bf384
1 parent de5eee1
commit 17bf384
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...ava/com/vi/migrationtool/keycloak/UpdateTenantIdAttributeBasedOnConsultantAgencyTask.java b/...ava/com/vi/migrationtool/keycloak/UpdateTenantIdAttributeBasedOnConsultantAgencyTask.java
@@ -63,6 +63,9 @@ public void execute(Database database) {
                   adviceSeekerTenant.getUserId(),
                   adviceSeekerTenant.getTenantId());
 
+              keycloakUserService.updateUserCustomAttribute(
+                  "tenantId", adviceSeekerTenant.getTenantId(), adviceSeekerTenant.getUserId());
+
               userServiceJdbcTemplate.update(
                   "UPDATE user SET tenant_id = ? WHERE user_id = ?",
                   adviceSeekerTenant.getTenantId(),
@@ -73,8 +76,6 @@ public void execute(Database database) {
                   adviceSeekerTenant.getTenantId(),
                   adviceSeekerTenant.getUserId());
 
-              keycloakUserService.updateUserCustomAttribute(
-                  "tenantId", adviceSeekerTenant.getTenantId(), adviceSeekerTenant.getUserId());
 
               log.info(
                   "Successfully set tenantId for {} with id {} to {}",