[open-ils-commits] [GIT] Evergreen ILS branch rel_2_11 updated. 5da54ca031ee81e348c68e6a96d239838de33bee

[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_2_11 has been updated
       via  5da54ca031ee81e348c68e6a96d239838de33bee (commit)
      from  6badeb7a7ce8ab4172808343b1e88e068d1fc524 (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 5da54ca031ee81e348c68e6a96d239838de33bee
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