Opened 9 years ago

Closed 9 years ago

#4827 closed Bug (fixed)

Open image dialog on existing element cause error

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Dialogs Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

Environment

3.1.x branch

Reproducing Procedures

  1. Open 'replacebyclass' sample page and load the document with the following source:
    <p>
    <img alt="" src="http://dev.fckeditor.net/chrome/site/logos.gif" /></p>
    
  2. click on the image and open 'Image' dialog;
  • Actual Result: JavaScript exception thrown.

Attachments (1)

4827.patch (1.5 KB) - added by Garry Yao 9 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Regression of [4685].
Ticket test of that is telling the same thing:

http://ckeditor.t/tt/4246/2.html

Changed 9 years ago by Garry Yao

Attachment: 4827.patch added

comment:2 Changed 9 years ago by Garry Yao

Keywords: Review? added

We need to make sure the preview is well setup before setup dialog contents.

comment:3 Changed 9 years ago by Garry Yao

Keywords: 3.0.2 added

comment:4 Changed 9 years ago by Garry Yao

Correction, should be regression of [4665].

comment:5 Changed 9 years ago by Garry Yao

Keywords: 3.0.2 removed

comment:6 Changed 9 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:7 Changed 9 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4787].

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