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

[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  136e731a0dcd6379b7cbbe9111f8a17edb7f96de (commit)
      from  70b406c7a60cfd2417cbc3b08e089e94dfed3697 (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 136e731a0dcd6379b7cbbe9111f8a17edb7f96de
Author: Galen Charlton <gmc at equinoxinitiative.org>
Date:   Tue Jan 17 16:58:39 2017 -0500

    LP#1657282: fix redirect of one-hit metarecord searches
    
    This patches fixes a bug where, when performing a metarecord
    ("Group Formats and Editions") search, one-hit result sets
    get redirected to a "/eg/opac/record/0.0..." page, which
    results in a "Bad Request" error being shown to the user.
    
    To test:
    
    [1] Perform a metarecord search that would result in exactly
        one metarecord search. Observe that the browser displays
        a "Bad Request" error page.
    [2] Apply the patch and perform step 1 again. This time, the
        search should succeed. Note that if the metarecord contains
        more than one component, a results page with one grouped
        hit is displayed; if the metarecord has exactly one component,
        that bib's record page is displayed.
    
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>

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 7ee9cfa..48c218a 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
@@ -496,9 +496,9 @@ sub load_rresults {
         my $stat = 0;
         if ($is_meta) {
             # if the MR has a single constituent record, it will
-            # be in array position 2 of the result blob.
+            # be in array position 4 of the result blob.
             # otherwise, we don't want to redirect anyway.
-            my $bre_id = $results->{ids}->[0]->[2];
+            my $bre_id = $results->{ids}->[0]->[4];
             $stat = $self->check_1hit_redirect([$bre_id]) if $bre_id;
         } else {
             my $stat = $self->check_1hit_redirect($rec_ids);

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

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


hooks/post-receive
-- 
Evergreen ILS