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/twentyten/loop-attachment.php

    r42343 r43571  
    2020    while ( have_posts() ) :
    2121        the_post();
    22     ?>
     22        ?>
    2323
    2424                <?php if ( ! empty( $post->post_parent ) ) : ?>
     
    2727                                                                /* translators: %s - title of parent post */
    2828                                                                printf( __( '<span class="meta-nav">&larr;</span> %s', 'twentyten' ), get_the_title( $post->post_parent ) );
    29                     ?>
     29                                                                ?>
    3030                    </a></p>
    3131                <?php endif; ?>
     
    4646                                )
    4747                            );
    48                             ?>
     48                        ?>
    4949                            <span class="meta-sep">|</span>
    5050                            <?php
     
    7878                        <div class="entry-content">
    7979                        <div class="entry-attachment">
    80     <?php
    81     if ( wp_attachment_is_image() ) :
    82         $attachments = array_values(
    83             get_children(
    84                 array(
    85                     'post_parent'    => $post->post_parent,
    86                     'post_status'    => 'inherit',
    87                     'post_type'      => 'attachment',
    88                     'post_mime_type' => 'image',
    89                     'order'          => 'ASC',
    90                     'orderby'        => 'menu_order ID',
     80        <?php
     81        if ( wp_attachment_is_image() ) :
     82            $attachments = array_values(
     83                get_children(
     84                    array(
     85                        'post_parent'    => $post->post_parent,
     86                        'post_status'    => 'inherit',
     87                        'post_type'      => 'attachment',
     88                        'post_mime_type' => 'image',
     89                        'order'          => 'ASC',
     90                        'orderby'        => 'menu_order ID',
     91                    )
    9192                )
    92             )
    93         );
    94         foreach ( $attachments as $k => $attachment ) {
    95             if ( $attachment->ID == $post->ID ) {
    96                 break;
     93            );
     94            foreach ( $attachments as $k => $attachment ) {
     95                if ( $attachment->ID == $post->ID ) {
     96                    break;
     97                }
    9798            }
    98         }
    9999
    100         // If there is more than 1 image attachment in a gallery
    101         if ( count( $attachments ) > 1 ) {
    102             $k++;
    103             if ( isset( $attachments[ $k ] ) ) {
    104                 // get the URL of the next image attachment
    105                 $next_attachment_url = get_attachment_link( $attachments[ $k ]->ID );
    106             } else {          // or get the URL of the first image attachment
    107                 $next_attachment_url = get_attachment_link( $attachments[0]->ID );
     100            // If there is more than 1 image attachment in a gallery
     101            if ( count( $attachments ) > 1 ) {
     102                $k++;
     103                if ( isset( $attachments[ $k ] ) ) {
     104                    // get the URL of the next image attachment
     105                    $next_attachment_url = get_attachment_link( $attachments[ $k ]->ID );
     106                } else {          // or get the URL of the first image attachment
     107                    $next_attachment_url = get_attachment_link( $attachments[0]->ID );
     108                }
     109            } else {
     110                // or, if there's only 1 image attachment, get the URL of the image
     111                $next_attachment_url = wp_get_attachment_url();
    108112            }
    109         } else {
    110             // or, if there's only 1 image attachment, get the URL of the image
    111             $next_attachment_url = wp_get_attachment_url();
    112         }
    113     ?>
     113            ?>
    114114                        <p class="attachment"><a href="<?php echo esc_url( $next_attachment_url ); ?>" title="<?php the_title_attribute(); ?>" rel="attachment">
    115115                                                                    <?php
     
    131131                                                                    $attachment_height = apply_filters( 'twentyten_attachment_height', 900 );
    132132                                                                    echo wp_get_attachment_image( $post->ID, array( $attachment_width, $attachment_height ) ); // filterable image width with, essentially, no limit for image height.
    133                             ?>
     133                                                                    ?>
    134134                            </a></p>
    135135
     
    138138                            <div class="nav-next"><?php next_image_link( false ); ?></div>
    139139                        </div><!-- #nav-below -->
    140     <?php else : ?>
     140        <?php else : ?>
    141141                        <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>
    142142    <?php endif; ?>
     
    146146                        if ( ! empty( $post->post_excerpt ) ) {
    147147                            the_excerpt();}
    148 ?>
     148                        ?>
    149149</div>
    150150
    151     <?php the_content( __( 'Continue reading <span class="meta-nav">&rarr;</span>', 'twentyten' ) ); ?>
    152     <?php
    153     wp_link_pages(
    154         array(
    155             'before' => '<div class="page-link">' . __( 'Pages:', 'twentyten' ),
    156             'after'  => '</div>',
    157         )
    158     );
    159     ?>
     151        <?php the_content( __( 'Continue reading <span class="meta-nav">&rarr;</span>', 'twentyten' ) ); ?>
     152        <?php
     153        wp_link_pages(
     154            array(
     155                'before' => '<div class="page-link">' . __( 'Pages:', 'twentyten' ),
     156                'after'  => '</div>',
     157            )
     158        );
     159        ?>
    160160
    161161                    </div><!-- .entry-content -->
     
    167167                </div><!-- #post-## -->
    168168
    169 <?php comments_template(); ?>
     169        <?php comments_template(); ?>
    170170
    171171<?php endwhile;
Note: See TracChangeset for help on using the changeset viewer.