From: Mike Rylander <mrylander@gmail.com>
Date: Fri, 4 Sep 2015 19:32:35 +0000 (-0400)
Subject: webstaff: Add confirmation before deleting copies/volumes
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2e9b5949998f189d044f897b8059a9b411b72f46;p=evergreen%2Fmasslnc.git

webstaff: Add confirmation before deleting copies/volumes

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
---

diff --git a/Open-ILS/src/templates/staff/cat/catalog/index.tt2 b/Open-ILS/src/templates/staff/cat/catalog/index.tt2
index 26c450f000..9635c5d994 100644
--- a/Open-ILS/src/templates/staff/cat/catalog/index.tt2
+++ b/Open-ILS/src/templates/staff/cat/catalog/index.tt2
@@ -16,6 +16,15 @@
 <script src="[% ctx.media_prefix %]/js/ui/default/staff/circ/services/holds.js"></script>
 [% INCLUDE 'staff/circ/share/hold_strings.tt2' %]
 <script src="[% ctx.media_prefix %]/js/ui/default/staff/cat/catalog/app.js"></script>
+<script>
+  angular.module('egCoreMod').run(['egStrings', function(s) {
+    s.CONFIRM_DELETE_COPIES_VOLUMES =
+      "[% l('Permanently delete selected copies and/or volumes from catalog?') %]";
+    s.CONFIRM_DELETE_COPIES_VOLUMES_MESSAGE =
+      "[% l('Will delete {{copies}} copies and {{volumes}} volumes') %]";
+  }])
+</script>
+
 [% END %]
 
 <div ng-view></div>
diff --git a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
index 1f77a0985f..4177943b0c 100644
--- a/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/catalog/app.js
@@ -226,9 +226,9 @@ function($scope , $routeParams , $location , $window , $q , egCore) {
 }])
 
 .controller('CatalogCtrl',
-       ['$scope','$routeParams','$location','$window','$q','egCore','egHolds','egCirc',
+       ['$scope','$routeParams','$location','$window','$q','egCore','egHolds','egCirc','egConfirmDialog',
         'egGridDataProvider','egHoldGridActions','$timeout','$modal','holdingsSvc','egUser',
-function($scope , $routeParams , $location , $window , $q , egCore , egHolds , egCirc, 
+function($scope , $routeParams , $location , $window , $q , egCore , egHolds , egCirc,  egConfirmDialog,
          egGridDataProvider , egHoldGridActions , $timeout , $modal , holdingsSvc , egUser) {
 
     // set record ID on page load if available...
@@ -494,15 +494,20 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e
         var cnHash = {};
         var perCnCopies = {};
 
+        var cn_count = 0;
+        var cp_count = 0;
+
         angular.forEach(
             $scope.holdingsGridControls.selectedItems(),
             function (item) {
                 if (vols && item.raw_call_number) {
                     cnHash[item.call_number.id] = egCore.idl.Clone(item.raw_call_number);
                     cnHash[item.call_number.id].isdeleted(1);
+                    cn_count++;
                 } else if (copies) {
                     angular.forEach(egCore.idl.Clone(item.raw), function (cp) {
                         cp.isdeleted(1);
+                        cp_count++;
                         var cn_id = cp.call_number().id();
                         if (!cnHash[cn_id]) {
                             cnHash[cn_id] = cp.call_number();
@@ -518,7 +523,10 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e
         );
 
         angular.forEach(perCnCopies, function (v, k) {
-            if (vols) cnHash[k].isdeleted(1);
+            if (vols) {
+                cnHash[k].isdeleted(1);
+                cn_count++;
+            }
             cnHash[k].copies(v);
         });
 
@@ -529,12 +537,18 @@ function($scope , $routeParams , $location , $window , $q , egCore , egHolds , e
 
         if (cnList.length == 0) return;
 
-        egCore.net.request(
-            'open-ils.cat',
-            'open-ils.cat.asset.volume.fleshed.batch.update.override',
-            egCore.auth.token(), cnList, 1, {}
-        ).then(function(update_count) {
-            $scope.holdingsGridDataProvider.refresh();
+        egConfirmDialog.open(
+            egCore.strings.CONFIRM_DELETE_COPIES_VOLUMES,
+            egCore.strings.CONFIRM_DELETE_COPIES_VOLUMES_MESSAGE,
+            {copies : cp_count, volumes : cn_count}
+        ).result.then(function() {
+            egCore.net.request(
+                'open-ils.cat',
+                'open-ils.cat.asset.volume.fleshed.batch.update.override',
+                egCore.auth.token(), cnList, 1, {}
+            ).then(function(update_count) {
+                $scope.holdingsGridDataProvider.refresh();
+            });
         });
     }
     $scope.selectedHoldingsCopyDelete = function () { $scope.selectedHoldingsDelete(false,true) }