PHP warning @ admin/settings/project_release [#97426]

I assume this is yet another PHP 5.x-related bug:

warning: array_merge() [function.array-merge]: Argument #1 is not an array in /Applications/MAMP/htdocs/drupal-4-7/modules/project/release/project_release.module on line 112.

The line in question seems to be:

    $form['#validate'] = array_merge($form['#validate'], array('project_release_settings_validate' => array()));

which looks fine to me?

Comment	File	Size	Author
#2	project_release_settings_php5.patch.txt	969 bytes	dww

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

dww

we/he/they

CreditAttribution: dww commented 14 November 2006 at 00:46

yup, php5 badness again. $form['#validate'] just isn't set in that case, most likely, and it's complaining. should be an easy fix.

Comment #2

dww

we/he/they

CreditAttribution: dww commented 18 November 2006 at 00:03

Assigned:	Unassigned	» dww
Status:	Active	» Needs review

File	Size
project_release_settings_php5.patch.txt	969 bytes

this works in php4. i assume it'll solve the problem in php5, but webchick, could you please verify?
thanks,
-derek

Comment #3

AjK CreditAttribution: AjK commented 19 November 2006 at 16:54

Status:

Needs review

» Reviewed & tested by the community

dww, sorry for the delay in review. Recreated the problem (in PHP5). Applied patch no problems. Fixes fault in PHP5, PHP4 continues to work as before.