Skip to content

Instantly share code, notes, and snippets.

@jeff

jeff/gist:944503

Created April 27, 2011 15:40
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save jeff/944503 to your computer and use it in GitHub Desktop.
Save jeff/944503 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
gistfile1.txt
Enabling OpenILS::WWW::AddedContent to return cover images based on more than just ISBN
Supporting legacy without making the New Awesome have to bend over backward
* make AddedContent superclass look for a use-time argument to enable "legacy" lookup-by-isbn support
* change apache handler to use legacy option
* create new apache handler at new url to use new default of lookup-by-record-id
for records: AC_HOST/opac/extras/ac/jacket/small/r/RECORD_ID
for metarecords: AC_HOST/opac/extras/ac/jacket/small/m/METARECORD_ID
Enabling lookups by record id
* modify opac to request at new url and with new argument -- no longer need to sanitize isbn
* modify AddedContent.pm superclass to look up keys (currently isbn and adding upc), sanitize
* make any required changes to modules Syndetics/ContentCafe/etc to support passing additional keys
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment