#1004 Отделить логику маппинга от клиентов + #1000 Добавить множественный маппинг сущностей в TimeStampedClient #31

Merged
on.nemtina merged 11 commits from feature-mapping into master 2025-03-03 09:44:27 +05:00
Collaborator
No description provided.
aa.shibalkin was assigned by rs.efremov 2025-02-24 15:57:32 +05:00
on.nemtina was assigned by rs.efremov 2025-02-24 15:57:32 +05:00
rs.efremov added 9 commits 2025-02-24 15:57:32 +05:00
on.nemtina approved these changes 2025-02-25 13:02:36 +05:00
@ -0,0 +52,4 @@
/// <inheritdoc/>
public async Task<IEnumerable<SetpointLogDto>> GetPart(DateTimeOffset dateBegin, int take, CancellationToken token)
{
var res = await setpointClient.GetPart(dateBegin, take, token);
Owner

пусть будет result, также как сейчас в остальных методах

пусть будет result, также как сейчас в остальных методах
@ -0,0 +70,4 @@
/// <inheritdoc/>
public void Dispose()
{
setpointClient.Dispose();
Owner

добавить GC.SuppressFinalize(this);

добавить GC.SuppressFinalize(this);
@ -0,0 +7,4 @@
namespace DD.Persistence.Client.Clients.Mapping.Clients;
/// <inheritdoc/>
public class TimestampedMappingClient(ITimestampedValuesClient client, Dictionary<Guid, Type>? mappingConfigs) : ITimestampedMappingClient
Owner

Нужно сделать отдельный сервис, который отвечает за кеширование маппингов, и зарегистрировать его в DI.

Нужно сделать отдельный сервис, который отвечает за кеширование маппингов, и зарегистрировать его в DI.
@ -0,0 +58,4 @@
return mappedDtos;
});
//var genericMapperType = typeof(TimestampedSetMapper<>).MakeGenericType(type);
Owner

Желательно комментарии стереть

Желательно комментарии стереть
@ -0,0 +95,4 @@
var value = jsonElement.Deserialize(property.PropertyType);
property.SetValue(entity, value);
}
catch (Exception) { }
Owner

Тут внутри блока Exception пусто, наверное, нужна какая-то обработка?

Тут внутри блока Exception пусто, наверное, нужна какая-то обработка?
@ -29,0 +31,4 @@
}
public static IServiceCollection AddPersistenceMapping(this IServiceCollection services, Dictionary<Guid, Type>? mappingConfigs)
Owner

Dictionary<Guid, Type>? mappingConfigs - не nullable

Dictionary<Guid, Type>? mappingConfigs - не nullable
aa.shibalkin added 1 commit 2025-02-27 14:01:35 +05:00
rs.efremov added 1 commit 2025-02-28 13:15:30 +05:00
on.nemtina merged commit 7330f9b6db into master 2025-03-03 09:44:27 +05:00
on.nemtina deleted branch feature-mapping 2025-03-03 09:44:27 +05:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: on.nemtina/persistence#31
No description provided.