Skip to content

Commit 7eeca89

Browse files
authored
Merge pull request #2761 from TechnologyEnhancedLearning/Develop/Features/TD-4324-IncludeOptionalintheFilteroptionsonthelearnercompetenciesmenu
TD-4324 Include "Optional" in the Filter options on the learner competencies menu
2 parents dd87364 + f76b8ba commit 7eeca89

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

DigitalLearningSolutions.Data/Enums/SelfAssessmentCompetencyFilter.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
{
33
public enum SelfAssessmentCompetencyFilter
44
{
5+
Optional = -11,
56
AwaitingConfirmation = -10,
67
PendingConfirmation = -9,
78
RequiresSelfAssessment = -8,

DigitalLearningSolutions.Web/Helpers/SupervisorCompetencyFilterHelper.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ private static void ApplyResponseStatusFilters(ref IEnumerable<Competency> compe
4848
|| (filters.Contains((int)SelfAssessmentCompetencyFilter.ConfirmationRejected) && c.AssessmentQuestions.Any(q => q.Verified.HasValue && q.SignedOff != true))
4949
|| (filters.Contains((int)SelfAssessmentCompetencyFilter.AwaitingConfirmation) && c.AssessmentQuestions.Any(q => q.Verified == null && q.Requested != null && q.UserIsVerifier == true))
5050
|| (filters.Contains((int)SelfAssessmentCompetencyFilter.Verified) && c.AssessmentQuestions.Any(q => q.Verified.HasValue && q.SignedOff == true))
51+
|| (filters.Contains((int)SelfAssessmentCompetencyFilter.Optional) && c.Optional )
5152
where (wordsInSearchText.Count() == 0 || searchTextMatchesGroup || searchTextMatchesCompetencyDescription || searchTextMatchesCompetencyName)
5253
&& (!appliedResponseStatusFilters.Any() || responseStatusFilterMatchesAnyQuestion)
5354
select c;
@@ -111,6 +112,7 @@ public static bool IsResponseStatusFilter(int filter)
111112
{
112113
var responseStatusFilters = new int[]
113114
{
115+
(int)SelfAssessmentCompetencyFilter.Optional,
114116
(int)SelfAssessmentCompetencyFilter.RequiresSelfAssessment,
115117
(int)SelfAssessmentCompetencyFilter.SelfAssessed,
116118
(int)SelfAssessmentCompetencyFilter.Verified,

0 commit comments

Comments
 (0)