[open-ils-commits] [GIT] Evergreen ILS branch rel_3_0 updated. e9e0b4e0998ea304b55b1fa5e974880a76c5f01e

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_3_0 has been updated
       via  e9e0b4e0998ea304b55b1fa5e974880a76c5f01e (commit)
      from  f8e98c9d481e603971e854b094dd3e9f2bd38e8b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e9e0b4e0998ea304b55b1fa5e974880a76c5f01e
Author: Jason Stephenson <jason at sigio.com>
Date:   Thu Apr 12 10:47:40 2018 -0400

    Lp 1735539: Fix Item Status ability to delete multiple copies.
    
    Use the notify handler instead of the resolve handler to collect our
    copies.  This ensures we get them all instead of just the last one.
    
    To reproduce the bug:
       1. Open Item Status in the web staff client.
       2. Load multiple items, perhaps by uploading a file of barcodes.
       3. Select more than 1 item from the list.
       4. Choose "Delete Items" from the Actions menu.
       5. Note that the dialog indicates that you will delete 1 copy.
       6. Click the Cancel button in the dialog.
    
    To test, after applying the fix branch:
       1. Open Item Status in the web staff client.
       2. Load multiple items, perhaps by uploading a file of barcodes.
       3. Select more than 1 item from the list.
       4. Choose "Delete Items" from the Actions menu.
       5. Note that that the dialog indicates you will delete X copies,
          where X is the number of items selected.
       6. Click OK and confirm that all copies were deleted.
       7. You can confirm 6 by reloading the interface and uploading the
          file of barcodes again.
    
    Signed-off-by: Jason Stephenson <jason at sigio.com>
    Signed-off-by: Jason Boyer <jboyer at library.in.gov>
    Signed-off-by: Dan Wells <dbw2 at calvin.edu>

diff --git a/Open-ILS/web/js/ui/default/staff/circ/services/item.js b/Open-ILS/web/js/ui/default/staff/circ/services/item.js
index 59d92a8..be05ed5 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/services/item.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/services/item.js
@@ -494,9 +494,7 @@ function(egCore , egCirc , $uibModal , $q , $timeout , $window , egConfirmDialog
         egCore.pcrud.search('acp',
             {deleted : 'f', id : items.map(function(el){return el.id;}) },
             { flesh : 1, flesh_fields : { acp : ['call_number'] } }
-        ).then(function(copy) {
-            copy_objects.push(copy);
-        }).then(function() {
+        ).then(function() {
 
             var cnHash = {};
             var perCnCopies = {};
@@ -546,6 +544,10 @@ function(egCore , egCirc , $uibModal , $q , $timeout , $window , egConfirmDialog
                     angular.forEach(items, function(cp){service.add_barcode_to_list(cp.barcode)});
                 });
             });
+        },
+        null,
+        function(copy) {
+            copy_objects.push(copy);
         });
     }
 

-----------------------------------------------------------------------

Summary of changes:
 .../web/js/ui/default/staff/circ/services/item.js  |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
Evergreen ILS