LP1798628: Copy alert counts in Holdings user/jboyer/lp1798628_holdings_alert_count
authorJason Boyer <jboyer@library.in.gov>
Thu, 17 Jan 2019 18:33:10 +0000 (13:33 -0500)
committerJason Boyer <jboyer@library.in.gov>
Thu, 17 Jan 2019 18:33:10 +0000 (13:33 -0500)
Currently the count of all copy alerts ever applied
to a copy is displayed in the Holdings view, which
can be quite confusing when clicking Manage and not
seeing the expected number of alerts. This branch
changes the display to only count the number of
events currently active since those are the only
ones available to manage.

Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Open-ILS/web/js/ui/default/staff/cat/services/holdings.js

index 476dbe2..f5e9665 100644 (file)
@@ -127,7 +127,9 @@ function(egCore , $q) {
 
                 // create virtual field for copy alert count
                 angular.forEach(svc.copies, function (cp) {
-                    if (cp.copy_alerts) cp.copy_alert_count = cp.copy_alerts.length;
+                    if (cp.copy_alerts) {
+                        cp.copy_alert_count = cp.copy_alerts.filter(function(aca) { return aca.ack_time == null ;}).length;
+                    }
                     else cp.copy_alert_count = 0;
                 });