Changeset 622

Timestamp:

2007-08-03 12:52:47 (18 months ago)

Author:

fredck

Message:

IE was throwing "Unspecified error" when opening _samples/default.html. Added a try/catch to avoid it.

Files:

• FCKeditor/trunk/editor/_source/classes/fckdomrange_ie.js (modified) (3 diffs)

FCKeditor/trunk/editor/_source/classes/fckdomrange_ie.js

@@ -135 +135 @@
 FCKDomRange.prototype._GetSelectionMarkerTag = function( toStart )
 {
+        var doc = this.Window.document
+        var selection = doc.selection ;
+
         // Get a range for the start boundary.
-        var oRange = this.Window.document.selection.createRange() ;
+        var oRange ;
+
+        // IE may throw an "unspecified error" on some cases (it happened when
+        // loading _samples/default.html), so try/catch.
+        try
+        {
+                oRange = selection.createRange() ;
+        }
+        catch (e)
+        {
+                return null ;
+        }
+
         oRange.collapse( toStart === true ) ;
 
@@ -142 +157 @@
         // This is known to happen when the editor window has not been selected before (See #933).
         // We need to avoid that.
-        if (oRange.parentElement().document != this.Window.document)
+        if ( oRange.parentElement().document != doc )
                 return null;
 
@@ -148 +163 @@
         var sMarkerId = 'fck_dom_range_temp_' + (new Date()).valueOf() + '_' + Math.floor(Math.random()*1000) ;
         oRange.pasteHTML( '<span id="' + sMarkerId + '"></span>' ) ;
-        return this.Window.document.getElementById( sMarkerId ) ;
+        return doc.getElementById( sMarkerId ) ;
 }