Opened 16 years ago

Closed 15 years ago

#1907 closed Bug (fixed)

JavaScript errors in Firefox 3.0b3

Reported by: Matthias Miller Owned by: Martin Kou
Priority: Normal Milestone: FCKeditor 2.6.3
Component: General Version: FCKeditor 2.5.1
Keywords: Review+ Cc:


I am getting sporadic errors when loading the editor in Firefox 3.0b3. The problem is that fckeditorapi.js is appending a SCRIPT element to the parent page to bootstrap the API. However, appending the SCRIPT element appears to be an asynchronous action in Firefox, so it's not always possible to immediately access oParentWindow.FCKeditorAPI. (I discovered this because the parent page sometimes needs to load a script that takes a second or two to fetch.)

A simple solution is to remove the check for FCKBrowserInfo.IsGecko19 added in r659, since the Firefox bug has been fixed.

As an aside, I don't know if this is an issue in Safari. If it is, you might consider something like this approach.

Attachments (1)

1907.patch (1.2 KB) - added by Martin Kou 15 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 16 years ago by Wojciech Olchawa

Keywords: Pending added


Could you provide information about what error are you experiencing? You've described the origin of the problem fully however I'm not able to specify what problem you are dealing with. Also could you please write what version of FCKeditor are you using?


comment:2 Changed 16 years ago by Matthias Miller

I encountered this with 2.5.1 (build 17566). Firefox threw an error because oParentWindow.FCKeditorAPI was not defined, then subsequently refused to load up the editor.

comment:3 Changed 16 years ago by Wojciech Olchawa

Version: FCKeditor 2.5.1

Hi again!

I'm using Firefox Beta 3 as well as you are but I didn't encounter the same problem as you did yet. However as you said it is a sporadic bug so it is hard to reproduce.

Does the problem also occur on our demo page?

Please let us know.


comment:4 Changed 16 years ago by Wojciech Olchawa

Resolution: invalid
Status: newclosed

Ticket expired.

comment:5 Changed 15 years ago by Alfonso Martínez de Lizarrondo

Milestone: FCKeditor 2.6.3
Resolution: invalid
Status: closedreopened

Reopening due to and targeting for further study.

comment:6 Changed 15 years ago by Martin Kou

Owner: set to Martin Kou
Status: reopenednew

Changed 15 years ago by Martin Kou

Attachment: 1907.patch added

comment:7 Changed 15 years ago by Martin Kou

Keywords: Review? added; Pending removed

comment:8 Changed 15 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:9 Changed 15 years ago by Martin Kou

Resolution: fixed
Status: newclosed

Fixed with [2188].

Click here for more info about our SVN system.

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