Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When I insert a video link from YouTube with the Video Filter Button in CKEditor, I get the following message:
"Strict warning: Only variables should be passed by reference in video_filter_dashboard_page() (line 387 of ...\sites\all\modules\video_filter\video_filter.module)."
Line 387 is the following:
print theme('video_filter_dashboard', array('form' => render(drupal_get_form('_video_filter_form'))));
I only have this problem on one site, on the other it never occured.
Comment | File | Size | Author |
---|---|---|---|
#3 | strict-warning-message-1662746-3.patch | 472 bytes | btopro |
Comments
Comment #1
joel_osc CreditAttribution: joel_osc commentedI am getting the same message...I think render requires a variable, so if you split it to lines as follows the message goes away:
Comment #2
acranius CreditAttribution: acranius commentedThanks, the module still works and the message doesn't appear anymore.
Setting status to closed (fixed).
Comment #3
btopro CreditAttribution: btopro commentedWriting about a solution in a thread and marking it fixed without a patch seems a poor way of handling a current error. I'm still reproducing this in the latest version of drupal and latest version of module. Here's a patch of the solution above, which does work just think it should make it into the module.
Comment #4
smira CreditAttribution: smira commentedclean patch fixes bug, please include in next release.
Thank you!
Comment #5
prston CreditAttribution: prston commentedPatch provided on comment #3 was not included in further releases.
So, the strict warning always appear on 3.1 on line 435 (same file). Comment #1 still works on this line.
Comment #6
smira CreditAttribution: smira commentedswitching status and priority hoping for commit
Comment #7
Yaron Tal CreditAttribution: Yaron Tal commentedBump
Patch fixes strict warning for me as well. Still applies to dev version.
Only difference in the patch is that the line number increased by one.
Comment #8
ultimateboy CreditAttribution: ultimateboy commentedThanks all. Committed.
Comment #11
SuperTyp CreditAttribution: SuperTyp commentedI still get this error in 7.31
Strict warning: Only variables should be passed by reference in video_filter_dashboard_page() (line 435 /sites/all/modules/video_filter/video_filter.module)
isn't it commited yet?
Comment #12
Northern_Girl CreditAttribution: Northern_Girl commentedI got the exact same message as SuperTyp. What is the status on this?
Thanks.