TD_5557_ added a null check to the query for the description propery whenever we update any competency group, so that the updated description value can be saved to the DB and show up.

ABSinhaa · ABSinhaa · commit 0fce67c7730e · 2025-05-01T15:14:04.000+01:00
diff --git a/DigitalLearningSolutions.Data/DataServices/FrameworkDataService.cs b/DigitalLearningSolutions.Data/DataServices/FrameworkDataService.cs
@@ -1042,7 +1042,7 @@ int adminId
             {
                 var numberOfAffectedRows = connection.Execute(
                     @"UPDATE CompetencyGroups SET Name = @name, UpdatedByAdminID = @adminId, Description = @description
-                    WHERE ID = @competencyGroupId AND (Name <> @name OR Description <> @description)",
+                    WHERE ID = @competencyGroupId AND (Name <> @name OR ISNULL(Description, '') <> ISNULL(@description, ''))",
                     new { name, adminId, competencyGroupId, description }
                 );
                 if (numberOfAffectedRows < 1)

Original file line number	Diff line number	Diff line change
`@@ -1042,7 +1042,7 @@ int adminId`
`1042`	`1042`	`{`
`1043`	`1043`	`var numberOfAffectedRows = connection.Execute(`
`1044`	`1044`	`@"UPDATE CompetencyGroups SET Name = @name, UpdatedByAdminID = @adminId, Description = @description`
`1045`		`- WHERE ID = @competencyGroupId AND (Name <> @name OR Description <> @description)",`
	`1045`	`+ WHERE ID = @competencyGroupId AND (Name <> @name OR ISNULL(Description, '') <> ISNULL(@description, ''))",`
`1046`	`1046`	`new { name, adminId, competencyGroupId, description }`
`1047`	`1047`	`);`
`1048`	`1048`	`if (numberOfAffectedRows < 1)`