hrynco/hrynco-ef
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: eliminate multi-save in batch Add and DeleteAsync(TEntityId) #10

Merged
agrynco merged 1 commits from IT-631-fix-delete into development 2026-05-06 12:45:10 +03:00
Conversation 0 Commits 1 Files Changed 1
+11 -2
agrynco commented 2026-05-06 12:45:01 +03:00
Owner
Copy Link
Copy Source
  • Add(TEntity[]) now passes save:false in the loop and calls SaveChanges once at the end
  • DeleteAsync(TEntityId) now calls DoRemove directly instead of Delete(TEntityId) to avoid the double-save from the sync overload chain

Ref: IT-631

- `Add(TEntity[])` now passes `save:false` in the loop and calls `SaveChanges` once at the end - `DeleteAsync(TEntityId)` now calls `DoRemove` directly instead of `Delete(TEntityId)` to avoid the double-save from the sync overload chain Ref: IT-631
agrynco added 1 commit 2026-05-06 12:45:01 +03:00
fix: eliminate multi-save in batch Add and DeleteAsync(TEntityId) 93461fd35e 
- Add(TEntity[]) now passes save:false in the loop and calls SaveChanges once at the end
- DeleteAsync(TEntityId) now calls DoRemove directly instead of Delete(TEntityId) to avoid the double-save from the sync overload chain

Ref: #IT-631
agrynco merged commit 1edcafdebd into development 2026-05-06 12:45:10 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agrynco (Anatolii Hrynchuk)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: hrynco/hrynco-ef#10
Delete Branch "IT-631-fix-delete"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?