Revert "LP#1208613: Reset lineitem cache when resetting table"
authorBill Erickson <berickxx@gmail.com>
Mon, 12 Jun 2017 21:19:02 +0000 (17:19 -0400)
committerBill Erickson <berickxx@gmail.com>
Thu, 21 Mar 2019 19:46:23 +0000 (15:46 -0400)
Avoid clearing the lineitem cache on each page of lineitems.  We need
the cache to for multi-page selection-to-PO support.  If the original
bugs addressed by this commit resurface, we can resolve it by
maintaining 2 caches, the stock liCache and a separate parallel cache
used just for multi-page actions.

https://bugs.launchpad.net/evergreen/+bug/1208613

https://kcevergreen.atlassian.net/browse/JBAS-673

This reverts commit 341babf860a20539529e29db252545031b01ed5f.

Open-ILS/web/js/ui/default/acq/common/li_table.js

index 3804f0c..a50eb98 100644 (file)
@@ -443,7 +443,6 @@ function AcqLiTable() {
     this.reset = function(keep_selectors) {
         while(self.tbody.childNodes[0])
             self.tbody.removeChild(self.tbody.childNodes[0]);
-        self.liCache = {};
         self.noteAcks = {};
         self.relCache = {};