Merge pull request #3303 from TechnologyEnhancedLearning/Develop/Fix/TD-5659-Duplicate-records-of-the-reviewer

TD-5659-Modify the SQL query to retrieve distinct records and avoid records for which a request has already been sent.

Author: rshrirohit

DigitalLearningSolutions.Data/DataServices/FrameworkDataService.cs

@@ -2066,7 +2066,7 @@ FROM   FrameworkComments
         public IEnumerable<CollaboratorDetail> GetReviewersForFrameworkId(int frameworkId)
         {
             return connection.Query<CollaboratorDetail>(
-                @"SELECT
+                @"SELECT DISTINCT
                         fc.ID,
                         fc.FrameworkID,
                         fc.AdminID,
@@ -2077,8 +2077,11 @@ public IEnumerable<CollaboratorDetail> GetReviewersForFrameworkId(int frameworkI
                     FROM FrameworkCollaborators fc
                     INNER JOIN AdminUsers AS au ON fc.AdminID = au.AdminID
                     LEFT OUTER JOIN FrameworkReviews ON fc.ID = FrameworkReviews.FrameworkCollaboratorID
-                    WHERE (fc.FrameworkID = @FrameworkID) AND (FrameworkReviews.ID IS NULL) AND (fc.IsDeleted=0) OR
-                            (fc.FrameworkID = @FrameworkID) AND (FrameworkReviews.Archived IS NOT NULL) AND (fc.IsDeleted=0)",
+                    WHERE ((fc.FrameworkID = @FrameworkID) AND (FrameworkReviews.ID IS NULL) AND (fc.IsDeleted=0)) OR
+                            ((fc.FrameworkID = @FrameworkID) AND (FrameworkReviews.Archived IS NOT NULL) AND (fc.IsDeleted=0))
+                            AND
+                            (fc.ID Not in ( Select FrameworkCollaboratorID from FrameworkReviews where FrameworkID = @FrameworkID AND 
+							FrameworkReviews.Archived IS NULL AND FrameworkCollaboratorID = fc.ID))",
                 new { frameworkId }
             );
         }