Upon installing I received the following message.

warning: Invalid argument supplied for foreach() in **/sites/default/modules/contrib/wysiwyg-6.x-1.x-dev/wysiwyg_editor.install on line 109.

Everything actually worked fine though. I think the problem is that TinyMCE.module had been previously installed but was disabled when I installed WYSIWYG.

From a quick look at the source it appears that wysiwyg_editor_migrate_tinymce() in the .install file detects the 'tinymce_settings' table and continues processing as if tinymce.module is currently enabled.

The line "foreach ($old_buttons as $old_button => $enabled) {" in wysiwyg_editor_update_5001() is where the actual error occurs.

Cheers.

CommentFileSizeAuthor
#1 wysiwyg-DRUPAL-5.foreach.patch826 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Status: Active » Fixed
FileSize
826 bytes

Committed attached patch that should fix this issue. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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