Opened 7 years ago

Last modified 5 years ago

#9227 confirmed Bug

OnTextChanged event required for CKEditor ASP.NET control doesn't fire

Reported by: Koen Van Exem Owned by:
Priority: Normal Milestone:
Component: Server : ASP.Net Version: 3.6.4
Keywords: HasPatch Cc:

Description

The event is fired in OnPreRender(). This is not the proper place to fire this event.

It should be fired in IPostBackDataHandler.RaisePostDataChangedEvent().

In attachment you can find a pach

Attachments (3)

CKEditorControl.cs.patch (913 bytes) - added by Koen Van Exem 7 years ago.
Ticket9227.zip (7.3 KB) - added by Marcus Lum 5 years ago.
Example of problem with current implementation
CKEditorControl.cs.2.patch (1.2 KB) - added by Marcus Lum 5 years ago.
Patch w/ removed unnecessary isChanged variable

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by Koen Van Exem

Attachment: CKEditorControl.cs.patch added

comment:1 Changed 7 years ago by Jakub Ś

Keywords: OnTextChanged removed
Status: newpending

Hi KoenVanExem,

Could you explain why is it important to move this event?
What sort of the problem has current code been causing for you?

comment:2 in reply to:  1 Changed 7 years ago by Koen Van Exem

Replying to j.swiderski:

Hi KoenVanExem,

Could you explain why is it important to move this event?
What sort of the problem has current code been causing for you?

When I used the editor control in a dynamic control the event didn't fire. (using new Control() instead of just dropping it on the design surface)

Using the IPostBackDataHandler is the standard way of doing it, and will make your control more robust (e.g. dynamic, partial postback, ...)

comment:3 Changed 7 years ago by Jakub Ś

Keywords: HasPatch added
Status: pendingconfirmed

Changed 5 years ago by Marcus Lum

Attachment: Ticket9227.zip added

Example of problem with current implementation

Changed 5 years ago by Marcus Lum

Attachment: CKEditorControl.cs.2.patch added

Patch w/ removed unnecessary isChanged variable

comment:4 Changed 5 years ago by Jakub Ś

#10651 was marked as duplicate.

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