In php versions < 5.3 adding an array to a non array can cause trouble. We've seen this in the machine_name_field_widget_form function. $element is empty so 

$element['value'] = $element;
$element['value'] += array(

causes the page to fail.
This can be easily fixed with a simple cast to an array.

CommentFileSizeAuthor
#2 machine_name-fatal-error-1847006-2.patch668 byteslliss

Comments

lliss’s picture

lliss commented

The actual error returned is:

Unsupported operand types in ***/machine_name/machine_name.module on line 51

lliss’s picture

lliss commented
Status: Needs work » Needs review
StatusFileSize
new machine_name-fatal-error-1847006-2.patch668 bytes

The attached patch should solve this issue.

ducktape’s picture

ducktape commented
Status: Needs review » Reviewed & tested by the community

That fixed it for me, thanks!

sproets’s picture

sproets commented

Yep, fixed it for me to!
Super!

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented

Ok, thank you for testing. I am very busy at moment but I try to commit the patch as soon as possible.

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed in branch 7.x-2.x: http://drupalcode.org/project/machine_name.git/commit/b65300c

dww’s picture

dww
we/he/they
commented
Version: 7.x-2.x-dev » 7.x-1.x-dev
Status: Fixed » Reviewed & tested by the community

So long as we have two branches (boo), we need this commit (and every other bug fix) in the 7.x-1.x branch, too. See #1952404-5: Support devel_generate for machine_name fields for why I think we should just merge 7.x-2.x back into 7.x-1.x and (for now) abandon work on 7.x-2.x.

Thanks,
-Derek

dww’s picture

dww
we/he/they
commented
Status: Reviewed & tested by the community » Fixed

7.x-2.x is now merged back into 7.x-1.x: #1971726: Merge 7.x-2.x branch back into 7.x-1.x and (for now) abandon 7.x-2.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.