Comment 2 for bug 1318029

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Bill's fix above appears to work OK for us. It fixes most of our Content Cafe problems. However, we do believe it will break OpenLibrary because OpenLibrary only accepts ISBN.

I am attaching a patch that actually applies and makes another slight change of moving the return check for the missing keyhash values inside the try block. This adds another check, but makes them more specific to what we're looking for.

Along the way, we've discovered that Evergreen appears to be using the testing API for Content Cafe. Their integration guide suggests doing a lot of other things, including send all of the identifiers for a record at once.

We think the module ought to be refactored to take advantage of the full API, but of course there's always the question of tuits.