[open-ils-commits] [GIT] Evergreen ILS branch master updated. f3ad285e081b1bae0e14d328943b510bd34c9d44

[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, master has been updated
       via  f3ad285e081b1bae0e14d328943b510bd34c9d44 (commit)
      from  dfd1fd9db6f56ac3c9e413b980c0a61f0424b28a (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 f3ad285e081b1bae0e14d328943b510bd34c9d44
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