I'm getting repeated logged errors of invalid argument supplied for foreach() for sites/all/modules/jquerymenu/jquerymenu.module on line 207.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | jquerymenu-invalid-argument-4-2722802.patch | 24.58 KB | Milan0 |
| #3 | jquerymenu-invalid-argument-3-2722802(2).patch | 24.58 KB | Milan0 |
Comments
Comment #1
silurius commentedDuplicated here under Drupal 6.16 & jquerymenu-6.x-3.0. For reference, here's the relevant line (207):
foreach ($items as $item) {And the section that it's in:
Comment #2
Milan0 commentedConfirmed, error also happening here same versions as described above.
Comment #3
Milan0 commentedComment #4
Milan0 commentedyuck..windows.
proper filenme now.
Comment #5
not_Dries_Buytaert commentedSince applying the patch and manually running '/admin/reports/status/run-cron', the log entries do not appear anymore. I am unsure which use cases (requiring this module) triggered the error. Nevertheless, I consider I successfully reviewed and tested this patch, which changes the code after line 204 in '.../sites/all/modules/jquerymenu/jquerymenu.module'
from:
to:
Please, could someone include this change into the next release?
Comment #6
PixelClever commentedOk the code here has been changed and committed to HEAD. I want to wait a little bit to see if there are any other things that need work before making a new release.
Comment #7
not_Dries_Buytaert commentedA pity that this patch just missed the release d.d. 2010-May-05 (http://drupal.org/node/790206). I also applied and successfully tested the change in that release.
Comment #8
PixelClever commentedComment #9
not_Dries_Buytaert commentedJust noticed that this issue was solved thanks to this release: jquerymenu 6.x-3.3 (http://drupal.org/node/799538). :-)