LP#1208613: Reset lineitem cache when resetting table
authorDan Wells <dbw2@calvin.edu>
Thu, 7 Jan 2016 16:47:43 +0000 (11:47 -0500)
committerGalen Charlton <gmc@esilibrary.com>
Mon, 14 Mar 2016 21:42:34 +0000 (17:42 -0400)
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>
Open-ILS/web/js/ui/default/acq/common/li_table.js

index 68d9a67..1f02110 100644 (file)
@@ -360,6 +360,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 = {};