search

neokoenig / jQuery-gridmanager

A way of building rows and grids with built in editable regions; requires jQuery, jQueryUI, Bootstrap 3.x, & optional TinyMCE or CKEditor
http://neokoenig.github.io/jQuery-gridmanager/
461 stars 129 forks source link

Gridmanager throws error with default options and tinyMCE #67

Open madc opened 9 years ago

madc commented 9 years ago

The gridmanager throws the following error, when tinyMCE is present on the page and tinymce.config is not overwritten.

TypeError: t.plugins.split is not a function

The problem here seems to be the default optinos for tinyMCE in gridmanager.js on line 1527. Here, tinymce.config.plugins is preset with an array instead a string. Removing [ and ] should solve this issue.

neokoenig commented 9 years ago

Which version of TinyMCE? I've only tested against 4.0

madc commented 9 years ago

The TinyMCE version is 4.0.12, jQuery version is 1.11.0. In the documentation it says, TinyMCE expects the plugins to be "a comma or space seperated string, or an array of strings".

I just saw, that it is working on your demo, so maybe this is a problem with running TinMCE < 4.1? Anyway removing the brackets should have no effect on newer versions of the editor and fix it for older ones.