[open-ils-commits] [GIT] Evergreen ILS branch rel_3_4 updated. c8fe0607192688421c796d3c1590189533659ed4

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_3_4 has been updated
       via  c8fe0607192688421c796d3c1590189533659ed4 (commit)
      from  8844d3265eb661c358c65b0eef57acc1ea7f3416 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit c8fe0607192688421c796d3c1590189533659ed4
Author: Jane Sandberg <sandbej at linnbenton.edu>
Date:   Tue Jul 30 13:54:10 2019 -0700

    LP1607922: Receiving items on page 2 of search results
    
    Previously, if a user selected a bunch of line items on two consecutive
    pages of acq search results, and tried to receive items from the second
    page, the operation would fail.
    
    This issue was caused by Evergreen caching IDs of selected line items
    from page 1, but not caching the data that went along with them.
    
    To test:
    
    1) Go to Acquisitions > General Search
    2) In the Select Search Field dropdown, choose LI - State.
    3) Set the state to on-order.
    4) Click Search.
    5) Click the check mark to select all results on the first page.
    6) Click the Next button.
    7) Click the check mark to select all results on the second page.
    8) On the top --Actions-- menu, select Mark Selected Line Items as
    Received.
    9) Note that the interface does nothing, and that there is a TypeError
    error message in the browser console.
    10) Apply this commit.
    11) Refresh your page completely and repeat steps 1-8.
    12) Note that the items on the second page are received.  Since they no
    longer meet the search criteria, Evergreen gives you the next page of
    results.
    
    For further testing, see similar test steps by John Yorio at
    https://bugs.launchpad.net/evergreen/+bug/1607922.
    
    Signed-off-by: Jane Sandberg <sandbej at linnbenton.edu>
    Signed-off-by: Chris Sharp <csharp at georgialibraries.org>

diff --git a/Open-ILS/web/js/ui/default/acq/common/li_table_pager.js b/Open-ILS/web/js/ui/default/acq/common/li_table_pager.js
index de24dcba68..02e4ef0825 100644
--- a/Open-ILS/web/js/ui/default/acq/common/li_table_pager.js
+++ b/Open-ILS/web/js/ui/default/acq/common/li_table_pager.js
@@ -26,7 +26,7 @@ function LiTablePager() {
     };
 
     this.show = function() {
-        this.liTable.reset(/* keep_selectors */ true);
+        this.liTable.reset();
         this.liTable.show("list");
     };
 

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/web/js/ui/default/acq/common/li_table_pager.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
Evergreen ILS