From 6c76986dc79f368b971a0bb49263c3ee62c8c7e4 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berick@esilibrary.com>
Date: Fri, 21 Jan 2011 16:09:07 -0500
Subject: [PATCH] use path-based bib IDs instead of GET params

---
 Open-ILS/src/perlmods/OpenILS/WWW/EGCatLoader.pm | 7 +++++--
 Open-ILS/web/templates/default/opac/results.tt2  | 2 +-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/src/perlmods/OpenILS/WWW/EGCatLoader.pm b/Open-ILS/src/perlmods/OpenILS/WWW/EGCatLoader.pm
index 98eff625d3..754616c13a 100644
--- a/Open-ILS/src/perlmods/OpenILS/WWW/EGCatLoader.pm
+++ b/Open-ILS/src/perlmods/OpenILS/WWW/EGCatLoader.pm
@@ -3,7 +3,7 @@ use strict; use warnings;
 use CGI;
 use XML::LibXML;
 use Digest::MD5 qw(md5_hex);
-use Apache2::Const -compile => qw(OK DECLINED HTTP_INTERNAL_SERVER_ERROR REDIRECT);
+use Apache2::Const -compile => qw(OK DECLINED HTTP_INTERNAL_SERVER_ERROR REDIRECT HTTP_BAD_REQUEST);
 use OpenSRF::AppSession;
 use OpenSRF::EX qw/:try/;
 use OpenSRF::Utils::Logger qw/$logger/;
@@ -317,8 +317,11 @@ sub load_rresults {
 sub load_rdetail {
     my $self = shift;
 
+    my $rec_id = $self->ctx->{page_args}->[0]
+        or return Apache2::Const::HTTP_BAD_REQUEST;
+
     $self->ctx->{record} = $self->editor->retrieve_biblio_record_entry([
-        $self->cgi->param('record'),
+        $rec_id,
         {
             flesh => 2, 
             flesh_fields => {
diff --git a/Open-ILS/web/templates/default/opac/results.tt2 b/Open-ILS/web/templates/default/opac/results.tt2
index a0e13a8b87..956e4eaac5 100644
--- a/Open-ILS/web/templates/default/opac/results.tt2
+++ b/Open-ILS/web/templates/default/opac/results.tt2
@@ -87,7 +87,7 @@
                     </td>
                     <td width='auto'>
                         <div>
-                            <a href='rdetail?record=[% rec.bre.id %]'>[% attrs.title %]</a>
+                            <a href='rdetail/[% rec.bre.id %]'>[% attrs.title %]</a>
                             <span style='padding-left:10px;'>[% rec.copy_counts.available %] / [% rec.copy_counts.visible %]</span>
                         </div>
                         <div>[% attrs.author %]</div>
-- 
2.43.2