lancommander-ref/LANCommander
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix updating for relationships with stale/untracked entities

Browse source
This commit is contained in:
Pat Hartl 2026-06-15 20:25:09 -05:00
parent 86e1d6075a
commit 5a29943d44
1 changed files with 16 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

20
LANCommander.Server.Data/UpdateEntityContext.cs
View file

@ -96,10 +96,22 @@ public class UpdateEntityContext<TEntity>
// Get the list of tracked entities from the context
var trackedEntities = _context.Set<TRelatedEntity>().Local;
// Replace entities with tracked instances to avoid duplicate tracking
var updatedTrackedEntities = updatedEntities
.Select(e => trackedEntities.FirstOrDefault(t => t.Id == e.Id) ?? e)
.ToList();
// Replace entities with tracked instances to avoid duplicate tracking.
// For entities that aren't already tracked locally, fetch the tracked
// instance from the database instead of attaching the detached incoming
// instance. Attaching a detached instance would pull in its nested
// navigation graph (e.g. Archive.StorageLocation), and two entities that
// share the same related key would trigger an identity-conflict.
var updatedTrackedEntities = new List<TRelatedEntity>();
foreach (var updatedEntity in updatedEntities)
{
var trackedEntity = trackedEntities.FirstOrDefault(t => t.Id == updatedEntity.Id)
?? await _context.Set<TRelatedEntity>().FirstOrDefaultAsync(t => t.Id == updatedEntity.Id)
?? updatedEntity;
updatedTrackedEntities.Add(trackedEntity);
}
// Update values for existing entities first
foreach (var existingEntity in existingEntities)