Changeset 46088 for trunk/src/wp-includes/class.wp-styles.php

Timestamp:

09/10/2019 07:22:07 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Docs: Fix placement of some duplicate hook references.

Hook documentation should be on the line directly above the line containing the do_action() or apply_filters() call. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of duplicate hook references.

Includes minor code layout fixes.

See #47110.

File:

• trunk/src/wp-includes/class.wp-styles.php (modified) (3 diffs)

trunk/src/wp-includes/class.wp-styles.php

@@ -200 +200 @@
         $title = isset( $obj->extra['title'] ) ? "title='" . esc_attr( $obj->extra['title'] ) . "'" : '';
 
+        $tag = "<link rel='$rel' id='$handle-css' $title href='$href' type='text/css' media='$media' />\n";
+
         /**
          * Filters the HTML link tag of an enqueued style.
@@ -212 +214 @@
          * @param string $media  The stylesheet's media attribute.
          */
-        $tag = apply_filters( 'style_loader_tag', "<link rel='$rel' id='$handle-css' $title href='$href' type='text/css' media='$media' />\n", $handle, $href, $media );
+        $tag = apply_filters( 'style_loader_tag', $tag, $handle, $href, $media );
 
         if ( 'rtl' === $this->text_direction && isset( $obj->extra['rtl'] ) && $obj->extra['rtl'] ) {
@@ -222 +224 @@
             }
 
+            $rtl_tag = "<link rel='$rel' id='$handle-rtl-css' $title href='$rtl_href' type='text/css' media='$media' />\n";
             /** This filter is documented in wp-includes/class.wp-styles.php */
-            $rtl_tag = apply_filters( 'style_loader_tag', "<link rel='$rel' id='$handle-rtl-css' $title href='$rtl_href' type='text/css' media='$media' />\n", $handle, $rtl_href, $media );
+            $rtl_tag = apply_filters( 'style_loader_tag', $rtl_tag, $handle, $rtl_href, $media );
 
             if ( $obj->extra['rtl'] === 'replace' ) {