Opened 12 years ago

Last modified 10 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 12 years ago.
Ticket9227.zip (7.3 KB) - added by Marcus Lum 10 years ago.
Example of problem with current implementation
CKEditorControl.cs.2.patch (1.2 KB) - added by Marcus Lum 10 years ago.
Patch w/ removed unnecessary isChanged variable

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by Koen Van Exem

Attachment: CKEditorControl.cs.patch added

comment:1 Changed 12 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 12 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 12 years ago by Jakub Ś

Keywords: HasPatch added
Status: pendingconfirmed

Changed 10 years ago by Marcus Lum

Attachment: Ticket9227.zip added

Example of problem with current implementation

Changed 10 years ago by Marcus Lum

Attachment: CKEditorControl.cs.2.patch added

Patch w/ removed unnecessary isChanged variable

comment:4 Changed 10 years ago by Jakub Ś

#10651 was marked as duplicate.

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