The amazon_search module is great, but the results are a little generic. I propose using theme('amazon_item') to display the results.

Also, like the asin module, the amazon api search should use "All" not "Blended" because it gives better results that way.

Patch forthcoming.

CommentFileSizeAuthor
#2 amazon_search_results-1321850-2.patch1.4 KBshawn dearmond
#1 amazon_search_results-1321850-1.patch1.43 KBshawn dearmond

Comments

shawn dearmond’s picture

shawn dearmond commented
Status: Active » Needs review
StatusFileSize
new amazon_search_results-1321850-1.patch1.43 KB

Here's a patch.

shawn dearmond’s picture

shawn dearmond commented
StatusFileSize
new amazon_search_results-1321850-2.patch1.4 KB

Here's a better patch. Changed ResponseGroup to Large instead of what it was before.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Fixed

I think it's quite nice.

Committed to 7.x-1.x: ef9ce6f

Now that you're back, I've also given you commit privileges and would love to have you be increasingly active here.

The things I ask of committers:
* Subscribe to all issues (so you know right away about problems)
* Work transparently in the issue queue; make sure there's an issue for everything you do and that people can follow it. Even if you can't get anybody to review, still do it all in public.
* Put your git hash for commits in the issue when fixing it.

Thanks so much!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.