Fix: View users org dropdown not consistently populating with selecte…

…d organization (#150)

* Prevent duplicates in dropdown from the view instead of in template

* Remove unused import from testing

apcd-cms/src/apps/view_users/templates/view_users.html

@@ -63,15 +63,6 @@ <h1>View Users</h1>
   window.location.href = `/administration/view-users/?filter=${filter}`;
   window.location.load();
 }
-
-/* remove duplicates from dropdown */
-
-const options = []
-
-document.querySelectorAll('#organizationFilter > option').forEach((option) => {
-    if (options.includes(option.value)) option.remove()
-    else options.push(option.value)
-})
 </script>
 
 {% endblock %}

apcd-cms/src/apps/view_users/views.py

@@ -42,7 +42,9 @@ def _set_user(usr):
         table_entries = []
         for user in user_content:
             table_entries.append(_set_user(user,))
-            context['filter_options'].append(user[4])
+            org_name = user[4]
+            if org_name not in context['filter_options']:  # prevent duplicates
+                context['filter_options'].append(user[4])
 
         filter = self.request.GET.get('filter')
 
@@ -53,5 +55,5 @@ def _set_user(usr):
 
         context.update(paginator(self.request, table_entries))
         context['pagination_url_namespaces'] = 'administration:view_users'
-        
+
         return context