I started getting this error in the log after upgrading the server to PHP 5.3. Drupal core and most modules were affected: #360605: PHP 5.3 Compatibility.

I fixed it simply by removing the reference on the $node argument of this function.

Comments

infojunkie’s picture

infojunkie
Location Vancouver
commented

Save for other providers including YouTube and blip.tv.

marktheshark’s picture

marktheshark commented
Priority: Normal » Critical

Still an issue with the latest development snapshot (6.x-1.x-dev).

Embedded youtube videos will not play, and the reference warning is shown within the page after a refresh.

Has this fix been commited?

aaron’s picture

aaron commented
Status: Active » Fixed

awesome, thanks @kratib! i fixed this for all the providers now. sorry, i missed this during my last round.

marktheshark’s picture

marktheshark commented
Status: Fixed » Needs work

It appears the fix is not in the latest snapshot available:

warning: Parameter 6 to emvideo_youtube_video() expected to be a reference, value given in C:\xampp\htdocs\drupal\sites\all\modules\emfield\emfield.module on line 593.
aaron’s picture

aaron commented
Status: Needs work » Fixed

snapshots are created every 12 hours. if you need it before that, you need to checkout directly from cvs.

aaron’s picture

aaron commented
Priority: Critical » Normal

certainly not critical

marktheshark’s picture

marktheshark commented

Well, considering that the module didn't work, I considered it a critical error.

Thanks for the information

aaron’s picture

aaron commented
Priority: Normal » Critical

ah, good point. sorry, @marktheshark :D

Status: Fixed » Closed (fixed)
Issue tags: -PHP 5.3

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

Issue tags: +PHP 5.3

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.