Ticket #3829: 3829.patch

_source/plugins/htmldataprocessor/plugin.js

@@ -91 +91 @@
         for ( i in blockLikeTags )
                 defaultDataBlockFilterRules.elements[ i ] = extendBlockForDisplay;
 
-        /**
-         * IE sucks with dynamic 'name' attribute after element is created, '_cke_saved_name' is used instead for this attribute.
-         */
-        var removeName = function( element )
-        {
-                var attribs = element.attributes;
-
-                if ( attribs._cke_saved_name )
-                        delete attribs.name;
-        };
-
         var defaultHtmlFilterRules =
                 {
                         elementNames :
@@ -124 +113 @@
 
                         elements :
                         {
+                                $ : function( element ) 
+                                {
+                                        // Remove duplicated attributes - #3789.
+                                        var attribs = element.attributes;
+
+                                        if ( attribs )
+                                        {
+                                                // IE sucks with dynamic 'name' attribute after element is created, '_cke_saved_name' is used instead for this attribute.
+                                                if ( attribs._cke_saved_name )
+                                                        delete attribs.name;
+                                                if ( attribs._cke_saved_href )
+                                                        delete attribs.href;
+                                                if ( attribs._cke_saved_src )
+                                                        delete attribs.src;
+                                        }
+                                },
+
                                 embed : function( element )
                                 {
                                         var parent = element.parent;
@@ -137 +143 @@
                                         }
                                 },
 
-                                img : function( element )
-                                {
-                                        var attribs = element.attributes;
-
-                                        if ( attribs._cke_saved_name )
-                                                delete attribs.name;
-                                        if ( attribs._cke_saved_src )
-                                                delete attribs.src;
-                                },
-
                                 a : function( element )
                                 {
-                                        var attribs = element.attributes;
-
-                                        if ( attribs._cke_saved_name )
-                                                delete attribs.name;
-                                        if ( attribs._cke_saved_href )
-                                                delete attribs.href;
-                                },
+                                        if ( !( element.children.length || element.attributes.name ) )
+                                                return false;
+                                }
+                        },
 
-                                input : removeName,
-                                textarea : removeName,
-                                select : removeName,
-                                form : removeName
-                        },
-
                         attributes :
                         {
                                 'class' : function( value, element )

CHANGES.html

@@ -62 +62 @@
                         </li>
                 <li><a href="http://dev.fckeditor.net/ticket/3821">#3821</a> : Fixed an issue with JAWS in which
                         toolbar items are read inconsistently between virtual cursor modes.</li>
+                <li><a href="http://dev.fckeditor.net/ticket/3829">#3829</a> : Fixed remove empty link on output data.</li>
         </ul>
         <h3>
                 CKEditor 3.0 RC</h3>