Refactor the ReactionManager.

modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Reactions/DefaultReactionDefinitionStore.cs

@@ -15,12 +15,12 @@ namespace Volo.CmsKit.Reactions
             Options = options.Value;
         }
 
-        public virtual Task<List<ReactionDefinition>> GetAvailableReactionsAsync(string entityType)
+        public virtual Task<List<ReactionDefinition>> GetReactionsAsync(string entityType = null)
         {
             return Task.FromResult(Options.Reactions.Values.ToList());
         }
 
-        public Task<ReactionDefinition> GetReactionOrNullAsync(string reactionName, string entityType)
+        public Task<ReactionDefinition> GetReactionOrNullAsync(string reactionName, string entityType = null)
         {
             return Task.FromResult(Options.Reactions.GetOrDefault(reactionName));
         }

modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Reactions/IReactionDefinitionStore.cs

@@ -1,5 +1,4 @@
-using System;
-using System.Collections.Generic;
+using System.Collections.Generic;
 using System.Threading.Tasks;
 using JetBrains.Annotations;
 
@@ -7,8 +6,8 @@ namespace Volo.CmsKit.Reactions
 {
     public interface IReactionDefinitionStore
     {
-        Task<List<ReactionDefinition>> GetAvailableReactionsAsync([CanBeNull] string entityType);
+        Task<List<ReactionDefinition>> GetReactionsAsync([CanBeNull] string entityType = null);
 
-        Task<ReactionDefinition> GetReactionOrNullAsync([NotNull] string reactionName, [CanBeNull] string entityType);
+        Task<ReactionDefinition> GetReactionOrNullAsync([NotNull] string reactionName, [CanBeNull] string entityType = null);
     }
 }

modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Reactions/ReactionManager.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using System.Threading.Tasks;
 using JetBrains.Annotations;
 using Volo.Abp;
@@ -19,80 +20,34 @@ namespace Volo.CmsKit.Reactions
             ReactionDefinitionStore = reactionDefinitionStore;
         }
 
-        public virtual async Task<List<ReactionDefinition>> GetAvailableReactionsAsync(
+        public virtual async Task<List<ReactionDefinition>> GetReactionsAsync(
             [CanBeNull] string entityType = null)
         {
-            return await ReactionDefinitionStore.GetAvailableReactionsAsync(entityType);
+            return await ReactionDefinitionStore.GetReactionsAsync(entityType);
         }
 
         public virtual async Task<List<ReactionSummary>> GetSummariesAsync(
             [NotNull] string entityType,
             [NotNull] string entityId)
         {
-            var queryResultItems = await UserReactionRepository.GetSummariesAsync(entityType, entityId);
-
-            var summaryDtos = new List<ReactionSummary>();
-
-            foreach (var queryResultItem in queryResultItems)
-            {
-                var summary = new ReactionSummary
-                {
-                    Count = queryResultItem.Count
-                };
-
-                //TODO: Get all definitions then filter here?
-                var reactionDefinition = await ReactionDefinitionStore
-                    .GetReactionOrNullAsync(
-                        queryResultItem.ReactionName,
-                        entityType
-                    );
-
-                if (reactionDefinition == null)
-                {
-                    continue;
-                }
-
-                summary.Reaction = reactionDefinition;
+            Check.NotNullOrWhiteSpace(entityType, nameof(entityType));
+            Check.NotNullOrWhiteSpace(entityId, nameof(entityId));
 
-                summaryDtos.Add(summary);
-            }
+            var userReactionCounts = (await UserReactionRepository.GetSummariesAsync(entityType, entityId))
+                .ToDictionary(x => x.ReactionName, x => x.Count);
 
-            return summaryDtos;
-        }
-
-        public virtual async Task<List<ReactionDefinition>> GetUserReactionsAsync(
-            Guid userId,
-            [NotNull] string entityType,
-            [NotNull] string entityId)
-        {
-            var userReactions = await UserReactionRepository
-                .GetListForUserAsync(
-                    userId,
-                    entityType,
-                    entityId
+            var reactions = await ReactionDefinitionStore
+                .GetReactionsAsync(
+                    entityType
                 );
 
-            var reactionDtos = new List<ReactionDefinition>();
-
-            foreach (var userReaction in userReactions)
-            {
-                //TODO: Get all definitions then filter here?
-                var reactionDefinition = await ReactionDefinitionStore
-                    .GetReactionOrNullAsync(
-                        userReaction.ReactionName,
-                        userReaction.EntityType
-                    );
-
-                if (reactionDefinition == null)
+            return reactions
+                .Select(reaction => new ReactionSummary
                 {
-                    await UserReactionRepository.DeleteAsync(userReaction);
-                    continue;
-                }
-
-                reactionDtos.Add(reactionDefinition);
-            }
-
-            return reactionDtos;
+                    Reaction = reaction,
+                    Count = userReactionCounts.GetOrDefault(reaction.Name)
+                })
+                .ToList();
         }
 
         public virtual async Task<UserReaction> CreateAsync(
@@ -129,6 +84,10 @@ namespace Volo.CmsKit.Reactions
             [NotNull] string entityId,
             [NotNull] string reactionName)
         {
+            Check.NotNullOrWhiteSpace(entityType, nameof(entityType));
+            Check.NotNullOrWhiteSpace(entityId, nameof(entityId));
+            Check.NotNullOrWhiteSpace(reactionName, nameof(reactionName));
+
             var existingReaction = await UserReactionRepository.FindAsync(userId, entityType, entityId, reactionName);
             if (existingReaction == null)
             {

modules/cms-kit/src/Volo.CmsKit.Public.Application/Volo/CmsKit/Reactions/ReactionPublicAppService.cs

@@ -27,36 +27,30 @@ namespace Volo.CmsKit.Reactions
 
         public virtual async Task<ListResultDto<ReactionWithSelectionDto>> GetForSelectionAsync(string entityType, string entityId)
         {
-            var reactionDefinitions = await ReactionManager
-                .GetAvailableReactionsAsync(
-                    entityType
-                );
-
-            var summaries =
-                (await ReactionManager.GetSummariesAsync(entityType, entityId))
-                .ToDictionary(x => x.Reaction.Name, x => x.Count);
+            var summaries = await ReactionManager.GetSummariesAsync(entityType, entityId);
 
-            var userReactions = await ReactionManager.GetUserReactionsAsync(
-                CurrentUser.GetId(),
-                entityType,
-                entityId
-            );
+            var userReactions = (await UserReactionRepository
+                .GetListForUserAsync(
+                    CurrentUser.GetId(),
+                    entityType,
+                    entityId
+                )).ToDictionary(x => x.ReactionName, x => x);
 
-            var reactionDtos = new List<ReactionWithSelectionDto>();
+            var reactionWithSelectionDtos = new List<ReactionWithSelectionDto>();
 
-            foreach (var reactionDefinition in reactionDefinitions)
+            foreach (var summary in summaries)
             {
-                reactionDtos.Add(
+                reactionWithSelectionDtos.Add(
                     new ReactionWithSelectionDto
                     {
-                        Reaction = ConvertToReactionDto(reactionDefinition),
-                        Count = summaries.GetOrDefault(reactionDefinition.Name),
-                        IsSelectedByCurrentUser = userReactions.Any(x => x.Name == reactionDefinition.Name)
+                        Reaction = ConvertToReactionDto(summary.Reaction),
+                        Count = summary.Count,
+                        IsSelectedByCurrentUser = userReactions.ContainsKey(summary.Reaction.Name)
                     }
                 );
             }
 
-            return new ListResultDto<ReactionWithSelectionDto>(reactionDtos);
+            return new ListResultDto<ReactionWithSelectionDto>(reactionWithSelectionDtos);
         }
 
         public virtual async Task CreateAsync(CreateReactionDto input)