Opened 8 years ago

Closed 8 years ago

#3825 closed Bug (fixed)

JS error when opening spellingcheck.

Reported by: tbo Owned by: Tobiasz Cudnik
Priority: Normal Milestone: CKEditor 3.0
Component: UI : Spell Checker Version: 3.0 RC
Keywords: Confirmed Gecko Review+ Cc: WebSpellChecker.net

Description

On http://ckeditor.com/ckeditor/3.0rc/_samples/ui_languages.html, when clicking on the spellingcheck button (abc), I get a javascript error on Firefox 3.0.11

window.opener is null tryLoad()tmpFrameset.html (line 30) onload(load )kWyyDmfg...phg%3D%3D (line 2) [Break on this error] var oParams = window.opener.oldFramesetPageParams ;

Attachments (1)

3825.patch (2.1 KB) - added by Tobiasz Cudnik 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Cc: WebSpellChecker.net added

comment:2 Changed 8 years ago by Tobiasz Cudnik

Component: GeneralUI : Spell Checker
Keywords: Confirmed added
Owner: set to Tobiasz Cudnik
Status: newassigned

comment:3 Changed 8 years ago by Tobiasz Cudnik

Keywords: Gecko Review? added

Regression on [3711]. "opener" variable have to be assigned without declaration. "window.opener = " won't assign in FF.

Patch reverts change from [3711] and fixes indentation.

Changed 8 years ago by Tobiasz Cudnik

Attachment: 3825.patch added

comment:4 Changed 8 years ago by Artur Formella

Keywords: Review+ added; Review? removed

Yes, opener is a global variable.

comment:5 Changed 8 years ago by Tobiasz Cudnik

Resolution: fixed
Status: assignedclosed

Fixed with [3796].

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