Make WordPress Core

Ticket #43187: 43187.12.diff

File 43187.12.diff, 9.5 KB (added by notnownikki, 6 years ago)
  • src/wp-includes/default-filters.php

     
    104104        add_filter( $filter, 'balanceTags', 50 );
    105105}
    106106
     107// Add proper rel values for links with target.
     108foreach ( array( 'pre_user_email', 'title_save_pre', 'pre_user_first_name', 'pre_term_description', 'pre_link_name', 'pre_link_target', 'pre_user_nickname', 'pre_comment_author_email', 'pre_link_notes', 'pre_user_last_name', 'pre_term_name', 'pre_user_display_name', 'pre_comment_content', 'pre_link_rel', 'content_save_pre', 'pre_user_description', 'content_filtered_save_pre', 'excerpt_save_pre', 'pre_comment_author_name', 'pre_link_description' ) as $filter ) {
     109        add_filter( $filter, 'wp_targeted_link_rel' );
     110};
     111
    107112// Format strings for display.
    108113foreach ( array( 'comment_author', 'term_name', 'link_name', 'link_description', 'link_notes', 'bloginfo', 'wp_title', 'widget_title' ) as $filter ) {
    109114        add_filter( $filter, 'wptexturize' );
  • src/wp-includes/formatting.php

     
    30243024}
    30253025
    30263026/**
     3027 * Adds rel noreferrer and noopener to all HTML A elements that have a target.
     3028 *
     3029 * @since 5.0.0
     3030 *
     3031 * @param string $text Content that may contain HTML A elements.
     3032 * @return string Converted content.
     3033 */
     3034function wp_targeted_link_rel( $text ) {
     3035        $text = preg_replace_callback( '|<a\s([^>]*target\s*=[^>]*)>|i', 'wp_targeted_link_rel_callback', $text );
     3036        return $text;
     3037}
     3038
     3039/**
     3040 * Callback to add rel="noreferrer noopener" string to HTML A element.
     3041 *
     3042 * Will not duplicate existing noreferrer and noopener values
     3043 * to prevent from invalidating the HTML.
     3044 *
     3045 * @since 5.0.0
     3046 *
     3047 * @param array $matches Single Match
     3048 * @return string HTML A Element with rel noreferrer noopener in addition to any existing values
     3049 */
     3050function wp_targeted_link_rel_callback( $matches ) {
     3051        $link_html = $matches[1];
     3052        $rel_match = array();
     3053
     3054        /**
     3055         * Filters the rel values that are added to links with `target` attribute.
     3056         *
     3057         * @since 5.0.0
     3058         *
     3059         * @param string The rel values.
     3060         * @param string $link_html The matched content of the link tag including all HTML attributes.
     3061         */
     3062        $rel = apply_filters( 'wp_targeted_link_rel', 'noopener noreferrer', $link_html );
     3063
     3064        // Value with delimiters, spaces around are optional.
     3065        $attr_regex  = '|rel\s*=\s*?(\\\\{0,1}["\'])(.*?)\\1|i';
     3066        preg_match( $attr_regex, $link_html, $rel_match );
     3067
     3068        if ( empty( $rel_match[0] ) ) {
     3069                // No delimiters, try with a single value and spaces, because `rel =  va"lue` is totally fine...
     3070                $attr_regex  = '|rel\s*=(\s*)([^\s]*)|i';
     3071                preg_match( $attr_regex, $link_html, $rel_match );
     3072        }
     3073
     3074        if ( ! empty( $rel_match[0] ) ) {
     3075                $parts = preg_split( '|\s+|', strtolower( $rel_match[2] ) );
     3076                $parts = array_map( 'esc_attr', $parts );
     3077                $needed = explode( ' ', $rel );
     3078                $parts = array_unique( array_merge( $parts, $needed ) );
     3079                $delimiter = trim( $rel_match[1] ) ? $rel_match[1] : '"';
     3080                $rel = 'rel=' . $delimiter . trim( implode( ' ', $parts ) ) . $delimiter;
     3081                $link_html = str_replace( $rel_match[0], $rel, $link_html );
     3082        } else {
     3083                $link_html .= " rel=\"$rel\"";
     3084        }
     3085
     3086        return "<a $link_html>";
     3087}
     3088
     3089/**
    30273090 * Convert one smiley code to the icon graphic file equivalent.
    30283091 *
    30293092 * Callback handler for convert_smilies().
  • tests/phpunit/tests/formatting/WPTargetedLinkRel.php

     
     1<?php
     2
     3/**
     4 * @group formatting
     5 */
     6class Tests_Targeted_Link_Rel extends WP_UnitTestCase {
     7
     8        public function test_add_to_links_with_target() {
     9                $content  = '<p>Links: <a href="/" target="_blank">No rel</a></p>';
     10                $expected = '<p>Links: <a href="/" target="_blank" rel="noopener noreferrer">No rel</a></p>';
     11                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     12        }
     13
     14        public function test_target_as_first_attribute() {
     15                $content  = '<p>Links: <a target="_blank" href="#">No rel</a></p>';
     16                $expected = '<p>Links: <a target="_blank" href="#" rel="noopener noreferrer">No rel</a></p>';
     17                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     18        }
     19
     20        public function test_add_to_existing_rel() {
     21                $content  = '<p>Links: <a href="/" rel="existing values" target="_blank">Existing rel</a></p>';
     22                $expected = '<p>Links: <a href="/" rel="existing values noopener noreferrer" target="_blank">Existing rel</a></p>';
     23                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     24        }
     25
     26        public function test_no_duplicate_values_added() {
     27                $content  = '<p>Links: <a href="/" rel="existing noopener values" target="_blank">Existing rel</a></p>';
     28                $expected = '<p>Links: <a href="/" rel="existing noopener values noreferrer" target="_blank">Existing rel</a></p>';
     29                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     30        }
     31
     32        public function test_rel_with_single_quote_delimiter() {
     33                $content  = '<p>Links: <a href="/" rel=\'existing values\' target="_blank">Existing rel</a></p>';
     34                $expected = '<p>Links: <a href="/" rel=\'existing values noopener noreferrer\' target="_blank">Existing rel</a></p>';
     35                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     36        }
     37
     38        public function test_rel_with_no_delimiter() {
     39                $content  = '<p>Links: <a href="/" rel=existing target="_blank">Existing rel</a></p>';
     40                $expected = '<p>Links: <a href="/" rel="existing noopener noreferrer" target="_blank">Existing rel</a></p>';
     41                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     42        }
     43
     44        public function test_rel_value_spaced_and_no_delimiter() {
     45                $content  = '<p>Links: <a href="/" rel = existing target="_blank">Existing rel</a></p>';
     46                $expected = '<p>Links: <a href="/" rel="existing noopener noreferrer" target="_blank">Existing rel</a></p>';
     47                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     48        }
     49
     50        public function test_rel_value_spaced_and_no_delimiter_and_values_to_escape() {
     51                $content  = '<p>Links: <a href="/" rel = existing"value target="_blank">Existing rel</a></p>';
     52                $expected = '<p>Links: <a href="/" rel="existing&quot;value noopener noreferrer" target="_blank">Existing rel</a></p>';
     53                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     54        }
     55
     56        public function test_escaped_quotes() {
     57                $content  = '<p>Links: <a href=\"/\" rel=\"existing values\" target=\"_blank\">Existing rel</a></p>';
     58                $expected = '<p>Links: <a href=\"/\" rel=\"existing values noopener noreferrer\" target=\"_blank\">Existing rel</a></p>';
     59                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     60        }
     61
     62        public function test_ignore_links_with_no_target() {
     63                $content  = '<p>Links: <a href="/" target="_blank">Change me</a> <a href="/">Do not change me</a></p>';
     64                $expected = '<p>Links: <a href="/" target="_blank" rel="noopener noreferrer">Change me</a> <a href="/">Do not change me</a></p>';
     65                $this->assertEquals( $expected, wp_targeted_link_rel( $content ) );
     66        }
     67}
  • tests/phpunit/tests/rest-api/rest-attachments-controller.php

    Property changes on: tests/phpunit/tests/formatting/WPTargetedLinkRel.php
    ___________________________________________________________________
    Added: svn:executable
    ## -0,0 +1 ##
    +*
    \ No newline at end of property
     
    10151015                                                'rendered' => '<a href="#">link</a>',
    10161016                                        ),
    10171017                                        'description' => array(
    1018                                                 'raw'      => '<a href="#" target="_blank">link</a>',
    1019                                                 'rendered' => '<p><a href="#" target="_blank">link</a></p>',
     1018                                                'raw'      => '<a href="#" target="_blank" rel="noopener noreferrer">link</a>',
     1019                                                'rendered' => '<p><a href="#" target="_blank" rel="noopener noreferrer">link</a></p>',
    10201020                                        ),
    10211021                                        'caption'     => array(
    1022                                                 'raw'      => '<a href="#" target="_blank">link</a>',
    1023                                                 'rendered' => '<p><a href="#" target="_blank">link</a></p>',
     1022                                                'raw'      => '<a href="#" target="_blank" rel="noopener noreferrer">link</a>',
     1023                                                'rendered' => '<p><a href="#" target="_blank" rel="noopener noreferrer">link</a></p>',
    10241024                                        ),
    10251025                                ),
    10261026                        ),
  • tests/phpunit/tests/rest-api/rest-posts-controller.php

     
    31853185                                                'rendered' => '<a href="#">link</a>',
    31863186                                        ),
    31873187                                        'content' => array(
    3188                                                 'raw'      => '<a href="#" target="_blank">link</a>',
    3189                                                 'rendered' => '<p><a href="#" target="_blank">link</a></p>',
     3188                                                'raw'      => '<a href="#" target="_blank" rel="noopener noreferrer">link</a>',
     3189                                                'rendered' => '<p><a href="#" target="_blank" rel="noopener noreferrer">link</a></p>',
    31903190                                        ),
    31913191                                        'excerpt' => array(
    3192                                                 'raw'      => '<a href="#" target="_blank">link</a>',
    3193                                                 'rendered' => '<p><a href="#" target="_blank">link</a></p>',
     3192                                                'raw'      => '<a href="#" target="_blank" rel="noopener noreferrer">link</a>',
     3193                                                'rendered' => '<p><a href="#" target="_blank" rel="noopener noreferrer">link</a></p>',
    31943194                                        ),
    31953195                                ),
    31963196                        ),