WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
02/04/2025 02:47:41 AM (5 months ago)
Author:
jorbin
Message:

HTML API: Fix extensibility of WP_HTML_Processor::next_token().

Break out logic from the next_token() method into a private method which may call itself recursively. This allows for subclasses to override the next_token() method and be assured that each call to next_token() corresponds with the consumption of one single token. This also parallels how WP_HTML_Tag_Processor::next_token() wraps a private base_class_next_token() method.

Reviewed by jonsurrell.
Merges [59285], [59364], and [59747] to 6.7 branch.

Props westonruter, jonsurrell, dmsnell, jorbin.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/6.7/tests/phpunit/tests/html-api/wpHtmlProcessor.php

    r59721 r59757  
    910910
    911911    /**
     912     * Data provider.
     913     *
     914     * @return array
     915     */
     916    public function data_html_processor_with_extended_next_token() {
     917        return array(
     918            'single_instance_per_tag'   => array(
     919                'html'                  => '
     920                    <html>
     921                        <head>
     922                            <meta charset="utf-8">
     923                            <title>Hello World</title>
     924                        </head>
     925                        <body>
     926                            <h1>Hello World!</h1>
     927                            <img src="example.png">
     928                            <p>Each tag should occur only once in this document.<!--Closing P tag omitted intentionally.-->
     929                            <footer>The end.</footer>
     930                        </body>
     931                    </html>
     932                ',
     933                'expected_token_counts' => array(
     934                    '+HTML'    => 1,
     935                    '+HEAD'    => 1,
     936                    '#text'    => 14,
     937                    '+META'    => 1,
     938                    '+TITLE'   => 1,
     939                    '-HEAD'    => 1,
     940                    '+BODY'    => 1,
     941                    '+H1'      => 1,
     942                    '-H1'      => 1,
     943                    '+IMG'     => 1,
     944                    '+P'       => 1,
     945                    '#comment' => 1,
     946                    '-P'       => 1,
     947                    '+FOOTER'  => 1,
     948                    '-FOOTER'  => 1,
     949                    '-BODY'    => 1,
     950                    '-HTML'    => 1,
     951                    ''         => 1,
     952                ),
     953            ),
     954
     955            'multiple_tag_instances'    => array(
     956                'html'                  => '
     957                    <html>
     958                        <body>
     959                            <h1>Hello World!</h1>
     960                            <p>First
     961                            <p>Second
     962                            <p>Third
     963                            <ul>
     964                                <li>1
     965                                <li>2
     966                                <li>3
     967                            </ul>
     968                        </body>
     969                    </html>
     970                ',
     971                'expected_token_counts' => array(
     972                    '+HTML' => 1,
     973                    '+HEAD' => 1,
     974                    '-HEAD' => 1,
     975                    '+BODY' => 1,
     976                    '#text' => 13,
     977                    '+H1'   => 1,
     978                    '-H1'   => 1,
     979                    '+P'    => 3,
     980                    '-P'    => 3,
     981                    '+UL'   => 1,
     982                    '+LI'   => 3,
     983                    '-LI'   => 3,
     984                    '-UL'   => 1,
     985                    '-BODY' => 1,
     986                    '-HTML' => 1,
     987                    ''      => 1,
     988                ),
     989            ),
     990
     991            'extreme_nested_formatting' => array(
     992                'html'                  => '
     993                    <html>
     994                        <body>
     995                            <p>
     996                                <strong><em><strike><i><b><u>FORMAT</u></b></i></strike></em></strong>
     997                            </p>
     998                        </body>
     999                    </html>
     1000                ',
     1001                'expected_token_counts' => array(
     1002                    '+HTML'   => 1,
     1003                    '+HEAD'   => 1,
     1004                    '-HEAD'   => 1,
     1005                    '+BODY'   => 1,
     1006                    '#text'   => 7,
     1007                    '+P'      => 1,
     1008                    '+STRONG' => 1,
     1009                    '+EM'     => 1,
     1010                    '+STRIKE' => 1,
     1011                    '+I'      => 1,
     1012                    '+B'      => 1,
     1013                    '+U'      => 1,
     1014                    '-U'      => 1,
     1015                    '-B'      => 1,
     1016                    '-I'      => 1,
     1017                    '-STRIKE' => 1,
     1018                    '-EM'     => 1,
     1019                    '-STRONG' => 1,
     1020                    '-P'      => 1,
     1021                    '-BODY'   => 1,
     1022                    '-HTML'   => 1,
     1023                    ''        => 1,
     1024                ),
     1025            ),
     1026        );
     1027    }
     1028
     1029    /**
     1030     * Ensures that subclasses to WP_HTML_Processor can do bookkeeping by extending the next_token() method.
     1031     *
     1032     * @ticket 62269
     1033     * @dataProvider data_html_processor_with_extended_next_token
     1034     */
     1035    public function test_ensure_next_token_method_extensibility( $html, $expected_token_counts ) {
     1036        require_once DIR_TESTDATA . '/html-api/token-counting-html-processor.php';
     1037
     1038        $processor = Token_Counting_HTML_Processor::create_full_parser( $html );
     1039        while ( $processor->next_tag() ) {
     1040            continue;
     1041        }
     1042
     1043        $this->assertEquals( $expected_token_counts, $processor->token_seen_count, 'Snapshot: ' . var_export( $processor->token_seen_count, true ) );
     1044    }
     1045
     1046    /**
    9121047     * Ensure that lowercased tag_name query matches tags case-insensitively.
    9131048     *
Note: See TracChangeset for help on using the changeset viewer.