From 6ecef85471542bd308d0041e098bd58a7e216455 Mon Sep 17 00:00:00 2001
From: Andrew Cain <macite@gmail.com>
Date: Wed, 24 May 2023 23:17:18 +1000
Subject: [PATCH] fix: portfolio task list performance

---
 src/app/common/filters/filters.coffee                  |  5 +++++
 .../tasks/project-tasks-list/project-tasks-list.coffee | 10 ++++++++++
 .../project-tasks-list/project-tasks-list.tpl.html     |  6 +++---
 src/app/units/states/portfolios/portfolios.tpl.html    |  8 --------
 4 files changed, 18 insertions(+), 11 deletions(-)

diff --git a/src/app/common/filters/filters.coffee b/src/app/common/filters/filters.coffee
index 7ff6a2ac1..8511c42c2 100644
--- a/src/app/common/filters/filters.coffee
+++ b/src/app/common/filters/filters.coffee
@@ -246,6 +246,11 @@ angular.module("doubtfire.common.filters", [])
     )
 )
 
+.filter('tasksForGroupset', ->
+  (input, groupSet) ->
+    return input unless input?
+    return input.filter (task) -> (task.definition.groupSet == groupSet) || (!task.definition.groupSet? && !groupSet?)
+)
 
 .filter('paginateAndSort', ($filter) ->
   (input, pagination, tableSort) ->
diff --git a/src/app/tasks/project-tasks-list/project-tasks-list.coffee b/src/app/tasks/project-tasks-list/project-tasks-list.coffee
index 5c91742bb..5e1dc4fd5 100644
--- a/src/app/tasks/project-tasks-list/project-tasks-list.coffee
+++ b/src/app/tasks/project-tasks-list/project-tasks-list.coffee
@@ -18,6 +18,16 @@ angular.module('doubtfire.tasks.project-tasks-list', [])
   controller: ($scope, $modal, newTaskService, analyticsService, gradeService) ->
     analyticsService.event 'Student Project View', "Showed Task Button List"
 
+    $scope.groupTasks = []
+
+    $scope.groupTasks.push.apply $scope.groupTasks, $scope.unit.groupSets.map (gs) ->
+      {
+        groupSet: gs,
+        name: gs.name
+      }
+
+    $scope.groupTasks.push {groupSet: null, name: 'Individual Work'}
+
     # functions from task service
     $scope.statusClass = newTaskService.statusClass
     $scope.statusText = newTaskService.statusText
diff --git a/src/app/tasks/project-tasks-list/project-tasks-list.tpl.html b/src/app/tasks/project-tasks-list/project-tasks-list.tpl.html
index 2dbbdbd49..2ef977a13 100644
--- a/src/app/tasks/project-tasks-list/project-tasks-list.tpl.html
+++ b/src/app/tasks/project-tasks-list/project-tasks-list.tpl.html
@@ -1,7 +1,7 @@
 <ul class="project-tasks-list">
-  <div ng-repeat="(groupSetId, relatedTasks) in project.tasks | groupBy: 'definition.groupSet.id'" class="group-set-tasks clearfix">
-    <h5 class="group-set-name" ng-hide="hideGroupSetName">{{groupSetName(groupSetId)}}</h5>
-    <li class="col-xs-4 col-lg-3" ng-repeat="task in relatedTasks | orderBy: ['definition.targetGrade','definition.seq']">
+  <div ng-repeat="grouping in groupTasks" class="group-set-tasks clearfix">
+    <h5 class="group-set-name" ng-hide="hideGroupSetName">{{grouping.name}}</h5>
+    <li class="col-xs-4 col-lg-3" ng-repeat="task in project.tasks | tasksForGroupset: grouping.groupSet | orderBy: ['definition.targetGrade','definition.seq']">
       <button type="button" class="col-xs-12 btn task-status {{statusClass(task.status)}}" ng-click="onSelect(task, 'Task Button List')" tooltip-popup-delay='200' tooltip-placement="bottom" tooltip-html-unsafe="<strong>{{task.definition.name}}:</strong><br>{{task.statusLabel()}}">
         <i class="fa fa-eye" ng-show="task.similarToCount - task.similarToDismissedCount > 0"></i> {{taskText(task)}}
       </button>
diff --git a/src/app/units/states/portfolios/portfolios.tpl.html b/src/app/units/states/portfolios/portfolios.tpl.html
index 9337672fc..144c56381 100644
--- a/src/app/units/states/portfolios/portfolios.tpl.html
+++ b/src/app/units/states/portfolios/portfolios.tpl.html
@@ -217,7 +217,6 @@ <h4 class="panel-title" ng-hide="selectedStudent">Portfolio Details</h4>
       <div class="panel-heading clearfix">
         <div class="pull-left">
           <h3 class="panel-title">Review Progress of {{selectedStudent.student.name}}</h3>
-          <!--/title-->
           Review the students progress through the unit's tasks.
         </div>
       </div>
@@ -230,7 +229,6 @@ <h3 class="panel-title">Review Progress of {{selectedStudent.student.name}}</h3>
       <div class="panel-heading clearfix">
         <div class="pull-left">
           <h3 class="panel-title">Review Portfolio of {{selectedStudent.student.name}}</h3>
-          <!--/title-->
           View or download portfolio for assessment.
         </div>
       </div>
@@ -266,12 +264,6 @@ <h4 class="panel-title">Grade for {{selectedStudent.student.name}}</h4>
             <div class="small" ng-hide="project.gradeRationale">Click to add one</div>
           </div>
           <div ng-if="editingRationale" class="clearfix">
-            <!-- <markdown-editor
-              height="200"
-              ng-model="project.gradeRationale"
-              placeholder="Edit rationale..."
-              autofocus="true">
-            </markdown-editor> -->
             <div
               contenteditable
               ng-model="project.gradeRationale"