Opened 15 years ago

Last modified 15 years ago

#3077 closed Bug

Tools: createClass incompleteness — at Version 4

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

Description (last modified by Garry Yao)

CKEDITOR.tools.createClass logic's not right in the following sense:

  • Generally class creation need to fix contructor property, which is wrong after prototype copying;
  • Private fields should be available in constructor;

Change History (8)

Changed 15 years ago by Garry Yao

Attachment: 3077.patch added

comment:1 Changed 15 years ago by Garry Yao

Description: modified (diff)
Keywords: Confirmed Review? added
Owner: set to Garry Yao
Status: newassigned

Changed 15 years ago by Garry Yao

Attachment: 3077_2.patch added

comment:2 Changed 15 years ago by Garry Yao

The proposed patch for private fields has impacted legacy codes which need to fix also.

Changed 15 years ago by Garry Yao

Unit Test Case

Changed 15 years ago by Garry Yao

Attachment: 3077_3.patch added

comment:3 Changed 15 years ago by Garry Yao

Cutting off unnecessary changes after discuss with Fred.

comment:4 Changed 15 years ago by Garry Yao

Description: modified (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy