Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.During install of a new site, changes in commit dc8726 seem to be causing the following error:
Invalid argument supplied for foreach() Service.php:144
Simple fix is to wrap the foreach loop with a is_array() conditional.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | Service.php_.diff | 738 bytes | lieb |
Comments
Comment #1
decipheredhttp://drupalcode.org/sandbox/deciphered/2005700.git/shortlog/refs/heads...
Patch: http://drupalcode.org/sandbox/deciphered/2005700.git/patch/82a56dc32068e...
Comment #2
realityloop commentedDeciphered's patch has fixed the issue for me
Comment #3
anarcat commentedyum yum! i love RTBC patches! :) committed!
unfortunately, it's incomplete: i was a bad boy and made that error in a bunch of places in Service.php... maybe we want to initialize that array instead of doing foreach loops?
Comment #4
lieb commentedThis one initializes $_config as an array() . Tested with provison-install and provision-delete.
Comment #5
lieb commentedI attached a patch in #4 but did not set the status to 'needs review' so I am doing so now.
Comment #6
anarcat commentedthanks, that looks better.
i had forgotten to push the other patch, which should have been reverted, so it won't need to be now. :) great! congrats on your first patch in, lieb!