WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (3 years ago)
Author:
pento
Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

  • Multiline function calls must now put each parameter on a new line.
  • Auto-formatting files is now part of the grunt precommit script.
  • Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-content/themes/twentyeleven/showcase.php

    r42343 r43571  
    2929                    ?>
    3030
    31                 <?php
     31                    <?php
    3232                    /*
    3333                     * We are using a heading by rendering the_content
    3434                     * If we have content for this page, let's display it.
    3535                     */
    36                 if ( '' != get_the_content() ) {
    37                     get_template_part( 'content', 'intro' );
    38                 }
    39                 ?>
     36                    if ( '' != get_the_content() ) {
     37                        get_template_part( 'content', 'intro' );
     38                    }
     39                    ?>
    4040
    4141                <?php endwhile; ?>
     
    7878                            $header_image_width = HEADER_IMAGE_WIDTH;
    7979                        }
    80                     ?>
     80                        ?>
    8181
    8282                    <div class="featured-posts">
    8383                    <h1 class="showcase-heading"><?php _e( 'Featured Post', 'twentyeleven' ); ?></h1>
    8484
    85                 <?php
    86                 // Let's roll.
    87                 while ( $featured->have_posts() ) :
    88                     $featured->the_post();
    89 
    90                     // Increase the counter.
    91                     $counter_slider++;
    92 
    93                     /*
    94                     * We're going to add a class to our featured post for featured images
    95                     * by default it'll have the feature-text class.
    96                     */
    97                     $feature_class = 'feature-text';
    98 
    99                     if ( has_post_thumbnail() ) {
    100                         // ... but if it has a featured image let's add some class
    101                         $feature_class = 'feature-image small';
    102 
    103                         // Hang on. Let's check this here image out.
    104                         $image = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), array( $header_image_width, $header_image_width ) );
    105 
    106                         // Is it bigger than or equal to our header?
    107                         if ( $image[1] >= $header_image_width ) {
    108                             // If bigger, let's add a BIGGER class. It's EXTRA classy now.
    109                             $feature_class = 'feature-image large';
    110                         }
    111                     }
    112                     ?>
     85                        <?php
     86                        // Let's roll.
     87                        while ( $featured->have_posts() ) :
     88                            $featured->the_post();
     89
     90                            // Increase the counter.
     91                            $counter_slider++;
     92
     93                            /*
     94                            * We're going to add a class to our featured post for featured images
     95                            * by default it'll have the feature-text class.
     96                            */
     97                            $feature_class = 'feature-text';
     98
     99                            if ( has_post_thumbnail() ) {
     100                                // ... but if it has a featured image let's add some class
     101                                $feature_class = 'feature-image small';
     102
     103                                // Hang on. Let's check this here image out.
     104                                $image = wp_get_attachment_image_src( get_post_thumbnail_id( $post->ID ), array( $header_image_width, $header_image_width ) );
     105
     106                                // Is it bigger than or equal to our header?
     107                                if ( $image[1] >= $header_image_width ) {
     108                                    // If bigger, let's add a BIGGER class. It's EXTRA classy now.
     109                                    $feature_class = 'feature-image large';
     110                                }
     111                            }
     112                            ?>
    113113
    114114                    <section class="featured-post <?php echo esc_attr( $feature_class ); ?>" id="featured-post-<?php echo esc_attr( $counter_slider ); ?>">
    115115
    116                     <?php
    117                         /*
    118                         * If the thumbnail is as big as the header image
    119                         * make it a large featured post, otherwise render it small
    120                         */
    121                     if ( has_post_thumbnail() ) {
    122                         if ( $image[1] >= $header_image_width ) {
    123                             $thumbnail_size = 'large-feature';
    124                         } else {
    125                             $thumbnail_size = 'small-feature';
    126                         }
    127                         ?>
     116                            <?php
     117                                /*
     118                                * If the thumbnail is as big as the header image
     119                                * make it a large featured post, otherwise render it small
     120                                */
     121                            if ( has_post_thumbnail() ) {
     122                                if ( $image[1] >= $header_image_width ) {
     123                                    $thumbnail_size = 'large-feature';
     124                                } else {
     125                                    $thumbnail_size = 'small-feature';
     126                                }
     127                                ?>
    128128                        <a href="<?php the_permalink(); ?>" title="<?php echo esc_attr( sprintf( __( 'Permalink to %s', 'twentyeleven' ), the_title_attribute( 'echo=0' ) ) ); ?>" rel="bookmark"><?php the_post_thumbnail( $thumbnail_size ); ?></a>
    129                             <?php
    130                     }
    131                         ?>
    132                         <?php get_template_part( 'content', 'featured' ); ?>
     129                                <?php
     130                            }
     131                            ?>
     132                            <?php get_template_part( 'content', 'featured' ); ?>
    133133                    </section>
    134                 <?php endwhile; ?>
    135 
    136                     <?php
    137                     // Show slider only if we have more than one featured post.
    138                     if ( $featured->post_count > 1 ) :
    139                     ?>
     134                        <?php endwhile; ?>
     135
     136                        <?php
     137                        // Show slider only if we have more than one featured post.
     138                        if ( $featured->post_count > 1 ) :
     139                            ?>
    140140                    <nav class="feature-slider">
    141141                    <ul>
    142                     <?php
    143 
    144                     // Reset the counter so that we end up with matching elements
    145                     $counter_slider = 0;
    146 
    147                     // Begin from zero
    148                     rewind_posts();
    149 
    150                     // Let's roll again.
    151                     while ( $featured->have_posts() ) :
    152                         $featured->the_post();
    153                         $counter_slider++;
    154                         if ( 1 == $counter_slider ) {
    155                             $class = ' class="active"';
    156                         } else {
    157                             $class = '';
    158                         }
    159                     ?>
     142                            <?php
     143
     144                            // Reset the counter so that we end up with matching elements
     145                            $counter_slider = 0;
     146
     147                            // Begin from zero
     148                            rewind_posts();
     149
     150                            // Let's roll again.
     151                            while ( $featured->have_posts() ) :
     152                                $featured->the_post();
     153                                $counter_slider++;
     154                                if ( 1 == $counter_slider ) {
     155                                    $class = ' class="active"';
     156                                } else {
     157                                    $class = '';
     158                                }
     159                                ?>
    160160                    <li><a href="#featured-post-<?php echo esc_attr( $counter_slider ); ?>" title="<?php echo esc_attr( sprintf( __( 'Featuring: %s', 'twentyeleven' ), the_title_attribute( 'echo=0' ) ) ); ?>"<?php echo $class; ?>></a></li>
    161                     <?php endwhile; ?>
     161                        <?php endwhile; ?>
    162162                    </ul>
    163163                    </nav>
     
    207207                    while ( $recent->have_posts() ) :
    208208                        $recent->the_post();
    209                     ?>
     209                        ?>
    210210
    211211                        <li class="entry-title">
     
    216216                        </li>
    217217
    218                     <?php
     218                        <?php
    219219                    endwhile;
    220220
     
    231231                        <?php
    232232                        the_widget(
    233                             'Twenty_Eleven_Ephemera_Widget', '', array(
     233                            'Twenty_Eleven_Ephemera_Widget',
     234                            '',
     235                            array(
    234236                                'before_title' => '<h3 class="widget-title">',
    235237                                'after_title'  => '</h3>',
Note: See TracChangeset for help on using the changeset viewer.