Changeset 5578 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/formatting.php

Timestamp:

06/15/2017 10:48:35 PM (7 years ago)

Author:

coffee2code

Message:

developer.wordpress.org: Change regex for @see linking of hooks to recognize characters and HTML entities that may be present in hook names.

This is valid {@see 'comment_$new_status_$comment->comment_type'} but failed to be recognized as a hook due to the presence of & and ; (from the encoded value of >) and the -.

Example: ​https://developer.wordpress.org/reference/functions/wp_transition_comment_status/

Fixes #2880.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/formatting.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/formatting.php

@@ -177 +177 @@
 
         // Link to hook: {@see 'pre_get_search_form'}
-        elseif ( 1 === preg_match( '/^(?:\'|(?:‘))([\$\w]+)(?:\'|(?:’))$/', $link, $hook ) ) {
+        elseif ( 1 === preg_match( '/^(?:\'|(?:‘))([\$\w-&;]+)(?:\'|(?:’))$/', $link, $hook ) ) {
             if ( ! empty( $hook[1] ) ) {
                 $link = '<a href="' .
                         get_post_type_archive_link( 'wp-parser-hook' ) .
-                        sanitize_key( $hook[1] ) . '/' .
+                        sanitize_key( html_entity_decode( $hook[1] ) ) . '/' .
                         '">' . esc_html( $link ) . '</a>';
             }