From ad7e1b91f8c433dd737da404d8ba1f24183ac32b Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@esilibrary.com>
Date: Mon, 26 Jan 2015 19:54:41 +0000
Subject: [PATCH] LP#1402797 make it "delete selected" rather than "delete all"

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 Open-ILS/src/templates/staff/cat/bucket/copy/index.tt2  | 2 +-
 Open-ILS/src/templates/staff/cat/bucket/copy/t_view.tt2 | 2 +-
 Open-ILS/web/js/ui/default/staff/cat/bucket/copy/app.js | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/Open-ILS/src/templates/staff/cat/bucket/copy/index.tt2 b/Open-ILS/src/templates/staff/cat/bucket/copy/index.tt2
index 8a81f6d13a..259beea6f0 100644
--- a/Open-ILS/src/templates/staff/cat/bucket/copy/index.tt2
+++ b/Open-ILS/src/templates/staff/cat/bucket/copy/index.tt2
@@ -12,7 +12,7 @@
 <script>
   angular.module('egCoreMod').run(['egStrings', function(s) {
     s.CONFIRM_DELETE_COPY_BUCKET_ITEMS_FROM_CATALOG =
-      "[% l('Are you sure you want to delete all items in bucket from catalog?') %]";
+      "[% l('Are you sure you want to delete selected items in bucket from catalog?') %]";
   }])
 </script>
 [% END %]
diff --git a/Open-ILS/src/templates/staff/cat/bucket/copy/t_view.tt2 b/Open-ILS/src/templates/staff/cat/bucket/copy/t_view.tt2
index e9446bd171..07e5f23f86 100644
--- a/Open-ILS/src/templates/staff/cat/bucket/copy/t_view.tt2
+++ b/Open-ILS/src/templates/staff/cat/bucket/copy/t_view.tt2
@@ -12,7 +12,7 @@
 
   <eg-grid-action label="[% l('Remove Selected Copies') %]" 
     handler="detachCopies"></eg-grid-action>
-  <eg-grid-action label="[% l('Delete All from Catalog') %]" 
+  <eg-grid-action label="[% l('Delete Selected Copies from Catalog') %]" 
     handler="deleteCopiesFromCatalog"></eg-grid-action>
 
   <eg-grid-field path="id" required hidden></eg-grid-field>
diff --git a/Open-ILS/web/js/ui/default/staff/cat/bucket/copy/app.js b/Open-ILS/web/js/ui/default/staff/cat/bucket/copy/app.js
index 3a9bf0ef07..8f6e3f654c 100644
--- a/Open-ILS/web/js/ui/default/staff/cat/bucket/copy/app.js
+++ b/Open-ILS/web/js/ui/default/staff/cat/bucket/copy/app.js
@@ -449,19 +449,19 @@ function($scope,  $q , $routeParams,  bucketSvc, egCore,
         return $q.all(promises).then(drawBucket);
     }
 
-    $scope.deleteCopiesFromCatalog = function() {
+    $scope.deleteCopiesFromCatalog = function(copies) {
         egConfirmDialog.open(
             egCore.strings.CONFIRM_DELETE_COPY_BUCKET_ITEMS_FROM_CATALOG,
             '', {}
         ).result.then(function() {
             var fleshed_copies = [];
             var promises = [];
-            angular.forEach(bucketSvc.currentBucket.items(), function(i) {
+            angular.forEach(copies, function(i) {
                 promises.push(
                     egCore.net.request(
                         'open-ils.search',
                         'open-ils.search.asset.copy.fleshed2.retrieve',
-                        i.target_copy()
+                        i.id
                     ).then(function(copy) {
                         copy.ischanged(1);
                         copy.isdeleted(1);
-- 
2.11.0