Added content by record ID

[Jeff Godin]

This is a wishlist bug.

Some added content sources support keys other than ISBN, such as UPC.

Local image overrides are currently only possible with ISBN. In some cases, this means that a fake ISBN needs to be added to a record in order to supply a local image.

There is benefit to be had in being able to request AC by record ID, and having the AC handlers extract any keys they like. Extracting ISBN and UPC makes sense for most, and can be done in a common place.

The idea is to implement in a way which is backward compatible -- not breaking any previously working URLs. If desired, an option may be supplied in the future to turn off the old-style URLs.

An example, for a jacket image:

old style: /opac/extras/ac/jacket/medium/ISBN
new style: /opac/extras/ac/jacket/medium/r/RECORD_ID

Changes will affect OpenILS::WWW::AddedContent, some/all modules under that namespace, and the OPAC.

Updated working branch to follow.

[Michael Peters (mrpeters)]

I've had the pleasure of testing this out and it is FANTASTIC! The coverage for things like DVD's, CD's, etc. really goes up within our Syndetics subscription. So much more added content will become available to patrons with this code and a quality AC provider.

[Yamil (ysuarez)]

Do you have a public "working" git branch I could look at so I can hack one of my test VMs to test out this code? I tried looking, but I don't think I found it.

Thanks in advance,
Yamil

[Jeff Godin (jgodin)]

Updated working branch is user/jeff/ac_by_record_id_2013 located here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeff/ac_by_record_id_2013

[Jeff Godin (jgodin)]

Still needed before pullrequest: release notes and an Apache config change to support 1x1 blank.png on 404 for jackets in general.

[Jeff Godin (jgodin)]

Added Apache config examples and release notes and force-pushed rebased branch to user/jeff/ac_by_record_id_2013_rebase

Adding pullrequest tag.

[Ben Shum (bshum)]

Been using the UPC stuff for quite awhile now, and the ISSN stuff looks to be fine as well.

Thanks to Yamil for providing Syndetics help and testing as well. And thanks Jeff for the fine features.

Pushed to master at last!