From b8c348afb69dece28383a2a39fa1988a103bbddb Mon Sep 17 00:00:00 2001
From: Developer 02 <developer02@didaloghe.onmicrosoft.com>
Date: Thu, 28 Aug 2025 17:52:24 +0200
Subject: [PATCH] refactor(ReadHistoryQueryHandler): update to use
 IRepository<EnvelopeHistory>

---
 .../Queries/Read/ReadHistoryQueryHandler.cs   | 32 +++++++++++--------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/EnvelopeGenerator.Application/Histories/Queries/Read/ReadHistoryQueryHandler.cs b/EnvelopeGenerator.Application/Histories/Queries/Read/ReadHistoryQueryHandler.cs
index ddc88001..cff4c4c7 100644
--- a/EnvelopeGenerator.Application/Histories/Queries/Read/ReadHistoryQueryHandler.cs
+++ b/EnvelopeGenerator.Application/Histories/Queries/Read/ReadHistoryQueryHandler.cs
@@ -1,7 +1,9 @@
 using AutoMapper;
-using EnvelopeGenerator.Application.Interfaces.Repositories;
+using DigitalData.Core.Abstraction.Application.Repository;
 using DigitalData.Core.Exceptions;
+using EnvelopeGenerator.Domain.Entities;
 using MediatR;
+using Microsoft.EntityFrameworkCore;
 
 namespace EnvelopeGenerator.Application.Histories.Queries.Read;
 
@@ -10,20 +12,18 @@ namespace EnvelopeGenerator.Application.Histories.Queries.Read;
 /// </summary>
 public class ReadHistoryQueryHandler : IRequestHandler<ReadHistoryQuery, IEnumerable<ReadHistoryResponse>>
 {
-    [Obsolete("Use IRepository")]
-    private readonly IEnvelopeHistoryRepository _repository;
+    private readonly IRepository<EnvelopeHistory> _repo;
 
     private readonly IMapper _mapper;
 
     /// <summary>
     /// 
     /// </summary>
-    /// <param name="repository"></param>
+    /// <param name="repo"></param>
     /// <param name="mapper"></param>
-    [Obsolete("Use IRepository")]
-    public ReadHistoryQueryHandler(IEnvelopeHistoryRepository repository, IMapper mapper)
+    public ReadHistoryQueryHandler(IRepository<EnvelopeHistory> repo, IMapper mapper)
     {
-        _repository = repository;
+        _repo = repo;
         _mapper = mapper;
     }
 
@@ -31,16 +31,20 @@ public class ReadHistoryQueryHandler : IRequestHandler<ReadHistoryQuery, IEnumer
     /// 
     /// </summary>
     /// <param name="request"></param>
-    /// <param name="cancellationToken"></param>
+    /// <param name="cancel"></param>
     /// <returns></returns>
     /// <exception cref="NotFoundException"></exception>
-    public async Task<IEnumerable<ReadHistoryResponse>> Handle(ReadHistoryQuery request, CancellationToken cancellationToken)
+    public async Task<IEnumerable<ReadHistoryResponse>> Handle(ReadHistoryQuery request, CancellationToken cancel = default)
     {
-        var hists = await _repository.ReadAsync(request.EnvelopeId, status: request.Status is null ? null : request.Status);
+        var query = _repo.ReadOnly().Where(h => h.EnvelopeId == request.EnvelopeId);
+        if (request.Status is not null)
+            query = query.Where(h => h.Status == request.Status);
 
-        if (!hists.Any())
-            throw new NotFoundException();
-                
-        return _mapper.Map<IEnumerable<ReadHistoryResponse>>(hists);
+        var hists = await query.ToListAsync(cancel);
+
+        if (hists.Count == 0)
+            return _mapper.Map<IEnumerable<ReadHistoryResponse>>(hists);
+
+        throw new NotFoundException();
     }
 }
\ No newline at end of file