From: Dan Wells <dbw2@calvin.edu>
Date: Thu, 7 Jan 2016 16:47:43 +0000 (-0500)
Subject: LP#1208613: Reset lineitem cache when resetting table
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=b2047e5dc4f19bbfc61628dc52698e3970bd5aa6;p=Evergreen.git

LP#1208613: Reset lineitem cache when resetting table

Stale cache entries in this table were causing the "expand all"
functionality to break.  This might be a nuclear option, but as far as
I could see, this "cache" gets repopulated on every pagination anyway,
so I don't think we're losing anything.

To test:

[1] Select a large list from the Acquisitions --> My Selection List
    screen.
[2] Click on the Expand All button and the items on the page expand
    with more information.
[3] Click on the Next link to bring up page 2 of the list.
[4] Click on the Expand All button and verify that additional
    information is displayed.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Erica Rohlfs <erohlfs@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---

diff --git a/Open-ILS/web/js/ui/default/acq/common/li_table.js b/Open-ILS/web/js/ui/default/acq/common/li_table.js
index af33b034cb..455caafb31 100644
--- a/Open-ILS/web/js/ui/default/acq/common/li_table.js
+++ b/Open-ILS/web/js/ui/default/acq/common/li_table.js
@@ -359,6 +359,7 @@ 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 = {};