Merge pull request #388 from TechnologyEnhancedLearning/Develop/fixes/TD-4205-Track-Portal-administrator-User-ID-4--Activities-on-Prod-BD

ArunimaGeorge · web-flow · commit 850669fd7c1e · 2024-07-08T11:56:05.000+01:00
TD-4205: Added user authentication check before recording Article/Image resource activity.
diff --git a/LearningHub.Nhs.WebUI/Controllers/ResourceController.cs b/LearningHub.Nhs.WebUI/Controllers/ResourceController.cs
@@ -180,7 +180,7 @@ public async Task<IActionResult> Index(int resourceReferenceId, bool? acceptSens
             }
 
             // For article/image resources, immediately record the resource activity for this user.
-            if ((resource.ResourceTypeEnum == ResourceTypeEnum.Article || resource.ResourceTypeEnum == ResourceTypeEnum.Image) && (!resource.SensitiveContent))
+            if ((resource.ResourceTypeEnum == ResourceTypeEnum.Article || resource.ResourceTypeEnum == ResourceTypeEnum.Image) && (!resource.SensitiveContent) && this.User.Identity.IsAuthenticated)
             {
                 var activity = new CreateResourceActivityViewModel()
                 {

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ public async Task<IActionResult> Index(int resourceReferenceId, bool? acceptSens`
`180`	`180`	`}`
`181`	`181`
`182`	`182`	`// For article/image resources, immediately record the resource activity for this user.`
`183`		`- if ((resource.ResourceTypeEnum == ResourceTypeEnum.Article \|\| resource.ResourceTypeEnum == ResourceTypeEnum.Image) && (!resource.SensitiveContent))`
	`183`	`+ if ((resource.ResourceTypeEnum == ResourceTypeEnum.Article \|\| resource.ResourceTypeEnum == ResourceTypeEnum.Image) && (!resource.SensitiveContent) && this.User.Identity.IsAuthenticated)`
`184`	`184`	`{`
`185`	`185`	`var activity = new CreateResourceActivityViewModel()`
`186`	`186`	`{`