From 2c86860c5ee4645d2edb44c56112c52c31d9e483 Mon Sep 17 00:00:00 2001 From: Peter Wilson Date: Thu, 22 Nov 2018 00:24:48 +0000 Subject: [PATCH] Formatting: Revert pre-save filter adding `rel="noopener"`. Removes filters adding `rel="noopener"` to links targeting `_blank`. Previous implementation could introduce blank `rel` attributes and could corrupt JSON data when saving via the customizer. See #43187. Reverts [43732] and [43733] from the 5.0 branch. Built from https://develop.svn.wordpress.org/branches/5.0@43930 git-svn-id: http://core.svn.wordpress.org/branches/5.0@43762 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/default-filters.php | 15 -------- wp-includes/formatting.php | 63 --------------------------------- wp-includes/version.php | 2 +- 3 files changed, 1 insertion(+), 79 deletions(-) diff --git a/wp-includes/default-filters.php b/wp-includes/default-filters.php index dfc0508f05..c1393cb066 100644 --- a/wp-includes/default-filters.php +++ b/wp-includes/default-filters.php @@ -118,21 +118,6 @@ foreach ( array( 'content_save_pre', 'excerpt_save_pre', 'comment_save_pre', 'pr add_filter( $filter, 'balanceTags', 50 ); } -// Add proper rel values for links with target. -foreach ( array( - 'title_save_pre', - 'content_save_pre', - 'excerpt_save_pre', - 'content_filtered_save_pre', - 'pre_comment_content', - 'pre_term_description', - 'pre_link_description', - 'pre_link_notes', - 'pre_user_description', -) as $filter ) { - add_filter( $filter, 'wp_targeted_link_rel' ); -}; - // Format strings for display. foreach ( array( 'comment_author', 'term_name', 'link_name', 'link_description', 'link_notes', 'bloginfo', 'wp_title', 'widget_title' ) as $filter ) { add_filter( $filter, 'wptexturize' ); diff --git a/wp-includes/formatting.php b/wp-includes/formatting.php index a3f47b9c1d..f5ac4accf0 100644 --- a/wp-includes/formatting.php +++ b/wp-includes/formatting.php @@ -2773,69 +2773,6 @@ function wp_rel_nofollow_callback( $matches ) { return ""; } -/** - * Adds rel noreferrer and noopener to all HTML A elements that have a target. - * - * @param string $text Content that may contain HTML A elements. - * @return string Converted content. - */ -function wp_targeted_link_rel( $text ) { - // Don't run (more expensive) regex if no links with targets. - if ( stripos( $text, 'target' ) !== false && stripos( $text, ']*target\s*=[^>]*)>|i', 'wp_targeted_link_rel_callback', $text ); - } - - return $text; -} - -/** - * Callback to add rel="noreferrer noopener" string to HTML A element. - * - * Will not duplicate existing noreferrer and noopener values - * to prevent from invalidating the HTML. - * - * @param array $matches Single Match - * @return string HTML A Element with rel noreferrer noopener in addition to any existing values - */ -function wp_targeted_link_rel_callback( $matches ) { - $link_html = $matches[1]; - $rel_match = array(); - - /** - * Filters the rel values that are added to links with `target` attribute. - * - * @since 5.0.0 - * - * @param string The rel values. - * @param string $link_html The matched content of the link tag including all HTML attributes. - */ - $rel = apply_filters( 'wp_targeted_link_rel', 'noopener noreferrer', $link_html ); - - // Value with delimiters, spaces around are optional. - $attr_regex = '|rel\s*=\s*?(\\\\{0,1}["\'])(.*?)\\1|i'; - preg_match( $attr_regex, $link_html, $rel_match ); - - if ( empty( $rel_match[0] ) ) { - // No delimiters, try with a single value and spaces, because `rel = va"lue` is totally fine... - $attr_regex = '|rel\s*=(\s*)([^\s]*)|i'; - preg_match( $attr_regex, $link_html, $rel_match ); - } - - if ( ! empty( $rel_match[0] ) ) { - $parts = preg_split( '|\s+|', strtolower( $rel_match[2] ) ); - $parts = array_map( 'esc_attr', $parts ); - $needed = explode( ' ', $rel ); - $parts = array_unique( array_merge( $parts, $needed ) ); - $delimiter = trim( $rel_match[1] ) ? $rel_match[1] : '"'; - $rel = 'rel=' . $delimiter . trim( implode( ' ', $parts ) ) . $delimiter; - $link_html = str_replace( $rel_match[0], $rel, $link_html ); - } else { - $link_html .= " rel=\"$rel\""; - } - - return ""; -} - /** * Convert one smiley code to the icon graphic file equivalent. * diff --git a/wp-includes/version.php b/wp-includes/version.php index cfc11247da..3770cc94a4 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -4,7 +4,7 @@ * * @global string $wp_version */ -$wp_version = '5.0-beta5-43929'; +$wp_version = '5.0-beta5-43930'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.