2025-07-30 09:48:05 -03:00
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 )
{
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.
2025-08-09 17:39:21 -03:00
// 1. Obtener todos los suscriptores en una sola consulta
2025-07-30 09:48:05 -03:00
var suscriptores = await _suscriptorRepository . GetAllAsync ( nombreFilter , nroDocFilter , soloActivos ) ;
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.
2025-08-09 17:39:21 -03:00
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 ;
2025-07-30 09:48:05 -03:00
}
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 ) ;
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.
2025-08-09 17:39:21 -03:00
2025-07-30 09:48:05 -03:00
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." ) ;
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.
2025-08-09 17:39:21 -03:00
2025-07-30 09:48:05 -03:00
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." ) ;
}
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.
2025-08-09 17:39:21 -03:00
2025-07-30 09:48:05 -03:00
// 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 ( ) ;
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.
2025-08-09 17:39:21 -03:00
2025-07-30 09:48:05 -03:00
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." ) ;
}
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.
2025-08-09 17:39:21 -03:00
2025-07-30 09:48:05 -03:00
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 ) ;
}
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.
2025-08-09 17:39:21 -03:00
catch ( Exception ex )
2025-07-30 09:48:05 -03:00
{
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 ) ;
}
}
}