I'm evaluating Drupal for use in our company, working through the "Using Drupal" O'Reilly book.

Mac: 10.6.2
Apache: 2.2.13
PHP: 5.3.0
mySQL: 5.1.40
Drupal: 6.14
Amazon Module: 6.x-1.0-rc1

Every page using the Amazon Module has the following warning displayed:

warning: Parameter 1 to amazon_media_amazon_item_load() expected to be a reference, value given in /Library/WebServer/Documents/includes/module.inc on line 471.

I had a similar warning in regards to the CSS Injector module and Mac/Safari and it involved removing an '&' in a function. Is there a quick fix for this issue?

Comments

rfay’s picture

Thanks for the report - I try to make sure none of this happens, and run with E_NOTIFY on all the time, and haven't seen it.

Please give me some details:

What version of PHP are you running? (You can attach the output of phpinfo if you'd like)
What version of Drupal?
What kind of page are you viewing? (You say "every page" but I'll need to know if that's every page on a drupal site, or what).

rfay’s picture

Title: amazon_media_amazon_item_load()...line 471 » PHP 5.3 compatibility

Sorry - just saw your attached information.

I haven't tried running PHP 5.3 yet. The vast majority of Drupal is not yet ready for PHP 5.3. Core just had a patch that mostly fixes the issues, and most of contrib does not.

rfay’s picture

OK, you were right - just an & that didn't need to be.

Committed to HEAD: will be in next dev version. Please test and make sure your issue is resolved.

http://drupal.org/cvs?commit=303876

Thanks for the report!

rfay’s picture

Status: Active » Fixed
dfellner’s picture

Made the fix to amazon_media.module and the warning is gone.

Thanks for your help!

Status: Fixed » Closed (fixed)

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