Merge pull request #1491 from TechnologyEnhancedLearning/Develop/Fixes/TD-6342-Issue-missing-the-'My-contributions'-link-on-Moodletestenv-for-some-users

TD-6342: Added cache logic to store menu items.

Author: swapnamol-abraham

OpenAPI/LearningHub.Nhs.OpenApi.Services/Services/UserGroupService.cs

@@ -54,6 +54,11 @@ public class UserGroupService : IUserGroupService
         /// </summary>
         private IUserGroupAttributeRepository userGroupAttributeRepository;
 
+        /// <summary>
+        /// The caching service.
+        /// </summary>
+        private readonly ICachingService cachingService;
+
         /// <summary>
         /// Initializes a new instance of the <see cref="UserGroupService"/> class.
         /// </summary>
@@ -64,14 +69,16 @@ public class UserGroupService : IUserGroupService
         /// <param name="scopeRepository">The scope repository.</param>
         /// <param name="userGroupAttributeRepository">The user group attribute repository.</param>
         /// <param name="mapper">The mapper.</param>
+        /// <param name="cachingService">The caching service.</param>
         public UserGroupService(
             ICatalogueService catalogueService,
             IUserGroupRepository userGroupRepository,
             IUserUserGroupRepository userUserGroupRepository,
             IScopeRepository scopeRepository,
             IRoleUserGroupRepository roleUserGroupRepository,
             IUserGroupAttributeRepository userGroupAttributeRepository,
-            IMapper mapper)
+            IMapper mapper,
+            ICachingService cachingService)
         {
             this.catalogueService = catalogueService;
             this.userGroupRepository = userGroupRepository;
@@ -80,6 +87,7 @@ public UserGroupService(
             this.roleUserGroupRepository = roleUserGroupRepository;
             this.userGroupAttributeRepository = userGroupAttributeRepository;
             this.mapper = mapper;
+            this.cachingService = cachingService;
         }
 
         /// <summary>
@@ -106,7 +114,21 @@ public async Task<UserGroup> GetByIdAsync(int id, bool includeRoles)
         /// <inheritdoc />
         public async Task<bool> UserHasCatalogueContributionPermission(int userId)
         {
-            var userRoleGroups = await this.roleUserGroupRepository.GetRoleUserGroupViewModelsByUserId(userId);
+
+            string cacheKey = $"{userId}:AllRolesWithPermissions";
+            var userRoleGroupsInCache = await this.cachingService.GetAsync<List<RoleUserGroupViewModel>>(cacheKey);
+            var userRoleGroups = new List<RoleUserGroupViewModel>();
+
+            if (userRoleGroupsInCache.ResponseEnum == CacheReadResponseEnum.Found)
+            {
+                userRoleGroups = userRoleGroupsInCache.Item;
+            }
+            else
+            {
+                userRoleGroups = await this.roleUserGroupRepository.GetRoleUserGroupViewModelsByUserId(userId);
+                await this.cachingService.SetAsync($"{userId}:AllRolesWithPermissions", userRoleGroups);
+            }
+
             if (userRoleGroups != null && userRoleGroups.Any(r => r.RoleEnum == RoleEnum.Editor))
             {
                 return true;