Ticket #2411: 2411.patch

editor/dialog/fck_anchor.html

@@ -124 +124 @@
                 if ( FCKBrowserInfo.IsIE )
                         aNewAnchors.push( oEditor.FCK.InsertElement( '<a name="' + FCKTools.HTMLEncode( sNewName ).replace( '"', '&quot;' ) + '">' ) ) ;
                 else
-                {
-                        var n = oEditor.FCK.InsertElement( 'a' ) ;
-                        n.name = sNewName ;
-                        aNewAnchors.push( n ) ;
-                }
+                        aNewAnchors.push( oEditor.FCK.InsertElement( 'a' ) ) ;
         }
         else
         {
@@ -142 +138 @@
         {
                 oAnchor = aNewAnchors[i] ;
 
+                // Set the name
+                if ( FCKBrowserInfo.IsIE )
+                {
+                        // Setting anchor names directly in IE will trash the HTML code stored
+                        // in FCKTempBin after undos. See #2263.
+                        var replaceAnchor = oEditor.FCK.EditorDocument.createElement( '<a name="' +
+                                        FCKTools.HTMLEncode( sNewName ).replace( '"', '&quot;' ) + '">' ) ;
+                        oEditor.FCKDomTools.MoveChildren( oAnchor, replaceAnchor ) ;
+                        oAnchor.parentNode.replaceChild( replaceAnchor, oAnchor ) ;
+                        oAnchor = replaceAnchor ;
+                }
+                else
+                        oAnchor.name = sNewName ;
+
                 // IE does require special processing to show the Anchor's image
                 // Opera doesn't allow to select empty anchors
                 if ( FCKBrowserInfo.IsIE || FCKBrowserInfo.IsOpera )