Attachment 36010 Details for Bug 12858 – Bug 12858: Add error handling to Syndetics Index

[patch] Bug 12858: Add error handling to Syndetics Index

Bug-12858-Add-error-handling-to-Syndetics-Index.patch (text/plain), 1.51 KB, created by Chris Cormack on 2015-02-19 08:44:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2015-02-19 08:44:55 UTC

Size: 1.51 KB

patch

obsolete

>From e385ffda01a9478117fe81d9ed2469f9cb5369a2 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Tue, 2 Sep 2014 09:14:03 +0000
>Subject: [PATCH] Bug 12858: Add error handling to Syndetics Index
>
>* Syndetics routines include a statement to check that the returned
>  content from Syndetics is xml. The get_syndetics_index routine was
>  missing this check and so when a 'not found' html page was returned
>  the opac-detail page would take a long time to load whilst xml::simple
>  attempted to parse the large html document.
>
>  Test Plan
>  1. Enable Syndetics indexes on opac.
>  2. Head over to an opac-detail page on the opac.
>  3. Remove the ISBN from the same item in the staff client.
>  4. Reload the opac-detail page for the item noticeing a much slower
>  page load.
>  5. Apply the patch
>  6. Reload the opac-detail page for the item and note that it now laods
>  in a reasonable timescale again.
>
>Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
>---
> C4/External/Syndetics.pm | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/C4/External/Syndetics.pm b/C4/External/Syndetics.pm
>index 3d518da..2d14674 100644
>--- a/C4/External/Syndetics.pm
>+++ b/C4/External/Syndetics.pm
>@@ -66,6 +66,9 @@ sub get_syndetics_index {
>     my ( $isbn,$upc,$oclc ) = @_;
> 
>     my $response = _fetch_syndetics_content('INDEX.XML', $isbn, $upc, $oclc);
>+    unless ($response->content_type =~ /xml/) {
>+        return;
>+    }
> 
>     my $content = $response->content;
>     my $xmlsimple = XML::Simple->new();
>-- 
>2.1.0

Actions: View | Diff | Splinter Review

Attachments on bug 12858: 31318 | 36010 | 36072 | 36073 | 36074