Opened 13 years ago
Closed 13 years ago
#8644 closed Bug (fixed)
Issue in Dialog plugin - declaring i as global variable wrongly
Reported by: | benny | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.3 |
Component: | UI : Dialogs | Version: | 3.6.2 |
Keywords: | Cc: |
Description
The file ckeditor/_source/plugins/dialog/plugin.js, line 228 where a single letter variable “i” is used without “var” thereby making it appear under the global scope. That causes problem in the minified ckeditor.js because "i" is referred as CKEDITOR.config when init the instance. After method to removing the cached tabs of dialog is called, the new editor instance can not be created because the "i" is a int value instead of CKEDITOR.config object.
The steps to reproduce:
- Create a editor instacne.
- Open one dialog plugin like images, link or table and close.
- Create another editor instance in the same page
- It fails to create another editor instance with js error 'z is undefined or not an object' in ckeditor.js
Change History (1)
comment:1 Changed 13 years ago by
Milestone: | → CKEditor 3.6.3 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Thanks for the report, I've fixed with with microchange [7351].
Just a note with regards to the reproduction steps (they might be obvious but anyway):
config.removeDialogTabs = 'flash:advanced';
so the block of code is executed.