Files
GestionIntegralWeb/Backend/GestionIntegral.Api/Services/Suscripciones/AjusteService.cs
dmolinari 5a806eda38 Feat: Mejora UI de Cuenta Corriente y corrige colores en email de aviso
Este commit introduce significativas mejoras de usabilidad en la página de gestión de ajustes del suscriptor y corrige la representación visual de los ajustes en el email de notificación mensual.

###  Nuevas Características y Mejoras de UI

- **Nuevos Filtros en Cuenta Corriente del Suscriptor:**
    - Se han añadido nuevos filtros desplegables en la página de "Cuenta Corriente" para filtrar los ajustes por **Estado** (`Pendiente`, `Aplicado`, `Anulado`) y por **Tipo** (`Crédito`, `Débito`).
    - Esta mejora permite a los usuarios encontrar registros específicos de manera mucho más rápida y eficiente, especialmente para suscriptores con un largo historial de ajustes.

- **Visualización Mejorada del Estado de Ajuste:**
    - La columna "Estado" en la tabla de ajustes ahora muestra el número de factura oficial (ej. `A-0001-12345`) si un ajuste ha sido aplicado y la factura ya está numerada.
    - Si la factura aún no tiene un número oficial, se muestra una referencia al ID interno (ej. `ID Interno #64`) para mantener la trazabilidad.
    - Para soportar esto, se ha enriquecido el `AjusteDto` en el backend para incluir el `NumeroFacturaAplicado`.

### 🐛 Corrección y Refactorización

- **Corrección de Colores en Email de Aviso:**
    - Se han invertido los colores de los montos de ajuste en el email de aviso mensual enviado al cliente para alinearlos con la perspectiva del usuario.
    - **Créditos** (descuentos a favor del cliente) ahora se muestran en **verde** (positivo).
    - **Débitos** (cargos extra) ahora se muestran en **rojo** (negativo).
    - Este cambio mejora drásticamente la claridad del resumen de cuenta y evita posibles confusiones.

### ⚙️ Cambios Técnicos de Soporte

- Se ha añadido el método `GetByIdsAsync` al `IFacturaRepository` para optimizar la obtención de datos de múltiples facturas en una sola consulta, evitando el problema N+1.
- El `AjusteService` ha sido actualizado para utilizar este nuevo método y poblar eficientemente la información de la factura en el DTO de ajuste que se envía al frontend.
2025-08-09 18:16:56 -03:00

225 lines
11 KiB
C#

using GestionIntegral.Api.Data;
using GestionIntegral.Api.Data.Repositories.Suscripciones;
using GestionIntegral.Api.Data.Repositories.Usuarios;
using GestionIntegral.Api.Dtos.Suscripciones;
using GestionIntegral.Api.Models.Suscripciones;
using System.Data;
using GestionIntegral.Api.Data.Repositories.Distribucion;
namespace GestionIntegral.Api.Services.Suscripciones
{
public class AjusteService : IAjusteService
{
private readonly IAjusteRepository _ajusteRepository;
private readonly ISuscriptorRepository _suscriptorRepository;
private readonly IUsuarioRepository _usuarioRepository;
private readonly IEmpresaRepository _empresaRepository;
private readonly IFacturaRepository _facturaRepository;
private readonly DbConnectionFactory _connectionFactory;
private readonly ILogger<AjusteService> _logger;
public AjusteService(
IAjusteRepository ajusteRepository,
ISuscriptorRepository suscriptorRepository,
IUsuarioRepository usuarioRepository,
IEmpresaRepository empresaRepository,
IFacturaRepository facturaRepository,
DbConnectionFactory connectionFactory,
ILogger<AjusteService> logger)
{
_ajusteRepository = ajusteRepository;
_suscriptorRepository = suscriptorRepository;
_usuarioRepository = usuarioRepository;
_empresaRepository = empresaRepository;
_facturaRepository = facturaRepository;
_connectionFactory = connectionFactory;
_logger = logger;
}
private async Task<AjusteDto?> MapToDto(Ajuste ajuste)
{
if (ajuste == null) return null;
var usuario = await _usuarioRepository.GetByIdAsync(ajuste.IdUsuarioAlta);
var empresa = await _empresaRepository.GetByIdAsync(ajuste.IdEmpresa);
return new AjusteDto
{
IdAjuste = ajuste.IdAjuste,
IdSuscriptor = ajuste.IdSuscriptor,
IdEmpresa = ajuste.IdEmpresa,
NombreEmpresa = empresa?.Nombre ?? "N/A",
FechaAjuste = ajuste.FechaAjuste.ToString("yyyy-MM-dd"),
TipoAjuste = ajuste.TipoAjuste,
Monto = ajuste.Monto,
Motivo = ajuste.Motivo,
Estado = ajuste.Estado,
IdFacturaAplicado = ajuste.IdFacturaAplicado,
FechaAlta = ajuste.FechaAlta.ToString("yyyy-MM-dd HH:mm"),
NombreUsuarioAlta = usuario != null ? $"{usuario.Nombre} {usuario.Apellido}" : "N/A"
};
}
public async Task<IEnumerable<AjusteDto>> ObtenerAjustesPorSuscriptor(int idSuscriptor, DateTime? fechaDesde, DateTime? fechaHasta)
{
var ajustes = await _ajusteRepository.GetAjustesPorSuscriptorAsync(idSuscriptor, fechaDesde, fechaHasta);
if (!ajustes.Any())
{
return Enumerable.Empty<AjusteDto>();
}
// 1. Recolectar IDs de usuarios, empresas Y FACTURAS
var idsUsuarios = ajustes.Select(a => a.IdUsuarioAlta).Distinct().ToList();
var idsEmpresas = ajustes.Select(a => a.IdEmpresa).Distinct().ToList();
var idsFacturas = ajustes.Where(a => a.IdFacturaAplicado.HasValue)
.Select(a => a.IdFacturaAplicado!.Value)
.Distinct().ToList();
// 2. Obtener todos los datos necesarios en consultas masivas
var usuariosTask = _usuarioRepository.GetByIdsAsync(idsUsuarios);
var empresasTask = _empresaRepository.GetAllAsync(null, null);
var facturasTask = _facturaRepository.GetByIdsAsync(idsFacturas);
await Task.WhenAll(usuariosTask, empresasTask, facturasTask);
// 3. Convertir a diccionarios para búsqueda rápida
var usuariosDict = (await usuariosTask).ToDictionary(u => u.Id);
var empresasDict = (await empresasTask).ToDictionary(e => e.IdEmpresa);
var facturasDict = (await facturasTask).ToDictionary(f => f.IdFactura);
// 4. Mapear en memoria, ahora con la información de la factura disponible
var dtos = ajustes.Select(ajuste =>
{
usuariosDict.TryGetValue(ajuste.IdUsuarioAlta, out var usuario);
empresasDict.TryGetValue(ajuste.IdEmpresa, out var empresa);
// Buscar la factura en el diccionario si el ajuste está aplicado
facturasDict.TryGetValue(ajuste.IdFacturaAplicado ?? 0, out var factura);
return new AjusteDto
{
IdAjuste = ajuste.IdAjuste,
IdSuscriptor = ajuste.IdSuscriptor,
IdEmpresa = ajuste.IdEmpresa,
NombreEmpresa = empresa?.Nombre ?? "N/A",
FechaAjuste = ajuste.FechaAjuste.ToString("yyyy-MM-dd"),
TipoAjuste = ajuste.TipoAjuste,
Monto = ajuste.Monto,
Motivo = ajuste.Motivo,
Estado = ajuste.Estado,
IdFacturaAplicado = ajuste.IdFacturaAplicado,
NumeroFacturaAplicado = factura?.NumeroFactura,
FechaAlta = ajuste.FechaAlta.ToString("yyyy-MM-dd HH:mm"),
NombreUsuarioAlta = usuario != null ? $"{usuario.Nombre} {usuario.Apellido}" : "N/A"
};
});
return dtos;
}
public async Task<(AjusteDto? Ajuste, string? Error)> CrearAjusteManual(CreateAjusteDto createDto, int idUsuario)
{
var suscriptor = await _suscriptorRepository.GetByIdAsync(createDto.IdSuscriptor);
if (suscriptor == null)
{
return (null, "El suscriptor especificado no existe.");
}
var empresa = await _empresaRepository.GetByIdAsync(createDto.IdEmpresa);
if (empresa == null)
{
return (null, "La empresa especificada no existe.");
}
var nuevoAjuste = new Ajuste
{
IdSuscriptor = createDto.IdSuscriptor,
IdEmpresa = createDto.IdEmpresa,
FechaAjuste = createDto.FechaAjuste.Date,
TipoAjuste = createDto.TipoAjuste,
Monto = createDto.Monto,
Motivo = createDto.Motivo,
IdUsuarioAlta = idUsuario
};
using var connection = _connectionFactory.CreateConnection();
await (connection as System.Data.Common.DbConnection)!.OpenAsync();
using var transaction = connection.BeginTransaction();
try
{
var ajusteCreado = await _ajusteRepository.CreateAsync(nuevoAjuste, transaction);
if (ajusteCreado == null) throw new DataException("Error al crear el registro de ajuste.");
transaction.Commit();
_logger.LogInformation("Ajuste manual ID {IdAjuste} creado para Suscriptor ID {IdSuscriptor} por Usuario ID {IdUsuario}", ajusteCreado.IdAjuste, ajusteCreado.IdSuscriptor, idUsuario);
var dto = await MapToDto(ajusteCreado);
return (dto, null);
}
catch (Exception ex)
{
try { transaction.Rollback(); } catch { }
_logger.LogError(ex, "Error al crear ajuste manual para Suscriptor ID {IdSuscriptor}", createDto.IdSuscriptor);
return (null, "Error interno al registrar el ajuste.");
}
}
public async Task<(bool Exito, string? Error)> AnularAjuste(int idAjuste, int idUsuario)
{
using var connection = _connectionFactory.CreateConnection();
await (connection as System.Data.Common.DbConnection)!.OpenAsync();
using var transaction = connection.BeginTransaction();
try
{
var ajuste = await _ajusteRepository.GetByIdAsync(idAjuste);
if (ajuste == null) return (false, "Ajuste no encontrado.");
if (ajuste.Estado != "Pendiente") return (false, $"No se puede anular un ajuste en estado '{ajuste.Estado}'.");
var exito = await _ajusteRepository.AnularAjusteAsync(idAjuste, idUsuario, transaction);
if (!exito) throw new DataException("No se pudo anular el ajuste.");
transaction.Commit();
_logger.LogInformation("Ajuste ID {IdAjuste} anulado por Usuario ID {IdUsuario}", idAjuste, idUsuario);
return (true, null);
}
catch (Exception ex)
{
try { transaction.Rollback(); } catch { }
_logger.LogError(ex, "Error al anular ajuste ID {IdAjuste}", idAjuste);
return (false, "Error interno al anular el ajuste.");
}
}
public async Task<(bool Exito, string? Error)> ActualizarAjuste(int idAjuste, UpdateAjusteDto updateDto)
{
using var connection = _connectionFactory.CreateConnection();
await (connection as System.Data.Common.DbConnection)!.OpenAsync();
using var transaction = connection.BeginTransaction();
try
{
var ajuste = await _ajusteRepository.GetByIdAsync(idAjuste);
if (ajuste == null) return (false, "Ajuste no encontrado.");
if (ajuste.Estado != "Pendiente") return (false, $"No se puede modificar un ajuste en estado '{ajuste.Estado}'.");
var empresa = await _empresaRepository.GetByIdAsync(updateDto.IdEmpresa);
if (empresa == null) return (false, "La empresa especificada no existe.");
ajuste.IdEmpresa = updateDto.IdEmpresa;
ajuste.FechaAjuste = updateDto.FechaAjuste;
ajuste.TipoAjuste = updateDto.TipoAjuste;
ajuste.Monto = updateDto.Monto;
ajuste.Motivo = updateDto.Motivo;
var actualizado = await _ajusteRepository.UpdateAsync(ajuste, transaction);
if (!actualizado) throw new DataException("La actualización falló o el ajuste ya no estaba pendiente.");
transaction.Commit();
_logger.LogInformation("Ajuste ID {IdAjuste} actualizado.", idAjuste);
return (true, null);
}
catch (Exception ex)
{
try { transaction.Rollback(); } catch { }
_logger.LogError(ex, "Error al actualizar ajuste ID {IdAjuste}", idAjuste);
return (false, "Error interno al actualizar el ajuste.");
}
}
}
}