Ticket #3983: 3983.patch

_source/plugins/toolbar/plugin.js

@@ -127 +127 @@
 
                                                 for ( var r = 0 ; r < toolbar.length ; r++ )
                                                 {
-                                                        var row = toolbar[ r ],
-                                                                toolbarId = 'cke_' + CKEDITOR.tools.getNextNumber(),
+                                                        var row = toolbar[ r ];
+
+                                                        // It's better to check if the row object is really
+                                                        // available because it's a common mistake to leave
+                                                        // an extra comma in the toolbar definition
+                                                        // settings, which leads on the editor not loading
+                                                        // at all in IE. (#3983)
+                                                        if ( !row )
+                                                                continue;
+
+                                                        var toolbarId = 'cke_' + CKEDITOR.tools.getNextNumber(),
                                                                 toolbarObj = { id : toolbarId, items : [] };
 
                                                         if ( groupStarted )

CHANGES.html

@@ -153 +153 @@
                 <li><a href="http://dev.fckeditor.net/ticket/3793">#3793</a> : Combined images into sprites.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/3783">#3783</a> : Fixed indenting command in table cells create collapsed paragraph.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/3968">#3968</a> : About dialog layout was broken with IE+Standards+RTL.</li>
-                <li><a href="http://dev.fckeditor.net/ticket/3991">#3991</a> : In IE quirks, text was not visible in v2 and office2003 skins.</li>              
+                <li><a href="http://dev.fckeditor.net/ticket/3991">#3991</a> : In IE quirks, text was not visible in v2 and office2003 skins.</li>
+                <li><a href="http://dev.fckeditor.net/ticket/3983">#3983</a> : In IE, we&#39;ll now 
+                        silently ignore wrong toolbar definition settings which have extra commas being 
+                        left around.</li>               
         </ul>
         <h3>
                 CKEditor 3.0 RC</h3>