Update project to latest tag parer-kettle-2.1.1

CHANGELOG.md

@@ -1,4 +1,9 @@
 
+## 2.1.1 (10-12-2024)
+
+### Bugfix: 1
+- [#34679](https://parermine.regione.emilia-romagna.it/issues/34679) Correzioni segnalazioni sonarqube
+
 ## 2.1.0 (11-11-2024)
 
 ### Bugfix: 2

RELEASE-NOTES.md

@@ -1,11 +1,4 @@
-## 2.1.0 (11-11-2024)
+## 2.1.1 (10-12-2024)
 
-### Bugfix: 2
-- [#34198](https://parermine.regione.emilia-romagna.it/issues/34198) Lista di risultati troncati dalla chiamata allo storico delle trasformazioni
-- [#34064](https://parermine.regione.emilia-romagna.it/issues/34064) Gestione del carattere + nel nome dell'oggetto
-
-### Novità: 1
-- [#34451](https://parermine.regione.emilia-romagna.it/issues/34451) Rimuovere il parametro XF_KETTLE_DB_PASSWORD dal report di trasformazione
-
-### SUE: 1
-- [#34063](https://parermine.regione.emilia-romagna.it/issues/34063) Modifica all'url di AWS nella configurazione di kettle server (tutti gli ambienti)
+### Bugfix: 1
+- [#34679](https://parermine.regione.emilia-romagna.it/issues/34679) Correzioni segnalazioni sonarqube

parer-kettle-jpa/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>it.eng.parer</groupId>
         <artifactId>parer-kettle</artifactId>
-        <version>2.1.1-SNAPSHOT</version>
+        <version>2.1.1</version>
     </parent>
     <artifactId>parer-kettle-jpa</artifactId>
     <name>Parer Kettle Persistence</name>

parer-kettle-jpa/src/main/java/it/eng/parer/kettle/jpa/MonExecTrasf.java

@@ -44,7 +44,7 @@ public class MonExecTrasf implements Serializable {
 
     public enum STATO_TRASFORMAZIONE {
         IN_CODA_TRASFORMAZIONE, TRASFORMAZIONE_IN_CORSO, TRASFORMAZIONE_TERMINATA, ERRORE_TRASFORMAZIONE
-    };
+    }
 
     @Id
     @SequenceGenerator(name = "MON_EXEC_TRASF_IDEXECTRASF_GENERATOR", sequenceName = "SMON_EXEC_TRASF", allocationSize = 1)
@@ -87,7 +87,7 @@ public enum STATO_TRASFORMAZIONE {
     private String nmKsInstance;
 
     public MonExecTrasf() {
-
+        // per usi futuri.
     }
 
     public long getIdExecTrasf() {

parer-kettle-jpa/src/main/java/it/eng/parer/kettle/lite/jpa/MonLog.java

@@ -68,6 +68,7 @@ public class MonLog implements Serializable {
     private String flEscapeXml;
 
     public MonLog() {
+        // per usi futuri
     }
 
     public long getIdMonLog() {

parer-kettle-model/pom.xml

@@ -4,13 +4,13 @@
     <parent>
         <groupId>it.eng.parer</groupId>
         <artifactId>parer-kettle</artifactId> 
-        <version>2.1.1-SNAPSHOT</version>
+        <version>2.1.1</version>
     </parent>
     <artifactId>parer-kettle-model</artifactId>
     <name>Parer Kettle Model</name>
     <description>Data Transfer Object utilizzati dal progetto Parer Kettle.</description>
     <packaging>jar</packaging>
-    
+
     <dependencies>
         <dependency>
             <groupId>javax.xml.bind</groupId>

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/AbstractEsito.java

@@ -38,10 +38,10 @@ public abstract class AbstractEsito implements Serializable {
      */
     public enum ESITO_SINTETICO {
         OK, KO, CODA_PIENA
-    };
+    }
 
     protected String dettagli;
-    protected Esito.ESITO_SINTETICO esitoSintetico;
+    protected ESITO_SINTETICO esitoSintetico;
 
     public String getDettaglio() {
         return dettagli;
@@ -52,11 +52,11 @@ public void setDettaglio(String dettagli) {
     }
 
     @XmlElement(nillable = false, required = true)
-    public Esito.ESITO_SINTETICO getEsitoSintetico() {
+    public ESITO_SINTETICO getEsitoSintetico() {
         return esitoSintetico;
     }
 
-    public void setEsitoSintetico(Esito.ESITO_SINTETICO esitoSintetico) {
+    public void setEsitoSintetico(ESITO_SINTETICO esitoSintetico) {
         this.esitoSintetico = esitoSintetico;
     }
 }

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/EsitoListaParametri.java

@@ -37,7 +37,7 @@ public void setParameters(List<Parametro> parameters) {
 
     public void addParameter(Parametro parameter) {
         if (this.parameters == null)
-            this.parameters = new ArrayList<Parametro>();
+            this.parameters = new ArrayList<>();
 
         this.parameters.add(parameter);
     }

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/EsitoStatusCodaTrasformazione.java

@@ -24,9 +24,9 @@
  * @author Cappelli_F
  */
 public class EsitoStatusCodaTrasformazione extends AbstractEsito {
-    List<StatoTrasformazione> trasformazioniInCorso;
-    List<StatoTrasformazione> trasformazioniInCoda;
-    List<StatoTrasformazione> storicoTrasformazioni;
+    private List<StatoTrasformazione> trasformazioniInCorso;
+    private List<StatoTrasformazione> trasformazioniInCoda;
+    private List<StatoTrasformazione> storicoTrasformazioni;
 
     public List<StatoTrasformazione> getTrasformazioniInCorso() {
         return trasformazioniInCorso;

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/KettleJob.java

@@ -41,7 +41,7 @@ public class KettleJob implements Serializable {
     private String versione;
 
     public KettleJob() {
-
+        // per usi futuri
     }
 
     public DataHandler getJobDescriptor() {

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/KettleTransformation.java

@@ -41,7 +41,7 @@ public class KettleTransformation implements Serializable {
     private String versione;
 
     public KettleTransformation() {
-
+        // per usi futuri
     }
 
     public DataHandler getTransformationDescriptor() {

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/Parametro.java

@@ -70,8 +70,7 @@ public String toString() {
 
     @Override
     public int hashCode() {
-        int hash = 7;
-        return hash;
+        return super.hashCode();
     }
 
     @Override
@@ -82,17 +81,15 @@ public boolean equals(Object obj) {
         if (obj == null) {
             return false;
         }
+
         if (getClass() != obj.getClass()) {
             return false;
         }
+
         final Parametro other = (Parametro) obj;
-        if (!Objects.equals(this.nomeParametro, other.nomeParametro)) {
-            return false;
-        }
-        if (!Objects.equals(this.valoreParametro, other.valoreParametro)) {
-            return false;
-        }
-        return true;
+
+        return Objects.equals(this.nomeParametro, other.nomeParametro)
+                && Objects.equals(this.valoreParametro, other.valoreParametro);
     }
 
 }

parer-kettle-model/src/main/java/it/eng/parer/kettle/model/Trasformazione.java

@@ -29,12 +29,12 @@ public class Trasformazione implements Serializable {
 
     private static final long serialVersionUID = -2359465051207551433L;
     private String nomeTrasformazione;
-    private long IdOggettoPing;
+    private long idOggettoPing;
     private List<Parametro> parametri;
     private long idTrasfReport;
 
     public Trasformazione() {
-
+        // per usi futuri
     }
 
     public String getNomeTrasformazione() {
@@ -46,11 +46,11 @@ public void setNomeTrasformazione(String nomeTrasformazione) {
     }
 
     public long getIdOggettoPing() {
-        return IdOggettoPing;
+        return idOggettoPing;
     }
 
-    public void setIdOggettoPing(long IdOggettoPing) {
-        this.IdOggettoPing = IdOggettoPing;
+    public void setIdOggettoPing(long idOggettoPing) {
+        this.idOggettoPing = idOggettoPing;
     }
 
     public List<Parametro> getParametri() {

parer-kettle-rest-client/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>it.eng.parer</groupId>
         <artifactId>parer-kettle</artifactId>
-        <version>2.1.1-SNAPSHOT</version>
+        <version>2.1.1</version>
     </parent>
     <artifactId>parer-kettle-rest-client</artifactId>
     <name>Parer Kettle REST Client</name>

parer-kettle-rest/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>it.eng.parer</groupId>
         <artifactId>parer-kettle</artifactId>
-        <version>2.1.1-SNAPSHOT</version>
+        <version>2.1.1</version>
     </parent>
     <artifactId>parer-kettle-rest</artifactId>
     <name>Parer Kettle REST Service</name>

parer-kettle-rest/src/main/java/it/eng/parer/kettle/rest/TrasformazioniServiceImpl.java

@@ -20,7 +20,6 @@
 import it.eng.parer.kettle.model.StatoTrasformazione;
 import it.eng.parer.kettle.rest.client.TrasformazioniService;
 import it.eng.parer.kettle.service.DataService;
-import it.eng.parer.kettle.service.GestoreTrasformazioni;
 import java.util.List;
 
 /**
@@ -50,14 +49,9 @@
  */
 public class TrasformazioniServiceImpl implements TrasformazioniService {
 
-    private GestoreTrasformazioni gestoreTrasformazioni;
     private String versioneSoftware;
     private DataService dataService;
 
-    public void setGestoreTrasformazioni(GestoreTrasformazioni gestoreTrasformazioni) {
-        this.gestoreTrasformazioni = gestoreTrasformazioni;
-    }
-
     public void setDataService(DataService dataService) {
         this.dataService = dataService;
     }
@@ -68,12 +62,7 @@ public void setVersioneSoftware(String versioneSoftware) {
 
     @Override
     public List<StatoTrasformazione> getTrasformazioni(String status) {
-        if (status != null) {
-
-        }
-
-        List<StatoTrasformazione> ottieniStatoTrasformazioniAttive = dataService.ottieniTrasformazioniAttive();
-        return ottieniStatoTrasformazioniAttive;
+        return dataService.ottieniTrasformazioniAttive();
     }
 
     @Override

parer-kettle-server/pom.xml

@@ -3,7 +3,7 @@
     <parent>
         <groupId>it.eng.parer</groupId>
         <artifactId>parer-kettle</artifactId> 
-        <version>2.1.1-SNAPSHOT</version>
+        <version>2.1.1</version>
     </parent>
 
     <artifactId>kettle-server</artifactId>

parer-kettle-server/src/main/java/it/eng/parer/kettle/server/Constants.java

@@ -22,8 +22,12 @@
  * @author Cappelli_F
  */
 public class Constants {
-    public final static String XF_OBJECT_STORAGE_KEY = "XF_OBJECT_STORAGE_KEY";
-    public final static String XF_OBJECT_STORAGE_BUCKET = "XF_OBJECT_STORAGE_BUCKET";
-    public final static String XF_INPUT_FILE_NAME = "XF_INPUT_FILE_NAME";
-    public final static String XF_TMP_DIR = "XF_TMP_DIR";
+    private Constants() {
+        // solo per non far lamentare sonarqube
+    }
+
+    public static final String XF_OBJECT_STORAGE_KEY = "XF_OBJECT_STORAGE_KEY";
+    public static final String XF_OBJECT_STORAGE_BUCKET = "XF_OBJECT_STORAGE_BUCKET";
+    public static final String XF_INPUT_FILE_NAME = "XF_INPUT_FILE_NAME";
+    public static final String XF_TMP_DIR = "XF_TMP_DIR";
 }

parer-kettle-server/src/main/java/it/eng/parer/kettle/server/config/CXFConfig.java

@@ -54,11 +54,9 @@ public class CXFConfig {
     public static final String SOAP_TRASFORMAZIONI = "/soap/trasformazioni";
     public static final String REST_TRASFORMAZIONI_V1 = "/rest/v1";
 
-    @Value("true")
-    private boolean cxfDebug;
+    private final boolean cxfDebug = true;
 
-    @Value("true")
-    private boolean mtomEnabled;
+    private final boolean mtomEnabled = true;
 
     @Value("${build.version}")
     private String versioneSoftware;
@@ -87,7 +85,7 @@ private void addLoggers() {
 
     // --------------- SOAP
     @Bean
-    public TrasformazioniSoapService soapTrasformazioniService() {
+    public TrasformazioniSoapService soapTrasformazioniService(GestoreTrasformazioni gestoreTrasformazioni) {
         TrasformazioniSoapServiceImpl service = new TrasformazioniSoapServiceImpl();
         service.setGestoreTrasformazioni(gestoreTrasformazioni);
         service.setDataService(dataService);
@@ -101,7 +99,7 @@ private void enableMTOM(EndpointImpl endpoint) {
 
     @Bean
     public Endpoint createSoapTrasformazioniEndpoint() {
-        EndpointImpl endpoint = new EndpointImpl(bus, soapTrasformazioniService());
+        EndpointImpl endpoint = new EndpointImpl(bus, soapTrasformazioniService(gestoreTrasformazioni));
         endpoint.publish(SOAP_TRASFORMAZIONI);
         enableMTOM(endpoint);
         return endpoint;
@@ -111,7 +109,6 @@ public Endpoint createSoapTrasformazioniEndpoint() {
     @Bean
     public TrasformazioniService restTrasformazioniService() {
         TrasformazioniServiceImpl service = new TrasformazioniServiceImpl();
-        service.setGestoreTrasformazioni(gestoreTrasformazioni);
         service.setVersioneSoftware(versioneSoftware);
         return service;
     }

parer-kettle-server/src/main/java/it/eng/parer/kettle/server/config/KettleEnvironmentConfig.java

@@ -105,7 +105,7 @@ public void init() {
                 sb.append("\n\n");
             }
         }
-        LOGGER.info(sb.toString());
+        LOGGER.info("{}", sb);
     }
 
     @PreDestroy

parer-kettle-server/src/main/java/it/eng/parer/kettle/server/config/LitePersistanceConfig.java

@@ -49,23 +49,15 @@ public class LitePersistanceConfig {
     @Autowired
     private Environment env;
 
-    // @Bean
-    // public DataSourceInitializer liteDatasourceInitializer(@Qualifier("liteDatasource") DataSource datasource) {
-    // ResourceDatabasePopulator resourceDatabasePopulator = new ResourceDatabasePopulator();
-    // resourceDatabasePopulator.addScript(new ClassPathResource("schema-monlog.sql"));
-    //
-    // DataSourceInitializer dataSourceInitializer = new DataSourceInitializer();
-    // dataSourceInitializer.setDataSource(datasource);
-    // dataSourceInitializer.setDatabasePopulator(resourceDatabasePopulator);
-    // return dataSourceInitializer;
-    // }
-
     @Bean
     public DataSource liteDatasource() throws NamingException {
         String repoJNDI = env.getProperty("jndi.kettle_lite_ds");
-        DataSource dataSource = (DataSource) new JndiTemplate().lookup(repoJNDI);
 
-        return dataSource;
+        if (repoJNDI != null) {
+            return (DataSource) new JndiTemplate().lookup(repoJNDI);
+        } else {
+            throw new NamingException("jndi.kettle_lite_ds not found.");
+        }
     }
 
     // Bean per @EnableJpaRepositories
@@ -74,7 +66,7 @@ public LocalContainerEntityManagerFactoryBean liteEntityManagerFactory() throws
 
         final LocalContainerEntityManagerFactoryBean em = new LocalContainerEntityManagerFactoryBean();
         em.setDataSource(liteDatasource());
-        em.setPackagesToScan(new String[] { "it.eng.parer.kettle.lite.jpa" });
+        em.setPackagesToScan("it.eng.parer.kettle.lite.jpa");
         em.setJpaVendorAdapter(new HibernateJpaVendorAdapter());
         em.setJpaProperties(additionalProperties());