nns/food-market
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

fix(employees): правка сотрудника с привязкой к точке падала 500 (EF8 nav-collection)

Browse source 
DbUpdateConcurrencyException «0 rows affected» при PUT сотрудника, у которого
есть EmployeeRetailPointAssignment. Замена nav RemoveRange+Clear+Add на
ExecuteDeleteAsync по EmployeeId + DbSet.Add (как в Supplies/Enters/Losses.Update).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
nns 2026-10-08 02:10:11 +05:00
parent 24646e0649
commit aedcd3f83a
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/food-market.api/Controllers/Organizations/EmployeesController.cs
View file

@ -283,12 +283,15 @@ public async Task<IActionResult> Update(Guid id, [FromBody] EmployeeInput input,
if (e.IsActive != nowActive) await SetLinkedUserActiveAsync(e.UserId, nowActive, ct);
e.IsActive = nowActive;
// Replace assignments wholesale
_db.EmployeeRetailPointAssignments.RemoveRange(e.RetailPointAssignments);
e.RetailPointAssignments.Clear();
// Replace assignments wholesale. ExecuteDelete + DbSet.Add вместо
// nav-collection RemoveRange+Clear+Add: иначе EF8 путается на
// nav-collection и UPDATE employees падает с DbUpdateConcurrencyException
// «0 rows affected» (у сотрудника с привязкой к точке правка была
// невозможна). Тот же паттерн, что в Supplies/Enters/Losses/Demands.Update.
await _db.EmployeeRetailPointAssignments.Where(a => a.EmployeeId == e.Id).ExecuteDeleteAsync(ct);
foreach (var rpId in input.RetailPointIds ?? [])
e.RetailPointAssignments.Add(new EmployeeRetailPointAssignment
{ OrganizationId = orgId, RetailPointId = rpId });
_db.EmployeeRetailPointAssignments.Add(new EmployeeRetailPointAssignment
{ OrganizationId = orgId, EmployeeId = e.Id, RetailPointId = rpId });
await _db.SaveChangesAsync(ct);