On a fresh install I get the error

Warning: array_merge() [function.array-merge]: Argument #2 is not an array in _webform_conditional_possible_trigger_components() (line 419 of .../sites/all/modules/webform_conditional/webform_conditional.module).

Warning: array_merge() [function.array-merge]: Argument #3 is not an array in _webform_conditional_possible_trigger_components() (line 419 of /home/plymouthphoto/jaeruberto/sites/all/modules/webform_conditional/webform_conditional.module).

on the Webform Edit Component page.

CommentFileSizeAuthor
#5 1135998_5-warning_argument_2_is_not_an_array.patch744 bytesrobertom

Comments

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented

I got the same error on a fresh d7 install

turneight’s picture

turneight commented

Hi all,
from the php manual:
The behavior of array_merge() was modified in PHP 5. Unlike PHP 4, array_merge() now only accepts parameters of type array. However, you can use typecasting to merge other types.

I changed the line 419 in webform_conditional.module 

- $get_component_types = array_merge($trigger_types, 'pagebreak', 'fieldset');
+ $get_component_types = array_merge($trigger_types, array('pagebreak'), array('fieldset'));

so it works

MrPhilbert’s picture

MrPhilbert commented

Thank you. Great save!

ahkiam’s picture

ahkiam commented

Thank you. I had the same error and this fixed it.

robertom’s picture

robertom commented
Status: Active » Needs review
StatusFileSize
new 1135998_5-warning_argument_2_is_not_an_array.patch744 bytes
bayousoft’s picture

bayousoft commented

nice fix ! !

brightbold’s picture

brightbold
she/her
Primary language English
Location Boston, MA
commented

Awesome. This patch solved the problem for me.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Needs review » Closed (fixed)

@robertom,
Thanks for finding this. I had fixed this in D6 but not D7.
This is now in D7. Sorry, didn't get your "handle" in the on the commit message which I try to do.