Opened 3 years ago

Closed 3 years ago

#13790 closed Bug (fixed)

Allow for the iframe having been removed already

Reported by: Szymon Kupś Owned by: Szymon Kupś
Priority: Normal Milestone: CKEditor 4.5.5
Component: General Version: 4.5.3
Keywords: Cc:

Description

This ticket is created in order to handle pull request https://github.com/ckeditor/ckeditor-dev/pull/200

When integrating ckeditor in Odoo, we encounter TypeError: iframe is null in http://localhost:9069/web_ckeditor4/static/lib/ckeditor/plugins/wysiwygarea/plugin.js:520 when the editor is created in a ajax popup. The editor is destroyed when the popup is closed, but by then the Odoo framework has already cleared out the DOM.

After we implemented error logging mechanism (#13632) we can handle this PR.

Change History (4)

comment:1 Changed 3 years ago by Szymon Kupś

Owner: set to Szymon Kupś
Status: newassigned

comment:2 Changed 3 years ago by Piotrek Koszuliński

Milestone: CKEditor 4.5.4CKEditor 4.5.5

comment:3 Changed 3 years ago by Szymon Kupś

I've pushed new branch:t/13790, it includes changes made in pull request plus some additional test. It will be merged into master branch after the release of CKEditor 4.5.4 (which is in progress).

comment:4 Changed 3 years ago by Szymon Kupś

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