Почему EF Core пытается вставить значения NULL при использовании специального ValueGenerator? ⇐ C#

[Anonymous]

У меня есть собственный ValueGenerator следующим образом:

Код: Выделить всё

public class UtcDateTimeGenerator : ValueGenerator
{
public override DateTime Next(EntityEntry entry)
{
// This method never seems to be called
return DateTime.UtcNow;
}

protected override object NextValue(EntityEntry entry)
{
// This one is called.
return DateTime.UtcNow;
}

public override bool GeneratesTemporaryValues => false;
}

Моя сущность:

Код: Выделить всё

public abstract class AggregateRoot
where TId : IComparable
{
public TId Id { get; set; }
}

public abstract class AuditedAggregateRoot : AggregateRoot
where TId : IComparable
{
public DateTime Created { get; internal set; }
public DateTime LastModified { get; internal set; }
}

public class Player : AuditedAggregateRoot
{
public string ExternalId { get; internal set; }
public string DisplayName { get; internal set; }
public string Email { get; internal set; }
public DateTime LastLogin { get; internal set; }

/// 
/// Required for Entity Framework Core
/// 
private Player()
{
}

public Player([NotNull]string externalId, [NotNull]string displayName, [NotNull]string email)
{
ExternalId = externalId;
DisplayName = displayName;
Email = email;
}
}

Конфигурация IEntityType моего объекта:

Код: Выделить всё

public abstract class AggregateRootTypeConfiguration : IEntityTypeConfiguration
where TEntity : AggregateRoot
where TKey : IComparable
{
public virtual void Configure(EntityTypeBuilder builder)
{
builder.HasKey(x => x.Id);
builder.Property(x => x.Id)
.HasValueGenerator()
.ValueGeneratedOnAdd()
.IsRequired();
}
}

public class AuditedAggregateRootTypeConfiguration : AggregateRootTypeConfiguration
where TEntity : AuditedAggregateRoot
where TKey : IComparable
{
public override void Configure(EntityTypeBuilder builder)
{
base.Configure(builder);
builder.Property(x => x.Created)
.HasValueGenerator()
.ValueGeneratedOnAdd()
.IsRequired();

builder.Property(x => x.LastModified)
.HasValueGenerator()
.ValueGeneratedOnAddOrUpdate()
.IsRequired();
}
}

public class PlayerEntityTypeConfiguration : AuditedAggregateRootTypeConfiguration
{
public override void Configure(EntityTypeBuilder builder)
{
base.Configure(builder);
builder.Property(x => x.Email)
.HasMaxLength(254)
.IsRequired();

builder.Property(x => x.ExternalId)
.HasMaxLength(200)
.IsRequired();

builder.Property(x => x.DisplayName)
.HasMaxLength(200)
.IsRequired();

builder.HasIndex(x => x.ExternalId).IsUnique();
builder.HasIndex(x => x.Email).IsUnique();
builder.HasIndex(x => x.DisplayName).IsUnique();
}
}

DbContext:

Код: Выделить всё

public class MyDbContext : DbContext
{
public EmpiresDbContext(DbContextOptions dbContextOptions) : base(dbContextOptions)
{
}

public DbSet Players { get; set;  }

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
modelBuilder.ApplyConfigurationsFromAssembly(typeof(EmpiresDbContext).Assembly);
base.OnModelCreating(modelBuilder);
}

/*
* Overrides below to fix EF Core not calling ValueGenerator's on update
* See: https://github.com/dotnet/efcore/issues/19765#issuecomment-617679987
*/

public override int SaveChanges(bool acceptAllChangesOnSuccess)
{
GenerateOnUpdate();
return base.SaveChanges(acceptAllChangesOnSuccess);
}

public override Task SaveChangesAsync(
bool acceptAllChangesOnSuccess, CancellationToken cancellationToken = default)
{
GenerateOnUpdate();
return base.SaveChangesAsync(acceptAllChangesOnSuccess, cancellationToken);
}

private void GenerateOnUpdate()
{
foreach (var entityEntry in ChangeTracker.Entries())
{
foreach (var propertyEntry in entityEntry.Properties)
{
var property = propertyEntry.Metadata;
var valueGeneratorFactory =
property.GetValueGeneratorFactory();
var generatedOnUpdate = (property.ValueGenerated & ValueGenerated.OnUpdate)
== ValueGenerated.OnUpdate;
if (!generatedOnUpdate || valueGeneratorFactory == null)
{
continue;
}

var valueGenerator = valueGeneratorFactory.Invoke(
property,
entityEntry.Metadata);
propertyEntry.CurrentValue = valueGenerator.Next(entityEntry);
}
}
}
}

Я внедряю MyDbContext в службу, а затем в метод CreatePlayer:

Код: Выделить всё

var player = new Player(externalId, displayName, email);
_dbContext.Players.Add(player);
await _dbContext.SaveChangesAsync();

Однако при вызове await _dbContext.SaveChangesAsync(); я получаю исключение:

SqlException: невозможно вставить значение NULL в столбец LastModified

Я проверил с помощью точки останова, что свойствоEntry.CurrentValue = valueGenerator.Next(entityEntry); строка в Dbcontext обрабатывается и что она присваивает правильное значение свойству LastModified.
С трассировкой SQL Я вижу, что сгенерированный SQL выглядит следующим образом:

Код: Выделить всё

exec sp_executesql N'SET NOCOUNT ON;
INSERT INTO [Players] ([Id], [Created], [DisplayName], [Email], [ExternalId], [LastLogin])
VALUES (@p0, @p1, @p2, @p3, @p4, @p5);
SELECT [LastModified]
FROM [Players]
WHERE @@ROWCOUNT = 1 AND [Id] = @p0;

',N'@p0 uniqueidentifier,@p1 datetime2(7),@p2 nvarchar(200),@p3 nvarchar(254),@p4 nvarchar(4000),@p5 datetime2(7)',@p0='C17E4EC8-CDD6-458A-8CEF-08D8C5D6F63A',@p1='2021-01-31 11:00:27.1068666',@p2=N'Kyr',@p3=N'test@test.test',@p4=N'my-iDp-id-removed-for-security',@p5='0001-01-01 00:00:00'

Как видно выше, столбец LastModified даже не упоминается.


Подробнее здесь: https://stackoverflow.com/questions/659 ... egenerator