Feat: Implementar modificación de Tiradas y mejorar UX/UI
All checks were successful
Optimized Build and Deploy / remote-build-and-deploy (push) Successful in 12m54s

**Backend:**
- Se ha añadido el endpoint `PUT /api/tiradas` para manejar la modificación de una Tirada, identificada por su clave única (fecha, idPublicacion, idPlanta).
- Se implementó un mecanismo de actualización granular para las secciones de la tirada (`bob_RegPublicaciones`), reemplazando la estrategia anterior de "eliminar todo y recrear".
  - La nueva lógica reconcilia el estado entrante con el de la base de datos, realizando operaciones individuales de `INSERT`, `UPDATE` y `DELETE` para cada sección.
  - Esto mejora significativamente el rendimiento y proporciona un historial de auditoría mucho más preciso.
- Se añadieron los DTOs `UpdateTiradaRequestDto` y `UpdateDetalleSeccionTiradaDto` para soportar el nuevo payload de modificación.
- Se expandieron los repositorios `IRegPublicacionSeccionRepository` y `IPubliSeccionRepository` con métodos para operaciones granulares (`UpdateAsync`, `DeleteByIdAsync`, `GetByIdsAndPublicacionAsync`).

**Frontend:**
- El componente `TiradaFormModal` ha sido refactorizado para funcionar tanto en modo "Crear" como en modo "Editar", recibiendo una prop `tiradaToEdit`.
- Se implementó una lógica de carga asíncrona robusta que obtiene los datos completos de una tirada antes de abrir el modal en modo edición.

**Mejoras de UI/UX:**
- Se ha rediseñado el layout de la lista de tiradas en `GestionarTiradasPage`:
  - Los botones de acción (Editar, Borrar) y los datos clave (chips de ejemplares y páginas) ahora se encuentran en una cabecera estática.
  - Estos elementos permanecen fijos en la parte superior y no se desplazan al expandir el acordeón, mejorando la consistencia visual.
- Se ha mejorado la tabla de secciones dentro del `TiradaFormModal`:
  - El botón "+ AGREGAR SECCIÓN" ahora está fijo en la parte inferior de la tabla, permaneciendo siempre visible incluso cuando la lista de secciones tiene scroll.
  - Al agregar una nueva sección, la lista se desplaza automáticamente hacia abajo para mostrar la nueva fila.
This commit is contained in:
2025-07-23 14:05:58 -03:00
parent 052141a45b
commit 28c1b88a92
14 changed files with 884 additions and 320 deletions

View File

@@ -22,7 +22,8 @@ namespace GestionIntegral.Api.Controllers.Impresion
// Permisos para Tiradas (IT001 a IT003)
private const string PermisoVerTiradas = "IT001";
private const string PermisoRegistrarTirada = "IT002";
private const string PermisoEliminarTirada = "IT003"; // Asumo que se refiere a eliminar una tirada completa (cabecera y detalles)
private const string PermisoEliminarTirada = "IT003";
private const string PermisoModificarTirada = "IT004";
public TiradasController(ITiradaService tiradaService, ILogger<TiradasController> logger)
{
@@ -83,6 +84,43 @@ namespace GestionIntegral.Api.Controllers.Impresion
return StatusCode(StatusCodes.Status201Created, tiradaCreada);
}
[HttpPut]
[ProducesResponseType(typeof(TiradaDto), StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status400BadRequest)]
[ProducesResponseType(StatusCodes.Status403Forbidden)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
public async Task<IActionResult> ModificarTirada(
[FromQuery, BindRequired] DateTime fecha,
[FromQuery, BindRequired] int idPublicacion,
[FromQuery, BindRequired] int idPlanta,
[FromBody] UpdateTiradaRequestDto updateDto)
{
if (!TienePermiso(PermisoModificarTirada)) return Forbid();
if (!ModelState.IsValid) return BadRequest(ModelState);
var userId = GetCurrentUserId();
if (userId == null) return Unauthorized();
var (tiradaActualizada, error) = await _tiradaService.ModificarTiradaCompletaAsync(fecha, idPublicacion, idPlanta, updateDto, userId.Value);
if (error != null)
{
// Chequear si el error es porque no se encontró la tirada.
if (error.StartsWith("No se encontró la tirada"))
{
return NotFound(new { message = error });
}
return BadRequest(new { message = error });
}
if (tiradaActualizada == null)
{
return StatusCode(StatusCodes.Status500InternalServerError, "Error interno al modificar la tirada.");
}
return Ok(tiradaActualizada);
}
// DELETE: api/tiradas
// Se identifica la tirada a eliminar por su combinación única de Fecha, IdPublicacion, IdPlanta
[HttpDelete]

View File

@@ -9,10 +9,11 @@ namespace GestionIntegral.Api.Data.Repositories.Distribucion
{
Task<IEnumerable<PubliSeccion>> GetByPublicacionIdAsync(int idPublicacion, bool? soloActivas = null);
Task<PubliSeccion?> GetByIdAsync(int idSeccion);
Task<IEnumerable<PubliSeccion>> GetByIdsAndPublicacionAsync(IEnumerable<int> idsSeccion, int idPublicacion, bool? soloActivas = null);
Task<PubliSeccion?> CreateAsync(PubliSeccion nuevaSeccion, int idUsuario, IDbTransaction transaction);
Task<bool> UpdateAsync(PubliSeccion seccionAActualizar, int idUsuario, IDbTransaction transaction);
Task<bool> DeleteAsync(int idSeccion, int idUsuario, IDbTransaction transaction);
Task<bool> DeleteByPublicacionIdAsync(int idPublicacion, int idUsuarioAuditoria, IDbTransaction transaction); // Ya existe
Task<bool> DeleteByPublicacionIdAsync(int idPublicacion, int idUsuarioAuditoria, IDbTransaction transaction);
Task<bool> ExistsByNameInPublicacionAsync(string nombre, int idPublicacion, int? excludeIdSeccion = null);
Task<bool> IsInUseAsync(int idSeccion); // Verificar en bob_RegPublicaciones, bob_StockBobinas
Task<IEnumerable<(PubliSeccionHistorico Historial, string NombreUsuarioModifico)>> GetHistorialAsync(

View File

@@ -169,6 +169,32 @@ namespace GestionIntegral.Api.Data.Repositories.Distribucion
return rowsAffected == 1;
}
public async Task<IEnumerable<PubliSeccion>> GetByIdsAndPublicacionAsync(IEnumerable<int> idsSeccion, int idPublicacion, bool? soloActivas = null)
{
if (idsSeccion == null || !idsSeccion.Any())
{
return Enumerable.Empty<PubliSeccion>();
}
var sqlBuilder = new StringBuilder(@"
SELECT Id_Seccion AS IdSeccion, Id_Publicacion AS IdPublicacion, Nombre, Estado
FROM dbo.dist_dtPubliSecciones
WHERE Id_Publicacion = @IdPublicacionParam AND Id_Seccion IN @IdsSeccionParam");
var parameters = new DynamicParameters();
parameters.Add("IdPublicacionParam", idPublicacion);
parameters.Add("IdsSeccionParam", idsSeccion);
if (soloActivas.HasValue)
{
sqlBuilder.Append(" AND Estado = @EstadoParam");
parameters.Add("EstadoParam", soloActivas.Value);
}
using var connection = _cf.CreateConnection();
return await connection.QueryAsync<PubliSeccion>(sqlBuilder.ToString(), parameters);
}
public async Task<bool> DeleteAsync(int idSeccion, int idUsuario, IDbTransaction transaction)
{
var actual = await transaction.Connection!.QuerySingleOrDefaultAsync<PubliSeccion>(

View File

@@ -1,3 +1,5 @@
// --- FICHERO MODIFICADO: IRegTiradaRepository.cs ---
using GestionIntegral.Api.Models.Impresion;
using System;
using System.Collections.Generic;
@@ -6,11 +8,12 @@ using System.Threading.Tasks;
namespace GestionIntegral.Api.Data.Repositories.Impresion
{
public interface IRegTiradaRepository // Para bob_RegTiradas
public interface IRegTiradaRepository
{
Task<RegTirada?> GetByIdAsync(int idRegistro);
Task<IEnumerable<RegTirada>> GetByCriteriaAsync(DateTime? fecha, int? idPublicacion, int? idPlanta);
Task<RegTirada?> CreateAsync(RegTirada nuevaTirada, int idUsuario, IDbTransaction transaction);
Task<bool> UpdateAsync(RegTirada tiradaAActualizar, int idUsuario, IDbTransaction transaction);
Task<bool> DeleteAsync(int idRegistro, int idUsuario, IDbTransaction transaction); // Si se borra el registro principal
Task<bool> DeleteByFechaPublicacionPlantaAsync(DateTime fecha, int idPublicacion, int idPlanta, int idUsuario, IDbTransaction transaction);
Task<RegTirada?> GetByFechaPublicacionPlantaAsync(DateTime fecha, int idPublicacion, int idPlanta, IDbTransaction? transaction = null);
@@ -27,9 +30,11 @@ namespace GestionIntegral.Api.Data.Repositories.Impresion
public interface IRegPublicacionSeccionRepository // Para bob_RegPublicaciones
{
Task<RegPublicacionSeccion?> GetByIdAsync(int idTirada);
Task<IEnumerable<RegPublicacionSeccion>> GetByFechaPublicacionPlantaAsync(DateTime fecha, int idPublicacion, int idPlanta);
Task<RegPublicacionSeccion?> CreateAsync(RegPublicacionSeccion nuevaSeccionTirada, int idUsuario, IDbTransaction transaction);
Task<bool> UpdateAsync(RegPublicacionSeccion seccionAActualizar, int idUsuario, IDbTransaction transaction);
Task<bool> DeleteByIdAsync(int idTirada, int idUsuario, IDbTransaction transaction);
Task<bool> DeleteByFechaPublicacionPlantaAsync(DateTime fecha, int idPublicacion, int idPlanta, int idUsuario, IDbTransaction transaction);
// Podría tener un DeleteByIdAsync si se permite borrar secciones individuales de una tirada
}
}

View File

@@ -83,6 +83,42 @@ namespace GestionIntegral.Api.Data.Repositories.Impresion
return inserted;
}
public async Task<bool> UpdateAsync(RegTirada tiradaAActualizar, int idUsuario, IDbTransaction transaction)
{
// 1. Obtener el estado actual para guardarlo en el historial
const string sqlSelectActual = "SELECT * FROM dbo.bob_RegTiradas WHERE Id_Registro = @IdRegistro";
var estadoActual = await transaction.Connection!.QuerySingleOrDefaultAsync<RegTirada>(sqlSelectActual, new { IdRegistro = tiradaAActualizar.IdRegistro }, transaction);
if (estadoActual == null)
{
throw new KeyNotFoundException("No se encontró el registro de tirada a actualizar para generar el historial.");
}
// 2. Guardar el estado PREVIO en el historial
const string sqlHistorico = @"INSERT INTO dbo.bob_RegTiradas_H (Id_Registro, Ejemplares, Id_Publicacion, Fecha, Id_Planta, Id_Usuario, FechaMod, TipoMod)
VALUES (@IdRegistroParam, @EjemplaresParam, @IdPublicacionParam, @FechaParam, @IdPlantaParam, @IdUsuarioParam, @FechaModParam, @TipoModParam);";
await transaction.Connection!.ExecuteAsync(sqlHistorico, new
{
IdRegistroParam = estadoActual.IdRegistro,
EjemplaresParam = estadoActual.Ejemplares,
IdPublicacionParam = estadoActual.IdPublicacion,
FechaParam = estadoActual.Fecha,
IdPlantaParam = estadoActual.IdPlanta,
IdUsuarioParam = idUsuario,
FechaModParam = DateTime.Now,
TipoModParam = "Modificado"
}, transaction);
// 3. Actualizar el registro principal
const string sqlUpdate = @"
UPDATE dbo.bob_RegTiradas
SET Ejemplares = @Ejemplares
WHERE Id_Registro = @IdRegistro;";
var rowsAffected = await transaction.Connection!.ExecuteAsync(sqlUpdate, tiradaAActualizar, transaction);
return rowsAffected == 1;
}
public async Task<bool> DeleteAsync(int idRegistro, int idUsuario, IDbTransaction transaction)
{
var actual = await GetByIdAsync(idRegistro); // No necesita TX aquí ya que es solo para historial
@@ -276,6 +312,66 @@ namespace GestionIntegral.Api.Data.Repositories.Impresion
return inserted;
}
public async Task<RegPublicacionSeccion?> GetByIdAsync(int idTirada)
{
const string sql = @"SELECT Id_Tirada AS IdTirada, Id_Publicacion AS IdPublicacion, Id_Seccion AS IdSeccion, CantPag, Fecha, Id_Planta AS IdPlanta
FROM dbo.bob_RegPublicaciones WHERE Id_Tirada = @IdTiradaParam";
using var connection = _cf.CreateConnection();
return await connection.QuerySingleOrDefaultAsync<RegPublicacionSeccion>(sql, new { IdTiradaParam = idTirada });
}
public async Task<bool> UpdateAsync(RegPublicacionSeccion seccionAActualizar, int idUsuario, IDbTransaction transaction)
{
// Obtener estado PREVIO para historial
var actual = await GetByIdAsync(seccionAActualizar.IdTirada);
if (actual == null) throw new KeyNotFoundException("No se encontró la sección de tirada a actualizar.");
// Insertar en historial con tipo "Modificado"
const string sqlHistorico = @"INSERT INTO dbo.bob_RegPublicaciones_H (Id_Tirada, Id_Publicacion, Id_Seccion, CantPag, Fecha, Id_Planta, Id_Usuario, FechaMod, TipoMod)
VALUES (@IdTirada, @IdPublicacion, @IdSeccion, @CantPag, @Fecha, @IdPlanta, @IdUsuario, GETDATE(), 'Modificado');";
await transaction.Connection!.ExecuteAsync(sqlHistorico, new
{
actual.IdTirada,
actual.IdPublicacion,
actual.IdSeccion,
actual.CantPag,
actual.Fecha,
actual.IdPlanta,
IdUsuario = idUsuario
}, transaction);
// Actualizar el registro
const string sqlUpdate = "UPDATE dbo.bob_RegPublicaciones SET CantPag = @CantPag WHERE Id_Tirada = @IdTirada";
var rowsAffected = await transaction.Connection!.ExecuteAsync(sqlUpdate, seccionAActualizar, transaction);
return rowsAffected == 1;
}
public async Task<bool> DeleteByIdAsync(int idTirada, int idUsuario, IDbTransaction transaction)
{
// Obtener estado PREVIO para historial
var actual = await GetByIdAsync(idTirada);
if (actual == null) return false; // Ya no existe, no hacemos nada.
// Insertar en historial con tipo "Eliminado"
const string sqlHistorico = @"INSERT INTO dbo.bob_RegPublicaciones_H (Id_Tirada, Id_Publicacion, Id_Seccion, CantPag, Fecha, Id_Planta, Id_Usuario, FechaMod, TipoMod)
VALUES (@IdTirada, @IdPublicacion, @IdSeccion, @CantPag, @Fecha, @IdPlanta, @IdUsuario, GETDATE(), 'Eliminado');";
await transaction.Connection!.ExecuteAsync(sqlHistorico, new
{
actual.IdTirada,
actual.IdPublicacion,
actual.IdSeccion,
actual.CantPag,
actual.Fecha,
actual.IdPlanta,
IdUsuario = idUsuario
}, transaction);
// Eliminar el registro
const string sqlDelete = "DELETE FROM dbo.bob_RegPublicaciones WHERE Id_Tirada = @IdTirada";
var rowsAffected = await transaction.Connection!.ExecuteAsync(sqlDelete, new { IdTirada = idTirada }, transaction);
return rowsAffected == 1;
}
public async Task<bool> DeleteByFechaPublicacionPlantaAsync(DateTime fecha, int idPublicacion, int idPlanta, int idUsuario, IDbTransaction transaction)
{
var seccionesAEliminar = await GetByFechaPublicacionPlantaAsync(fecha.Date, idPublicacion, idPlanta); // No necesita TX, es SELECT

View File

@@ -0,0 +1,17 @@
using System.ComponentModel.DataAnnotations;
namespace GestionIntegral.Api.Dtos.Impresion
{
public class UpdateDetalleSeccionTiradaDto
{
// ID del registro en bob_RegPublicaciones.
// Si es 0 o null, es una nueva sección a añadir.
public int IdRegPublicacionSeccion { get; set; }
[Required]
public int IdSeccion { get; set; }
[Required, Range(1, 1000)]
public int CantPag { get; set; }
}
}

View File

@@ -0,0 +1,15 @@
using System.ComponentModel.DataAnnotations;
namespace GestionIntegral.Api.Dtos.Impresion
{
public class UpdateTiradaRequestDto
{
[Required, Range(1, int.MaxValue)]
public int Ejemplares { get; set; }
[Required]
// No necesitamos MinLength(1), ya que una tirada podría quedar sin secciones si el usuario las borra todas.
// Por ahora lo quitamos para más flexibilidad.
public List<UpdateDetalleSeccionTiradaDto> Secciones { get; set; } = new List<UpdateDetalleSeccionTiradaDto>();
}
}

View File

@@ -10,6 +10,7 @@ namespace GestionIntegral.Api.Services.Impresion
{
Task<IEnumerable<TiradaDto>> ObtenerTiradasAsync(DateTime? fecha, int? idPublicacion, int? idPlanta);
Task<(TiradaDto? TiradaCreada, string? Error)> RegistrarTiradaCompletaAsync(CreateTiradaRequestDto createDto, int idUsuario);
Task<(TiradaDto? TiradaActualizada, string? Error)> ModificarTiradaCompletaAsync(DateTime fecha, int idPublicacion, int idPlanta, UpdateTiradaRequestDto updateDto, int idUsuario); // <-- AÑADIDO
Task<(bool Exito, string? Error)> EliminarTiradaCompletaAsync(DateTime fecha, int idPublicacion, int idPlanta, int idUsuario);
Task<IEnumerable<RegTiradaHistorialDto>> ObtenerRegTiradasHistorialAsync(
DateTime? fechaDesde, DateTime? fechaHasta,

View File

@@ -182,6 +182,104 @@ namespace GestionIntegral.Api.Services.Impresion
}
}
public async Task<(TiradaDto? TiradaActualizada, string? Error)> ModificarTiradaCompletaAsync(DateTime fecha, int idPublicacion, int idPlanta, UpdateTiradaRequestDto updateDto, int idUsuario)
{
// 1. Validar que la tirada a modificar exista.
var tiradaExistente = await _regTiradaRepository.GetByFechaPublicacionPlantaAsync(fecha.Date, idPublicacion, idPlanta);
if (tiradaExistente == null)
{
return (null, "No se encontró la tirada que intenta modificar.");
}
// 2. Validaciones de DTO (secciones válidas, etc.)
var idsSeccionesUnicasDto = updateDto.Secciones.Select(s => s.IdSeccion).Distinct();
if (idsSeccionesUnicasDto.Any()) // Solo validar si se enviaron secciones
{
var seccionesValidasDb = await _publiSeccionRepository.GetByIdsAndPublicacionAsync(idsSeccionesUnicasDto, idPublicacion, soloActivas: true);
if (seccionesValidasDb.Count() != idsSeccionesUnicasDto.Count())
{
var idsEncontrados = seccionesValidasDb.Select(s => s.IdSeccion).ToHashSet();
var idsFaltantes = string.Join(", ", idsSeccionesUnicasDto.Where(id => !idsEncontrados.Contains(id)));
return (null, $"Las siguientes secciones no son válidas, no pertenecen a la publicación o no están activas: {idsFaltantes}.");
}
}
using var connection = _connectionFactory.CreateConnection();
if (connection is System.Data.Common.DbConnection dbConn) await dbConn.OpenAsync(); else connection.Open();
using var transaction = connection.BeginTransaction();
try
{
// 3. Actualizar registro principal (bob_RegTiradas) si cambió el número de ejemplares
if (tiradaExistente.Ejemplares != updateDto.Ejemplares)
{
tiradaExistente.Ejemplares = updateDto.Ejemplares;
bool actualizado = await _regTiradaRepository.UpdateAsync(tiradaExistente, idUsuario, transaction);
if (!actualizado) throw new DataException("No se pudo actualizar el registro principal de la tirada.");
}
// 4. Lógica de reconciliación de secciones
// 4.1. Obtener estado actual de las secciones en la BBDD
var seccionesActualesDb = await _regPublicacionSeccionRepository.GetByFechaPublicacionPlantaAsync(fecha.Date, idPublicacion, idPlanta);
// 4.2. Procesar Adiciones y Actualizaciones
foreach (var seccionDto in updateDto.Secciones)
{
if (seccionDto.IdRegPublicacionSeccion == 0) // Es una nueva sección
{
var nuevaRegPubSeccion = new RegPublicacionSeccion
{
IdPublicacion = idPublicacion,
IdSeccion = seccionDto.IdSeccion,
CantPag = seccionDto.CantPag,
Fecha = fecha.Date,
IdPlanta = idPlanta
};
await _regPublicacionSeccionRepository.CreateAsync(nuevaRegPubSeccion, idUsuario, transaction);
}
else // Es una sección existente, verificar si hay cambios
{
var seccionDb = seccionesActualesDb.FirstOrDefault(s => s.IdTirada == seccionDto.IdRegPublicacionSeccion);
if (seccionDb == null) throw new InvalidOperationException($"La sección con ID de registro {seccionDto.IdRegPublicacionSeccion} no pertenece a esta tirada.");
if (seccionDb.CantPag != seccionDto.CantPag) // Solo actualizamos si cambió la cantidad de páginas
{
seccionDb.CantPag = seccionDto.CantPag;
await _regPublicacionSeccionRepository.UpdateAsync(seccionDb, idUsuario, transaction);
}
}
}
// 4.3. Procesar Eliminaciones
var idsSeccionesEntrantes = updateDto.Secciones
.Where(s => s.IdRegPublicacionSeccion != 0)
.Select(s => s.IdRegPublicacionSeccion)
.ToHashSet();
var seccionesAEliminar = seccionesActualesDb
.Where(s => !idsSeccionesEntrantes.Contains(s.IdTirada));
foreach (var seccionParaBorrar in seccionesAEliminar)
{
await _regPublicacionSeccionRepository.DeleteByIdAsync(seccionParaBorrar.IdTirada, idUsuario, transaction);
}
transaction.Commit();
_logger.LogInformation("Tirada completa modificada (granular) para Pub ID {IdPub}, Fecha {Fecha}, por Usuario ID {UserId}.", idPublicacion, fecha.Date, idUsuario);
// 5. Devolver el DTO actualizado (tendríamos que volver a consultar o construirlo)
var tiradaActualizadaResult = await ObtenerTiradasAsync(fecha.Date, idPublicacion, idPlanta);
return (tiradaActualizadaResult.FirstOrDefault(), null);
}
catch (Exception ex)
{
try { transaction.Rollback(); } catch { }
_logger.LogError(ex, "Error en ModificarTiradaCompletaAsync (granular) para Pub ID {IdPub}, Fecha {Fecha}", idPublicacion, fecha);
return (null, $"Error interno al modificar la tirada: {ex.Message}");
}
}
public async Task<(bool Exito, string? Error)> EliminarTiradaCompletaAsync(DateTime fecha, int idPublicacion, int idPlanta, int idUsuario)
{
// Verificar que la tirada principal exista