Our TinyMCE implementation implodes all settings arrays into comma separated strings, including settings provided by plugins.
This makes it impossible to use nested arrays in hook_wysiwyg_plugin() implementations for TinyMCE, as seen in #769830: paint Web integration module.

The issue can be worked around using the new hook_wysiwyg_editor_settings_alter() available in Wysiwyg 2.1 until this is fixed. (Example in the referenced issue.)

CommentFileSizeAuthor
#2 wysiwyg-tinymce-781086.patch951 bytestwod

Comments

twod’s picture

twod
Primary language Swedish
Location Sweden
commented

The offending code is at the end of wysiwyg_tinymce_settings()

  // Convert the config values into the form expected by TinyMCE.
  foreach ($settings as $key => $value) {
    if (is_bool($value)) {
      continue;
    }
    if (is_array($value)) {
      $settings[$key] = implode(',', $settings[$key]);
    }
  }

  return $settings;
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Active » Needs review
StatusFileSize
new wysiwyg-tinymce-781086.patch951 bytes

This patch makes sure only the settings we use in the implementation are imploded to a comma separated values string.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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