Se introduce una nueva funcionalidad de reporte crucial para la logística y se realiza una refactorización mayor del sistema de ajustes para garantizar la correcta imputación contable. ### ✨ Nuevas Características - **Nuevo Reporte de Distribución de Suscripciones (RR011):** - Se añade un nuevo reporte en PDF que genera un listado de todas las suscripciones activas en un rango de fechas. - El reporte está diseñado para el equipo de reparto, incluyendo datos clave como nombre del suscriptor, dirección, teléfono, días de entrega y observaciones. - Se implementa el endpoint, la lógica de servicio, la consulta a la base de datos y el template de QuestPDF en el backend. - Se crea la página correspondiente en el frontend (React) con su selector de fechas y se añade la ruta y el enlace en el menú de reportes. ### 🔄 Refactorización Mayor - **Asociación de Ajustes a Empresas:** - Se refactoriza por completo la entidad `Ajuste` para incluir una referencia obligatoria a una `IdEmpresa`. - **Motivo:** Corregir un error crítico en la lógica de negocio donde los ajustes de un suscriptor se aplicaban a la primera factura generada, sin importar a qué empresa correspondía el ajuste. - Se provee un script de migración SQL para actualizar el esquema de la base de datos (`susc_Ajustes`). - Se actualizan todos los DTOs, repositorios y servicios (backend) para manejar la nueva relación. - Se modifica el `FacturacionService` para que ahora aplique los ajustes pendientes correspondientes a cada empresa dentro de su bucle de facturación. - Se actualiza el formulario de creación/edición de ajustes en el frontend (React) para incluir un selector de empresa obligatorio. ### ⚡️ Optimizaciones de Rendimiento - **Solución de N+1 Queries:** - Se optimiza el método `ObtenerTodos` en `SuscriptorService` para obtener todas las formas de pago en una única consulta en lugar de una por cada suscriptor. - Se optimiza el método `ObtenerAjustesPorSuscriptor` en `AjusteService` para obtener todos los nombres de usuarios y empresas en dos consultas masivas, en lugar de N consultas individuales. - Se añade el método `GetByIdsAsync` al `IUsuarioRepository` y su implementación para soportar esta optimización. ### 🐛 Corrección de Errores - Se corrigen múltiples errores en el script de migración de base de datos (uso de `GO` dentro de transacciones, error de "columna inválida"). - Se soluciona un error de SQL (`INSERT` statement) en `AjusteRepository` que impedía la creación de nuevos ajustes. - Se corrige un bug en el `AjusteService` que causaba que el nombre de la empresa apareciera como "N/A" en la UI debido a un mapeo incompleto en el método optimizado. - Se corrige la lógica de generación de emails en `FacturacionService` para mostrar correctamente el nombre de la empresa en cada sección del resumen de cuenta.
		
			
				
	
	
		
			250 lines
		
	
	
		
			11 KiB
		
	
	
	
		
			C#
		
	
	
	
	
	
			
		
		
	
	
			250 lines
		
	
	
		
			11 KiB
		
	
	
	
		
			C#
		
	
	
	
	
	
| using GestionIntegral.Api.Data;
 | |
| using GestionIntegral.Api.Data.Repositories.Suscripciones;
 | |
| using GestionIntegral.Api.Dtos.Suscripciones;
 | |
| using GestionIntegral.Api.Models.Suscripciones;
 | |
| using System.Data;
 | |
| 
 | |
| namespace GestionIntegral.Api.Services.Suscripciones
 | |
| {
 | |
|     public class SuscriptorService : ISuscriptorService
 | |
|     {
 | |
|         private readonly ISuscriptorRepository _suscriptorRepository;
 | |
|         private readonly IFormaPagoRepository _formaPagoRepository;
 | |
|         private readonly DbConnectionFactory _connectionFactory;
 | |
|         private readonly ILogger<SuscriptorService> _logger;
 | |
| 
 | |
|         public SuscriptorService(
 | |
|             ISuscriptorRepository suscriptorRepository,
 | |
|             IFormaPagoRepository formaPagoRepository,
 | |
|             DbConnectionFactory connectionFactory,
 | |
|             ILogger<SuscriptorService> logger)
 | |
|         {
 | |
|             _suscriptorRepository = suscriptorRepository;
 | |
|             _formaPagoRepository = formaPagoRepository;
 | |
|             _connectionFactory = connectionFactory;
 | |
|             _logger = logger;
 | |
|         }
 | |
| 
 | |
|         // Helper para mapear Modelo -> DTO, enriqueciendo con el nombre de la forma de pago
 | |
|         private async Task<SuscriptorDto?> MapToDto(Suscriptor suscriptor)
 | |
|         {
 | |
|             if (suscriptor == null) return null;
 | |
| 
 | |
|             var formaPago = await _formaPagoRepository.GetByIdAsync(suscriptor.IdFormaPagoPreferida);
 | |
| 
 | |
|             return new SuscriptorDto
 | |
|             {
 | |
|                 IdSuscriptor = suscriptor.IdSuscriptor,
 | |
|                 NombreCompleto = suscriptor.NombreCompleto,
 | |
|                 Email = suscriptor.Email,
 | |
|                 Telefono = suscriptor.Telefono,
 | |
|                 Direccion = suscriptor.Direccion,
 | |
|                 TipoDocumento = suscriptor.TipoDocumento,
 | |
|                 NroDocumento = suscriptor.NroDocumento,
 | |
|                 CBU = suscriptor.CBU,
 | |
|                 IdFormaPagoPreferida = suscriptor.IdFormaPagoPreferida,
 | |
|                 NombreFormaPagoPreferida = formaPago?.Nombre ?? "Desconocida",
 | |
|                 Observaciones = suscriptor.Observaciones,
 | |
|                 Activo = suscriptor.Activo
 | |
|             };
 | |
|         }
 | |
| 
 | |
|         public async Task<IEnumerable<SuscriptorDto>> ObtenerTodos(string? nombreFilter, string? nroDocFilter, bool soloActivos)
 | |
|         {
 | |
|             // 1. Obtener todos los suscriptores en una sola consulta
 | |
|             var suscriptores = await _suscriptorRepository.GetAllAsync(nombreFilter, nroDocFilter, soloActivos);
 | |
|             if (!suscriptores.Any())
 | |
|             {
 | |
|                 return Enumerable.Empty<SuscriptorDto>();
 | |
|             }
 | |
| 
 | |
|             // 2. Obtener todas las formas de pago en una sola consulta
 | |
|             // y convertirlas a un diccionario para una búsqueda rápida (O(1) en lugar de O(n)).
 | |
|             var formasDePago = (await _formaPagoRepository.GetAllAsync())
 | |
|                                 .ToDictionary(fp => fp.IdFormaPago);
 | |
| 
 | |
|             // 3. Mapear en memoria, evitando múltiples llamadas a la base de datos.
 | |
|             var dtos = suscriptores.Select(s =>
 | |
|             {
 | |
|                 // Busca la forma de pago en el diccionario en memoria.
 | |
|                 formasDePago.TryGetValue(s.IdFormaPagoPreferida, out var formaPago);
 | |
| 
 | |
|                 return new SuscriptorDto
 | |
|                 {
 | |
|                     IdSuscriptor = s.IdSuscriptor,
 | |
|                     NombreCompleto = s.NombreCompleto,
 | |
|                     Email = s.Email,
 | |
|                     Telefono = s.Telefono,
 | |
|                     Direccion = s.Direccion,
 | |
|                     TipoDocumento = s.TipoDocumento,
 | |
|                     NroDocumento = s.NroDocumento,
 | |
|                     CBU = s.CBU,
 | |
|                     IdFormaPagoPreferida = s.IdFormaPagoPreferida,
 | |
|                     NombreFormaPagoPreferida = formaPago?.Nombre ?? "Desconocida", // Asigna el nombre
 | |
|                     Observaciones = s.Observaciones,
 | |
|                     Activo = s.Activo
 | |
|                 };
 | |
|             });
 | |
| 
 | |
|             return dtos;
 | |
|         }
 | |
| 
 | |
|         public async Task<SuscriptorDto?> ObtenerPorId(int id)
 | |
|         {
 | |
|             var suscriptor = await _suscriptorRepository.GetByIdAsync(id);
 | |
|             if (suscriptor == null)
 | |
|                 return null;
 | |
|             return await MapToDto(suscriptor);
 | |
|         }
 | |
| 
 | |
|         public async Task<(SuscriptorDto? Suscriptor, string? Error)> Crear(CreateSuscriptorDto createDto, int idUsuario)
 | |
|         {
 | |
|             // Validación de Lógica de Negocio
 | |
|             if (await _suscriptorRepository.ExistsByDocumentoAsync(createDto.TipoDocumento, createDto.NroDocumento))
 | |
|             {
 | |
|                 return (null, "Ya existe un suscriptor con el mismo tipo y número de documento.");
 | |
|             }
 | |
| 
 | |
|             var formaPago = await _formaPagoRepository.GetByIdAsync(createDto.IdFormaPagoPreferida);
 | |
|             if (formaPago == null || !formaPago.Activo)
 | |
|             {
 | |
|                 return (null, "La forma de pago seleccionada no es válida o está inactiva.");
 | |
|             }
 | |
|             if (formaPago.RequiereCBU && string.IsNullOrWhiteSpace(createDto.CBU))
 | |
|             {
 | |
|                 return (null, "El CBU es obligatorio para la forma de pago seleccionada.");
 | |
|             }
 | |
| 
 | |
|             var nuevoSuscriptor = new Suscriptor
 | |
|             {
 | |
|                 NombreCompleto = createDto.NombreCompleto,
 | |
|                 Email = createDto.Email,
 | |
|                 Telefono = createDto.Telefono,
 | |
|                 Direccion = createDto.Direccion,
 | |
|                 TipoDocumento = createDto.TipoDocumento,
 | |
|                 NroDocumento = createDto.NroDocumento,
 | |
|                 CBU = createDto.CBU,
 | |
|                 IdFormaPagoPreferida = createDto.IdFormaPagoPreferida,
 | |
|                 Observaciones = createDto.Observaciones,
 | |
|                 IdUsuarioAlta = idUsuario
 | |
|             };
 | |
| 
 | |
|             using var connection = _connectionFactory.CreateConnection();
 | |
|             await (connection as System.Data.Common.DbConnection)!.OpenAsync();
 | |
|             using var transaction = connection.BeginTransaction();
 | |
| 
 | |
|             try
 | |
|             {
 | |
|                 var suscriptorCreado = await _suscriptorRepository.CreateAsync(nuevoSuscriptor, transaction);
 | |
|                 if (suscriptorCreado == null) throw new DataException("La creación en el repositorio devolvió null.");
 | |
| 
 | |
|                 transaction.Commit();
 | |
|                 _logger.LogInformation("Suscriptor ID {IdSuscriptor} creado por Usuario ID {IdUsuario}.", suscriptorCreado.IdSuscriptor, idUsuario);
 | |
| 
 | |
|                 var dtoCreado = await MapToDto(suscriptorCreado);
 | |
|                 return (dtoCreado, null);
 | |
|             }
 | |
|             catch (Exception ex)
 | |
|             {
 | |
|                 try { transaction.Rollback(); } catch { }
 | |
|                 _logger.LogError(ex, "Error al crear suscriptor: {Nombre}", createDto.NombreCompleto);
 | |
|                 return (null, $"Error interno al crear el suscriptor: {ex.Message}");
 | |
|             }
 | |
|         }
 | |
| 
 | |
|         public async Task<(bool Exito, string? Error)> Actualizar(int id, UpdateSuscriptorDto updateDto, int idUsuario)
 | |
|         {
 | |
|             var suscriptorExistente = await _suscriptorRepository.GetByIdAsync(id);
 | |
|             if (suscriptorExistente == null) return (false, "Suscriptor no encontrado.");
 | |
| 
 | |
|             if (await _suscriptorRepository.ExistsByDocumentoAsync(updateDto.TipoDocumento, updateDto.NroDocumento, id))
 | |
|             {
 | |
|                 return (false, "El tipo y número de documento ya pertenecen a otro suscriptor.");
 | |
|             }
 | |
| 
 | |
|             var formaPago = await _formaPagoRepository.GetByIdAsync(updateDto.IdFormaPagoPreferida);
 | |
|             if (formaPago == null || !formaPago.Activo)
 | |
|             {
 | |
|                 return (false, "La forma de pago seleccionada no es válida o está inactiva.");
 | |
|             }
 | |
|             if (formaPago.RequiereCBU && string.IsNullOrWhiteSpace(updateDto.CBU))
 | |
|             {
 | |
|                 return (false, "El CBU es obligatorio para la forma de pago seleccionada.");
 | |
|             }
 | |
| 
 | |
|             // Mapeo DTO -> Modelo
 | |
|             suscriptorExistente.NombreCompleto = updateDto.NombreCompleto;
 | |
|             suscriptorExistente.Email = updateDto.Email;
 | |
|             suscriptorExistente.Telefono = updateDto.Telefono;
 | |
|             suscriptorExistente.Direccion = updateDto.Direccion;
 | |
|             suscriptorExistente.TipoDocumento = updateDto.TipoDocumento;
 | |
|             suscriptorExistente.NroDocumento = updateDto.NroDocumento;
 | |
|             suscriptorExistente.CBU = updateDto.CBU;
 | |
|             suscriptorExistente.IdFormaPagoPreferida = updateDto.IdFormaPagoPreferida;
 | |
|             suscriptorExistente.Observaciones = updateDto.Observaciones;
 | |
|             suscriptorExistente.IdUsuarioMod = idUsuario;
 | |
|             suscriptorExistente.FechaMod = DateTime.Now;
 | |
| 
 | |
|             using var connection = _connectionFactory.CreateConnection();
 | |
|             await (connection as System.Data.Common.DbConnection)!.OpenAsync();
 | |
|             using var transaction = connection.BeginTransaction();
 | |
| 
 | |
|             try
 | |
|             {
 | |
|                 var actualizado = await _suscriptorRepository.UpdateAsync(suscriptorExistente, transaction);
 | |
|                 if (!actualizado) throw new DataException("La actualización en el repositorio devolvió false.");
 | |
| 
 | |
|                 transaction.Commit();
 | |
|                 _logger.LogInformation("Suscriptor ID {IdSuscriptor} actualizado por Usuario ID {IdUsuario}.", id, idUsuario);
 | |
|                 return (true, null);
 | |
|             }
 | |
|             catch (Exception ex)
 | |
|             {
 | |
|                 try { transaction.Rollback(); } catch { }
 | |
|                 _logger.LogError(ex, "Error al actualizar suscriptor ID: {IdSuscriptor}", id);
 | |
|                 return (false, $"Error interno al actualizar: {ex.Message}");
 | |
|             }
 | |
|         }
 | |
| 
 | |
|         private async Task<(bool Exito, string? Error)> CambiarEstadoActivo(int id, bool activar, int idUsuario)
 | |
|         {
 | |
|             var suscriptor = await _suscriptorRepository.GetByIdAsync(id);
 | |
|             if (suscriptor == null) return (false, "Suscriptor no encontrado.");
 | |
| 
 | |
|             if (!activar && await _suscriptorRepository.IsInUseAsync(id))
 | |
|             {
 | |
|                 return (false, "No se puede desactivar un suscriptor con suscripciones activas.");
 | |
|             }
 | |
| 
 | |
|             using var connection = _connectionFactory.CreateConnection();
 | |
|             await (connection as System.Data.Common.DbConnection)!.OpenAsync();
 | |
|             using var transaction = connection.BeginTransaction();
 | |
| 
 | |
|             try
 | |
|             {
 | |
|                 var actualizado = await _suscriptorRepository.ToggleActivoAsync(id, activar, idUsuario, transaction);
 | |
|                 if (!actualizado) throw new DataException("No se pudo cambiar el estado del suscriptor.");
 | |
| 
 | |
|                 transaction.Commit();
 | |
|                 _logger.LogInformation("El estado del Suscriptor ID {IdSuscriptor} se cambió a {Estado} por el Usuario ID {IdUsuario}.", id, activar ? "Activo" : "Inactivo", idUsuario);
 | |
|                 return (true, null);
 | |
|             }
 | |
|             catch (Exception ex)
 | |
|             {
 | |
|                 try { transaction.Rollback(); } catch { }
 | |
|                 _logger.LogError(ex, "Error al cambiar estado del suscriptor ID: {IdSuscriptor}", id);
 | |
|                 return (false, $"Error interno: {ex.Message}");
 | |
|             }
 | |
|         }
 | |
| 
 | |
|         public Task<(bool Exito, string? Error)> Desactivar(int id, int idUsuario)
 | |
|         {
 | |
|             return CambiarEstadoActivo(id, false, idUsuario);
 | |
|         }
 | |
| 
 | |
|         public Task<(bool Exito, string? Error)> Activar(int id, int idUsuario)
 | |
|         {
 | |
|             return CambiarEstadoActivo(id, true, idUsuario);
 | |
|         }
 | |
|     }
 | |
| } |