Has been shown to me as red drupal set message "error" after running the beta2 updates.

warning: Invalid argument supplied for foreach() in /srv/www/web123/html/includes/module.inc on line 300.

I believe this happens by module_enable('l10n_drupal'); in 6016. The module is not active after the update, but I had local packs and the settings seems to be converted and $enabled must have been TRUE.

Upgrade steps for repro:
1. Removed dependency of server module in community
2. Enabled l10n_server module
3. Ran the updates of beta2
4. Added dependency of server module in community back.

CommentFileSizeAuthor
#3 module-update.patch815 bytesgábor hojtsy

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Postponed (maintainer needs more info)

I've implemented and run the updates while in development and have not seen these errors. I just saved an l.d.o dump yesterday to run the beta2 updates with (l.d.o still runs beta1), so will attempt to reproduce there as well. Until then, any other reproductions are welcome, so we can get more info.

hass’s picture

hass commented

You may need 1.x as basis...

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Postponed (maintainer needs more info) » Fixed
StatusFileSize
new module-update.patch815 bytes

Ok, run update tests another time. The issue is in the module module_enable() call in the update. It should have an array, not a single string. Committing this fix. Thanks for the report.

Status: Fixed » Closed (fixed)

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