TD-5138_ Staff count was incorrect on the dashboard and doesn't match when staff list is clicked to navigate the list.

DigitalLearningSolutions.Data/DataServices/SupervisorDataService.cs

@@ -126,13 +126,35 @@ public SupervisorDataService(IDbConnection connection, ILogger<SupervisorDataSer
         public DashboardData? GetDashboardDataForAdminId(int adminId)
         {
             return connection.Query<DashboardData>(
-                @"  SELECT (SELECT COUNT(sd.ID) AS StaffCount
-                        FROM SupervisorDelegates sd
-                        LEFT OUTER JOIN users u
-                            ON u.id = sd.DelegateUserID
-                            AND u.Active = 1
-                        WHERE  (sd.SupervisorAdminID = @adminId) 
-	                        AND (sd.Removed IS NULL)) AS StaffCount,
+                @" SELECT (SELECT COUNT(sd.ID) AS StaffCount
+                        FROM   CustomPrompts AS cp6 
+	                    RIGHT OUTER JOIN CustomPrompts AS cp5 
+	                    RIGHT OUTER JOIN DelegateAccounts AS da
+	                    RIGHT OUTER JOIN SupervisorDelegates AS sd 
+	                    INNER JOIN AdminUsers AS au 
+		                    ON sd.SupervisorAdminID = au.AdminID 
+	                    INNER JOIN Centres AS ct 
+		                    ON au.CentreID = ct.CentreID 
+	                    ON da.CentreID = ct.CentreID 
+		                    AND da.UserID = sd.DelegateUserID 
+	                    LEFT OUTER JOIN Users AS u 
+	                    LEFT OUTER JOIN JobGroups AS jg 
+		                    ON u.JobGroupID = jg.JobGroupID
+	                    ON da.UserID = u.ID 
+	                    LEFT OUTER JOIN CustomPrompts AS cp1 
+		                    ON ct.CustomField1PromptID = cp1.CustomPromptID 
+	                    LEFT OUTER JOIN CustomPrompts AS cp2 
+		                    ON ct.CustomField2PromptID = cp2.CustomPromptID 
+	                    LEFT OUTER JOIN CustomPrompts AS cp3 
+		                    ON ct.CustomField3PromptID = cp3.CustomPromptID 
+	                    LEFT OUTER JOIN CustomPrompts AS cp4 
+		                    ON ct.CustomField4PromptID = cp4.CustomPromptID 
+	                    ON cp5.CustomPromptID = ct.CustomField5PromptID 
+	                    ON cp6.CustomPromptID = ct.CustomField6PromptID 
+	                    LEFT OUTER JOIN AdminAccounts AS au2 
+		                    ON da.UserID = au2.UserID AND da.CentreID = au2.CentreID
+                    WHERE (sd.SupervisorAdminID = @adminId) AND (sd.Removed IS NULL) AND 
+                     (u.ID = da.UserID OR sd.DelegateUserID IS NULL)) AS StaffCount,
                          (SELECT COUNT(ID) AS StaffCount
                          FROM SupervisorDelegates AS SupervisorDelegates_1
                          WHERE (SupervisorAdminID = @adminId)

DigitalLearningSolutions.Data/Models/Frameworks/FrameworkCompetency.cs

@@ -14,6 +14,7 @@ public class FrameworkCompetency : BaseSearchableItem
         public int AssessmentQuestions { get; set; }
         public int CompetencyLearningResourcesCount { get; set; }
         public string? FrameworkName { get; set; }
+        public bool? AlwaysShowDescription { get; set; }
 
         public override string SearchableName
         {

DigitalLearningSolutions.Data/Models/Frameworks/Import/BulkCompetency.cs

@@ -0,0 +1,19 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace DigitalLearningSolutions.Data.Models.Frameworks.Import
+{
+    public class BulkCompetency
+    {
+        public int? ID { get; set; }
+        public string? CompetencyGroup { get; set; }
+        public string? GroupDescription { get; set; }
+        public string? Competency { get; set; }
+        public string? CompetencyDescription { get; set; }
+        public bool? AlwaysShowDescription { get; set; }
+        public string? FlagsCsv { get; set; }
+    }
+}

DigitalLearningSolutions.Data/Models/Frameworks/Import/CompetencyTableRow.cs

@@ -1,53 +1,69 @@
 namespace DigitalLearningSolutions.Data.Models.Frameworks.Import
 {
     using ClosedXML.Excel;
-    using System;
-    using System.Collections.Generic;
-    using System.Text;
+
     public enum RowStatus
     {
         NotYetProcessed,
         Skipped,
-        CompetencyGroupInserted,
         CompetencyGroupAndCompetencyInserted,
-        CompetencyInserted
+        CompetencyInserted,
+        CompetencyUpdated,
+        CompetencyGroupInserted,
+        CompetencyGroupUpdated,
+        CompetencyGroupAndCompetencyUpdated,
+        InvalidAlwaysShowDescription,
+        InvalidId
     }
-    public class CompetencyTableRow
+    public class CompetencyTableRow : BulkCompetency
     {
         public CompetencyTableRow(IXLTable table, IXLRangeRow row)
         {
             string? FindFieldValue(string name)
             {
-                var colNumber = table.FindColumn(col => col.FirstCell().Value.ToString()?.ToLower() == name).ColumnNumber();
+                var colNumber = table.FindColumn(col => col.FirstCell().Value.ToString()?.ToLower() == name.ToLower()).ColumnNumber();
                 return row.Cell(colNumber).GetValue<string?>();
             }
 
             RowNumber = row.RowNumber();
-            CompetencyGroupName = FindFieldValue("competency group");
-            CompetencyName = FindFieldValue("competency name");
-            CompetencyDescription = FindFieldValue("competency description");
+            ID = row.Cell(1).GetValue<int?>();
+            CompetencyGroup = row.Cell(2).GetValue<string?>();
+            GroupDescription = row.Cell(3).GetValue<string?>();
+            Competency = row.Cell(4).GetValue<string?>();
+            CompetencyDescription = row.Cell(5).GetValue<string?>();
+            AlwaysShowDescriptionRaw = FindFieldValue("AlwaysShowDescription");
+            AlwaysShowDescription = bool.TryParse(AlwaysShowDescriptionRaw, out var hasPrn) ? hasPrn : (bool?)null;
+            FlagsCsv = FindFieldValue("FlagsCSV");
             RowStatus = RowStatus.NotYetProcessed;
         }
         public int RowNumber { get; set; }
-        public string? CompetencyGroupName { get; set; }
-        public string? CompetencyName { get; set; }
-        public string? CompetencyDescription { get; set; }
+        public int CompetencyOrderNumber { get; set; }
+        public string? AlwaysShowDescriptionRaw { get; set; }
         public ImportCompetenciesResult.ErrorReason? Error { get; set; }
         public RowStatus RowStatus { get; set; }
+        public bool Reordered { get; set; } = false;
         public bool Validate()
         {
-            if (string.IsNullOrEmpty(CompetencyName))
+            if (string.IsNullOrEmpty(Competency))
             {
                 Error = ImportCompetenciesResult.ErrorReason.MissingCompetencyName;
             }
-            else if (CompetencyGroupName?.Length > 255)
+            else if (CompetencyGroup?.Length > 255)
             {
                 Error = ImportCompetenciesResult.ErrorReason.TooLongCompetencyGroupName;
             }
-            else if (CompetencyName.Length > 500)
+            else if (Competency.Length > 500)
             {
                 Error = ImportCompetenciesResult.ErrorReason.TooLongCompetencyName;
             }
+            else if (!string.IsNullOrWhiteSpace(AlwaysShowDescriptionRaw) && !bool.TryParse(AlwaysShowDescriptionRaw, out _))
+            {
+                Error = ImportCompetenciesResult.ErrorReason.InvalidAlwaysShowDescription;
+            }
+            else if (RowStatus == RowStatus.InvalidId)
+            {
+                Error = ImportCompetenciesResult.ErrorReason.InvalidId;
+            }
 
             return !Error.HasValue;
         }

DigitalLearningSolutions.Data/Models/Frameworks/Import/ImportCompetenciesResult.cs

@@ -8,10 +8,11 @@ public class ImportCompetenciesResult
     {
         public enum ErrorReason
         {
+            InvalidId,
             MissingCompetencyName,
-            TooLongCompetencyGroupName,
             TooLongCompetencyName,
-            AlreadyExists
+            TooLongCompetencyGroupName,
+            InvalidAlwaysShowDescription
         }
         public ImportCompetenciesResult() { }
 
@@ -20,16 +21,39 @@ IReadOnlyCollection<CompetencyTableRow> competencyTableRows
         )
         {
             ProcessedCount = competencyTableRows.Count;
-            CompetenciesInsertedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.CompetencyInserted | dr.RowStatus == RowStatus.CompetencyGroupAndCompetencyInserted);
-            CompetencyGroupsInsertedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.CompetencyGroupInserted | dr.RowStatus == RowStatus.CompetencyGroupAndCompetencyInserted);
-            SkippedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.Skipped);
+            CompetencyAddedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.CompetencyInserted | dr.RowStatus == RowStatus.CompetencyGroupAndCompetencyInserted);
+            CompetencyUpdatedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.CompetencyUpdated | dr.RowStatus == RowStatus.CompetencyGroupAndCompetencyUpdated);
+            GroupAddedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.CompetencyGroupInserted | dr.RowStatus == RowStatus.CompetencyGroupAndCompetencyInserted);
+            SkippedCount = competencyTableRows.Count(dr => dr.RowStatus == RowStatus.Skipped && dr.Reordered == false);
+            CompetencyReorderedCount = competencyTableRows.Count(dr => dr.Reordered == true);
             Errors = competencyTableRows.Where(dr => dr.Error.HasValue).Select(dr => (dr.RowNumber, dr.Error!.Value));
+            FlagCount = competencyTableRows
+                .Where(row => !string.IsNullOrWhiteSpace(row.FlagsCsv))
+                .SelectMany(static row => row.FlagsCsv.Split(',', StringSplitOptions.RemoveEmptyEntries))
+                .Count();
+            DistinctFlagsCount = competencyTableRows
+                .Where(row => !string.IsNullOrWhiteSpace(row.FlagsCsv))
+                .SelectMany(row => row.FlagsCsv.Split(',', StringSplitOptions.RemoveEmptyEntries))
+                .Select(flag => flag.Trim())
+                .Distinct()
+                .Count();
+            CompetencyGroupCount = competencyTableRows
+                .Where(row => !string.IsNullOrWhiteSpace(row.CompetencyGroup))
+                .Select(static row => row.CompetencyGroup)
+                .Distinct()
+                .Count();
         }
 
         public IEnumerable<(int RowNumber, ErrorReason Reason)>? Errors { get; set; }
         public int ProcessedCount { get; set; }
-        public int CompetenciesInsertedCount { get; set; }
-        public int CompetencyGroupsInsertedCount { get; set; }
+        public int CompetencyAddedCount { get; set; }
+        public int CompetencyUpdatedCount { get; set; }
+        public int CompetencyReorderedCount { get; set; }
+        public int GroupAddedCount { get; set; }
+        public int GroupUpdatedCount { get; set; }
         public int SkippedCount { get; set; }
+        public int FlagCount { get; set; }
+        public int DistinctFlagsCount { get; set; }
+        public int CompetencyGroupCount { get; set; }
     }
 }

DigitalLearningSolutions.Web.Tests/Controllers/Frameworks/FrameworkControllerTests.cs

@@ -12,6 +12,8 @@
     using Microsoft.Extensions.Logging;
     using NUnit.Framework;
     using GDS.MultiPageFormData;
+    using DigitalLearningSolutions.Data.Utilities;
+    using Microsoft.AspNetCore.Hosting;
 
     public partial class FrameworkControllerTests
     {
@@ -28,6 +30,8 @@ public partial class FrameworkControllerTests
         private ILearningHubApiClient learningHubApiClient = null!;
         private ISearchSortFilterPaginateService searchSortFilterPaginateService = null!;
         private IMultiPageFormService multiPageFormService = null!;
+        private IClockUtility clockUtility = null!;
+        private IWebHostEnvironment webHostEnvironment = null!;
 
         [SetUp]
         public void SetUp()
@@ -42,7 +46,8 @@ public void SetUp()
             learningHubApiClient = A.Fake<ILearningHubApiClient>();
             searchSortFilterPaginateService = A.Fake<ISearchSortFilterPaginateService>();
             multiPageFormService = A.Fake<IMultiPageFormService>();
-
+            clockUtility = A.Fake<ClockUtility>();
+            webHostEnvironment = A.Fake<IWebHostEnvironment>();
             A.CallTo(() => config["CurrentSystemBaseUrl"]).Returns(BaseUrl);
 
             var user = new ClaimsPrincipal(
@@ -65,7 +70,9 @@ public void SetUp()
                 competencyLearningResourcesService,
                 learningHubApiClient,
                 searchSortFilterPaginateService,
-                multiPageFormService
+                multiPageFormService,
+                clockUtility,
+                webHostEnvironment
             )
             {
                 ControllerContext = new ControllerContext { HttpContext = new DefaultHttpContext { User = user } },

DigitalLearningSolutions.Web.Tests/ServiceFilter/RedirectToErrorEmptySessionDataTests.cs

@@ -22,6 +22,8 @@
     using NUnit.Framework;
     using GDS.MultiPageFormData;
     using GDS.MultiPageFormData.Enums;
+    using DigitalLearningSolutions.Data.Utilities;
+    using Microsoft.AspNetCore.Hosting;
 
     public class RedirectToErrorEmptySessionDataTests
     {
@@ -41,7 +43,9 @@ public void Setup()
                         A.Fake<ICompetencyLearningResourcesService>(),
                         A.Fake<ILearningHubApiClient>(),
                         A.Fake<ISearchSortFilterPaginateService>(),
-                        A.Fake<IMultiPageFormService>()
+                        A.Fake<IMultiPageFormService>(),
+                        A.Fake<IClockUtility>(),
+                        A.Fake<IWebHostEnvironment>()
                     )
                     .WithDefaultContext().WithMockTempData();
 

DigitalLearningSolutions.Web/Controllers/FrameworksController/Competencies.cs

@@ -71,7 +71,7 @@ public IActionResult AddEditFrameworkCompetencyGroup(int frameworkId, Competency
              (competencyGroupBase.Description), adminId);
                 return new RedirectResult(Url.Action("ViewFramework", new { tabname = "Structure", frameworkId }) + "#fcgroup-" + frameworkCompetencyGroupId.ToString());
             }
-            var newCompetencyGroupId = frameworkService.InsertCompetencyGroup(competencyGroupBase.Name, SanitizerHelper.SanitizeHtmlData(competencyGroupBase.Description), adminId);
+            var newCompetencyGroupId = frameworkService.InsertCompetencyGroup(competencyGroupBase.Name, SanitizerHelper.SanitizeHtmlData(competencyGroupBase.Description), adminId, frameworkId);
             if (newCompetencyGroupId > 0)
             {
                 var newFrameworkCompetencyGroupId = frameworkService.InsertFrameworkCompetencyGroup(newCompetencyGroupId, frameworkId, adminId);

DigitalLearningSolutions.Web/Controllers/FrameworksController/Frameworks.cs

@@ -716,6 +716,20 @@ public IActionResult ViewFramework(string tabname, int frameworkId, int? framewo
             return View("Developer/Framework", model);
         }
 
+        [Route("/Framework/{frameworkId}/Structure/PrintLayout")]
+        public IActionResult PrintLayout(int frameworkId) {
+            var adminId = GetAdminId();
+            var detailFramework = frameworkService.GetFrameworkDetailByFrameworkId(frameworkId, adminId);
+            var routeData = new Dictionary<string, string> { { "frameworkId", detailFramework?.ID.ToString() } };
+            var model = new FrameworkViewModel()
+            {
+                DetailFramework = detailFramework,
+            };
+            model.FrameworkCompetencyGroups = frameworkService.GetFrameworkCompetencyGroups(frameworkId).ToList();
+            model.CompetencyFlags = frameworkService.GetCompetencyFlagsByFrameworkId(frameworkId, null, selected: true);
+            model.FrameworkCompetencies = frameworkService.GetFrameworkCompetenciesUngrouped(frameworkId);
+            return View("Developer/FrameworkPrintLayout", model);
+        }
         [ResponseCache(CacheProfileName = "Never")]
         public IActionResult InsertFramework()
         {

DigitalLearningSolutions.Web/Controllers/FrameworksController/FrameworksController.cs

@@ -10,6 +10,8 @@
     using Microsoft.AspNetCore.Mvc;
     using Microsoft.Extensions.Logging;
     using GDS.MultiPageFormData;
+    using DigitalLearningSolutions.Data.Utilities;
+    using Microsoft.AspNetCore.Hosting;
 
     [Authorize(Policy = CustomPolicies.UserFrameworksAdminOnly)]
     [SetDlsSubApplication(nameof(DlsSubApplication.Frameworks))]
@@ -25,7 +27,8 @@ public partial class FrameworksController : Controller
         private readonly ILearningHubApiClient learningHubApiClient;
         private readonly ISearchSortFilterPaginateService searchSortFilterPaginateService;
         private readonly IMultiPageFormService multiPageFormService;
-
+        private readonly IClockUtility clockUtility;
+        private readonly IWebHostEnvironment webHostEnvironment;
         public FrameworksController(
             IFrameworkService frameworkService,
             ICommonService commonService,
@@ -35,7 +38,9 @@ public FrameworksController(
             ICompetencyLearningResourcesService competencyLearningResourcesService,
             ILearningHubApiClient learningHubApiClient,
             ISearchSortFilterPaginateService searchSortFilterPaginateService,
-            IMultiPageFormService multiPageFormService
+            IMultiPageFormService multiPageFormService,
+            IClockUtility clockUtility,
+            IWebHostEnvironment webHostEnvironment
         )
         {
             this.frameworkService = frameworkService;
@@ -47,6 +52,8 @@ IMultiPageFormService multiPageFormService
             this.learningHubApiClient = learningHubApiClient;
             this.searchSortFilterPaginateService = searchSortFilterPaginateService;
             this.multiPageFormService = multiPageFormService;
+            this.clockUtility = clockUtility;
+            this.webHostEnvironment = webHostEnvironment;
         }
 
         private int? GetCentreId()