Feat: Implementa auditoría de envíos masivos y mejora de procesos
Se introduce un sistema completo para auditar los envíos masivos de correos durante el cierre mensual y se refactoriza la interfaz de usuario de procesos para una mayor claridad y escalabilidad. Además, se mejora la lógica de negocio para la gestión de bajas de suscripciones. ### ✨ Nuevas Características - **Auditoría de Envíos Masivos (Cierre Mensual):** - Se crea una nueva tabla `com_LotesDeEnvio` para registrar cada ejecución del proceso de facturación mensual. - El `FacturacionService` ahora crea un "lote" al iniciar el cierre, registra el resultado de cada envío de email individual asociándolo a dicho lote, y actualiza las estadísticas finales (enviados, fallidos) al terminar. - Se implementa un nuevo `LotesEnvioController` con un endpoint para consultar los detalles de cualquier lote de envío histórico. ### 🔄 Refactorización y Mejoras - **Rediseño de la Página de Procesos:** - La antigua página "Facturación" se renombra a `CierreYProcesosPage` y se rediseña completamente utilizando una interfaz de Pestañas (Tabs). - **Pestaña "Procesos Mensuales":** Aisla las acciones principales (Generar Cierre, Archivo de Débito, Procesar Respuesta), mostrando un resumen del resultado del último envío. - **Pestaña "Historial de Cierres":** Muestra una tabla con todos los lotes de envío pasados y permite al usuario ver los detalles de cada uno en un modal. - **Filtros para el Historial de Cierres:** - Se añaden filtros por Mes y Año a la pestaña de "Historial de Cierres", permitiendo al usuario buscar y auditar procesos pasados de manera eficiente. El filtrado se realiza en el backend para un rendimiento óptimo. - **Lógica de `FechaFin` Obligatoria para Bajas:** - Se implementa una regla de negocio crucial: al cambiar el estado de una suscripción a "Pausada" o "Cancelada", ahora es obligatorio establecer una `FechaFin`. - **Frontend:** El modal de suscripciones ahora gestiona esto automáticamente, haciendo el campo `FechaFin` requerido y visible según el estado seleccionado. - **Backend:** Se añade una validación en `SuscripcionService` como segunda capa de seguridad para garantizar la integridad de los datos. ### 🐛 Corrección de Errores - **Reporte de Distribución:** Se corrigió un bug en la generación del PDF donde la columna de fecha no mostraba la "Fecha de Baja" para las suscripciones finalizadas. Ahora se muestra la fecha correcta según la sección (Altas o Bajas). - **Errores de Compilación y Dependencias:** Se solucionaron varios errores de compilación en el backend, principalmente relacionados con la falta de registro de los nuevos repositorios (`ILoteDeEnvioRepository`, `IEmailLogService`, etc.) en el contenedor de inyección de dependencias (`Program.cs`). - **Errores de Tipado en Frontend:** Se corrigieron múltiples errores de TypeScript en `CierreYProcesosPage` debidos a la inconsistencia entre `PascalCase` (C#) y `camelCase` (JSON/TypeScript), asegurando un mapeo correcto de los datos de la API.
This commit is contained in:
@@ -0,0 +1,40 @@
|
||||
using GestionIntegral.Api.Dtos.Comunicaciones;
|
||||
using GestionIntegral.Api.Services.Comunicaciones;
|
||||
using Microsoft.AspNetCore.Authorization;
|
||||
using Microsoft.AspNetCore.Mvc;
|
||||
|
||||
namespace GestionIntegral.Api.Controllers.Comunicaciones
|
||||
{
|
||||
[Route("api/lotes-envio")]
|
||||
[ApiController]
|
||||
[Authorize]
|
||||
public class LotesEnvioController : ControllerBase
|
||||
{
|
||||
private readonly IEmailLogService _emailLogService;
|
||||
|
||||
public LotesEnvioController(IEmailLogService emailLogService)
|
||||
{
|
||||
_emailLogService = emailLogService;
|
||||
}
|
||||
|
||||
// GET: api/lotes-envio/123/detalles
|
||||
[HttpGet("{idLote:int}/detalles")]
|
||||
[ProducesResponseType(typeof(IEnumerable<EmailLogDto>), StatusCodes.Status200OK)]
|
||||
[ProducesResponseType(StatusCodes.Status403Forbidden)]
|
||||
[ProducesResponseType(StatusCodes.Status404NotFound)]
|
||||
public async Task<IActionResult> GetDetallesLote(int idLote)
|
||||
{
|
||||
// Reutilizamos un permiso existente, ya que esta es una función de auditoría relacionada.
|
||||
var tienePermiso = User.IsInRole("SuperAdmin") || User.HasClaim(c => c.Type == "permission" && c.Value == "SU006");
|
||||
if (!tienePermiso)
|
||||
{
|
||||
return Forbid();
|
||||
}
|
||||
|
||||
var detalles = await _emailLogService.ObtenerDetallesPorLoteId(idLote);
|
||||
|
||||
// Devolvemos OK con un array vacío si no hay resultados, el frontend lo manejará.
|
||||
return Ok(detalles);
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -49,63 +49,103 @@ namespace GestionIntegral.Api.Controllers.Reportes.PdfTemplates
|
||||
{
|
||||
container.PaddingTop(10).Column(column =>
|
||||
{
|
||||
column.Spacing(20);
|
||||
foreach (var empresa in Model.DatosAgrupados)
|
||||
column.Spacing(20); // Espacio entre elementos principales (sección de altas y sección de bajas)
|
||||
|
||||
// --- Sección 1: Altas y Activas ---
|
||||
column.Item().Column(colAltas =>
|
||||
{
|
||||
column.Item().Element(c => ComposeEmpresa(c, empresa));
|
||||
colAltas.Item().Text("Altas y Suscripciones Activas en el Período").Bold().FontSize(14).Underline();
|
||||
colAltas.Item().PaddingBottom(10).Text("Listado de suscriptores que deben recibir entregas en el período seleccionado.");
|
||||
|
||||
if (!Model.DatosAgrupadosAltas.Any())
|
||||
{
|
||||
colAltas.Item().PaddingTop(10).Text("No se encontraron suscripciones activas para este período.").Italic();
|
||||
}
|
||||
else
|
||||
{
|
||||
foreach (var empresa in Model.DatosAgrupadosAltas)
|
||||
{
|
||||
colAltas.Item().Element(c => ComposeTablaEmpresa(c, empresa, esBaja: false));
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
// --- Sección 2: Bajas ---
|
||||
if (Model.DatosAgrupadosBajas.Any())
|
||||
{
|
||||
column.Item().PageBreak(); // Salto de página para separar las secciones
|
||||
column.Item().Column(colBajas =>
|
||||
{
|
||||
colBajas.Item().Text("Bajas de Suscripciones en el Período").Bold().FontSize(14).Underline().FontColor(Colors.Red.Medium);
|
||||
colBajas.Item().PaddingBottom(10).Text("Listado de suscriptores cuya suscripción finalizó. NO se les debe entregar a partir de su 'Fecha de Baja'.");
|
||||
|
||||
foreach (var empresa in Model.DatosAgrupadosBajas)
|
||||
{
|
||||
colBajas.Item().Element(c => ComposeTablaEmpresa(c, empresa, esBaja: true));
|
||||
}
|
||||
});
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
void ComposeEmpresa(IContainer container, GrupoEmpresa empresa)
|
||||
void ComposeTablaEmpresa(IContainer container, GrupoEmpresa empresa, bool esBaja)
|
||||
{
|
||||
container.Column(column =>
|
||||
{
|
||||
// Cabecera de la EMPRESA (ej. EL DIA)
|
||||
column.Item().Background(Colors.Grey.Lighten2).Padding(5).Text(empresa.NombreEmpresa).Bold().FontSize(12);
|
||||
column.Item().Column(colPub =>
|
||||
|
||||
// Contenedor para las tablas de las publicaciones de esta empresa
|
||||
column.Item().PaddingTop(5).Column(colPub =>
|
||||
{
|
||||
colPub.Spacing(10);
|
||||
colPub.Spacing(10); // Espacio entre cada tabla de publicación
|
||||
foreach (var publicacion in empresa.Publicaciones)
|
||||
{
|
||||
colPub.Item().Element(c => ComposePublicacion(c, publicacion));
|
||||
colPub.Item().Element(c => ComposeTablaPublicacion(c, publicacion, esBaja));
|
||||
}
|
||||
});
|
||||
});
|
||||
}
|
||||
|
||||
void ComposePublicacion(IContainer container, GrupoPublicacion publicacion)
|
||||
void ComposeTablaPublicacion(IContainer container, GrupoPublicacion publicacion, bool esBaja)
|
||||
{
|
||||
container.Table(table =>
|
||||
// Se envuelve la tabla en una columna para poder ponerle un título simple arriba.
|
||||
container.Column(column =>
|
||||
{
|
||||
table.ColumnsDefinition(columns =>
|
||||
column.Item().PaddingLeft(2).PaddingBottom(2).Text(publicacion.NombrePublicacion).SemiBold().FontSize(10);
|
||||
column.Item().Table(table =>
|
||||
{
|
||||
columns.RelativeColumn(2.5f); // Nombre
|
||||
columns.RelativeColumn(3); // Dirección
|
||||
columns.RelativeColumn(1.5f); // Teléfono
|
||||
columns.RelativeColumn(1.5f); // Días
|
||||
columns.RelativeColumn(2.5f); // Observaciones
|
||||
table.ColumnsDefinition(columns =>
|
||||
{
|
||||
columns.RelativeColumn(2.5f); // Nombre
|
||||
columns.RelativeColumn(3); // Dirección
|
||||
columns.RelativeColumn(1.5f); // Teléfono
|
||||
columns.ConstantColumn(65); // Fecha Inicio / Baja
|
||||
columns.RelativeColumn(1.5f); // Días
|
||||
columns.RelativeColumn(2.5f); // Observaciones
|
||||
});
|
||||
|
||||
table.Header(header =>
|
||||
{
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Suscriptor").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Dirección").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Teléfono").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text(esBaja ? "Fecha de Baja" : "Fecha Inicio").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Días Entrega").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Observaciones").SemiBold();
|
||||
});
|
||||
|
||||
foreach (var item in publicacion.Suscripciones)
|
||||
{
|
||||
table.Cell().Padding(2).Text(item.NombreSuscriptor);
|
||||
table.Cell().Padding(2).Text(item.Direccion);
|
||||
table.Cell().Padding(2).Text(item.Telefono ?? "-");
|
||||
var fecha = esBaja ? item.FechaFin : item.FechaInicio;
|
||||
table.Cell().Padding(2).Text(fecha?.ToString("dd/MM/yyyy") ?? "-");
|
||||
table.Cell().Padding(2).Text(item.DiasEntrega);
|
||||
table.Cell().Padding(2).Text(item.Observaciones ?? "-");
|
||||
}
|
||||
});
|
||||
|
||||
table.Header(header =>
|
||||
{
|
||||
header.Cell().ColumnSpan(5).Background(Colors.Grey.Lighten4).Padding(3)
|
||||
.Text(publicacion.NombrePublicacion).SemiBold().FontSize(10);
|
||||
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Suscriptor").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Dirección").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Teléfono").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Días Entrega").SemiBold();
|
||||
header.Cell().BorderBottom(1).Padding(2).Text("Observaciones").SemiBold();
|
||||
});
|
||||
|
||||
foreach (var item in publicacion.Suscripciones)
|
||||
{
|
||||
table.Cell().Padding(2).Text(item.NombreSuscriptor);
|
||||
table.Cell().Padding(2).Text(item.Direccion);
|
||||
table.Cell().Padding(2).Text(item.Telefono ?? "-");
|
||||
table.Cell().Padding(2).Text(item.DiasEntrega);
|
||||
table.Cell().Padding(2).Text(item.Observaciones ?? "-");
|
||||
}
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1727,16 +1727,16 @@ namespace GestionIntegral.Api.Controllers
|
||||
{
|
||||
if (!TienePermiso(PermisoVerReporteDistSuscripciones)) return Forbid();
|
||||
|
||||
var (data, error) = await _reportesService.ObtenerReporteDistribucionSuscripcionesAsync(fechaDesde, fechaHasta);
|
||||
var (altas, bajas, error) = await _reportesService.ObtenerReporteDistribucionSuscripcionesAsync(fechaDesde, fechaHasta);
|
||||
if (error != null) return BadRequest(new { message = error });
|
||||
if (data == null || !data.Any())
|
||||
if ((altas == null || !altas.Any()) && (bajas == null || !bajas.Any()))
|
||||
{
|
||||
return NotFound(new { message = "No se encontraron suscripciones activas para el período seleccionado." });
|
||||
return NotFound(new { message = "No se encontraron suscripciones activas ni bajas para el período seleccionado." });
|
||||
}
|
||||
|
||||
try
|
||||
{
|
||||
var viewModel = new DistribucionSuscripcionesViewModel(data)
|
||||
var viewModel = new DistribucionSuscripcionesViewModel(altas ?? Enumerable.Empty<DistribucionSuscripcionDto>(), bajas ?? Enumerable.Empty<DistribucionSuscripcionDto>())
|
||||
{
|
||||
FechaDesde = fechaDesde.ToString("dd/MM/yyyy"),
|
||||
FechaHasta = fechaHasta.ToString("dd/MM/yyyy"),
|
||||
|
||||
@@ -1,5 +1,4 @@
|
||||
using GestionIntegral.Api.Dtos.Comunicaciones;
|
||||
using GestionIntegral.Api.Dtos.Suscripciones;
|
||||
using GestionIntegral.Api.Services.Comunicaciones;
|
||||
using GestionIntegral.Api.Services.Suscripciones;
|
||||
using Microsoft.AspNetCore.Authorization;
|
||||
@@ -54,11 +53,10 @@ namespace GestionIntegral.Api.Controllers.Suscripciones
|
||||
return NoContent();
|
||||
}
|
||||
|
||||
// POST: api/facturacion/{idFactura}/enviar-factura-pdf
|
||||
[HttpPost("{idFactura:int}/enviar-factura-pdf")]
|
||||
public async Task<IActionResult> EnviarFacturaPdf(int idFactura)
|
||||
{
|
||||
if (!TienePermiso("SU009")) return Forbid();
|
||||
if (!TienePermiso(PermisoEnviarEmail)) return Forbid();
|
||||
var userId = GetCurrentUserId();
|
||||
if (userId == null) return Unauthorized();
|
||||
var (exito, error, emailDestino) = await _facturacionService.EnviarFacturaPdfPorEmail(idFactura, userId.Value);
|
||||
@@ -72,31 +70,6 @@ namespace GestionIntegral.Api.Controllers.Suscripciones
|
||||
return Ok(new { message = mensajeExito });
|
||||
}
|
||||
|
||||
// POST: api/facturacion/{anio}/{mes}/suscriptor/{idSuscriptor}/enviar-aviso
|
||||
[HttpPost("{anio:int}/{mes:int}/suscriptor/{idSuscriptor:int}/enviar-aviso")]
|
||||
[ProducesResponseType(StatusCodes.Status200OK)]
|
||||
[ProducesResponseType(StatusCodes.Status400BadRequest)]
|
||||
[ProducesResponseType(StatusCodes.Status404NotFound)]
|
||||
public async Task<IActionResult> EnviarAvisoPorEmail(int anio, int mes, int idSuscriptor)
|
||||
{
|
||||
// Usamos el permiso de enviar email
|
||||
if (!TienePermiso("SU009")) return Forbid();
|
||||
|
||||
var (exito, error) = await _facturacionService.EnviarAvisoCuentaPorEmail(anio, mes, idSuscriptor);
|
||||
|
||||
if (!exito)
|
||||
{
|
||||
if (error != null && (error.Contains("no encontrada") || error.Contains("no es válido")))
|
||||
{
|
||||
return NotFound(new { message = error });
|
||||
}
|
||||
return BadRequest(new { message = error });
|
||||
}
|
||||
|
||||
return Ok(new { message = "Email consolidado para el suscriptor ha sido enviado a la cola de procesamiento." });
|
||||
}
|
||||
|
||||
// GET: api/facturacion/{anio}/{mes}
|
||||
[HttpGet("{anio:int}/{mes:int}")]
|
||||
public async Task<IActionResult> GetFacturas(
|
||||
int anio, int mes,
|
||||
@@ -111,7 +84,6 @@ namespace GestionIntegral.Api.Controllers.Suscripciones
|
||||
return Ok(resumenes);
|
||||
}
|
||||
|
||||
|
||||
[HttpPost("{anio:int}/{mes:int}")]
|
||||
public async Task<IActionResult> GenerarFacturacion(int anio, int mes)
|
||||
{
|
||||
@@ -119,16 +91,34 @@ namespace GestionIntegral.Api.Controllers.Suscripciones
|
||||
var userId = GetCurrentUserId();
|
||||
if (userId == null) return Unauthorized();
|
||||
if (anio < 2020 || mes < 1 || mes > 12) return BadRequest(new { message = "El año y el mes proporcionados no son válidos." });
|
||||
var (exito, mensaje, facturasGeneradas) = await _facturacionService.GenerarFacturacionMensual(anio, mes, userId.Value);
|
||||
|
||||
var (exito, mensaje, resultadoEnvio) = await _facturacionService.GenerarFacturacionMensual(anio, mes, userId.Value);
|
||||
|
||||
if (!exito) return StatusCode(StatusCodes.Status500InternalServerError, new { message = mensaje });
|
||||
return Ok(new { message = mensaje, facturasGeneradas });
|
||||
|
||||
return Ok(new { message = mensaje, resultadoEnvio });
|
||||
}
|
||||
|
||||
[HttpGet("historial-lotes-envio")]
|
||||
[ProducesResponseType(typeof(IEnumerable<LoteDeEnvioHistorialDto>), StatusCodes.Status200OK)]
|
||||
public async Task<IActionResult> GetHistorialLotesEnvio([FromQuery] int? anio, [FromQuery] int? mes)
|
||||
{
|
||||
if (!TienePermiso("SU006")) return Forbid();
|
||||
var historial = await _facturacionService.ObtenerHistorialLotesEnvio(anio, mes);
|
||||
return Ok(historial);
|
||||
}
|
||||
|
||||
// Endpoint para el historial de envíos de una factura individual
|
||||
[HttpGet("{idFactura:int}/historial-envios")]
|
||||
[ProducesResponseType(typeof(IEnumerable<EmailLogDto>), StatusCodes.Status200OK)]
|
||||
public async Task<IActionResult> GetHistorialEnvios(int idFactura)
|
||||
{
|
||||
var historial = await _emailLogService.ObtenerHistorialPorReferencia("Factura-" + idFactura);
|
||||
if (!TienePermiso(PermisoGestionarFacturacion)) return Forbid(); // Reutilizamos el permiso
|
||||
|
||||
// Construimos la referencia que se guarda en el log
|
||||
string referencia = $"Factura-{idFactura}";
|
||||
var historial = await _emailLogService.ObtenerHistorialPorReferencia(referencia);
|
||||
|
||||
return Ok(historial);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user