Piccole migliorie su CaricoProdFinLav

ems-engine/src/main/java/it/integry/ems/document/controller/DocumentProdController.java

@@ -70,18 +70,10 @@ public class DocumentProdController {
     public @ResponseBody
     ServiceRestResponse generateCaricoProdFinLavJson(HttpServletRequest request,
                                                      @RequestParam(CommonConstants.PROFILE_DB) String configuration,
-                                                     @RequestBody CaricoProdFinLavDTO inputData) {
+                                                     @RequestBody CaricoProdFinLavDTO inputData) throws Exception {
-        ServiceRestResponse response;
 
-        try {
+        documentProdService.caricoProdFinLav(inputData);
-            documentProdService.caricoProdFinLav(inputData);
+        return ServiceRestResponse.createPositiveResponse();
-            response = ServiceRestResponse.createPositiveResponse();
-        } catch (Exception e) {
-            logger.error(request.getRequestURI(), e);
-            response = new ServiceRestResponse(EsitoType.KO, configuration, e);
-        }
-
-        return response;
     }
 
 
@@ -178,7 +170,7 @@ public class DocumentProdController {
                                                @RequestParam(CommonConstants.PROFILE_DB) String configuration,
                                                @RequestParam String codMdep,
                                                @RequestParam String postura,
-                                               @RequestParam (required = false) String partitaMag,
+                                               @RequestParam(required = false) String partitaMag,
                                                @RequestParam Date dataMov) {
         try {
             olioProductionService.allineaGiacenzaPosture(dataMov, codMdep, postura, partitaMag);
@@ -324,8 +316,8 @@ public class DocumentProdController {
     @RequestMapping(value = EmsRestConstants.PATH_AGGIORNA_CRITERIO_PROD_PIANIFICAZIONE, method = RequestMethod.POST)
     public @ResponseBody
     ServiceRestResponse aggiornaCriterioProdPianificazione(HttpServletRequest request,
-                                              @RequestParam(CommonConstants.PROFILE_DB) String configuration,
+                                                           @RequestParam(CommonConstants.PROFILE_DB) String configuration,
-                                              @RequestBody JsonNode inputBody) {
+                                                           @RequestBody JsonNode inputBody) {
         ServiceRestResponse response;
 
         try {

ems-engine/src/main/java/it/integry/ems/document/dto/CaricoProdFinLavDTO.java

@@ -1,6 +1,7 @@
 package it.integry.ems.document.dto;
 
 import it.integry.ems.adapter.XmlDateAdapter;
+import it.integry.ems_model.utility.UtilityString;
 
 import javax.xml.bind.annotation.XmlElement;
 import javax.xml.bind.annotation.XmlElementRef;
@@ -90,6 +91,10 @@ public class CaricoProdFinLavDTO {
         this.gestione = gestione;
     }
 
+    public boolean isEffettuaScaricoMateriali() {
+        return "S".equalsIgnoreCase(UtilityString.isNull(effettuaScaricoMateriali, "S"));
+    }
+
     public String getEffettuaScaricoMateriali() {
         return effettuaScaricoMateriali;
     }
@@ -112,11 +117,24 @@ public class CaricoProdFinLavDTO {
         return effettuaCaricoProdotto;
     }
 
+
     @XmlElement(name = "EFFETTUA_CARICO_PRODOTTO")
     public void setEffettuaCaricoProdotto(String effettuaCaricoProdotto) {
         this.effettuaCaricoProdotto = effettuaCaricoProdotto;
     }
 
+
+
+    public boolean isEffettuaCaricoProdotto() {
+        return "S".equalsIgnoreCase(UtilityString.isNull(effettuaCaricoProdotto, "S"));
+    }
+
+    public void setEffettuaCaricoProdottoBoolean(boolean effettuaCaricoProdotto) {
+        this.effettuaCaricoProdotto = effettuaCaricoProdotto ? "S" : "N";
+    }
+
+
+
     public String getCreaCaricoDaCollo() {
         return creaCaricoDaCollo;
     }
@@ -126,6 +144,10 @@ public class CaricoProdFinLavDTO {
         this.creaCaricoDaCollo = creaCaricoDaCollo;
     }
 
+    public boolean isCreaCaricoDaCollo() {
+        return "S".equalsIgnoreCase(UtilityString.isNull(creaCaricoDaCollo, "N"));
+    }
+
     public String getCodAnag() {
         return codAnag;
     }
@@ -284,6 +306,10 @@ public class CaricoProdFinLavDTO {
         this.preparatoDa = preparatoDa;
     }
 
+    public boolean isCreaScaricoDaCollo() {
+        return "S".equalsIgnoreCase(UtilityString.isNull(creaScaricoDaCollo, "N"));
+    }
+
     public String getCreaScaricoDaCollo() {
         return creaScaricoDaCollo;
     }
@@ -303,6 +329,10 @@ public class CaricoProdFinLavDTO {
         return this;
     }
 
+    public boolean isCreaRientroDaCollo() {
+        return "S".equalsIgnoreCase(UtilityString.isNull(creaRientroDaCollo, "N"));
+    }
+
     public String getCreaRientroDaCollo() {
         return creaRientroDaCollo;
     }
@@ -340,6 +370,15 @@ public class CaricoProdFinLavDTO {
         return this;
     }
 
+    public boolean isGenerateJson() {
+        return "S".equalsIgnoreCase(generateJson);
+    }
+
+    public CaricoProdFinLavDTO setGenerateJsonBoolean(boolean generateJson) {
+        this.generateJson = generateJson ? "S" : "N";
+        return this;
+    }
+
     public Integer getNumDoc() {
         return numDoc;
     }

ems-engine/src/main/java/it/integry/ems/document/service/DocumentProdService.java

@@ -37,10 +37,12 @@ import org.springframework.stereotype.Service;
 import javax.validation.constraints.NotNull;
 import java.math.BigDecimal;
 import java.math.RoundingMode;
-import java.sql.*;
+import java.sql.Connection;
+import java.sql.PreparedStatement;
+import java.sql.ResultSet;
+import java.sql.Statement;
 import java.text.SimpleDateFormat;
 import java.util.*;
-import java.util.Date;
 import java.util.stream.Collectors;
 
 @Service
@@ -667,11 +669,8 @@ public class DocumentProdService {
             }
         }
 
-        String generateJson = setupGest.getSetup(multiDBTransactionManager.getPrimaryConnection(), "CARICO_SCARICO_PF", "SETUP", "GENERATE_JSON");
+        boolean generateJson = setupGest.getSetupBoolean(multiDBTransactionManager.getPrimaryConnection(), "CARICO_SCARICO_PF", "SETUP", "GENERATE_JSON");
-        if (UtilityString.streNull(generateJson).length() == 0) {
+        carico.setGenerateJsonBoolean(generateJson);
-            generateJson = "S";
-        }
-        carico.setGenerateJson(generateJson);
 
         if (carico.getDataDoc() == null) {
             String dataDocType = setupGest.getSetup("PVM", "MONITORAGGIO_LINEE_V2", "DATA_DOC_TYPE");
@@ -2301,20 +2300,20 @@ public class DocumentProdService {
 
         // Generazione carico prodotto finito da collo o da ordine
 
-        if ("S".equalsIgnoreCase(UtilityString.isNull(carico.getEffettuaCaricoProdotto(), "S"))) {
+        if (carico.isEffettuaCaricoProdotto()) {
-            if ("S".equalsIgnoreCase(UtilityString.isNull(carico.getCreaCaricoDaCollo(), "N"))) {
+            if (carico.isCreaCaricoDaCollo()) {
                 arrayEntity.addAll(generateCaricoDaCollo(carico));
             } else {
                 generateCaricoProdottoFinito(carico, arrayEntity);
             }
         }
         // Generazione scarico materiali
-        if ("S".equalsIgnoreCase(UtilityString.isNull(carico.getEffettuaScaricoMateriali(), "S"))) {
+        if (carico.isEffettuaScaricoMateriali()) {
             /*Se configurato effettua lo scarico di alcune materie prime direttamente da collo di versamento, generando il collo di versamento ed il relativo scarico*/
             //generateScaricoMaterialiDaColloVers(carico, arrayEntity);
 
             /*Genera scarico da eventuali colli di versamento non ancora agganciati*/
-            if ("S".equalsIgnoreCase(UtilityString.isNull(carico.getCreaScaricoDaCollo(), "N"))) {
+            if (carico.isCreaScaricoDaCollo()) {
                 generateScaricoDaCollo(carico, arrayEntity);
             }
 
@@ -2329,7 +2328,7 @@ public class DocumentProdService {
         }
 
         // Generazione rientro prodotti da collo
-        if ("S".equalsIgnoreCase(UtilityString.isNull(carico.getCreaRientroDaCollo(), "N"))) {
+        if (carico.isCreaRientroDaCollo()) {
             arrayEntity.addAll(this.generateRientroDaCollo(carico));
         }