id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
10603	With multiple editors, after attaching an image in the first, the second editor is broken	Tim Dionne		"We use CKEDITOR with some custom plugins.  We have a plugin that replaces the image plugin with our own.  When the image button is pressed, our custom dialog is rendered, and along the way, CKEDITOR.dialog() is called.  In there, some code is executed depending on whether you have configured it to remove some dialog tabs (in config.removeDialogTabs), which we have done.  And in that code, there are 2 for loops that use a non-local variable 'i'.  This is causing problems for us because of the way the CKEDITOR.config object is minified.  In fact, it is minified to the variable 'i'.  So, when this code executes, the config object gets blown away, and reassigned to an integer because of the non-local scope.  We often have more than one rich editor on the page, and when this happens, all new editors are hosed because the config objects for them are prototyped from i.  Seems fairly dangerous to use non-local variables in this code.  I see that most other for loops declare i in the local scope using the 'var' keyword.

I have coded a fix and will attach as an svn diff.

I created the patch from trunk, but it exists in previous versions as well.  We are using 3.6.2."	Bug	closed	Normal		UI : Dialogs	3.6.2	invalid