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

Fix creation of entities with many-to-many relationships

Browse source 
Ref #414
This commit is contained in:
Pat Hartl 2026-06-27 11:28:39 -05:00
parent a778a56b18
commit f412f308a2
1 changed files with 13 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

15
LANCommander.Server.Services/_BaseDatabaseService.cs
View file

@ -280,9 +280,20 @@ namespace LANCommander.Server.Services
var currentUser = await GetCurrentUserAsync(context);
var newEntity = Activator.CreateInstance<T>();
// Assign a stable primary key up front. Relationships (including
// many-to-many joins) are wired up below while the entity is still
// tracked, before it is added. If the key were left empty until
// AddAsync generated one, the key would change mid-save and EF would
// fail to fix up join foreign keys when multiple related entities are
// attached at once (e.g. adding a tool with multiple games).
if (addedEntity.Id == Guid.Empty)
addedEntity.Id = Guid.NewGuid();
newEntity.Id = addedEntity.Id;
context.Entry(newEntity).CurrentValues.SetValues(addedEntity);
newEntity.CreatedOn = DateTime.UtcNow;
newEntity.CreatedById = currentUser?.Id;