Ticket #31441: 31441.18.patch

src/wp-includes/js/tinymce/plugins/wptextpattern/plugin.js

@@ -87 +87 @@
                                 }
                         }
 
+                        if ( ! child.nodeValue ) {
+                                child = child.nextSibling;
+                        }
+
                         if ( child !== node ) {
                                 return;
                         }
@@ -109 +113 @@
                                 editor.undoManager.add();
 
                                 editor.undoManager.transact( function() {
+                                        var $$parent;
+
                                         if ( replace ) {
                                                 $$( node ).replaceWith( document.createTextNode( replace ) );
                                         } else  {
-                                                $$( node.parentNode ).empty().append( '<br>' );
+                                                $$parent = $$( node.parentNode );
+
+                                                $$( node ).remove();
+
+                                                if ( ! $$parent.html() ) {
+                                                        $$parent.append( '<br>' );
+                                                }
                                         }
 
                                         editor.selection.setCursorLocation( parent );

tests/qunit/wp-includes/js/tinymce/plugins/wptextpattern/plugin.js

@@ -77 +77 @@
                 }, assert.async() );
         } );
 
-        QUnit.test( 'Ordered list with content.', function( assert ) {
+        QUnit.test( 'Ordered list with content. (1)', function( assert ) {
                 editor.setContent( '<p><strong>test</strong></p>' );
                 editor.selection.setCursorLocation();
 
@@ -86 +86 @@
                 }, assert.async() );
         } );
 
+        QUnit.test( 'Ordered list with content. (2)', function( assert ) {
+                editor.setContent( '<p><strong>test</strong></p>' );
+                editor.selection.setCursorLocation( editor.$( 'p' )[0], 0 );
+
+                type( '* ', function() {
+                        assert.equal( editor.getContent(), '<ul>\n<li><strong>test</strong></li>\n</ul>' );
+                }, assert.async() );
+        } );
+
         QUnit.test( 'Only transform inside a P tag.', function( assert ) {
                 editor.setContent( '<h1>test</h1>' );
                 editor.selection.setCursorLocation();