on.nemtina/persistence
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

featute/ChangeLog #6

Merged
on.nemtina merged 25 commits from featute/ChangeLog into master 2024-12-09 17:44:48 +05:00
Conversation 0 Commits 25 Files Changed 42 +17 -22
1 changed files with 17 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 42fe4dee50 - Show all commits

11
Persistence.Repository/Repositories/ChangeLogRepository.cs
View File

@ -101,8 +101,7 @@ public class ChangeLogRepository : IChangeLogRepository
var result = 0; var result = 0;
using var transaction = await db.Database.BeginTransactionAsync(token); using var transaction = await db.Database.BeginTransactionAsync(token);
ng.frolov commented 2024-12-05 13:42:50 +05:00
Outdated
Review
Copy Link

Не тот тип эксепшена.
Лучше использовать ArgumentException, а в middleware его перехватить и вернуть пользователю 400

Не тот тип эксепшена. Лучше использовать ArgumentException, а в middleware его перехватить и вернуть пользователю 400
👍 1
try
ng.frolov commented 2024-12-09 11:26:02 +05:00
Outdated
Review
Copy Link

try-catch тут и дальше не нужен. Rollback ролбэк будет вызван финализатором транзакции, если не был вызван комит

try-catch тут и дальше не нужен. Rollback ролбэк будет вызван финализатором транзакции, если не был вызван комит
{
foreach (var dto in dtos) foreach (var dto in dtos)
{ {
var updatedEntity = updatedEntities.GetValueOrDefault(dto.Id); var updatedEntity = updatedEntities.GetValueOrDefault(dto.Id);
@ -123,12 +122,8 @@ public class ChangeLogRepository : IChangeLogRepository
await transaction.CommitAsync(token); await transaction.CommitAsync(token);
return result; return result;
}
catch
{
await transaction.RollbackAsync(token);
throw;
}
} }
public async Task<PaginationContainer<DataWithWellDepthAndSectionDto>> GetByDate( public async Task<PaginationContainer<DataWithWellDepthAndSectionDto>> GetByDate(