Posted by infojunkie on October 6, 2009 at 11:03pm
Jump to:
| Project: | Embedded Media Field |
| Version: | 6.x-1.9 |
| Component: | Embedded Video Field |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | PHP 5.3 |
Issue Summary
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
#1
Save for other providers including YouTube and blip.tv.
#2
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?
#3
awesome, thanks @kratib! i fixed this for all the providers now. sorry, i missed this during my last round.
#4
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.#5
snapshots are created every 12 hours. if you need it before that, you need to checkout directly from cvs.
#6
certainly not critical
#7
Well, considering that the module didn't work, I considered it a critical error.
Thanks for the information
#8
ah, good point. sorry, @marktheshark :D
#9
Automatically closed -- issue fixed for 2 weeks with no activity.