Sonar: Refactor partialUpdate method to reduce its Cognitive Complexity (#32081)

Author: qmonmert

generators/spring-boot/templates/src/main/java/_package_/_entityPackage_/_partials_entity_/patch_template.ejs

@@ -38,13 +38,9 @@ _%>
        <%= mapper %>.partialUpdate(existing<%= entityClass %>, <%= instanceName %>);
   <%_ } else { _%>
     <%_ for (const field of fields.filter(field => !field.id && !field.transient)) { _%>
-    if (<%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>() != null) {
-        existing<%= entityClass %>.set<%= field.fieldInJavaBeanMethod %>(<%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>());
-    }
+        updateIfPresent(existing<%= entityClass %>::set<%= field.fieldInJavaBeanMethod %>, <%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>());
       <%_ if (field.fieldWithContentType) { _%>
-    if (<%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>ContentType() != null) {
-        existing<%= entityClass %>.set<%= field.fieldInJavaBeanMethod %>ContentType(<%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>ContentType());
-    }
+        updateIfPresent(existing<%= entityClass %>::set<%= field.fieldInJavaBeanMethod %>ContentType, <%= instanceName %>.get<%= field.fieldInJavaBeanMethod %>ContentType());
       <%_ } _%>
     <%_ } %>
   <% } %>

generators/spring-boot/templates/src/main/java/_package_/_entityPackage_/service/impl/_entityClass_ServiceImpl.java.ejs

@@ -75,6 +75,9 @@ import java.util.List;
 <%_ if (!reactive) { _%>
 import java.util.Optional;
 <%_ } _%>
+<%_ if (updatableEntity && !dtoMapstruct && fields.some(field => !field.id && !field.transient)) { _%>
+import java.util.function.Consumer;
+<%_ } _%>
 <%_ if (primaryKey.typeUUID) { _%>
 import java.util.UUID;
 <%_ } _%>
@@ -315,4 +318,12 @@ public class <%= serviceClassName %><% if (serviceImpl) { %> implements <%= enti
   <%_ } } _%>
     }
 <%_ } _%>
+<%_ if (updatableEntity && !dtoMapstruct && fields.some(field => !field.id && !field.transient)) { _%>
+
+    private <T> void updateIfPresent(Consumer<T> setter, T value) {
+        if (value != null) {
+            setter.accept(value);
+        }
+    }
+<%_ } _%>
 }

generators/spring-boot/templates/src/main/java/_package_/_entityPackage_/web/rest/_entityClass_Resource.java.ejs

@@ -120,6 +120,9 @@ import java.util.Optional;
 <%_ if (primaryKey.hasUUID) { _%>
 import java.util.UUID;
 <%_ } _%>
+<%_ if (updatableEntity && !viaService && !dtoMapstruct && fields.some(field => !field.id && !field.transient)) { _%>
+import java.util.function.Consumer;
+<%_ } _%>
 <%_ if ((searchEngineAny || fieldsContainNoOwnerOneToOne) && !reactive) { _%>
   <%_ if (!viaService) { _%>
 import java.util.stream.Collectors;
@@ -518,4 +521,12 @@ public class <%= entityClass %>Resource {
   <%_ } _%>
 <%- include('../../_partials_entity_/search_template', {viaService}); -%>
 <%_ } _%>
+<%_ if (updatableEntity && !viaService && !dtoMapstruct && fields.some(field => !field.id && !field.transient)) { _%>
+
+    private <T> void updateIfPresent(Consumer<T> setter, T value) {
+        if (value != null) {
+            setter.accept(value);
+        }
+    }
+<%_ } _%>
 }