Opened 11 years ago

Last modified 11 years ago

#2927 closed Bug

CKEDITOR.event duplicate dom event registration — at Version 2

Reported by: Garry Yao Owned by: Garry Yao
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Confirmed Review+ Cc:

Description (last modified by Garry Yao)

Event delegation registration with CKEDITOR.event.implementOnis currently incorrect, when this method is invoked on prototype object of a constructor, then instances of this constructor will share the same private model, which is wrong. This would result in duplicate invocation of event listeners from two different object.

Functonal TestCase

Tested with test-events-duplicate.path

Change History (4)

comment:1 Changed 11 years ago by Garry Yao

Keywords: Confirmed added
Owner: set to Garry Yao
Status: newassigned

comment:2 Changed 11 years ago by Garry Yao

Description: modified (diff)

Reduced reason was found, update description accordingly.

Changed 11 years ago by Garry Yao

Attachment: test-events-duplicate.patch added

Functional testcase

Changed 11 years ago by Garry Yao

Attachment: 2927.patch added
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy