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

[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  0765ecb89776374b692413a41285ed7001e7f07b (commit)
      from  767a65660d0b6f5a3b04243e0d7b6270f3c89b63 (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 0765ecb89776374b692413a41285ed7001e7f07b
Author: Dan Scott <dan at coffeecode.net>
Date:   Sun Apr 8 18:13:26 2012 -0400

    TPAC: Protect against requests for non-existent records
    
    If a record request is received for a record that does not exist (say,
    if 100 records have been loaded and someone submits a request for
    /eg/opac/record/399), rather than dying with a server error, return
    immediately.
    
    Eventually we'll want to provide an explicit "record was not
    found error" in the TPAC record details; for now the TPAC just displays
    zero bib detail.
    
    Signed-off-by: Dan Scott <dan at coffeecode.net>
    Signed-off-by: Thomas Berezansky <tsbere at mvlc.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
index 6031eac..a283f41 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
@@ -235,6 +235,10 @@ sub get_records_and_facets {
         success_handler => sub {
             my($self, $req) = @_;
             my $data = $req->{response}->[0]->content;
+
+            # Protect against requests for non-existent records
+            return unless $data->{'unapi.bre'};
+
             my $xml = XML::LibXML->new->parse_string($data->{'unapi.bre'})->documentElement;
 
             # Protect against legacy invalid MARCXML that might not have a 901c

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

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


hooks/post-receive
-- 
Evergreen ILS