Ticket #25387: 25387.patch

src/wp-includes/class-wp-embed.php

@@ -336 +336 @@
 
                 // Find URLs that are on their own line.
                 $content = preg_replace_callback( '|^(\s*)(https?://[^\s"]+)(\s*)$|im', array( $this, 'autoembed_callback' ), $content );
+                $content = preg_replace_callback( '|(<p[^>]*>\s*)(https?://[^\s<]+)(\s*<\/p>)|i', array( $this, 'autoembed_callback' ), $content );
 
                 // Put the line breaks back.
                 return str_replace( '<!-- wp-line-break -->', "\n", $content );

tests/phpunit/tests/media.php

@@ -158 +158 @@
                 $this->assertEquals( $content, $result );
         }
 
+        function data_autoembed() {
+                return array(
+
+                        // Should embed
+                        array(
+'https://w.org',
+'[embed]'
+                        ),
+                        array(
+'test
+ https://w.org
+test',
+'test
+ [embed]
+test'
+                        ),
+                        array(
+'<p class="test">https://w.org</p>',
+'<p class="test">[embed]</p>'
+                        ),
+                        array(
+'<p> https://w.org </p>',
+'<p> [embed] </p>'
+                        ),
+                        array(
+'<p>test
+https://w.org
+test</p>',
+'<p>test
+[embed]
+test</p>'
+                        ),
+
+                        // Should NOT embed
+                        array(
+'test https://w.org</p>'
+                        ),
+                        array(
+'<span>https://w.org</a>'
+                        ),
+                        array(
+'<a href=" https://w.org ">
+https://w.org
+</a>'
+                        )
+                );
+        }
+
+        /**
+         * @dataProvider data_autoembed
+         */
+        function test_autoembed( $content, $result = null ) {
+                $wp_embed = new Test_Autoembed;
+
+                $this->assertEquals( $wp_embed->autoembed( $content ), $result ? $result : $content );
+        }
+
         function test_wp_prepare_attachment_for_js() {
                 // Attachment without media
                 $id = wp_insert_attachment(array(
@@ -1612 +1669 @@
                 $this->assertSame( $expected, get_image_send_to_editor( $id, $caption, $title, $align, $url, $rel, $size, $alt ) );
         }
 }
+
+/**
+ * Helper class for `test_autoembed`.
+ */
+class Test_Autoembed extends WP_Embed {
+        public function shortcode( $attr, $url = '' ) {
+                return '[embed]';
+        }
+}