PHP 5.3 returns true on isset($arrayname['hash']) when $arrayname is a string. This causes an invalid argument supplied for foreach error in base.inc on line 215.

CommentFileSizeAuthor
#2 fix_invalid_argument_for_foreach-1224730-2.patch1.07 KBpdrake
#1 fix_invalid_argument_for_foreach-1224730-1.patch575 bytespdrake

Comments

pdrake’s picture

pdrake commented
StatusFileSize
new fix_invalid_argument_for_foreach-1224730-1.patch575 bytes

Patch for 7.x-3.x.

pdrake’s picture

pdrake commented
StatusFileSize
new fix_invalid_argument_for_foreach-1224730-2.patch1.07 KB

This also occurs on line 268. The attached patch fixes both.

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs work

Update status.

I set this to needs work, because this actually shows a developing error. Contains has to be an array by definition, at least from my perspective, so if someone is doing it wrong, it should be triggered.
Could you find out which option_definition uses 'contains' but has just a string? This would really help.

mustanggb’s picture

mustanggb commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)