Changeset 42343 for trunk/src/wp-content/themes/twentytwelve/inc/custom-header.php

Timestamp:

11/30/2017 11:09:33 PM (8 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

• trunk/src/wp-content/themes/twentytwelve/inc/custom-header.php (modified) (4 diffs)

trunk/src/wp-content/themes/twentytwelve/inc/custom-header.php

@@ -55 +55 @@
 function twentytwelve_custom_header_fonts() {
     $font_url = twentytwelve_get_font_url();
-    if ( ! empty( $font_url ) )
+    if ( ! empty( $font_url ) ) {
         wp_enqueue_style( 'twentytwelve-fonts', esc_url_raw( $font_url ), array(), null );
+    }
 }
 add_action( 'admin_print_styles-appearance_page_custom-header', 'twentytwelve_custom_header_fonts' );
@@ -71 +72 @@
 
     // If no custom options for text are set, let's bail
-    if ( $text_color == get_theme_support( 'custom-header', 'default-text-color' ) )
+    if ( $text_color == get_theme_support( 'custom-header', 'default-text-color' ) ) {
         return;
+    }
 
     // If we get this far, we have custom styles.
@@ -79 +81 @@
     <?php
         // Has the text been hidden?
-        if ( ! display_header_text() ) :
+    if ( ! display_header_text() ) :
     ?>
-        .site-title,
-        .site-description {
-            position: absolute;
-            clip: rect(1px 1px 1px 1px); /* IE7 */
-            clip: rect(1px, 1px, 1px, 1px);
-        }
+    .site-title,
+    .site-description {
+        position: absolute;
+        clip: rect(1px 1px 1px 1px); /* IE7 */
+        clip: rect(1px, 1px, 1px, 1px);
+    }
     <?php
-        // If the user has set a custom color for the text, use that.
+    // If the user has set a custom color for the text, use that.
         else :
     ?>
@@ -156 +158 @@
         <h1 class="displaying-header-text"><a id="name" style="<?php echo esc_attr( $style ); ?>" onclick="return false;" href="<?php echo esc_url( home_url( '/' ) ); ?>"><?php bloginfo( 'name' ); ?></a></h1>
         <h2 id="desc" class="displaying-header-text" style="<?php echo esc_attr( $style ); ?>"><?php bloginfo( 'description' ); ?></h2>
-        <?php $header_image = get_header_image();
-        if ( ! empty( $header_image ) ) : ?>
+        <?php
+        $header_image = get_header_image();
+        if ( ! empty( $header_image ) ) :
+        ?>
             <img src="<?php echo esc_url( $header_image ); ?>" class="header-image" width="<?php echo esc_attr( get_custom_header()->width ); ?>" height="<?php echo esc_attr( get_custom_header()->height ); ?>" alt="" />
         <?php endif; ?>
     </div>
-<?php }
+<?php
+}