Feat: Implementa Reporte de Distribución de Suscripciones y Refactoriza Gestión de Ajustes
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.
This commit is contained in:
@@ -51,10 +51,42 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
|
||||
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);
|
||||
var dtosTasks = suscriptores.Select(s => MapToDto(s));
|
||||
var dtos = await Task.WhenAll(dtosTasks);
|
||||
return dtos.Where(dto => dto != null).Select(dto => dto!);
|
||||
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)
|
||||
@@ -108,7 +140,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
|
||||
transaction.Commit();
|
||||
_logger.LogInformation("Suscriptor ID {IdSuscriptor} creado por Usuario ID {IdUsuario}.", suscriptorCreado.IdSuscriptor, idUsuario);
|
||||
|
||||
|
||||
var dtoCreado = await MapToDto(suscriptorCreado);
|
||||
return (dtoCreado, null);
|
||||
}
|
||||
@@ -124,7 +156,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
{
|
||||
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.");
|
||||
@@ -139,7 +171,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
{
|
||||
return (false, "El CBU es obligatorio para la forma de pago seleccionada.");
|
||||
}
|
||||
|
||||
|
||||
// Mapeo DTO -> Modelo
|
||||
suscriptorExistente.NombreCompleto = updateDto.NombreCompleto;
|
||||
suscriptorExistente.Email = updateDto.Email;
|
||||
@@ -156,7 +188,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
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);
|
||||
@@ -183,7 +215,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
{
|
||||
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();
|
||||
@@ -197,7 +229,7 @@ namespace GestionIntegral.Api.Services.Suscripciones
|
||||
_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)
|
||||
catch (Exception ex)
|
||||
{
|
||||
try { transaction.Rollback(); } catch { }
|
||||
_logger.LogError(ex, "Error al cambiar estado del suscriptor ID: {IdSuscriptor}", id);
|
||||
|
||||
Reference in New Issue
Block a user