Closed (fixed)
Project:
SWF Tools
Version:
5.x-1.1
Component:
SWF Tools
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Aug 2007 at 09:47 UTC
Updated:
12 May 2010 at 21:00 UTC
The Wijering family of players use the 'displayheight' option as a flag as well as a value. It presents turns on the display of the playlist (even if set to 0). Its absence menas no playlist. When it was being set to 0 (zero) the "_wijering_flashvars" would chomp it and it would not make it to the otherside. This can be fixed by a small change to the comparison operator:
This:
if (!$setting || $setting == 'default') {
unset($defaults[$category][$key]);
}
should be:
if ($setting == '' || $setting == 'default') {
unset($defaults[$category][$key]);
}
due to the way PHP see "" and "0" and NULL as the same in logic operators
word
Comments
Comment #1
Stuart Greenfield commentedFixed in DRUPAL-5 branch. Not sure when there will be a new release from it, but it's in place and ready.
Similar fix was already in place on SWF Tools 6.x