Report refactoring

.gitignore

@@ -10,3 +10,4 @@ nb-configuration.xml
 .idea/
 *.iml
 /newrelic
+src/main/resources/**/*.jasper

src/main/java/sic/controller/CuentaCorrienteController.java

@@ -23,6 +23,7 @@
 import sic.service.IClienteService;
 import sic.service.ICuentaCorrienteService;
 import sic.service.IProveedorService;
+import sic.util.FormatoReporte;
 
 @RestController
 @RequestMapping("/api/v1")
@@ -119,7 +120,7 @@ public ResponseEntity<byte[]> getReporteCuentaCorriente(
                 cuentaCorrienteService.getReporteCuentaCorrienteCliente(
                         cuentaCorrienteService.getCuentaCorrientePorCliente(
                                 clienteService.getClienteNoEliminadoPorId(criteria.getIdCliente())),
-                        formato);
+                        FormatoReporte.XLSX);
         headers.setContentLength(reporteXls.length);
         return new ResponseEntity<>(reporteXls, headers, HttpStatus.OK);
       }
@@ -130,7 +131,7 @@ public ResponseEntity<byte[]> getReporteCuentaCorriente(
                 cuentaCorrienteService.getReporteCuentaCorrienteCliente(
                         cuentaCorrienteService.getCuentaCorrientePorCliente(
                                 clienteService.getClienteNoEliminadoPorId(criteria.getIdCliente())),
-                        formato);
+                        FormatoReporte.PDF);
         return new ResponseEntity<>(reportePDF, headers, HttpStatus.OK);
       }
       default -> throw new BusinessServiceException(messageSource.getMessage(
@@ -148,32 +149,32 @@ public ResponseEntity<byte[]> getReporteListaDeCuentasCorrienteClientePorCriteri
     headers.setCacheControl("must-revalidate, post-check=0, pre-check=0");
     return switch (formato) {
       case "xlsx" ->
-        this.getReporteListaDeCuentasCorrienteClienteEnXlsx(headers, criteria, claims, formato);
+        this.getReporteListaDeCuentasCorrienteClienteEnXlsx(headers, criteria, claims);
       case "pdf" ->
-        this.getReporteListaDeCuentasCorrienteClienteEnPdf(headers, criteria, claims, formato);
+        this.getReporteListaDeCuentasCorrienteClienteEnPdf(headers, criteria, claims);
       default -> throw new BusinessServiceException(
               messageSource.getMessage("mensaje_formato_no_valido", null, Locale.getDefault()));
     };
   }
 
   private ResponseEntity<byte[]> getReporteListaDeCuentasCorrienteClienteEnXlsx(
-          HttpHeaders headers, BusquedaCuentaCorrienteClienteCriteria criteria, Claims claims, String formato) {
+          HttpHeaders headers, BusquedaCuentaCorrienteClienteCriteria criteria, Claims claims) {
     headers.setContentType(new MediaType("application", "vnd.ms-excel"));
     headers.set(CONTENT_DISPOSITION_HEADER, "attachment; filename=ListaClientes.xlsx");
     byte[] reporteXls =
             cuentaCorrienteService.getReporteListaDeCuentasCorrienteClientePorCriteria(
-                    criteria, (int) claims.get(ID_USUARIO), formato);
+                    criteria, (int) claims.get(ID_USUARIO), FormatoReporte.XLSX);
     headers.setContentLength(reporteXls.length);
     return new ResponseEntity<>(reporteXls, headers, HttpStatus.OK);
   }
 
   private ResponseEntity<byte[]> getReporteListaDeCuentasCorrienteClienteEnPdf(
-          HttpHeaders headers, BusquedaCuentaCorrienteClienteCriteria criteria, Claims claims, String formato) {
+          HttpHeaders headers, BusquedaCuentaCorrienteClienteCriteria criteria, Claims claims) {
     headers.setContentType(MediaType.APPLICATION_PDF);
     headers.add(CONTENT_DISPOSITION_HEADER, "attachment; filename=ListaClientes.pdf");
     byte[] reportePDF =
             cuentaCorrienteService.getReporteListaDeCuentasCorrienteClientePorCriteria(
-                    criteria, (int) claims.get(ID_USUARIO), formato);
+                    criteria, (int) claims.get(ID_USUARIO), FormatoReporte.PDF);
     return new ResponseEntity<>(reportePDF, headers, HttpStatus.OK);
   }
 }

src/main/java/sic/controller/ProductoController.java

@@ -15,6 +15,7 @@
 import sic.modelo.dto.*;
 import sic.service.*;
 import sic.exception.BusinessServiceException;
+import sic.util.FormatoReporte;
 
 import javax.persistence.EntityNotFoundException;
 
@@ -126,12 +127,10 @@ public void getListaDePrecios(
           @RequestBody BusquedaProductoCriteria criteria,
           @PathVariable long idSucursal,
           @RequestParam(required = false) String formato) {
-    if (formato == null || formato.isEmpty()) {
-      formato = "xlsx";
-    }
+    if (formato == null || formato.isEmpty()) formato = "pdf";
     switch (formato) {
-      case "xlsx" -> productoService.getListaDePreciosEnXls(criteria, idSucursal);
-      case "pdf" -> productoService.getListaDePreciosEnPdf(criteria, idSucursal);
+      case "xlsx" -> productoService.procesarReporteListaDePrecios(criteria, idSucursal, FormatoReporte.XLSX);
+      case "pdf" -> productoService.procesarReporteListaDePrecios(criteria, idSucursal, FormatoReporte.PDF);
       default -> throw new BusinessServiceException(
               messageSource.getMessage("mensaje_formato_no_valido", null, Locale.getDefault()));
     }

src/main/java/sic/service/ICuentaCorrienteService.java

@@ -6,7 +6,7 @@
 import sic.modelo.*;
 import sic.modelo.criteria.BusquedaCuentaCorrienteClienteCriteria;
 import sic.modelo.criteria.BusquedaCuentaCorrienteProveedorCriteria;
-
+import sic.util.FormatoReporte;
 import java.math.BigDecimal;
 import java.util.List;
 
@@ -18,11 +18,9 @@ public interface ICuentaCorrienteService {
 
   BigDecimal getSaldoCuentaCorriente(long idCliente);
 
-  CuentaCorrienteCliente guardarCuentaCorrienteCliente(
-      CuentaCorrienteCliente cuentaCorrienteCliente);
+  CuentaCorrienteCliente guardarCuentaCorrienteCliente(CuentaCorrienteCliente cuentaCorrienteCliente);
 
-  CuentaCorrienteProveedor guardarCuentaCorrienteProveedor(
-      CuentaCorrienteProveedor cuentaCorrienteProveedor);
+  CuentaCorrienteProveedor guardarCuentaCorrienteProveedor(CuentaCorrienteProveedor cuentaCorrienteProveedor);
 
   void validarReglasDeNegocio(CuentaCorriente cuentaCorriente);
 
@@ -50,8 +48,10 @@ Page<CuentaCorrienteProveedor> buscarCuentaCorrienteProveedor(
 
   void asentarEnCuentaCorriente(Remito remito, TipoDeOperacion tipo);
 
-  byte[] getReporteCuentaCorrienteCliente(
-      CuentaCorrienteCliente cuentaCorrienteCliente, String formato);
+  byte[] getReporteCuentaCorrienteCliente(CuentaCorrienteCliente cuentaCorrienteCliente, FormatoReporte formato);
+
+  byte[] getReporteListaDeCuentasCorrienteClientePorCriteria(
+          BusquedaCuentaCorrienteClienteCriteria criteria, long idUsuarioLoggedIn, FormatoReporte formato);
 
   List<RenglonCuentaCorriente> getUltimosDosMovimientos(CuentaCorriente cuentaCorriente);
 
@@ -65,11 +65,7 @@ byte[] getReporteCuentaCorrienteCliente(
 
   RenglonCuentaCorriente getRenglonCuentaCorrienteDeRemito(Remito remito, boolean eliminado);
 
-  byte[] getReporteListaDeCuentasCorrienteClientePorCriteria(
-          BusquedaCuentaCorrienteClienteCriteria criteria, long idUsuarioLoggedIn, String formato);
-
-  BooleanBuilder getBuilder(
-      BusquedaCuentaCorrienteClienteCriteria criteria, long idUsuarioLoggedIn);
+  BooleanBuilder getBuilder(BusquedaCuentaCorrienteClienteCriteria criteria, long idUsuarioLoggedIn);
 
   List<CuentaCorrienteCliente> buscarCuentasCorrienteClienteParaReporte(
       BusquedaCuentaCorrienteClienteCriteria criteria, long idUsuarioLoggedIn);

src/main/java/sic/service/IProductoService.java

@@ -11,6 +11,7 @@
 import sic.modelo.dto.*;
 import sic.modelo.embeddable.CantidadProductoEmbeddable;
 import sic.modelo.embeddable.PrecioProductoEmbeddable;
+import sic.util.FormatoReporte;
 
 public interface IProductoService {
 
@@ -38,9 +39,11 @@ void actualizarStockNotaCredito(
 
   Page<Producto> buscarProductos(BusquedaProductoCriteria criteria, Long idSucursal);
 
-  Page<Producto> buscarProductosDeCatalogoParaUsuario(BusquedaProductoCriteria criteria, Long idSucursal, Long isSucursal);
+  Page<Producto> buscarProductosDeCatalogoParaUsuario(
+          BusquedaProductoCriteria criteria, Long idSucursal, Long isSucursal);
 
-  Page<Producto> buscarProductosDeCatalogoParaVenta(BusquedaProductoCriteria criteria, Long idSucursal, Long idUsuario, Long idCliente);
+  Page<Producto> buscarProductosDeCatalogoParaVenta(
+          BusquedaProductoCriteria criteria, Long idSucursal, Long idUsuario, Long idCliente);
 
   void marcarFavoritos(Page<Producto> productos, long idUsuario);
 
@@ -75,17 +78,17 @@ BigDecimal calcularGananciaPorcentaje(
 
   Producto getProductoPorCodigo(String codigo);
 
-  Producto getProductoPorDescripcion(String descripciona);
+  Producto getProductoPorDescripcion(String descripcion);
 
   Producto getProductoNoEliminadoPorId(long idProducto);
 
   BigDecimal calcularValorStock(BusquedaProductoCriteria criteria);
 
-  void getListaDePreciosEnXls(BusquedaProductoCriteria criteria, long idSucursal);
+  void procesarReporteListaDePrecios(BusquedaProductoCriteria criteria, long idSucursal, FormatoReporte formato);
 
-  void getListaDePreciosEnPdf(BusquedaProductoCriteria criteria, long idSucursal);
+  void enviarListaDeProductosPorEmail(String mailTo, byte[] listaDeProductos, FormatoReporte formato);
 
-  void enviarListaDeProductosPorEmail(String mailTo, byte[] listaDeProductos, String formato);
+  byte[] getReporteListaDePrecios(List<Producto> productos, FormatoReporte formato);
 
   Producto guardar(NuevoProductoDTO producto, long idMedida, long idRubro, long idProveedor);
 
@@ -119,7 +122,8 @@ BigDecimal calcularGananciaPorcentaje(
 
   void validarLongitudDeArrays(int longitudIds, int longitudCantidades);
 
-  ProductoFaltanteDTO construirNuevoProductoFaltante(Producto producto, BigDecimal cantidadSolicitada, BigDecimal cantidadDisponible, long idSucursal);
+  ProductoFaltanteDTO construirNuevoProductoFaltante(
+          Producto producto, BigDecimal cantidadSolicitada, BigDecimal cantidadDisponible, long idSucursal);
 
   PrecioProductoEmbeddable construirPrecioProductoEmbeddable(ProductoDTO productoDTO);