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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

brooke_heaton created an issue. See original summary.

brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented
FileSize
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 CreditAttribution: brooke_heaton as a volunteer commented
FileSize
invalide_argument_supplied_foreach_304640_3.patch1023 bytes

Fix patch.

brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented
FileSize
invalide_argument_supplied_foreach_304640_3.patch1.52 KB
brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer commented
FileSize
invalid_argument_supplied_foreach_304640_4.patch2.14 KB

Update patch.

brooke_heaton’s picture

brooke_heaton CreditAttribution: brooke_heaton as a volunteer 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 CreditAttribution: KarenS at Lullabot 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.