Bug 8679

Summary: Remove usage of Amazon API
Product: Koha Reporter: Owen Leonard <oleonard>
Component: Architecture, internals, and plumbingAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: jonathan.druart+koha, katrin.fischer, nengard, paul.poulain
Version: 3.10   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 8679 - Remove usage of Amazon API
[SIGNED-OFF] Bug 8679 - Remove usage of Amazon API
Bug 8679 [REVISED] Remove usage of Amazon API
[SIGNED-OFF] Bug 8679 [REVISED] Remove usage of Amazon API

Description Owen Leonard 2012-08-23 17:21:14 UTC
Koha hasn't been updated to use the new Amazon API and no one appears to want to do so. Anyone choosing to undertake the task would appear to do so at their own risk, given the Amazon.com Product Advertising API License Agreement (https://affiliate-program.amazon.com/gp/advertising/api/detail/agreement.html) says that Amazon must approve each system's request for API access and:

"Unsuitable applications include those that: ... do not have as their principal purpose advertising and marketing the Amazon Site and driving sales of products and services on the Amazon Site."

Instead of continuing to ship new versions of Koha with broken Amazon integration I think we should remove the functionality which depends on the API.
Comment 1 Katrin Fischer 2012-08-23 17:52:08 UTC
Comment 2 Owen Leonard 2012-08-23 19:37:13 UTC Comment hidden (obsolete)
Comment 3 Nicole C. Engard 2012-08-23 19:49:03 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2012-08-27 12:34:22 UTC
QA Comments:
All seems good.
But there is 1 occurrence left of C4::External::Amazon in the unit test : t/db_dependent/Amazon.t

Marked as Failed QA
Comment 5 Owen Leonard 2012-08-28 12:33:09 UTC Comment hidden (obsolete)
Comment 6 Nicole C. Engard 2012-08-28 12:42:21 UTC
Created attachment 11864 [details] [review]
[SIGNED-OFF] Bug 8679 [REVISED] Remove usage of Amazon API

This patch removes the AmazonReviews and AmazonSimilarItems
features from the OPAC and staff client. With on Amazon
feature remaining, cover images, the *AmazonEnabled preference
is also removed in favor of checking the *AmazonCoverImages
preference. Two other system preferences, AWSAccessKeyID and
AWSPrivateKey are removed as they were required only by the
removed features.

Handling of book cover images from Amazon is unchanged.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>

Turned on amazon covers in opac and staff client and all
worked as expected. Then tested to make sure other cover image
services still worked and they do.

Signing off.
Comment 7 Jonathan Druart 2012-08-28 13:10:24 UTC
Marked as Passed QA
Comment 8 Paul Poulain 2012-08-29 15:47:19 UTC
Patch pushed. I suggest that it's not backported to 3.8 even it's it's a bugfix, because some hidden problem could be hard to detect.

I checked that there is no more occurrences of the removed sysprefs or subs, could not find any problem