Opened 7 years ago

Closed 7 years ago

#7014 closed Bug (fixed)

Samples - Ajax - error on page, when Create/Remove button is pressed one by one

Reported by: Krzysztof Studnik Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.1
Component: General Version: 3.5.1
Keywords: IE6 Cc:

Description

Test case

  • Open Ajax sample in IE6
  • Press Create editor
  • Press Remove editor, right after (before editor is enabled)
  • Press Create editor

Expected result
Editor is loaded after the second press of Enable button.

Actual result Error is thrown

line 141
char 237
error: 'element' is null or not an object
code: 0

Error does not occur in previous versions of samples.
Error does not occur in other browsers.

Probably IE6 is much slower in refreshing new ajax sample.

Attachments (1)

7014.patch (2.7 KB) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by Garry Yao

Status: newconfirmed

Regression of [6335].

comment:2 Changed 7 years ago by Wiktor Walc

I was able to trigger a JS error in all versions of the editor (starting from 3.0) using the Ajax sample ("theme is null or not an object", "Permission denied" and other random issues).

Changed 7 years ago by Garry Yao

Attachment: 7014.patch added

comment:3 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

We should make sure no error when request to destroy editor before editor finished loading.

comment:4 Changed 7 years ago by Wiktor Walc

Milestone: CKEditor 3.5.1

comment:5 Changed 7 years ago by Krzysztof Studnik

Patch solved problem on my side.

comment:6 Changed 7 years ago by Wiktor Walc

Pls check #7012 as well while reviewing this one.

comment:7 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

#7012 is also fixed with this patch.

comment:8 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6366].

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