Bug 12858: Add error handling to Syndetics Index
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Tue, 2 Sep 2014 09:14:03 +0000 (09:14 +0000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Fri, 20 Feb 2015 13:22:46 +0000 (10:22 -0300)
commitacc7da6a5f93e12704acdfd7981481c9ef07904b
treef93e6cdf4eff388fffe117bbeed8565b95e51445
parenta303cdec4871d3d3740d419c972eea7faf8e5a8a
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>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/External/Syndetics.pm