Ticket #6641: 6641_2.patch

_source/plugins/styles/plugin.js

@@ -1204 +1204 @@
                 if ( element )
                         element.copyAttributes( el );
 
-                return setupElement( el, style );
+                el = setupElement( el, style );
+
+                // Avoid ID duplication.
+                if ( targetDocument.getCustomData( 'doc_processing_style' ) && el.hasAttribute( 'id' ) )
+                        el.removeAttribute( 'id' );
+                else
+                        targetDocument.setCustomData( 'doc_processing_style', 1 );
+
+                return el;
         }
 
         function setupElement( el, style )
@@ -1438 +1446 @@
                 }
                 else
                         selection.selectBookmarks( bookmarks );
+
+                document.removeCustomData( 'doc_processing_style' );
         }
 })();
 

_source/plugins/link/dialogs/link.js

@@ -1161 +1161 @@
                 },
                 onOk : function()
                 {
-                        var attributes = { href : 'javascript:void(0)/*' + CKEDITOR.tools.getNextNumber() + '*/' },
+                        var attributes = {},
                                 removeAttributes = [],
-                                data = { href : attributes.href },
+                                data = {},
                                 me = this,
                                 editor = this.getParentEditor();
 
@@ -1283 +1283 @@
                                                 removeAttributes.push( attrName );
                                 };
 
-                                if ( this._.selectedElement )
-                                        advAttr( 'advId', 'id' );
+                                advAttr( 'advId', 'id' );
                                 advAttr( 'advLangDir', 'dir' );
                                 advAttr( 'advAccessKey', 'accessKey' );
                                 advAttr( 'advName', 'name' );
@@ -1312 +1311 @@
                                         selection.selectRanges( ranges );
                                 }
 
+                                // Browser need the "href" fro copy/paste link to work. (#6641)
+                                attributes.href = attributes[ 'data-cke-saved-href' ];
+
                                 // Apply style.
                                 var style = new CKEDITOR.style( { element : 'a', attributes : attributes } );
                                 style.type = CKEDITOR.STYLE_INLINE;             // need to override... dunno why.
                                 style.apply( editor.document );
-
-                                // Id. Apply only to the first link.
-                                if ( data.adv && data.adv.advId )
-                                {
-                                        var links = this.getParentEditor().document.$.getElementsByTagName( 'a' );
-                                        for ( i = 0 ; i < links.length ; i++ )
-                                        {
-                                                if ( links[i].href == attributes.href )
-                                                {
-                                                        links[i].id = data.adv.advId;
-                                                        break;
-                                                }
-                                        }
-                                }
-                        }
+                        }
                         else
                         {
                                 // We're only editing an existing link, so just overwrite the attributes.