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.
I get errors in the log after I enabled this. Was using flvplayer and xspf playlist to test flv file which worked fine.
array_merge() [function.array-merge]: Argument #2 is not an array in C:\web\apache\htdocs\home\sites\all\modules\swfobject_api\swfobject_api.module on line 92.
Invalid argument supplied for foreach() in C:\web\apache\htdocs\home\sites\all\modules\swfobject_api\swfobject_api.module on line 137.
I also got errors for flvplayer.
There was no uninstall so I'll have to do it by hand now.
Comments
Comment #1
sopia CreditAttribution: sopia commentedbump!
Comment #2
arthurf CreditAttribution: arthurf commentedI don't think this was the fault of swfobject, but it should do a better job of not tossing errors when $params is null. I've added some fixes that should prevent this error- please see the latest version
Comment #3
bflora CreditAttribution: bflora commentedCould you say what the fault is? I'm getting similar errors and I have no clue even where to begin understanding them.
Comment #4
a_c_m CreditAttribution: a_c_m commentedI get the following error on 1.2-15
Just dealing with array_merge() error first.
I added "print_r(array('base_params'=>$base_params, 'params'=>$params)); " to line 100(ie just before the merge statement), which gave :
Params is a string "> <".
Vars is a string too "> ".
Seems like there needs to be some input validation, i added
to the top of the function, but its just a hack.
Re-opened the issue.
Comment #5
jhaggenburg CreditAttribution: jhaggenburg commentedThat seemed to fix it for me too, thanks!
Comment #6
arthurf CreditAttribution: arthurf commentedCan you give me a debug_print_backtrace so that I can see what is calling that without an array? It just seems odd to me that that would be happening- I don't doubt you, but I'm wondering where either I messed up, or what module is giving it grief.
Comment #7
a_c_m CreditAttribution: a_c_m commentedAs my code is quite patched now from the stock, i've done my best to re-create the situation.
FLV media player needs to be set to use flash vars (with none set).
On further testing, it seems my patch http://drupal.org/node/258387 fixes this issue.
a_c_m