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.
+1
Created attachment 11807 [details] [review] Bug 8679 - 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. The Amazon module is removed. Two other system preferences, AWSAccessKeyID and AWSPrivateKey are removed as they were required only by Amazon.pm. Handling of book cover images from Amazon is unchanged.
Created attachment 11808 [details] [review] [SIGNED-OFF] Bug 8679 - 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. The Amazon module is removed. Two other system preferences, AWSAccessKeyID and AWSPrivateKey are removed as they were required only by Amazon.pm. 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.
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
Created attachment 11863 [details] [review] 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.
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.
Marked as Passed QA
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