Changeset 31266 for trunk/src/wp-content/themes/twentyten/header.php

Timestamp:

01/22/2015 12:41:58 AM (10 years ago)

Author:

lancewillett

Message:

Twenty Ten: fix escaping and minor code style issues. See #29127.

File:

• trunk/src/wp-content/themes/twentyten/header.php (modified) (3 diffs)

trunk/src/wp-content/themes/twentyten/header.php

@@ -31 +31 @@
     // Add a page number if necessary:
     if ( ( $paged >= 2 || $page >= 2 ) && ! is_404() )
-        echo ' | ' . sprintf( __( 'Page %s', 'twentyten' ), max( $paged, $page ) );
+        echo esc_html( ' | ' . sprintf( __( 'Page %s', 'twentyten' ), max( $paged, $page ) ) );
 
     ?></title>
@@ -63 +63 @@
                 <<?php echo $heading_tag; ?> id="site-title">
                     <span>
-                        <a href="<?php echo home_url( '/' ); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a>
+                        <a href="<?php echo esc_url( home_url( '/' ) ); ?>" title="<?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?>" rel="home"><?php bloginfo( 'name' ); ?></a>
                     </span>
                 </<?php echo $heading_tag; ?>>
@@ -97 +97 @@
                         }
                     ?>
-                        <img src="<?php header_image(); ?>" width="<?php echo $header_image_width; ?>" height="<?php echo $header_image_height; ?>" alt="" />
+                        <img src="<?php header_image(); ?>" width="<?php echo esc_attr( $header_image_width ); ?>" height="<?php echo esc_attr( $header_image_height ); ?>" alt="" />
                     <?php endif; ?>
             </div><!-- #branding -->