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);
|
|
}
|
|
}
|
|
} |