Opened 14 years ago
Last modified 14 years ago
#6203 confirmed Bug
Uncaught exception: TypeError: Cannot convert 'K' to object on editor.destroy();
Reported by: | Tony | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.4 |
Keywords: | HasPatch | Cc: |
Description
I receive the following error when trying to destroy a CKeditor 3.4 instance using a plugin :
Uncaught exception: TypeError: Cannot convert 'K' to object Error thrown at line 92, column 1741 in <anonymous function: getSnapshotData>(): return K.getFrameDocument().getBody().getHtml(); called from line 114, column 4746 in <anonymous function: init>(o): o.data=l(n).getSnapshotData(); called via Function.prototype.call() from line 6, column 3258 in <anonymous function: on>(o, p, q, r): var s={name:d,sender:this,editor:o,data:p,listenerData:g,stop:q,cancel:r,removeListener:function(){l.removeListener(d,e);}}; called via Function.prototype.call() from line 6, column 4027 in <anonymous function: fire>(h, i, j): d=f=false; called via Function.prototype.call() from unknown location in <anonymous function: fire>(b, c): /* no source available */ called from line 24, column 4875 in <anonymous function: getSnapshot>(): var l=this.fire('getSnapshot'); called from line 87, column 1760 in <anonymous function: j.undo.Image>(r): var s=r.getSnapshot(),t=s&&r.getSelection(); called from line 88, column 831 in <anonymous function: save>(r, s, t): s=new l(v.editor); called from line 87, column 277 in v(w): s.save(); called via Function.prototype.call() from line 6, column 3258 in <anonymous function: on>(o, p, q, r): var s={name:d,sender:this,editor:o,data:p,listenerData:g,stop:q,cancel:r,removeListener:function(){l.removeListener(d,e);}};
I guess that CKeditor's snapshot attempt is being fired after the editor has been destroyed? I have attached a small test plugin to make it easy to reproduce. The error was caught using Opera. The editor does destroy but the error message means not cleanly at a guess.
Attachments (2)
Change History (5)
Changed 14 years ago by
Attachment: | editorClose.zip added |
---|
comment:1 Changed 14 years ago by
Keywords: | destroy cannot convert 'K' removed |
---|
comment:2 Changed 14 years ago by
Status: | new → confirmed |
---|
It's impossible to guarantee absolute safety on destroyed instance since we're using a closure for referencing the "editor" variable all through our codes, considering this not a big issue we should just claim this as a convention in our API documentation.
Note: All instance methods of "CKEDITOR.editor" invocation is invalid after editor instance destroyed with "CKEDITOR.editor::destroy".
But we should at least error free for the attached TC.
Changed 14 years ago by
Attachment: | 6203.patch added |
---|
comment:3 Changed 14 years ago by
Keywords: | HasPatch added |
---|
editor close test plugin for CKeditor