Invalid argument supplied for foreach() taxonomy_place.module:49

This occurs on install because there are no set fields. Need to add an !empty check on fields variable.

CommentFileSizeAuthor
#5 invalid_argument_supplied_foreach_304640_4.patch2.14 KBbrooke_heaton
#4 invalide_argument_supplied_foreach_304640_3.patch1.52 KBbrooke_heaton
#3 invalide_argument_supplied_foreach_304640_3.patch1023 bytesbrooke_heaton
#2 invalide_argument_supplied_foreach_304640_2.patch359 bytesbrooke_heaton

Comments

brooke_heaton created an issue. See original summary.

brooke_heaton’s picture

brooke_heaton commented
StatusFileSize
new invalide_argument_supplied_foreach_304640_2.patch359 bytes

Patch taxonomy_place.module to add !empty on $fields to prevent invalid argument warning.

brooke_heaton’s picture

brooke_heaton commented
StatusFileSize
new invalide_argument_supplied_foreach_304640_3.patch1023 bytes

Fix patch.

brooke_heaton’s picture

brooke_heaton commented
StatusFileSize
new invalide_argument_supplied_foreach_304640_3.patch1.52 KB
brooke_heaton’s picture

brooke_heaton commented
StatusFileSize
new invalid_argument_supplied_foreach_304640_4.patch2.14 KB

Update patch.

brooke_heaton’s picture

brooke_heaton commented
Status: Active » Needs review

  • KarenS committed 58de6ac on 8.x-1.x authored by brooke_heaton 
    Issue #3043640 by brooke_heaton: Invalid argument supplied for foreach()
karens’s picture

karens commented
Status: Needs review » Fixed

I don't think this will be a problem now that another patch added initial configuration for the settings, but it never hurts to check so I'm adding it. Thanks!

Status: Fixed » Closed (fixed)

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