From 87932175271b6dda4e59f2893888fa0f850547ec Mon Sep 17 00:00:00 2001
From: Timo Bejan <timo.bejan.ext@iqser.com>
Date: Mon, 24 Jan 2022 20:38:03 +0200
Subject: [PATCH] removed dumb condition

---
 .../DossierTemplatePersistenceService.java       | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/service/persistence/DossierTemplatePersistenceService.java b/persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/service/persistence/DossierTemplatePersistenceService.java
index fbb7b2807..05c43b7b1 100644
--- a/persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/service/persistence/DossierTemplatePersistenceService.java
+++ b/persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/service/persistence/DossierTemplatePersistenceService.java
@@ -6,9 +6,7 @@ import com.iqser.red.service.persistence.management.v1.processor.exception.Confl
 import com.iqser.red.service.persistence.management.v1.processor.exception.NotFoundException;
 import com.iqser.red.service.persistence.management.v1.processor.service.persistence.repository.DossierTemplateRepository;
 import com.iqser.red.service.persistence.service.v1.api.model.dossiertemplate.CreateOrUpdateDossierTemplateRequest;
-import com.iqser.red.service.persistence.service.v1.api.model.dossiertemplate.DossierTemplate;
 import com.iqser.red.service.persistence.service.v1.api.model.dossiertemplate.DossierTemplateStatus;
-
 import lombok.RequiredArgsConstructor;
 import org.springframework.beans.BeanUtils;
 import org.springframework.stereotype.Service;
@@ -38,7 +36,7 @@ public class DossierTemplatePersistenceService {
             Optional<DossierTemplateEntity> dossierTemplate = dossierTemplateRepository.findById(createOrUpdateDossierRequest.getDossierTemplateId());
             if (dossierTemplate.isPresent()) {
 
-                if(!dossierTemplate.get().getName().equals(createOrUpdateDossierRequest.getName())){
+                if (!dossierTemplate.get().getName().equals(createOrUpdateDossierRequest.getName())) {
                     validateDossierTemplateNameIsUnique(createOrUpdateDossierRequest.getName());
                 }
 
@@ -71,18 +69,18 @@ public class DossierTemplatePersistenceService {
 
     private DossierTemplateStatus computeDossierTemplateStatus(DossierTemplateEntity dossierTemplate) {
         var legalBasis = legalBasisMappingPersistenceService.getLegalBasisMapping(dossierTemplate.getId());
-        if (legalBasis==null || legalBasis.isEmpty()) {
+        if (legalBasis == null || legalBasis.isEmpty()) {
             return DossierTemplateStatus.INCOMPLETE;
         }
 
         var rules = rulesPersistenceService.getRules(dossierTemplate.getId());
-        if (rules==null || rules.getVersion()==1) {
+        if (rules == null) {
             return DossierTemplateStatus.INCOMPLETE;
         }
 
         var now = OffsetDateTime.now();
-        boolean isNotInRange = !now.isAfter(dossierTemplate.getValidFrom()!=null ? dossierTemplate.getValidFrom() : OffsetDateTime.MIN) || !now.isBefore(dossierTemplate.getValidTo()!=null ? dossierTemplate.getValidTo() : OffsetDateTime.MAX);
-        if(isNotInRange) {
+        boolean isNotInRange = !now.isAfter(dossierTemplate.getValidFrom() != null ? dossierTemplate.getValidFrom() : OffsetDateTime.MIN) || !now.isBefore(dossierTemplate.getValidTo() != null ? dossierTemplate.getValidTo() : OffsetDateTime.MAX);
+        if (isNotInRange) {
             return DossierTemplateStatus.INACTIVE;
         }
 
@@ -90,9 +88,9 @@ public class DossierTemplatePersistenceService {
     }
 
 
-    public void validateDossierTemplateNameIsUnique(String templateName){
+    public void validateDossierTemplateNameIsUnique(String templateName) {
         getAllDossierTemplates().forEach(existing -> {
-            if(existing.getName().equals(templateName)){
+            if (existing.getName().equals(templateName)) {
                 throw new ConflictException("DossierTemplate name must be unique");
             }
         });