KMAIN 1836 Ability to remove copies
authorBill Erickson <berickxx@gmail.com>
Tue, 29 Dec 2015 18:29:38 +0000 (13:29 -0500)
committerBill Erickson <berickxx@gmail.com>
Thu, 21 Mar 2019 19:46:23 +0000 (15:46 -0400)
Removed reverse method from dojo query, use query to grab last
(item count variable) items instead

Signed-off-by: Michael Glass <mglass@catalystitservices.com>
Cross-ported by Bill.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Open-ILS/web/js/ui/default/acq/common/li_table.js

index fa68955..957995c 100644 (file)
@@ -2456,7 +2456,7 @@ function AcqLiTable() {
             // delete rows if necessary
             var diff = self.copyCount() - count;
             if(diff > 0) {
-                var rows = dojo.query('tr', self.copyTbody).reverse().slice(0, diff);
+                var rows = dojo.query('tr', self.copyTbody).slice(diff); // Fixed Here
                 if(confirm(dojo.string.substitute(localeStrings.DELETE_LI_COPIES_CONFIRM, [diff]))) {
                     dojo.forEach(rows, function(row) {self.deleteCopy(row); });
                 } else {