From 8a35f5696ec923c960ba207bbd2fdba4854aec95 Mon Sep 17 00:00:00 2001
From: Claire Peters <clairepeters@g.harvard.edu>
Date: Wed, 22 Nov 2023 15:29:47 -0800
Subject: [PATCH] active users only on projectuser table

---
 .../templates/project/project_detail.html     | 35 +++----------------
 coldfront/core/project/views.py               |  3 +-
 2 files changed, 7 insertions(+), 31 deletions(-)

diff --git a/coldfront/core/project/templates/project/project_detail.html b/coldfront/core/project/templates/project/project_detail.html
index 1da913b84..86f4f3585 100644
--- a/coldfront/core/project/templates/project/project_detail.html
+++ b/coldfront/core/project/templates/project/project_detail.html
@@ -97,8 +97,8 @@ <h3 class="d-inline"><i class="fas fa-file-invoice-dollar" aria-hidden="true"></
             <th scope="col">Resource Type</th>
             <th scope="col">Location</th>
             <th scope="col">User Count</th>
-            <th scope="col">Space (TB)</th>
-            <th scope="col">Used (TB)</th>
+            <th scope="col">Space</th>
+            <th scope="col">Used</th>
             <th scope="col">Monthly Cost</th>
             <th scope="col" class="nosort">Actions</th>
           </tr>
@@ -642,36 +642,11 @@ <h3 class="d-inline" id="research_outputs"><i class="far fa-newspaper" aria-hidd
     });
 
     $('#projectuser_table').DataTable({
-       'lengthMenu': [[10, 25, 50, -1], [10, 25, 50, 'All']],
-       'pageLength': 25,
-       'aoColumnDefs': [{
-         'bSortable': false,
-         'aTargets': ['nosort']
-       }],
+       "lengthMenu": [[10, 25, 50, -1], [10, 25, 50, "All"]],
+       "pageLength": 25,
        dom: 'B<"clear">lfrtip',
        order: [[ 3, "desc" ]],
-       buttons: [
-        {
-          name: 'toggleusers',
-          text: function() {
-              return $('#projectuser_table').attr('filter') == "on" ? 'Show All Users' : "Show Active Users"
-            },
-          action: function(e, dt, node, config) {
-            var table = $('#projectuser_table');
-            var filter = table.attr('filter') === "on" ? 'off' : "on";
-            document.querySelector('#projectuser_table').setAttribute('filter', filter);
-            if (filter == 'on') {
-              $.fn.dataTable.ext.search.push(
-                function(settings, data, dataIndex) {
-                    return $(dt.row(dataIndex).node()).attr('status') == "Active";
-                  });
-            } else {$.fn.dataTable.ext.search.pop();}
-            table.DataTable().draw();
-            this.text(filter == 'on' ? 'Show All Users' : "Show Active Users")
-          }
-        }
-      ]}
-    );
+    });
 
     $('#invoice_table').DataTable({
       'aoColumnDefs': [{
diff --git a/coldfront/core/project/views.py b/coldfront/core/project/views.py
index 085aef9b9..143d4e89c 100644
--- a/coldfront/core/project/views.py
+++ b/coldfront/core/project/views.py
@@ -180,7 +180,8 @@ def get_context_data(self, **kwargs):
             attributes_with_usage.remove(a)
 
         # Only show 'Active Users'
-        project_users = self.object.projectuser_set.order_by('user__username')
+        project_users = self.object.projectuser_set.filter(
+                    status__name='Active').order_by('user__username')
 
         context['mailto'] = 'mailto:' + ','.join([u.user.email for u in project_users])