The patch attached fixes the following problems:

  • Do not try to get image size (fails on some systems)
  • Invoke theme_amazon_filter through theme(), not directly, so it can be overloaded
  • Make filter work for more than one amazon filter tag in the body text
  • Make it output at least something, that is: use $product properties that actually exist
  • Get rid of the warning when no author exists

Additionally I'd like to say that the absence of any README or documentation made it rather hard to discover how this module works.

CommentFileSizeAuthor
amazon_filter.diff1.6 KBgerd riesselmann

Comments

willfe’s picture

willfe commented

Just a quick thumbs-up. This patch fixed every issue listed; the module's actually usable now :) Thanks!

mariuss’s picture

mariuss commented

I run into these same issues, unfortunately I found this patch only after I fixed them myself.

Please apply patch ASAP.

eaton’s picture

eaton commented

Both the 4.7 versions and the new 5 versions have been updated, though no official release is yet out. Check http://drupal.org/node/118902 and http://drupal.org/node/96504 for the latest code.

Apologies on the long wait for these patches; the module was originally released a a quick hack and I haven't had much time to update or even look at it. Recent work on the ASIN module got me back into the code, though.

eaton’s picture

eaton commented
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)