dmolinari/SIG-CM2.0
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

feat(udt-011): T400.20 — domain mutators accept explicit DateTime now param

Browse Source 
Remove DateTime.UtcNow calls from all With*/Deactivate/Reactivate/
CerrarVigencia/NuevaVersion domain methods. Caller (Application layer)
is now responsible for passing the UTC timestamp obtained via
_timeProvider.GetUtcNow().UtcDateTime.
This commit is contained in:
dmolinari
2026-04-18 10:12:03 -03:00
parent 3c264aa7a1
commit 4e1d8f69ab
6 changed files with 61 additions and 50 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
src/api/SIGCM2.Domain/Entities/IngresosBrutos.cs
View File

@@ -95,9 +95,13 @@ public sealed class IngresosBrutos
/// </summary>
/// <exception cref="InvalidOperationException">Si la predecesora ya está cerrada (VigenciaHasta != null).</exception>
/// <exception cref="ArgumentException">Si vigenciaDesde no es posterior a la predecesora, o nuevaAlicuota fuera de rango.</exception>
/// <summary>
/// <param name="now">Timestamp UTC provisto por el caller (Application layer via TimeProvider).</param>
/// </summary>
public (IngresosBrutos predecesoraCerrada, IngresosBrutos nuevaVersion) NuevaVersion(
decimal nuevaAlicuota,
DateOnly vigenciaDesde)
DateOnly vigenciaDesde,
DateTime now)
{
if (VigenciaHasta is not null)
throw new InvalidOperationException(
@@ -120,7 +124,7 @@ public sealed class IngresosBrutos
vigenciaHasta: vigenciaDesde.AddDays(-1),
predecesorId: PredecesorId,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
var nueva = ForCreation(
provincia: Provincia,
@@ -136,26 +140,26 @@ public sealed class IngresosBrutos
// ── Cosmetic mutators (NO WithAlicuota, NO WithProvincia) ─────────────────
/// <summary>Actualiza la descripción. Alicuota y Provincia permanecen inmutables.</summary>
public IngresosBrutos WithDescripcion(string descripcion)
public IngresosBrutos WithDescripcion(string descripcion, DateTime now)
=> new(Id, Provincia, descripcion, Alicuota, Activo,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Retorna instancia con Activo=false.</summary>
public IngresosBrutos Deactivate()
public IngresosBrutos Deactivate(DateTime now)
=> new(Id, Provincia, Descripcion, Alicuota, false,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Retorna instancia con Activo=true.</summary>
public IngresosBrutos Reactivate()
public IngresosBrutos Reactivate(DateTime now)
=> new(Id, Provincia, Descripcion, Alicuota, true,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>
/// Cierra la vigencia seteando VigenciaHasta. Usado por el handler de NuevaVersion.
/// </summary>
public IngresosBrutos CerrarVigencia(DateOnly vigenciaHasta)
public IngresosBrutos CerrarVigencia(DateOnly vigenciaHasta, DateTime now)
=> new(Id, Provincia, Descripcion, Alicuota, Activo,
VigenciaDesde, vigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, vigenciaHasta, PredecesorId, FechaCreacion, now);
// ── Private helpers ───────────────────────────────────────────────────────

10
src/api/SIGCM2.Domain/Entities/Medio.cs
View File

@@ -49,9 +49,9 @@ public sealed class Medio
/// <summary>
/// Returns a new instance with updated fields. Codigo is immutable (use BD UQ to enforce).
/// Sets FechaModificacion = UtcNow.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// </summary>
public Medio WithUpdatedProfile(string nombre, TipoMedio tipo, int? plataformaEmpresaId)
public Medio WithUpdatedProfile(string nombre, TipoMedio tipo, int? plataformaEmpresaId, DateTime now)
=> new(
id: Id,
codigo: Codigo,
@@ -60,9 +60,9 @@ public sealed class Medio
plataformaEmpresaId: plataformaEmpresaId,
activo: Activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
public Medio WithActivo(bool activo)
public Medio WithActivo(bool activo, DateTime now)
=> new(
id: Id,
codigo: Codigo,
@@ -71,5 +71,5 @@ public sealed class Medio
plataformaEmpresaId: PlataformaEmpresaId,
activo: activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
}

9
src/api/SIGCM2.Domain/Entities/PuntoDeVenta.cs
View File

@@ -52,8 +52,9 @@ public sealed class PuntoDeVenta
/// <summary>
/// Retorna una nueva instancia con nombre, numeroAFIP y descripcion actualizados.
/// MedioId es inmutable (enforce en BD).
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// </summary>
public PuntoDeVenta WithUpdatedProfile(string nombre, short numeroAFIP, string? descripcion)
public PuntoDeVenta WithUpdatedProfile(string nombre, short numeroAFIP, string? descripcion, DateTime now)
=> new(
id: Id,
medioId: MedioId,
@@ -62,9 +63,9 @@ public sealed class PuntoDeVenta
descripcion: descripcion,
activo: Activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
public PuntoDeVenta WithActivo(bool activo)
public PuntoDeVenta WithActivo(bool activo, DateTime now)
=> new(
id: Id,
medioId: MedioId,
@@ -73,5 +74,5 @@ public sealed class PuntoDeVenta
descripcion: Descripcion,
activo: activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
}

10
src/api/SIGCM2.Domain/Entities/Seccion.cs
View File

@@ -46,9 +46,9 @@ public sealed class Seccion
/// <summary>
/// Returns a new instance with updated fields. MedioId and Codigo are immutable.
/// Sets FechaModificacion = UtcNow.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// </summary>
public Seccion WithUpdatedProfile(string nombre, string tipo)
public Seccion WithUpdatedProfile(string nombre, string tipo, DateTime now)
=> new(
id: Id,
medioId: MedioId,
@@ -57,9 +57,9 @@ public sealed class Seccion
tipo: tipo,
activo: Activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
public Seccion WithActivo(bool activo)
public Seccion WithActivo(bool activo, DateTime now)
=> new(
id: Id,
medioId: MedioId,
@@ -68,5 +68,5 @@ public sealed class Seccion
tipo: Tipo,
activo: activo,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
}

33
src/api/SIGCM2.Domain/Entities/TipoDeIva.cs
View File

@@ -106,9 +106,14 @@ public sealed class TipoDeIva
/// </summary>
/// <exception cref="InvalidOperationException">Si la predecesora ya está cerrada (VigenciaHasta != null).</exception>
/// <exception cref="ArgumentException">Si vigenciaDesde no es posterior a la predecesora, o nuevoPorcentaje fuera de rango.</exception>
/// <summary>
/// Crea una nueva versión con el porcentaje actualizado.
/// <param name="now">Timestamp UTC provisto por el caller (Application layer via TimeProvider).</param>
/// </summary>
public (TipoDeIva predecesoraCerrada, TipoDeIva nuevaVersion) NuevaVersion(
decimal nuevoPorcentaje,
DateOnly vigenciaDesde)
DateOnly vigenciaDesde,
DateTime now)
{
if (VigenciaHasta is not null)
throw new InvalidOperationException(
@@ -132,7 +137,7 @@ public sealed class TipoDeIva
vigenciaHasta: vigenciaDesde.AddDays(-1),
predecesorId: PredecesorId,
fechaCreacion: FechaCreacion,
fechaModificacion: DateTime.UtcNow);
fechaModificacion: now);
var nueva = ForCreation(
codigo: Codigo,
@@ -149,36 +154,36 @@ public sealed class TipoDeIva
// ── Cosmetic mutators (sealed With* — NOT WithPorcentaje) ─────────────────
/// <summary>Actualiza la descripción. Porcentaje y vigencias permanecen inmutables.</summary>
public TipoDeIva WithDescripcion(string descripcion)
public TipoDeIva WithDescripcion(string descripcion, DateTime now)
=> new(Id, Codigo, descripcion, Porcentaje, AplicaIVA, Activo,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Actualiza el código. Porcentaje y vigencias permanecen inmutables.</summary>
public TipoDeIva WithCodigo(string codigo)
public TipoDeIva WithCodigo(string codigo, DateTime now)
=> new(Id, codigo, Descripcion, Porcentaje, AplicaIVA, Activo,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Actualiza la bandera AplicaIVA. Porcentaje permanece inmutable.</summary>
public TipoDeIva WithAplicaIVA(bool aplicaIVA)
public TipoDeIva WithAplicaIVA(bool aplicaIVA, DateTime now)
=> new(Id, Codigo, Descripcion, Porcentaje, aplicaIVA, Activo,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Retorna instancia con Activo=false.</summary>
public TipoDeIva Deactivate()
public TipoDeIva Deactivate(DateTime now)
=> new(Id, Codigo, Descripcion, Porcentaje, AplicaIVA, false,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>Retorna instancia con Activo=true.</summary>
public TipoDeIva Reactivate()
public TipoDeIva Reactivate(DateTime now)
=> new(Id, Codigo, Descripcion, Porcentaje, AplicaIVA, true,
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, VigenciaHasta, PredecesorId, FechaCreacion, now);
/// <summary>
/// Cierra la vigencia seteando VigenciaHasta. Usado por el handler de NuevaVersion.
/// </summary>
public TipoDeIva CerrarVigencia(DateOnly vigenciaHasta)
public TipoDeIva CerrarVigencia(DateOnly vigenciaHasta, DateTime now)
=> new(Id, Codigo, Descripcion, Porcentaje, AplicaIVA, Activo,
VigenciaDesde, vigenciaHasta, PredecesorId, FechaCreacion, DateTime.UtcNow);
VigenciaDesde, vigenciaHasta, PredecesorId, FechaCreacion, now);
// ── Private helpers ───────────────────────────────────────────────────────

25
src/api/SIGCM2.Domain/Entities/Usuario.cs
View File

@@ -76,9 +76,10 @@ public sealed class Usuario
/// <summary>
/// Returns a new instance with updated profile fields.
/// Sets FechaModificacion = UtcNow. Username and PasswordHash are immutable.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// Username and PasswordHash are immutable.
/// </summary>
public Usuario WithUpdatedProfile(string nombre, string apellido, string? email, string rol, bool activo)
public Usuario WithUpdatedProfile(string nombre, string apellido, string? email, string rol, bool activo, DateTime now)
=> new(
id: Id,
username: Username,
@@ -89,15 +90,15 @@ public sealed class Usuario
rol: rol,
permisosJson: PermisosJson,
activo: activo,
fechaModificacion: DateTime.UtcNow,
fechaModificacion: now,
ultimoLogin: UltimoLogin,
mustChangePassword: MustChangePassword);
/// <summary>
/// Returns a new instance with a new password hash and mustChangePassword flag.
/// Sets FechaModificacion = UtcNow.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// </summary>
public Usuario WithNewPasswordHash(string hash, bool mustChangePassword)
public Usuario WithNewPasswordHash(string hash, bool mustChangePassword, DateTime now)
=> new(
id: Id,
username: Username,
@@ -108,15 +109,15 @@ public sealed class Usuario
rol: Rol,
permisosJson: PermisosJson,
activo: Activo,
fechaModificacion: DateTime.UtcNow,
fechaModificacion: now,
ultimoLogin: UltimoLogin,
mustChangePassword: mustChangePassword);
/// <summary>
/// Returns a new instance with only the MustChangePassword flag changed.
/// Sets FechaModificacion = UtcNow.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// </summary>
public Usuario WithMustChangePassword(bool value)
public Usuario WithMustChangePassword(bool value, DateTime now)
=> new(
id: Id,
username: Username,
@@ -127,16 +128,16 @@ public sealed class Usuario
rol: Rol,
permisosJson: PermisosJson,
activo: Activo,
fechaModificacion: DateTime.UtcNow,
fechaModificacion: now,
ultimoLogin: UltimoLogin,
mustChangePassword: value);
/// <summary>
/// UDT-009: Returns a new instance with PermisosJson replaced.
/// Sets FechaModificacion = UtcNow.
/// Caller is responsible for passing the current UTC timestamp via TimeProvider.GetUtcNow().UtcDateTime.
/// Accepts raw JSON string so Domain stays free of Application dependencies.
/// </summary>
public Usuario WithPermisosJson(string permisosJson)
public Usuario WithPermisosJson(string permisosJson, DateTime now)
=> new(
id: Id,
username: Username,
@@ -147,7 +148,7 @@ public sealed class Usuario
rol: Rol,
permisosJson: permisosJson,
activo: Activo,
fechaModificacion: DateTime.UtcNow,
fechaModificacion: now,
ultimoLogin: UltimoLogin,
mustChangePassword: MustChangePassword);