Implement @slf4j for logging (#474)

* Implement @slf4j for logging

* Change some logging type from warn to info

* Disable logging file

src/main/java/sic/controller/ExceptionControllerAdvice.java

@@ -6,8 +6,7 @@
 import javax.persistence.EntityNotFoundException;
 import javax.validation.ConstraintViolation;
 import javax.validation.ConstraintViolationException;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.MessageSource;
 import org.springframework.http.HttpStatus;
@@ -19,9 +18,9 @@
 import sic.exception.ServiceException;
 
 @RestControllerAdvice
+@Slf4j
 public class ExceptionControllerAdvice {
 
-  private final Logger logger = LoggerFactory.getLogger(this.getClass());
   private final MessageSource messageSource;
 
   @Autowired
@@ -35,7 +34,7 @@ private String log(Exception ex) {
             + "\n(Transaction ID: "
             + LocalDateTime.now().atZone(ZoneId.systemDefault()).toInstant().toEpochMilli()
             + ")";
-    logger.error(mensaje, ex);
+    log.error(mensaje, ex);
     return ex.getMessage();
   }
 
@@ -51,7 +50,7 @@ public String handleConstraintViolationException(ConstraintViolationException ex
             "(Transaction ID: "
                 + LocalDateTime.now().atZone(ZoneId.systemDefault()).toInstant().toEpochMilli()
                 + ")");
-    logger.error(transactionID, ex);
+    log.error(transactionID, ex);
     return mensaje;
   }
 

src/main/java/sic/repository/CajaRepository.java

@@ -12,33 +12,33 @@
 import java.math.BigDecimal;
 import java.time.LocalDateTime;
 
-public interface CajaRepository
-    extends PagingAndSortingRepository<Caja, Long>,
+public interface CajaRepository extends
+        PagingAndSortingRepository<Caja, Long>,
         QuerydslPredicateExecutor<Caja>,
         CajaRepositoryCustom {
 
   @Query("SELECT c FROM Caja c WHERE c.idCaja = :idCaja AND c.eliminada = false")
   Caja findById(@Param("idCaja") long idCaja);
 
   @Query("SELECT c FROM Caja c WHERE c.sucursal.idSucursal = :idSucursal AND c.eliminada = false ORDER BY c.idCaja DESC")
-  Page<Caja> findTopBySucursalAndEliminadaOrderByIdCajaDesc(
-      @Param("idSucursal") long idSucursal, Pageable page);
+  Page<Caja> findTopBySucursalAndEliminadaOrderByIdCajaDesc(@Param("idSucursal") long idSucursal, Pageable page);
 
-  @Query(
-      "SELECT c FROM Caja c "
-          + "WHERE c.sucursal.idSucursal = :idSucursal AND c.eliminada = false AND c.estado = sic.modelo.EstadoCaja.ABIERTA "
+  @Query("SELECT c FROM Caja c "
+          + "WHERE c.sucursal.idSucursal = :idSucursal AND c.eliminada = false "
+          + "AND c.estado = sic.modelo.EstadoCaja.ABIERTA "
           + "ORDER BY c.idCaja DESC")
   Caja isUltimaCajaAbierta(@Param("idSucursal") long idSucursal);
 
-  @Query(
-      "SELECT c FROM Caja c "
-          + "WHERE c.sucursal.idSucursal = :idSucursal AND c.eliminada = false AND c.estado = sic.modelo.EstadoCaja.CERRADA "
+  @Query("SELECT c FROM Caja c "
+          + "WHERE c.sucursal.idSucursal = :idSucursal AND c.eliminada = false "
+          + "AND c.estado = sic.modelo.EstadoCaja.CERRADA "
           + "AND :fecha BETWEEN c.fechaApertura AND c.fechaCierre")
   Caja encontrarCajaCerradaQueContengaFechaEntreFechaAperturaYFechaCierre(
-      @Param("idSucursal") long idSucursal, @Param("fecha") LocalDateTime fecha);
+          @Param("idSucursal") long idSucursal, @Param("fecha") LocalDateTime fecha);
 
   @Modifying
-  @Query(
-      "UPDATE Caja c SET c.saldoSistema = c.saldoSistema + :monto WHERE c.idCaja = :idCaja AND c.estado = sic.modelo.EstadoCaja.CERRADA")
+  @Query("UPDATE Caja c "
+          + "SET c.saldoSistema = c.saldoSistema + :monto WHERE c.idCaja = :idCaja "
+          + "AND c.estado = sic.modelo.EstadoCaja.CERRADA")
   int actualizarSaldoSistema(@Param("idCaja") long idCaja, @Param("monto") BigDecimal monto);
 }

src/main/java/sic/repository/CarritoCompraRepository.java

@@ -12,28 +12,25 @@
 import sic.modelo.ItemCarritoCompra;
 import sic.modelo.Usuario;
 
-public interface CarritoCompraRepository
-    extends PagingAndSortingRepository<ItemCarritoCompra, Long> {
+public interface CarritoCompraRepository extends PagingAndSortingRepository<ItemCarritoCompra, Long> {
 
   Page<ItemCarritoCompra> findAllByUsuario(Usuario usuario, Pageable pageable);
 
   List<ItemCarritoCompra> findAllByUsuarioOrderByIdItemCarritoCompraDesc(Usuario usuario);
 
-  @Query(
-      "SELECT SUM(icc.cantidad) FROM ItemCarritoCompra icc WHERE icc.usuario.idUsuario = :idUsuario")
+  @Query("SELECT SUM(icc.cantidad) FROM ItemCarritoCompra icc WHERE icc.usuario.idUsuario = :idUsuario")
   BigDecimal getCantArticulos(@Param("idUsuario") long idUsuario);
 
   @Query("SELECT COUNT(icc) FROM ItemCarritoCompra icc WHERE icc.usuario.idUsuario = :idUsuario")
   Long getCantRenglones(@Param("idUsuario") long idUsuario);
 
   @Modifying
-  @Query(
-    "DELETE FROM ItemCarritoCompra icc WHERE icc.producto.idProducto = :idProducto")
+  @Query("DELETE FROM ItemCarritoCompra icc WHERE icc.producto.idProducto = :idProducto")
   void eliminarItem(@Param("idProducto") long idProducto);
 
   @Modifying
-  @Query(
-      "DELETE FROM ItemCarritoCompra icc WHERE icc.usuario.idUsuario = :idUsuario AND icc.producto.idProducto = :idProducto")
+  @Query("DELETE FROM ItemCarritoCompra icc "
+          + "WHERE icc.usuario.idUsuario = :idUsuario AND icc.producto.idProducto = :idProducto")
   void eliminarItemDelUsuario(@Param("idUsuario") long idUsuario, @Param("idProducto") long idProducto);
 
   @Modifying

src/main/java/sic/repository/ClienteRepository.java

@@ -10,21 +10,20 @@
 
 import java.util.List;
 
-public interface ClienteRepository
-    extends PagingAndSortingRepository<Cliente, Long>, QuerydslPredicateExecutor<Cliente> {
+public interface ClienteRepository extends
+        PagingAndSortingRepository<Cliente, Long>,
+        QuerydslPredicateExecutor<Cliente> {
 
   List<Cliente> findByIdFiscalAndEliminado(Long idFiscal, boolean eliminado);
 
   Cliente findByAndPredeterminadoAndEliminado(boolean predeterminado, boolean eliminado);
 
   boolean existsByAndPredeterminadoAndEliminado(boolean predeterminado, boolean eliminado);
 
-  @Query(
-      "SELECT c FROM Pedido p INNER JOIN p.cliente c WHERE p.idPedido = :idPedido AND c.eliminado = false")
+  @Query("SELECT c FROM Pedido p INNER JOIN p.cliente c WHERE p.idPedido = :idPedido AND c.eliminado = false")
   Cliente findClienteByIdPedido(@Param("idPedido") long idPedido);
 
-  @Query(
-      "SELECT c FROM Cliente c WHERE c.credencial.idUsuario = :idUsuario AND c.eliminado = false")
+  @Query("SELECT c FROM Cliente c WHERE c.credencial.idUsuario = :idUsuario AND c.eliminado = false")
   Cliente findClienteByIdUsuario(@Param("idUsuario") long idUsuario);
 
   Cliente findByCredencialAndEliminado(Usuario usuarioCredencial, boolean eliminado);
@@ -34,8 +33,7 @@ public interface ClienteRepository
   int desvincularClienteDeViajante(@Param("idUsuarioViajante") long idUsuarioViajante);
 
   @Modifying
-  @Query(
-      "UPDATE Cliente c SET c.credencial = null WHERE c.credencial.idUsuario = :idUsuarioCredencial")
+  @Query("UPDATE Cliente c SET c.credencial = null WHERE c.credencial.idUsuario = :idUsuarioCredencial")
   int desvincularClienteDeCredencial(@Param("idUsuarioCredencial") long idUsuarioCredencial);
 
   boolean existsByNroCliente(String nroCliente);

src/main/java/sic/repository/ConfiguracionSucursalRepository.java

@@ -6,23 +6,19 @@
 import org.springframework.data.repository.query.Param;
 import sic.modelo.ConfiguracionSucursal;
 
-public interface ConfiguracionSucursalRepository
-    extends PagingAndSortingRepository<ConfiguracionSucursal, Long> {
+public interface ConfiguracionSucursalRepository extends PagingAndSortingRepository<ConfiguracionSucursal, Long> {
 
-  @Query(
-      "SELECT configuracionSucursal.cantidadMaximaDeRenglonesEnFactura "
+  @Query("SELECT configuracionSucursal.cantidadMaximaDeRenglonesEnFactura "
           + "FROM Sucursal sucursal INNER JOIN sucursal.configuracionSucursal configuracionSucursal "
           + "WHERE sucursal.idSucursal = :idSucursal")
   int getCantidadMaximaDeRenglones(@Param("idSucursal") long idSucursal);
 
-  @Query(
-      "SELECT configuracionSucursal.facturaElectronicaHabilitada "
+  @Query("SELECT configuracionSucursal.facturaElectronicaHabilitada "
           + "FROM Sucursal sucursal INNER JOIN sucursal.configuracionSucursal configuracionSucursal "
           + "WHERE sucursal.idSucursal = :idSucursal")
   boolean isFacturaElectronicaHabilitada(@Param("idSucursal") long idSucursal);
 
   @Modifying
-  @Query(
-      "UPDATE ConfiguracionSucursal cs SET cs.predeterminada = false WHERE cs.predeterminada = true")
+  @Query("UPDATE ConfiguracionSucursal cs SET cs.predeterminada = false WHERE cs.predeterminada = true")
   void desmarcarSucursalPredeterminada();
 }

src/main/java/sic/repository/CuentaCorrienteClienteRepository.java

@@ -9,18 +9,16 @@
 
 import java.math.BigDecimal;
 
-public interface CuentaCorrienteClienteRepository
-    extends CuentaCorrienteRepository<CuentaCorrienteCliente>,
+public interface CuentaCorrienteClienteRepository extends
+        CuentaCorrienteRepository<CuentaCorrienteCliente>,
         QuerydslPredicateExecutor<CuentaCorrienteCliente> {
 
   CuentaCorrienteCliente findByClienteAndEliminada(Cliente cliente, boolean eliminada);
 
   @Modifying
-  @Query(
-      "UPDATE CuentaCorrienteCliente ccc SET ccc.eliminada = true WHERE ccc.cliente.idCliente = :idCliente")
+  @Query("UPDATE CuentaCorrienteCliente ccc SET ccc.eliminada = true WHERE ccc.cliente.idCliente = :idCliente")
   int eliminarCuentaCorrienteCliente(@Param("idCliente") long idCliente);
 
-  @Query(
-      "SELECT ccc.saldo FROM CuentaCorrienteCliente ccc WHERE ccc.cliente.idCliente = :idCliente")
+  @Query("SELECT ccc.saldo FROM CuentaCorrienteCliente ccc WHERE ccc.cliente.idCliente = :idCliente")
   BigDecimal getSaldoCuentaCorrienteCliente(@Param("idCliente") long idCliente);
 }

src/main/java/sic/repository/CuentaCorrienteProveedorRepository.java

@@ -8,14 +8,13 @@
 import sic.modelo.Sucursal;
 import sic.modelo.Proveedor;
 
-public interface CuentaCorrienteProveedorRepository
-    extends CuentaCorrienteRepository<CuentaCorrienteProveedor>,
+public interface CuentaCorrienteProveedorRepository extends
+        CuentaCorrienteRepository<CuentaCorrienteProveedor>,
         QuerydslPredicateExecutor<CuentaCorrienteProveedor> {
 
   CuentaCorrienteProveedor findByProveedorAndEliminada(Proveedor proveedor, boolean eliminada);
 
   @Modifying
-  @Query(
-      "UPDATE CuentaCorrienteProveedor ccp SET ccp.eliminada = true WHERE ccp.proveedor.idProveedor = :idProveedor")
+  @Query("UPDATE CuentaCorrienteProveedor ccp SET ccp.eliminada = true WHERE ccp.proveedor.idProveedor = :idProveedor")
   int eliminarCuentaCorrienteProveedor(@Param("idProveedor") long idProveedor);
 }

src/main/java/sic/repository/FacturaCompraRepository.java

@@ -3,7 +3,7 @@
 import org.springframework.data.querydsl.QuerydslPredicateExecutor;
 import sic.modelo.FacturaCompra;
 
-public interface FacturaCompraRepository
-    extends FacturaRepository<FacturaCompra>,
+public interface FacturaCompraRepository extends
+        FacturaRepository<FacturaCompra>,
         FacturaCompraRepositoryCustom,
         QuerydslPredicateExecutor<FacturaCompra> {}

src/main/java/sic/repository/FacturaVentaRepository.java

@@ -8,22 +8,21 @@
 
 import java.util.List;
 
-public interface FacturaVentaRepository
-    extends FacturaRepository<FacturaVenta>,
+public interface FacturaVentaRepository extends
+        FacturaRepository<FacturaVenta>,
         FacturaVentaRepositoryCustom,
         QuerydslPredicateExecutor<FacturaVenta> {
 
   @Modifying
-  @Query(
-          "UPDATE FacturaVenta fv SET fv.remito = :remito WHERE fv.idFactura = :idFactura")
+  @Query("UPDATE FacturaVenta fv SET fv.remito = :remito WHERE fv.idFactura = :idFactura")
   void modificarFacturaParaAgregarRemito(@Param("remito") Remito remito, @Param("idFactura") long idFactura);
 
   @Query("SELECT fv FROM FacturaVenta fv WHERE fv.remito = :remito")
   List<FacturaVenta> buscarFacturaPorRemito(@Param("remito") Remito remito);
 
-  @Query(
-      "SELECT max(fv.numFactura) FROM FacturaVenta fv "
-          + "WHERE fv.tipoComprobante = :tipoComprobante AND fv.numSerie = :numSerie AND fv.sucursal.idSucursal = :idSucursal")
+  @Query("SELECT max(fv.numFactura) FROM FacturaVenta fv "
+          + "WHERE fv.tipoComprobante = :tipoComprobante AND fv.numSerie = :numSerie "
+          + "AND fv.sucursal.idSucursal = :idSucursal")
   Long buscarMayorNumFacturaSegunTipo(
       @Param("tipoComprobante") TipoDeComprobante tipoComprobante,
       @Param("numSerie") long numSerie,

src/main/java/sic/repository/GastoRepository.java

@@ -10,15 +10,19 @@
 import sic.modelo.Sucursal;
 import sic.modelo.Gasto;
 
-public interface GastoRepository extends PagingAndSortingRepository<Gasto, Long>, QuerydslPredicateExecutor<Gasto>, GastoRepositoryCustom {
+public interface GastoRepository extends
+        PagingAndSortingRepository<Gasto, Long>,
+        QuerydslPredicateExecutor<Gasto>,
+        GastoRepositoryCustom {
 
     @Query("SELECT g FROM Gasto g " +
             "WHERE g.sucursal.idSucursal = :idSucursal " +
             "AND g.formaDePago.idFormaDePago = :idFormaDePago " +
             "AND g.fecha BETWEEN :desde AND :hasta AND g.eliminado = false")
     List<Gasto> getGastosEntreFechasPorFormaDePago(@Param("idSucursal") long idSucursal,
                                                    @Param("idFormaDePago") long idFormaDePago,
-                                                   @Param("desde") LocalDateTime desde, @Param("hasta") LocalDateTime hasta);
+                                                   @Param("desde") LocalDateTime desde,
+                                                   @Param("hasta") LocalDateTime hasta);
 
     Gasto findTopBySucursalOrderByNroGastoDesc(Sucursal sucursal);
 
@@ -28,20 +32,24 @@ List<Gasto> getGastosEntreFechasPorFormaDePago(@Param("idSucursal") long idSucur
             "WHERE g.sucursal.idSucursal = :idSucursal " +
             "AND g.formaDePago.idFormaDePago = :idFormaDePago " +
             "AND g.fecha BETWEEN :desde AND :hasta AND g.eliminado = false")
-    BigDecimal getTotalGastosEntreFechasPorFormaDePago(@Param("idSucursal") long idSucursal, @Param("idFormaDePago") long idFormaDePago,
-                                                       @Param("desde") LocalDateTime desde, @Param("hasta") LocalDateTime hasta);
+    BigDecimal getTotalGastosEntreFechasPorFormaDePago(@Param("idSucursal") long idSucursal,
+                                                       @Param("idFormaDePago") long idFormaDePago,
+                                                       @Param("desde") LocalDateTime desde,
+                                                       @Param("hasta") LocalDateTime hasta);
 
     @Query("SELECT SUM(g.monto) FROM Gasto g " +
             "WHERE g.sucursal.idSucursal = :idSucursal " +
             "AND g.formaDePago.afectaCaja = true " +
             "AND g.fecha BETWEEN :desde AND :hasta AND g.eliminado = false")
     BigDecimal getTotalGastosQueAfectanCajaEntreFechas(@Param("idSucursal") long idSucursal,
-                                                       @Param("desde") LocalDateTime desde, @Param("hasta") LocalDateTime hasta);
+                                                       @Param("desde") LocalDateTime desde,
+                                                       @Param("hasta") LocalDateTime hasta);
 
     @Query("SELECT SUM(g.monto) FROM Gasto g " +
             "WHERE g.sucursal.idSucursal = :idSucursal " +
             "AND g.fecha BETWEEN :desde AND :hasta AND g.eliminado = false")
     BigDecimal getTotalGastosEntreFechas(@Param("idSucursal") long idSucursal,
-                                         @Param("desde") LocalDateTime desde, @Param("hasta") LocalDateTime hasta);
+                                         @Param("desde") LocalDateTime desde,
+                                         @Param("hasta") LocalDateTime hasta);
 
 }

src/main/java/sic/repository/LocalidadRepository.java

@@ -8,8 +8,9 @@
 import sic.modelo.Localidad;
 import sic.modelo.Provincia;
 
-public interface LocalidadRepository
-    extends PagingAndSortingRepository<Localidad, Long>, QuerydslPredicateExecutor<Localidad> {
+public interface LocalidadRepository extends
+        PagingAndSortingRepository<Localidad, Long>,
+        QuerydslPredicateExecutor<Localidad> {
 
   @Query("SELECT l FROM Localidad l WHERE l.idLocalidad = :idLocalidad")
   Localidad findById(@Param("idLocalidad") long idLocalidad);

src/main/java/sic/repository/NotaCreditoRepository.java

@@ -6,13 +6,14 @@
 import sic.modelo.*;
 import java.util.List;
 
-public interface NotaCreditoRepository
-    extends NotaRepository<NotaCredito>, NotaCreditoRepositoryCustom, QuerydslPredicateExecutor<NotaCredito> {
+public interface NotaCreditoRepository extends
+        NotaRepository<NotaCredito>,
+        NotaCreditoRepositoryCustom,
+        QuerydslPredicateExecutor<NotaCredito> {
 
   List<NotaCredito> findAllByFacturaVentaAndEliminada(FacturaVenta factura, boolean eliminada);
 
-  @Query(
-      "SELECT max(nc.nroNota) FROM NotaCredito nc "
+  @Query("SELECT max(nc.nroNota) FROM NotaCredito nc "
           + "WHERE nc.tipoComprobante = :tipoComprobante "
           + "AND nc.serie = :serie "
           + "AND nc.sucursal.idSucursal = :idSucursal "

src/main/java/sic/repository/NotaDebitoRepository.java

@@ -5,11 +5,12 @@
 import org.springframework.data.repository.query.Param;
 import sic.modelo.*;
 
-public interface NotaDebitoRepository
-    extends NotaRepository<NotaDebito>, NotaDebitoRepositoryCustom, QuerydslPredicateExecutor<NotaDebito> {
+public interface NotaDebitoRepository extends
+        NotaRepository<NotaDebito>,
+        NotaDebitoRepositoryCustom,
+        QuerydslPredicateExecutor<NotaDebito> {
 
-  @Query(
-      "SELECT max(nd.nroNota) FROM NotaDebito nd "
+  @Query("SELECT max(nd.nroNota) FROM NotaDebito nd "
           + "WHERE nd.tipoComprobante = :tipoComprobante AND nd.serie = :serie AND nd.sucursal.idSucursal = :idSucursal "
           + "AND nd.cliente IS NOT null")
   Long buscarMayorNumNotaDebitoClienteSegunTipo(

src/main/java/sic/repository/NotaRepository.java

@@ -3,5 +3,4 @@
 import org.springframework.data.repository.PagingAndSortingRepository;
 import sic.modelo.Nota;
 
-public interface NotaRepository<T extends Nota>
-    extends PagingAndSortingRepository<T, Long> {}
+public interface NotaRepository<T extends Nota> extends PagingAndSortingRepository<T, Long> {}

src/main/java/sic/repository/PedidoRepository.java

@@ -8,11 +8,11 @@
 import org.springframework.data.repository.query.Param;
 import sic.modelo.*;
 
-public interface PedidoRepository
-    extends PagingAndSortingRepository<Pedido, Long>, QuerydslPredicateExecutor<Pedido> {
+public interface PedidoRepository extends
+        PagingAndSortingRepository<Pedido, Long>,
+        QuerydslPredicateExecutor<Pedido> {
 
-  Pedido findByNroPedidoAndSucursalAndEliminado(
-      long nroPedido, Sucursal sucursal, boolean eliminado);
+  Pedido findByNroPedidoAndSucursalAndEliminado(long nroPedido, Sucursal sucursal, boolean eliminado);
 
   boolean existsByNroPedidoAndSucursal(long nroPedido, Sucursal sucursal);
 

src/main/java/sic/repository/ProductoFavoritoRepository.java

@@ -10,8 +10,8 @@
 
 import java.util.List;
 
-public interface ProductoFavoritoRepository
-    extends PagingAndSortingRepository<ProductoFavorito, Long>,
+public interface ProductoFavoritoRepository extends
+        PagingAndSortingRepository<ProductoFavorito, Long>,
         QuerydslPredicateExecutor<ProductoFavorito> {
 
   List<ProductoFavorito> findAllByCliente(Cliente cliente);