Ticket #7494: 7494.patch

_source/core/htmlparser/fragment.js

@@ -112 +112 @@
                 }
 
                 /*
-                * Beside of simply append specified element to target, this function also takes
+                * Beside of simply append specified element to target, it also takes
                 * care of other dirty lifts like forcing block in body, trimming spaces at
                 * the block boundaries etc.
-                *
+                * 
                 * @param {Element} element  The element to be added as the last child of {@link target}.
                 * @param {Element} target The parent element to relieve the new node.
-                * @param {Boolean} [moveCurrent=false] Don't change the "currentNode" global unless
-                * there's a return point node specified on the element, otherwise move current onto {@link target} node.
                  */
-                function addElement( element, target, moveCurrent )
+                function addElement( element, target )
                 {
                         // Ignore any element that has already been added.
                         if ( element.previous !== undefined )
@@ -181 +179 @@
                                 delete element.returnPoint;
                         }
                         else
-                                currentNode = moveCurrent ? target : savedCurrent;
+                                currentNode = savedCurrent;
                 }
 
                 parser.onTagOpen = function( tagName, attributes, selfClosing )
@@ -260 +258 @@
                                                 // The most common case where we just need to close the
                                                 // current one and append the new one to the parent.
                                                 if ( currentNode.parent )
-                                                        addElement( currentNode, currentNode.parent, 1 );
+                                                {
+                                                        addElement( currentNode, currentNode.parent );
+                                                        currentNode = currentNode.parent;
+                                                }
                                                 // We've tried our best to fix the embarrassment here, while
                                                 // this element still doesn't find it's parent, mark it as
                                                 // orphan and show our tolerance to it.
@@ -398 +399 @@
                 // Send all pending BRs except one, which we consider a unwanted bogus. (#5293)
                 sendPendingBRs( !CKEDITOR.env.ie && 1 );
 
-                // Close all pending nodes, make sure return point is properly restored.
+                // Close all pending nodes.
                 while ( currentNode != fragment )
-                        addElement( currentNode, currentNode.parent, 1 );
+                {
+                        // Make sure return point is properly restored.
+                        var returnPoint = currentNode.returnPoint;
+                        addElement( currentNode, currentNode.parent );
+                        currentNode = returnPoint || currentNode.parent;
+                }
 
                 return fragment;
         };