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

[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  fc9b7ea11520aaf9f188e682daa9066d91870a1c (commit)
      from  b7d2ca71c16b892104538902efd8c7e501229efd (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 fc9b7ea11520aaf9f188e682daa9066d91870a1c
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date:   Tue May 30 15:51:23 2017 -0400

    LP#1694497: fix record links on 2nd+ page of grouped results
    
    This patch fixes a bug wherein the second (and subsequent)
    pages of search results in the public catalog (when group
    formats and editions is in effect) does not correctly generate
    links to hits that are not part of of a multi-bib metarecords.
    
    To test
    -------
    1. Keyword search concerto for "piano"
    2. Check the box for "Group formats and Editions"
    3. Click Page 2
    4. Click any result that is not a group (missing the parentheses after
       the title). You will get redirected to the my account page.
    5. Apply the patch and do steps #1-4 again. This time, you should
       end up on the record details page.
    
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Jason Boyer <jboyer at library.in.gov>
    Signed-off-by: Dan Wells <dbw2 at calvin.edu>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
index 40d63fb..72faa1c 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
@@ -516,7 +516,12 @@ sub load_rresults {
                 my $ses = OpenSRF::AppSession->create('open-ils.search');
                 $self->timelog("Firing off the multiclass query");
                 $args->{from_metarecord} = $rec_id;
+                # offset of main search does not apply to the MR
+                # constituents query
+                my $save_offset = $args->{offset};
+                $args->{offset} = 0;
                 my $req = $ses->request($method, $args, $query, 1);
+                $args->{offset} = $save_offset;
                 $meta_results = $req->gather(1);
                 $self->timelog("Returned from the multiclass query");
 

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

Summary of changes:
 .../perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)


hooks/post-receive
-- 
Evergreen ILS