Changeset 25021 for trunk/src/wp-content/themes/twentyfourteen/content-featured-post.php

Timestamp:

08/14/2013 04:38:01 PM (12 years ago)

Author:

lancewillett

Message:

Twenty Fourteen: first pass at matching code standards achieved with Twenty Thirteen development. See #24858, props obenland.

File:

• trunk/src/wp-content/themes/twentyfourteen/content-featured-post.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentyfourteen/content-featured-post.php

@@ -6 +6 @@
 ?>
 
-<article id="post-<?php the_ID(); ?>" <?php post_class( 'clearfix' ); ?>>
-    <a href="<?php the_permalink(); ?>" title="<?php echo esc_attr( sprintf( __( 'Permalink to %s', 'twentyfourteen' ), the_title_attribute( 'echo=0' ) ) ); ?>" rel="<?php the_ID(); ?>" class="attachment-featured-featured">
+<article id="post-<?php the_ID(); ?>" <?php post_class(); ?>>
+    <a class="attachment-featured-featured" href="<?php the_permalink(); ?>" title="<?php echo esc_attr( sprintf( __( 'Permalink to %s', 'twentyfourteen' ), the_title_attribute( 'echo=0' ) ) ); ?>" rel="<?php the_ID(); ?>">
         <?php
-            $images = get_children( array( 'post_parent' => $post->ID, 'post_type' => 'attachment', 'post_mime_type' => 'image', 'orderby' => 'menu_order', 'order' => 'ASC', 'numberposts' => 1 ) );
+            if ( has_post_thumbnail() ) :
+                the_post_thumbnail( 'featured-thumbnail-featured' );
 
-            if ( '' != get_the_post_thumbnail() ) :
-                the_post_thumbnail( 'featured-thumbnail-featured' );
-            elseif ( $images ) :
-                $image = array_shift( $images );
-                echo wp_get_attachment_image( $image->ID, 'featured-thumbnail-featured' );
-            else : ?>
-                <img src="<?php echo get_template_directory_uri(); ?>/images/placeholder.png" alt="" class="featured-thumbnail-featured" /><?php
+            else :
+                $images = get_children( array(
+                    'post_parent'    => get_the_ID(),
+                    'post_type'      => 'attachment',
+                    'post_mime_type' => 'image',
+                    'orderby'        => 'menu_order',
+                    'order'          => 'ASC',
+                    'numberposts'    => 1,
+                ) );
+
+                if ( $images ) :
+                    $image = array_shift( $images );
+                    echo wp_get_attachment_image( $image->ID, 'featured-thumbnail-featured' );
+
+                else : ?>
+                    <img class="featured-thumbnail-featured" src="<?php echo get_template_directory_uri(); ?>/images/placeholder.png" alt="" /><?php
+
+                endif;
             endif;
         ?>
@@ -24 +36 @@
     <div class="entry-wrap">
         <header class="entry-header">
+            <?php if ( in_array( 'category', get_object_taxonomies( get_post_type() ) ) && twentyfourteen_categorized_blog() ) : ?>
             <div class="entry-meta">
-                <?php
-                    /* translators: used between list items, there is a space after the comma */
-                    $categories_list = get_the_category_list( __( ', ', 'twentyfourteen' ) );
-                    if ( $categories_list && twentyfourteen_categorized_blog() ) :
-                ?>
-                <span class="cat-links">
-                    <?php echo $categories_list; ?>
-                </span>
-                <?php endif; // End if categories ?>
+                <span class="cat-links"><?php echo get_the_category_list( _x( ', ', 'Used between list items, there is a space after the comma.', 'twentyfourteen' ) ); ?></span>
             </div><!-- .entry-meta -->
-            <h1 class="entry-title"><a href="<?php the_permalink(); ?>" rel="bookmark"><?php the_title(); ?></a></h1>
+            <?php endif; ?>
+
+            <?php the_title( '<h1 class="entry-title"><a href="' . esc_url( get_permalink() ) . '" rel="bookmark">', '</a></h1>' ); ?>
         </header><!-- .entry-header -->
 
-        <div class="entry-summary clearfix">
+        <div class="entry-summary">
             <?php the_excerpt(); ?>
         </div><!-- .entry-summary -->
     </div>
-
-</article><!-- #post-<?php the_ID(); ?> -->
+</article><!-- #post-## -->