Ticket #47863: 47863.patch

src/wp-includes/shortcodes.php

@@ -498 +498 @@
 function shortcode_parse_atts( $text ) {
         $atts    = array();
         $pattern = get_shortcode_atts_regex();
-        $text    = preg_replace( "/[\x{00a0}\x{200b}]+/u", ' ', $text );
+        $text    = trim( preg_replace( "/[\x{00a0}\x{200b}]+/u", ' ', $text ) );
+
+        // Remove everything but attributes from shortcode
+        if ( preg_match( '/^\[\w+([^\]]*)\]/', $text, $matches ) ) {
+                if ( count( $matches ) === 2 ) {
+                        $text = trim( $matches[1], ' \/' );
+                }
+        }
+
         if ( preg_match_all( $pattern, $text, $match, PREG_SET_ORDER ) ) {
                 foreach ( $match as $m ) {
                         if ( ! empty( $m[1] ) ) {

tests/phpunit/tests/shortcode.php

@@ -972 +972 @@
                 );
                 $this->assertEquals( 'test-shortcode-tag', $this->tagname );
         }
+
+        public function data_shortcode_parse_atts() {
+
+                return [
+                        [
+                                '[unittest]',
+                                '',
+                        ],
+                        [
+                                '[unitest]Unit Test[/unittest]',
+                                '',
+                        ],
+                        [
+                                '[unittest title="unittest" link="https://unit.test/"]',
+                                [
+                                        'title' => 'unittest',
+                                        'link'  => 'https://unit.test/',
+                                ],
+                        ],
+                        [
+                                '[unittest title="unittest" link="https://unit.test/"/]',
+                                [
+                                        'title' => 'unittest',
+                                        'link'  => 'https://unit.test/',
+                                ],
+                        ],
+                        [
+                        '[unittest title="unittest" link="https://unit.test/"]Unit Test[/unittest]',
+                                [
+                                        'title' => 'unittest',
+                                        'link'  => 'https://unit.test/',
+                                ],
+                        ],
+                        [
+                                'title="unittest" link="https://unit.test/"',
+                                [
+                                        'title' => 'unittest',
+                                        'link'  => 'https://unit.test/',
+                                ],
+                        ],
+                ];
+
+        }
+
+        /**
+         * @covers ::shortcode_parse_atts
+         * @dataProvider data_shortcode_parse_atts
+         *
+         * @param $shortcode
+         * @param $expects
+         */
+        public function test_shortcode_parse_atts( $shortcode, $expects ) {
+
+                $atts = shortcode_parse_atts( $shortcode );
+
+                $this->assertSame( $expects, $atts );
+
+        }
 }