after moving drupal site from ubuntu 8.04 to ubuntu 10.04 sapo module began complaining with:

warning: Parameter 6 to emvideo_sapo_video() expected to be a reference, value given in /web/site/sites/all/modules/emfield/emfield.module on line 622.

CommentFileSizeAuthor
#1 media_sapo-remove_call_by_reference_in_emvideo_sapo_video-923446.1.patch553 bytesdxvargas

Comments

dxvargas’s picture

dxvargas commented
Status: Active » Needs review
StatusFileSize
new media_sapo-remove_call_by_reference_in_emvideo_sapo_video-923446.1.patch553 bytes

I confirm this error, that was causing the videos to not being displayed.
The following patch fixes the bug (right?).

I just removed the "&" from variable $node.
And also introduced $options variable, as in media_youtube module, because emfield_include_invoke() in emfield.module is sending one more argument then the previous existing ones.