Ticket #2024 (closed Bug: fixed)

Opened 16 months ago

Last modified 16 months ago

FCKDialog IE selection error when in source view

Reported by: letssurf Owned by: martinkou
Priority: High Milestone: FCKeditor 2.6
Component: UI : Dialogs Version: SVN (FCKeditor)
Keywords: Confirmed IE HasPatch Review+ Cc:

Description

When trying to display a dialog using the new FCKDialog while on IE while in source view an error occurs.

IE says "'selection' is null or not an object"

I managed to track this error down to /fckeditor/editor/_source/internals/fckdialog.js line 112.

I don't understand in internal workings of FCKEditor fully. I have managed to work out that it's due to it trying to access the Editor which isn't about while in source view.

Adding a check for the EditMode and using the EditingArea.Textarea to select the selection seems to fix the problem. I'm not 100% sure this is the correct fix so an FCKEditor core dev will have to read though the patch I've attached.

Attachments

fckdialog_ie_selection_source_view.patch (2.2 KB) - added by letssurf 16 months ago.
2024.patch (1.3 KB) - added by martinkou 16 months ago.

Change History

Changed 16 months ago by letssurf

Changed 16 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 16 months ago by martinkou

  • keywords Confirmed IE Review? added; dialogs ie selection removed

Hi letssurf,

Your observations and the general idea of your patch are correct. The patch can be further simplified though. I'm attaching a simplified patch for review.

Changed 16 months ago by martinkou

  • keywords HasPatch added

Changed 16 months ago by martinkou

Changed 16 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 16 months ago by letssurf

Works for me. Thanks guys :) very fast fix.

Changed 16 months ago by martinkou

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [1726].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.