From 5a36e4a0148fa22ad12cd75479b4636f8f9c74e4 Mon Sep 17 00:00:00 2001 From: Dan Wells Date: Thu, 7 Jan 2016 11:47:43 -0500 Subject: [PATCH] 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 Signed-off-by: Erica Rohlfs Signed-off-by: Galen Charlton --- Open-ILS/web/js/ui/default/acq/common/li_table.js | 1 + 1 file changed, 1 insertion(+) 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 68d9a672d6..1f0211005e 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 @@ -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 = {}; -- 2.43.2