Syndetic Solutions added content uses incorrect URL for Kirkus Reviews

Bug #984039 reported by Jeff Godin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

OpenILS::WWW::AddedContent::Syndetic uses an incorrect URL for Kirkus Reviews.

The correct "filename" portion of the URL for retrieving Kirkus Reviews is KIREVIEW.HTML or KIREVIEW.XML

Tested and confirmed with Bowker technical support as an error in their documentation, which they will fix.

See comment for a working branch that fixes this.

Revision history for this message
Jeff Godin (jgodin) wrote :

Fixed in branch user/jeff/lp984039_fix_syndetic_kirkus

This bugfix should be a simple (and useful) backport to all currently maintained versions.

tags: added: pullrequest
Revision history for this message
Michael Peters (mrpeters) wrote :

We installed this last evening @ Evergreen Indiana and it's working perfectly.

Thank you, Jeff, for making sure we're getting our money's worth :)

You can see it in action @ http://evergreen.lib.in.us/opac/en-US/skin/default/xml/rdetail.xml?r=11036621&t=harry potter stone&tp=keyword&d=0&f=at&hc=61&rt=keyword --- Click on "Reviews"

user/mrpeters-isl/lp984039_fix_syndetic_kirkus_signoff
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=31e9000b1d2455af073ab1265b6d7c117ab7542b

tags: added: signedoff
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Changed in evergreen:
milestone: none → 2.2.0beta3
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Pushed to master back through rel_2_0. Thanks all!

Changed in evergreen:
status: Confirmed → In Progress
status: In Progress → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.