From d633cd41c3060c118797ff0801ca4060f3bc8da2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sandro=20Mart=C3=ADnez?= Date: Fri, 27 Mar 2026 09:27:34 -0600 Subject: [PATCH] FINERACT-2558: Remove redundant gender update logic in client update flow --- .../ClientWritePlatformServiceJpaRepositoryImpl.java | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/service/ClientWritePlatformServiceJpaRepositoryImpl.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/service/ClientWritePlatformServiceJpaRepositoryImpl.java index ccf5cae3bdc..59ea3118e64 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/service/ClientWritePlatformServiceJpaRepositoryImpl.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/service/ClientWritePlatformServiceJpaRepositoryImpl.java @@ -564,7 +564,6 @@ public CommandProcessingResult updateClient(final Long clientId, final JsonComma } if (changes.containsKey(ClientApiConstants.genderIdParamName)) { - final Long newValue = command.longValueOfParameterNamed(ClientApiConstants.genderIdParamName); CodeValue gender = null; if (newValue != null) { @@ -585,15 +584,6 @@ public CommandProcessingResult updateClient(final Long clientId, final JsonComma clientForUpdate.updateSavingsProduct(savingsProductId); } - if (changes.containsKey(ClientApiConstants.genderIdParamName)) { - final Long newValue = command.longValueOfParameterNamed(ClientApiConstants.genderIdParamName); - CodeValue newCodeVal = null; - if (newValue != null) { - newCodeVal = this.codeValueRepository.findOneByCodeNameAndIdWithNotFoundDetection(ClientApiConstants.GENDER, newValue); - } - clientForUpdate.updateGender(newCodeVal); - } - if (changes.containsKey(ClientApiConstants.clientTypeIdParamName)) { final Long newValue = command.longValueOfParameterNamed(ClientApiConstants.clientTypeIdParamName); CodeValue newCodeVal = null;