Hi.

Love your code. It's so f* streamlined..

I think:

  foreach (module_implements('_amazon_item_clean_xml') as $module) {
    $function = $module .'_amazon_item_clean_xml';
    $function($item, $xml);
  }

should say:

  foreach (module_implements('amazon_item_clean_xml') as $module) {
    $function = $module .'_amazon_item_clean_xml';
    $function($item, $xml);
  }

Cheers,

Atle

Comments

eaton’s picture

eaton commented
Status: Active » Fixed

Awesome, thanks! I've fixed and will be rolling beta-2 this weekend.

Anonymous’s picture

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

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

osherl’s picture

osherl commented

Great module, just the right amount of features...

I am using the amazon_item_clean_xml hook to load additional elements and it works only if if I pass the $item variable by reference as follows:

function example_amazon_item_clean_xml(&$item, $xml) {
...

Is this the correct way of using the hook?
Thanks,
Laszlo

MGParisi’s picture

MGParisi commented

osherl, you posted to a closed item so your question will probably get ignored:(

Try making another post