Closed (fixed)
Project:
Video Filter
Version:
6.x-2.4
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Oct 2008 at 22:35 UTC
Updated:
2 Apr 2009 at 12:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
elanghout commentedI moved the code below (in video_filter.module):
// Pick random out of multiple sources separated by ','
if (strstr($video['source'], ',')) {
$sources = explode(',', $video['source']);
$random = array_rand($sources, 1);
$video['source'] = $sources[$random];
}
to just below:
function video_filter_process($text, $format = -1) {
if (preg_match_all('/\[video(\:(.+))?( .+)?\]/isU', $text, $matches_code)) {
foreach ($matches_code[0] as $ci => $code) {
$video = array(
'source' => $matches_code[2][$ci],
'width' => variable_get('video_filter_width_'.$format, 400),
'height' => variable_get('video_filter_height_'.$format, 400),
'autoplay' => variable_get('video_filter_autoplay_'.$format, 0),
);
And that works!
Comment #2
blackdog commented@elanghout - could you create a proper patch of this please?
Comment #3
blackdog commentedCommitted attached patch. Thanks elanghout.
Comment #4
FirstPersonShow-Kevin commentedI've not got shell access to my webhosting provider, so I downloaded the video_filter.module file and deleted the lines mentioned and added the ones you show as well. But I get the error:
Parse error: syntax error, unexpected T_ELSE in modules/video_filter/video_filter.module on line 162
?
cheers,
Kevin
Comment #5
blackdog commentedThat's odd. Can you try the -dev version that's listed on the project page?