Issue #586

Aggiunti campi stato e descrizioneStato ai dati restituiti dal servizio riconciliazioni delle API Backoffice.

Author: pintorig

integration-test/src/test/java/test/api/backoffice/v1/riconciliazioni/get/msg/riconciliazione-cumulativa-idflusso-response.json

@@ -6,6 +6,7 @@
    "data":"#notnull",
    "sct":"#notnull",
    "idFlusso":"#notnull",
+   "stato":"#notnull",
    "riscossioni":[
       {
          "idDominio":"#notnull",

integration-test/src/test/java/test/api/backoffice/v1/riconciliazioni/get/msg/riconciliazione-cumulativa-response.json

@@ -7,6 +7,7 @@
    "sct":"#notnull",
    "iuv":"##string",
    "idFlusso":"##string",
+   "stato":"#notnull",
    "riscossioni":[
       {
          "idDominio":"#notnull",

integration-test/src/test/java/test/api/backoffice/v1/riconciliazioni/get/msg/riconciliazione-singola-response.json

@@ -7,6 +7,7 @@
    "sct":"#notnull",
    "iuv":"##string",
    "idFlusso":"##string",
+   "stato":"#notnull",
    "riscossioni":[
       {
          "idDominio":"#notnull",

integration-test/src/test/java/test/api/backoffice/v1/riconciliazioni/get/msg/riconciliazione-singola-senza-rpt.json

@@ -7,6 +7,7 @@
    "sct":"#notnull",
    "iuv":"##string",
    "idFlusso":"##string",
+   "stato":"#notnull",
    "riscossioni":[
       {
          "idDominio":"#notnull",

integration-test/src/test/java/test/api/backoffice/v1/riconciliazioni/get/riconciliazioni-find-byStato.feature

@@ -0,0 +1,108 @@
+Feature: Validazione sintattica filtri di ricerca per stato
+
+Background:
+
+* callonce read('classpath:utils/common-utils.feature')
+* callonce read('classpath:configurazione/v1/anagrafica.feature')
+* def backofficeBaseurl = getGovPayApiBaseUrl({api: 'backoffice', versione: 'v1', autenticazione: 'basic'})
+* def nomeAPI = '/incassi'
+
+Scenario: Validazione sintattica filtri per stato
+
+# No filtri
+
+Given url backofficeBaseurl
+And path nomeAPI
+And headers gpAdminBasicAutenticationHeader
+When method get
+Then status 200
+And match response == 
+"""
+{
+	maxRisultati: '#number',
+	numRisultati: '#number',
+	numPagine: '#number',
+	risultatiPerPagina: 25,
+	pagina: 1,
+	prossimiRisultati: '#ignore',
+	risultati: '#array'
+}
+"""
+
+# Filtro stato = 'IN_ELABORAZIONE'
+
+Given url backofficeBaseurl
+And path nomeAPI
+And param stato = 'IN_ELABORAZIONE'
+And headers gpAdminBasicAutenticationHeader
+When method get
+Then status 200
+And match response == 
+"""
+{
+	maxRisultati: '#number',
+	numRisultati: '#number',
+	numPagine: '#number',
+	risultatiPerPagina: 25,
+	pagina: 1,
+	prossimiRisultati: '#ignore',
+	risultati: '#array'
+}
+"""
+
+# Filtro stato = 'ACQUISITO'
+
+Given url backofficeBaseurl
+And path nomeAPI
+And param stato = 'ACQUISITO'
+And headers gpAdminBasicAutenticationHeader
+When method get
+Then status 200
+And match response == 
+"""
+{
+	maxRisultati: '#number',
+	numRisultati: '#number',
+	numPagine: '#number',
+	risultatiPerPagina: 25,
+	pagina: 1,
+	prossimiRisultati: '#ignore',
+	risultati: '#array'
+}
+"""
+
+# Filtro stato = 'ERRORE'
+
+Given url backofficeBaseurl
+And path nomeAPI
+And param stato = 'ERRORE'
+And headers gpAdminBasicAutenticationHeader
+When method get
+Then status 200
+And match response == 
+"""
+{
+	maxRisultati: '#number',
+	numRisultati: '#number',
+	numPagine: '#number',
+	risultatiPerPagina: 25,
+	pagina: 1,
+	prossimiRisultati: '#ignore',
+	risultati: '#array'
+}
+"""
+
+# Filtro stato  non valido
+
+* def dataDaNonValida = 'STATO_NON_VALIDO'
+* def dataDaParamName = 'stato'
+
+Given url backofficeBaseurl
+And path nomeAPI
+And param dataDa = dataDaNonValida
+And headers gpAdminBasicAutenticationHeader
+When method get
+Then status 400
+
+* match response == { categoria: 'RICHIESTA', codice: 'SINTASSI', descrizione: 'Richiesta non valida', dettaglio: '#notnull' }
+* match response.dettaglio contains 'STATO_NON_VALIDO'

jars/core/src/main/java/it/govpay/core/dao/pagamenti/IncassiDAO.java

@@ -71,6 +71,7 @@ public ListaIncassiDTOResponse listaIncassi(ListaIncassiDTO listaIncassoDTO) thr
 			newFilter.setCodApplicazione(listaIncassoDTO.getIdA2A());
 			newFilter.setSct(listaIncassoDTO.getSct());
 			newFilter.setCodFlusso(listaIncassoDTO.getCodFlusso());
+			newFilter.setStato(listaIncassoDTO.getStato());
 
 			List<Incasso> findAll = new ArrayList<>();
 			if(listaIncassoDTO.getIuv() != null ) {

jars/core/src/main/java/it/govpay/core/dao/pagamenti/dto/ListaIncassiDTO.java

@@ -25,6 +25,7 @@
 import org.springframework.security.core.Authentication;
 
 import it.govpay.core.dao.anagrafica.dto.BasicFindRequestDTO;
+import it.govpay.model.Incasso.StatoIncasso;
 import it.govpay.orm.Incasso;
 
 public class ListaIncassiDTO extends BasicFindRequestDTO {
@@ -44,6 +45,7 @@ public ListaIncassiDTO(Authentication user) {
 	private boolean includiPagamenti;
 	private String codFlusso;
 	private String iuv;
+	private StatoIncasso stato;
 
 
 	public String getIdDominio() {
@@ -94,4 +96,10 @@ public String getIuv() {
 	public void setIuv(String iuv) {
 		this.iuv = iuv;
 	}
+	public StatoIncasso getStato() {
+		return stato;
+	}
+	public void setStato(StatoIncasso stato) {
+		this.stato = stato;
+	}
 }

jars/orm/src/main/java/it/govpay/bd/pagamento/filters/IncassoFilter.java

@@ -38,6 +38,7 @@
 
 import it.govpay.bd.AbstractFilter;
 import it.govpay.bd.ConnectionManager;
+import it.govpay.model.Incasso.StatoIncasso;
 import it.govpay.orm.Incasso;
 import it.govpay.orm.dao.jdbc.converter.IncassoFieldConverter;
 import it.govpay.orm.model.IncassoModel;
@@ -56,6 +57,7 @@ public class IncassoFilter extends AbstractFilter{
 	private String sct;
 	private String codFlusso = null;
 	private String iuv = null;
+	private StatoIncasso stato = null;
 
 	public IncassoFilter(IExpressionConstructor expressionConstructor) {
 		this(expressionConstructor,false);
@@ -197,6 +199,14 @@ public IExpression _toExpression() throws ServiceException {
 				addAnd = true;
 			}
 
+			if(this.stato != null) {
+				if(addAnd)
+					newExpression.and();
+				
+				newExpression.equals(Incasso.model().STATO, this.stato.toString());
+				addAnd = true;
+			}
+			
 			return newExpression;
 		} catch (NotImplementedException e) {
 			throw new ServiceException(e);
@@ -282,6 +292,10 @@ public ISQLQueryObject toWhereCondition(ISQLQueryObject sqlQueryObject) throws S
 				sqlQueryObject.addWhereCondition(true,converter.toColumn(model.IUV, true) + " = ? ");
 			}
 
+			if(this.stato != null) {
+				sqlQueryObject.addWhereCondition(true,converter.toColumn(model.STATO, true) + " = ? ");
+			}
+			
 			return sqlQueryObject;
 		} catch (ExpressionException e) {
 			throw new ServiceException(e);
@@ -347,6 +361,10 @@ public Object[] getParameters(ISQLQueryObject sqlQueryObject) throws ServiceExce
 			lst.add(this.iuv);
 		}
 
+		if(this.stato != null) {
+			lst.add(this.stato.toString());
+		}
+		
 		return lst.toArray(new Object[lst.size()]);
 	}
 
@@ -444,5 +462,13 @@ public String getIuv() {
 	public void setIuv(String iuv) {
 		this.iuv = iuv;
 	}
+
+	public StatoIncasso getStato() {
+		return stato;
+	}
+
+	public void setStato(StatoIncasso stato) {
+		this.stato = stato;
+	}
 
 }

wars/api-backoffice/src/main/java/it/govpay/backoffice/v1/Incassi.java

@@ -38,9 +38,9 @@ public Incassi() {
     @Path("/")
 
     @Produces({ "application/json" })
-    public Response findRiconciliazioni(@Context UriInfo uriInfo, @Context HttpHeaders httpHeaders, @QueryParam(value=Costanti.PARAMETRO_PAGINA) @DefaultValue(value="1") Integer pagina, @QueryParam(value=Costanti.PARAMETRO_RISULTATI_PER_PAGINA) @DefaultValue(value="25") Integer risultatiPerPagina, @QueryParam("ordinamento") String ordinamento, @QueryParam("dataDa") String dataDa, @QueryParam("dataA") String dataA, @QueryParam("idDominio") String idDominio, @QueryParam("metadatiPaginazione") @DefaultValue(value="true") Boolean metadatiPaginazione, @QueryParam("maxRisultati") @DefaultValue(value="true") Boolean maxRisultati, @QueryParam("sct") String sct, @QueryParam("idFlusso") String idFlusso, @QueryParam("iuv") String iuv){
+    public Response findRiconciliazioni(@Context UriInfo uriInfo, @Context HttpHeaders httpHeaders, @QueryParam(value=Costanti.PARAMETRO_PAGINA) @DefaultValue(value="1") Integer pagina, @QueryParam(value=Costanti.PARAMETRO_RISULTATI_PER_PAGINA) @DefaultValue(value="25") Integer risultatiPerPagina, @QueryParam("ordinamento") String ordinamento, @QueryParam("dataDa") String dataDa, @QueryParam("dataA") String dataA, @QueryParam("idDominio") String idDominio, @QueryParam("metadatiPaginazione") @DefaultValue(value="true") Boolean metadatiPaginazione, @QueryParam("maxRisultati") @DefaultValue(value="true") Boolean maxRisultati, @QueryParam("sct") String sct, @QueryParam("idFlusso") String idFlusso, @QueryParam("iuv") String iuv, @QueryParam("stato") String stato){
         this.buildContext();
-        return this.controller.findRiconciliazioni(this.getUser(), uriInfo, httpHeaders, pagina, risultatiPerPagina, ordinamento, dataDa, dataA, idDominio, metadatiPaginazione, maxRisultati, sct, idFlusso, iuv);
+        return this.controller.findRiconciliazioni(this.getUser(), uriInfo, httpHeaders, pagina, risultatiPerPagina, ordinamento, dataDa, dataA, idDominio, metadatiPaginazione, maxRisultati, sct, idFlusso, iuv,stato);
     }
 
     @GET

wars/api-backoffice/src/main/java/it/govpay/backoffice/v1/beans/Incasso.java

@@ -22,6 +22,8 @@
 "iuv",
 "idFlusso",
 "riscossioni",
+"stato",
+"descrizioneStato",
 })
 public class Incasso extends it.govpay.core.beans.JSONSerializable {
 
@@ -60,7 +62,13 @@ public class Incasso extends it.govpay.core.beans.JSONSerializable {
 
   @JsonProperty("idFlusso")
   private String idFlusso = null;
-
+  
+  @JsonProperty("stato")
+  private StatoIncasso stato = null;
+  
+  @JsonProperty("descrizioneStato")
+  private String descrizioneStato = null;
+  
   /**
    **/
   public Incasso dominio(DominioIndex dominio) {
@@ -212,7 +220,7 @@ public Incasso riscossioni(List<Riscossione> riscossioni) {
 
   @JsonProperty("riscossioni")
   public List<Riscossione> getRiscossioni() {
-    return this.riscossioni;
+    return riscossioni;
   }
   public void setRiscossioni(List<Riscossione> riscossioni) {
     this.riscossioni = riscossioni;
@@ -250,6 +258,37 @@ public void setIdFlusso(String idFlusso) {
     this.idFlusso = idFlusso;
   }
 
+  /**
+   **/
+  public Incasso stato(StatoIncasso stato) {
+    this.stato = stato;
+    return this;
+  }
+
+  @JsonProperty("stato")
+  public StatoIncasso getStato() {
+    return stato;
+  }
+  public void setStato(StatoIncasso stato) {
+    this.stato = stato;
+  }
+
+  /**
+   * Descrizione estesta dello stato della riconciliazione
+   **/
+  public Incasso descrizioneStato(String descrizioneStato) {
+    this.descrizioneStato = descrizioneStato;
+    return this;
+  }
+
+  @JsonProperty("descrizioneStato")
+  public String getDescrizioneStato() {
+    return descrizioneStato;
+  }
+  public void setDescrizioneStato(String descrizioneStato) {
+    this.descrizioneStato = descrizioneStato;
+  }
+
   @Override
   public boolean equals(java.lang.Object o) {
     if (this == o) {
@@ -270,12 +309,14 @@ public boolean equals(java.lang.Object o) {
         Objects.equals(sct, incasso.sct) &&
         Objects.equals(riscossioni, incasso.riscossioni) &&
     	Objects.equals(iuv, incasso.iuv) &&
-    	Objects.equals(idFlusso, incasso.idFlusso);
+    	Objects.equals(idFlusso, incasso.idFlusso) &&
+        Objects.equals(stato, incasso.stato) &&
+        Objects.equals(descrizioneStato, incasso.descrizioneStato);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(dominio, idIncasso, causale, importo, data, dataValuta, dataContabile, ibanAccredito, sct, riscossioni);
+    return Objects.hash(dominio, idIncasso, causale, importo, data, dataValuta, dataContabile, ibanAccredito, sct, riscossioni, iuv, idFlusso, stato, descrizioneStato);
   }
 
   public static Incasso parse(String json) throws IOException {
@@ -304,6 +345,8 @@ public String toString() {
     sb.append("    riscossioni: ").append(toIndentedString(riscossioni)).append("\n");
     sb.append("    iuv: ").append(this.toIndentedString(this.iuv)).append("\n");
     sb.append("    idFlusso: ").append(this.toIndentedString(this.idFlusso)).append("\n");
+    sb.append("    stato: ").append(toIndentedString(stato)).append("\n");
+    sb.append("    descrizioneStato: ").append(toIndentedString(descrizioneStato)).append("\n");
     sb.append("}");
     return sb.toString();
   }