Changeset 31266 for trunk/src/wp-content/themes/twentyten/loop-attachment.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/loop-attachment.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentyten/loop-attachment.php

@@ -81 +81 @@
         else
             // or get the URL of the first image attachment
-            $next_attachment_url = get_attachment_link( $attachments[ 0 ]->ID );
+            $next_attachment_url = get_attachment_link( $attachments[0]->ID );
     } else {
         // or, if there's only 1 image attachment, get the URL of the image
@@ -112 +112 @@
                         </div><!-- #nav-below -->
 <?php else : ?>
-                        <a href="<?php echo esc_url( wp_get_attachment_url() ); ?>" title="<?php the_title_attribute(); ?>" rel="attachment"><?php echo basename( get_permalink() ); ?></a>
+                        <a href="<?php echo esc_url( wp_get_attachment_url() ); ?>" title="<?php the_title_attribute(); ?>" rel="attachment"><?php echo esc_html( basename( get_permalink() ) ); ?></a>
 <?php endif; ?>
                         </div><!-- .entry-attachment -->
-                        <div class="entry-caption"><?php if ( !empty( $post->post_excerpt ) ) the_excerpt(); ?></div>
+                        <div class="entry-caption"><?php if ( ! empty( $post->post_excerpt ) ) the_excerpt(); ?></div>
 
 <?php the_content( __( 'Continue reading <span class="meta-nav">&rarr;</span>', 'twentyten' ) ); ?>