Merge pull request #949 from DFE-Digital/hotfix/change-for-merge

Hotfix/change for merge
DFE-Digital · Jan 17, 2025 · 7d45007 · 7d45007
2 parents 06aa523 + 7f47c7a
commit 7d45007
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -8,7 +8,7 @@ Web application to help schools plan a technology roadmap
 
 ## Requirements
 
-- .Net 8.0 and any supported IDE for DEV running.
+- .Net 8.0 and any supported IDE for DEV running. 
 
 ## Running Locally
 

diff --git a/src/Dfe.PlanTech.Infrastructure.Redis/RedisDependencyManager.cs b/src/Dfe.PlanTech.Infrastructure.Redis/RedisDependencyManager.cs
@@ -27,12 +27,16 @@ public Task RegisterDependenciesAsync<T>(IDatabase database, string key, T value
     private async Task GetAndSetDependencies<T>(IDatabase database, string key, T value)
     {
         var batch = database.CreateBatch();
-        var tasks = GetDependencies(value).Select(dependency => batch.SetAddAsync(GetDependencyKey(dependency), key, CommandFlags.FireAndForget)).ToArray();
+        var tasks = GetDependencies(value).Distinct()
+                                                     .Select(dependency => batch.SetAddAsync(GetDependencyKey(dependency), key, CommandFlags.FireAndForget))
+                                                     .ToArray();
+
         if (tasks.Length == 0)
         {
             // If the value has no dependencies (is empty) it should be invalidated when new content comes in
             tasks = tasks.Append(batch.SetAddAsync(EmptyCollectionDependencyKey, key, CommandFlags.FireAndForget)).ToArray();
         }
+
         batch.Execute();
         await Task.WhenAll(tasks);
     }