From 632addf8e84491c54cf4ca1a19cdac595403f214 Mon Sep 17 00:00:00 2001
From: Mohamed Nazem <109486038+nazem0@users.noreply.github.com>
Date: Wed, 19 Aug 2026 17:35:23 +0300
Subject: [PATCH] checking Use(Object/List)ProjectionMapper before resolving
 the mapper

---
 .../Services/AbstractKeyReadOnlyAppService.cs | 32 +++++++++++--------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/framework/src/Volo.Abp.Ddd.Application/Volo/Abp/Application/Services/AbstractKeyReadOnlyAppService.cs b/framework/src/Volo.Abp.Ddd.Application/Volo/Abp/Application/Services/AbstractKeyReadOnlyAppService.cs
index 0c53ac2705..f70953b2ac 100644
--- a/framework/src/Volo.Abp.Ddd.Application/Volo/Abp/Application/Services/AbstractKeyReadOnlyAppService.cs
+++ b/framework/src/Volo.Abp.Ddd.Application/Volo/Abp/Application/Services/AbstractKeyReadOnlyAppService.cs
@@ -44,11 +44,9 @@ public abstract class AbstractKeyReadOnlyAppService<TEntity, TGetOutputDto, TGet
 
     protected virtual string? GetListPolicyName { get; set; }
 
-    protected virtual IQueryProjectionMapper<TEntity, TGetOutputDto>? ObjectProjectionMapper =>
-        LazyServiceProvider.LazyGetService<IQueryProjectionMapper<TEntity, TGetOutputDto>>();
+    protected virtual IQueryProjectionMapper<TEntity, TGetOutputDto>? ObjectProjectionMapper { get; }
 
-    protected virtual IQueryProjectionMapper<TEntity, TGetListOutputDto>? ListProjectionMapper =>
-        LazyServiceProvider.LazyGetService<IQueryProjectionMapper<TEntity, TGetListOutputDto>>();
+    protected virtual IQueryProjectionMapper<TEntity, TGetListOutputDto>? ListProjectionMapper { get; }
 
     protected virtual bool UseObjectProjectionMapper =>
         ObjectProjectionMapper != null;
@@ -59,20 +57,28 @@ public abstract class AbstractKeyReadOnlyAppService<TEntity, TGetOutputDto, TGet
     protected AbstractKeyReadOnlyAppService(IReadOnlyRepository<TEntity> repository)
     {
         ReadOnlyRepository = repository;
+
+        ObjectProjectionMapper =
+            LazyServiceProvider.LazyGetService<
+                IQueryProjectionMapper<TEntity, TGetOutputDto>>();
+
+        ListProjectionMapper =
+            LazyServiceProvider.LazyGetService<
+                IQueryProjectionMapper<TEntity, TGetListOutputDto>>();
     }
 
     public virtual async Task<TGetOutputDto> GetAsync(TKey id)
     {
         await CheckGetPolicyAsync();
 
-        var projectionMapper = ObjectProjectionMapper;
-
-        if (UseObjectProjectionMapper && projectionMapper != null)
+        if (UseObjectProjectionMapper && ObjectProjectionMapper != null)
         {
             var query = await GetEntityByIdQueryAsync(id);
 
-            var dto = 
-                await AsyncExecuter.FirstOrDefaultAsync(projectionMapper.ProjectTo(query))
+            var dto =
+                await AsyncExecuter.FirstOrDefaultAsync(
+                    ObjectProjectionMapper.ProjectTo(query)
+                )
                 ?? throw new EntityNotFoundException(typeof(TEntity), id);
 
             return dto;
@@ -97,12 +103,10 @@ public abstract class AbstractKeyReadOnlyAppService<TEntity, TGetOutputDto, TGet
             query = ApplySorting(query, input);
             query = ApplyPaging(query, input);
 
-            var projectionMapper = ListProjectionMapper;
-
-            if (UseListProjectionMapper && projectionMapper != null)
+            if (UseListProjectionMapper && ListProjectionMapper != null)
             {
                 entityDtos = await AsyncExecuter.ToListAsync(
-                    projectionMapper.ProjectTo(query)
+                    ListProjectionMapper.ProjectTo(query)
                 );
             }
             else
@@ -124,7 +128,7 @@ public abstract class AbstractKeyReadOnlyAppService<TEntity, TGetOutputDto, TGet
     {
         throw new NotImplementedException(
             "Override this method to create the query used for getting an entity by id."
-            );
+        );
     }
 
     protected virtual async Task CheckGetPolicyAsync()