WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (3 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/class-wp-ms-themes-list-table.php

    r41683 r42343  
    3838        global $status, $page;
    3939
    40         parent::__construct( array(
    41             'plural' => 'themes',
    42             'screen' => isset( $args['screen'] ) ? $args['screen'] : null,
    43         ) );
     40        parent::__construct(
     41            array(
     42                'plural' => 'themes',
     43                'screen' => isset( $args['screen'] ) ? $args['screen'] : null,
     44            )
     45        );
    4446
    4547        $status = isset( $_REQUEST['theme_status'] ) ? $_REQUEST['theme_status'] : 'all';
    46         if ( !in_array( $status, array( 'all', 'enabled', 'disabled', 'upgrade', 'search', 'broken' ) ) )
     48        if ( ! in_array( $status, array( 'all', 'enabled', 'disabled', 'upgrade', 'search', 'broken' ) ) ) {
    4749            $status = 'all';
     50        }
    4851
    4952        $page = $this->get_pagenum();
     
    5154        $this->is_site_themes = ( 'site-themes-network' === $this->screen->id ) ? true : false;
    5255
    53         if ( $this->is_site_themes )
     56        if ( $this->is_site_themes ) {
    5457            $this->site_id = isset( $_REQUEST['id'] ) ? intval( $_REQUEST['id'] ) : 0;
    55     }
    56 
    57     /**
    58      *
     58        }
     59    }
     60
     61    /**
    5962     * @return array
    6063     */
     
    6568
    6669    /**
    67      *
    6870     * @return bool
    6971     */
    7072    public function ajax_user_can() {
    71         if ( $this->is_site_themes )
     73        if ( $this->is_site_themes ) {
    7274            return current_user_can( 'manage_sites' );
    73         else
     75        } else {
    7476            return current_user_can( 'manage_network_themes' );
    75     }
    76 
    77     /**
    78      *
     77        }
     78    }
     79
     80    /**
    7981     * @global string $status
    8082     * @global array $totals
     
    98100             * @param array $all An array of WP_Theme objects to display in the list table.
    99101             */
    100             'all' => apply_filters( 'all_themes', wp_get_themes() ),
    101             'search' => array(),
    102             'enabled' => array(),
     102            'all'      => apply_filters( 'all_themes', wp_get_themes() ),
     103            'search'   => array(),
     104            'enabled'  => array(),
    103105            'disabled' => array(),
    104             'upgrade' => array(),
    105             'broken' => $this->is_site_themes ? array() : wp_get_themes( array( 'errors' => true ) ),
     106            'upgrade'  => array(),
     107            'broken'   => $this->is_site_themes ? array() : wp_get_themes( array( 'errors' => true ) ),
    106108        );
    107109
    108110        if ( $this->is_site_themes ) {
    109111            $themes_per_page = $this->get_items_per_page( 'site_themes_network_per_page' );
    110             $allowed_where = 'site';
     112            $allowed_where   = 'site';
    111113        } else {
    112114            $themes_per_page = $this->get_items_per_page( 'themes_network_per_page' );
    113             $allowed_where = 'network';
     115            $allowed_where   = 'network';
    114116        }
    115117
     
    124126            if ( $maybe_update && isset( $current->response[ $key ] ) ) {
    125127                $themes['all'][ $key ]->update = true;
    126                 $themes['upgrade'][ $key ] = $themes['all'][ $key ];
    127             }
    128 
    129             $filter = $theme->is_allowed( $allowed_where, $this->site_id ) ? 'enabled' : 'disabled';
     128                $themes['upgrade'][ $key ]     = $themes['all'][ $key ];
     129            }
     130
     131            $filter                    = $theme->is_allowed( $allowed_where, $this->site_id ) ? 'enabled' : 'disabled';
    130132            $themes[ $filter ][ $key ] = $themes['all'][ $key ];
    131133        }
    132134
    133135        if ( $s ) {
    134             $status = 'search';
     136            $status           = 'search';
    135137            $themes['search'] = array_filter( array_merge( $themes['all'], $themes['broken'] ), array( $this, '_search_callback' ) );
    136138        }
    137139
    138140        $totals = array();
    139         foreach ( $themes as $type => $list )
     141        foreach ( $themes as $type => $list ) {
    140142            $totals[ $type ] = count( $list );
    141 
    142         if ( empty( $themes[ $status ] ) && !in_array( $status, array( 'all', 'search' ) ) )
     143        }
     144
     145        if ( empty( $themes[ $status ] ) && ! in_array( $status, array( 'all', 'search' ) ) ) {
    143146            $status = 'all';
     147        }
    144148
    145149        $this->items = $themes[ $status ];
     
    149153        $total_this_page = $totals[ $status ];
    150154
    151         wp_localize_script( 'updates', '_wpUpdatesItemCounts', array(
    152             'themes' => $totals,
    153             'totals' => wp_get_update_data(),
    154         ) );
     155        wp_localize_script(
     156            'updates', '_wpUpdatesItemCounts', array(
     157                'themes' => $totals,
     158                'totals' => wp_get_update_data(),
     159            )
     160        );
    155161
    156162        if ( $orderby ) {
    157163            $orderby = ucfirst( $orderby );
    158             $order = strtoupper( $order );
     164            $order   = strtoupper( $order );
    159165
    160166            if ( $orderby === 'Name' ) {
     
    169175        $start = ( $page - 1 ) * $themes_per_page;
    170176
    171         if ( $total_this_page > $themes_per_page )
     177        if ( $total_this_page > $themes_per_page ) {
    172178            $this->items = array_slice( $this->items, $start, $themes_per_page, true );
    173 
    174         $this->set_pagination_args( array(
    175             'total_items' => $total_this_page,
    176             'per_page' => $themes_per_page,
    177         ) );
     179        }
     180
     181        $this->set_pagination_args(
     182            array(
     183                'total_items' => $total_this_page,
     184                'per_page'    => $themes_per_page,
     185            )
     186        );
    178187    }
    179188
     
    185194    public function _search_callback( $theme ) {
    186195        static $term = null;
    187         if ( is_null( $term ) )
     196        if ( is_null( $term ) ) {
    188197            $term = wp_unslash( $_REQUEST['s'] );
     198        }
    189199
    190200        foreach ( array( 'Name', 'Description', 'Author', 'Author', 'AuthorURI' ) as $field ) {
    191201            // Don't mark up; Do translate.
    192             if ( false !== stripos( $theme->display( $field, false, true ), $term ) )
     202            if ( false !== stripos( $theme->display( $field, false, true ), $term ) ) {
    193203                return true;
    194         }
    195 
    196         if ( false !== stripos( $theme->get_stylesheet(), $term ) )
     204            }
     205        }
     206
     207        if ( false !== stripos( $theme->get_stylesheet(), $term ) ) {
    197208            return true;
    198 
    199         if ( false !== stripos( $theme->get_template(), $term ) )
     209        }
     210
     211        if ( false !== stripos( $theme->get_template(), $term ) ) {
    200212            return true;
     213        }
    201214
    202215        return false;
     
    217230        $b = $theme_b[ $orderby ];
    218231
    219         if ( $a == $b )
     232        if ( $a == $b ) {
    220233            return 0;
    221 
    222         if ( 'DESC' === $order )
     234        }
     235
     236        if ( 'DESC' === $order ) {
    223237            return ( $a < $b ) ? 1 : -1;
    224         else
     238        } else {
    225239            return ( $a < $b ) ? -1 : 1;
     240        }
    226241    }
    227242
     
    237252
    238253    /**
    239      *
    240254     * @return array
    241255     */
     
    249263
    250264    /**
    251      *
    252265     * @return array
    253266     */
    254267    protected function get_sortable_columns() {
    255268        return array(
    256             'name'         => 'name',
     269            'name' => 'name',
    257270        );
    258271    }
     
    270283
    271284    /**
    272      *
    273285     * @global array $totals
    274286     * @global string $status
     
    280292        $status_links = array();
    281293        foreach ( $totals as $type => $count ) {
    282             if ( !$count )
     294            if ( ! $count ) {
    283295                continue;
     296            }
    284297
    285298            switch ( $type ) {
     
    296309                    $text = _n( 'Update Available <span class="count">(%s)</span>', 'Update Available <span class="count">(%s)</span>', $count );
    297310                    break;
    298                 case 'broken' :
     311                case 'broken':
    299312                    $text = _n( 'Broken <span class="count">(%s)</span>', 'Broken <span class="count">(%s)</span>', $count );
    300313                    break;
    301314            }
    302315
    303             if ( $this->is_site_themes )
     316            if ( $this->is_site_themes ) {
    304317                $url = 'site-themes.php?id=' . $this->site_id;
    305             else
     318            } else {
    306319                $url = 'themes.php';
     320            }
    307321
    308322            if ( 'search' != $type ) {
    309                 $status_links[$type] = sprintf( "<a href='%s'%s>%s</a>",
    310                     esc_url( add_query_arg('theme_status', $type, $url) ),
     323                $status_links[ $type ] = sprintf(
     324                    "<a href='%s'%s>%s</a>",
     325                    esc_url( add_query_arg( 'theme_status', $type, $url ) ),
    311326                    ( $type === $status ) ? ' class="current" aria-current="page"' : '',
    312327                    sprintf( $text, number_format_i18n( $count ) )
     
    327342
    328343        $actions = array();
    329         if ( 'enabled' != $status )
     344        if ( 'enabled' != $status ) {
    330345            $actions['enable-selected'] = $this->is_site_themes ? __( 'Enable' ) : __( 'Network Enable' );
    331         if ( 'disabled' != $status )
     346        }
     347        if ( 'disabled' != $status ) {
    332348            $actions['disable-selected'] = $this->is_site_themes ? __( 'Disable' ) : __( 'Network Disable' );
     349        }
    333350        if ( ! $this->is_site_themes ) {
    334             if ( current_user_can( 'update_themes' ) )
     351            if ( current_user_can( 'update_themes' ) ) {
    335352                $actions['update-selected'] = __( 'Update' );
    336             if ( current_user_can( 'delete_themes' ) )
     353            }
     354            if ( current_user_can( 'delete_themes' ) ) {
    337355                $actions['delete-selected'] = __( 'Delete' );
     356            }
    338357        }
    339358        return $actions;
     
    343362     */
    344363    public function display_rows() {
    345         foreach ( $this->items as $theme )
     364        foreach ( $this->items as $theme ) {
    346365            $this->single_row( $theme );
     366        }
    347367    }
    348368
     
    355375     */
    356376    public function column_cb( $theme ) {
    357         $checkbox_id = 'checkbox_' . md5( $theme->get('Name') );
     377        $checkbox_id = 'checkbox_' . md5( $theme->get( 'Name' ) );
    358378        ?>
    359         <input type="checkbox" name="checked[]" value="<?php echo esc_attr( $theme->get_stylesheet() ) ?>" id="<?php echo $checkbox_id ?>" />
    360         <label class="screen-reader-text" for="<?php echo $checkbox_id ?>" ><?php _e( 'Select' ) ?>  <?php echo $theme->display( 'Name' ) ?></label>
     379        <input type="checkbox" name="checked[]" value="<?php echo esc_attr( $theme->get_stylesheet() ); ?>" id="<?php echo $checkbox_id; ?>" />
     380        <label class="screen-reader-text" for="<?php echo $checkbox_id; ?>" ><?php _e( 'Select' ); ?>  <?php echo $theme->display( 'Name' ); ?></label>
    361381        <?php
    362382    }
     
    379399
    380400        if ( $this->is_site_themes ) {
    381             $url = "site-themes.php?id={$this->site_id}&amp;";
     401            $url     = "site-themes.php?id={$this->site_id}&amp;";
    382402            $allowed = $theme->is_allowed( 'site', $this->site_id );
    383403        } else {
    384             $url = 'themes.php?';
     404            $url     = 'themes.php?';
    385405            $allowed = $theme->is_allowed( 'network' );
    386406        }
     
    388408        // Pre-order.
    389409        $actions = array(
    390             'enable' => '',
     410            'enable'  => '',
    391411            'disable' => '',
    392             'delete' => ''
     412            'delete'  => '',
    393413        );
    394414
    395415        $stylesheet = $theme->get_stylesheet();
    396         $theme_key = urlencode( $stylesheet );
     416        $theme_key  = urlencode( $stylesheet );
    397417
    398418        if ( ! $allowed ) {
    399419            if ( ! $theme->errors() ) {
    400                 $url = add_query_arg( array(
    401                     'action' => 'enable',
    402                     'theme'  => $theme_key,
    403                     'paged'  => $page,
    404                     's'      => $s,
    405                 ), $url );
     420                $url = add_query_arg(
     421                    array(
     422                        'action' => 'enable',
     423                        'theme'  => $theme_key,
     424                        'paged'  => $page,
     425                        's'      => $s,
     426                    ), $url
     427                );
    406428
    407429                if ( $this->is_site_themes ) {
     
    413435                }
    414436
    415                 $actions['enable'] = sprintf( '<a href="%s" class="edit" aria-label="%s">%s</a>',
     437                $actions['enable'] = sprintf(
     438                    '<a href="%s" class="edit" aria-label="%s">%s</a>',
    416439                    esc_url( wp_nonce_url( $url, 'enable-theme_' . $stylesheet ) ),
    417440                    esc_attr( $aria_label ),
     
    420443            }
    421444        } else {
    422             $url = add_query_arg( array(
    423                 'action' => 'disable',
    424                 'theme'  => $theme_key,
    425                 'paged'  => $page,
    426                 's'      => $s,
    427             ), $url );
     445            $url = add_query_arg(
     446                array(
     447                    'action' => 'disable',
     448                    'theme'  => $theme_key,
     449                    'paged'  => $page,
     450                    's'      => $s,
     451                ), $url
     452            );
    428453
    429454            if ( $this->is_site_themes ) {
     
    435460            }
    436461
    437             $actions['disable'] = sprintf( '<a href="%s" aria-label="%s">%s</a>',
     462            $actions['disable'] = sprintf(
     463                '<a href="%s" aria-label="%s">%s</a>',
    438464                esc_url( wp_nonce_url( $url, 'disable-theme_' . $stylesheet ) ),
    439465                esc_attr( $aria_label ),
     
    443469
    444470        if ( ! $allowed && current_user_can( 'delete_themes' ) && ! $this->is_site_themes && $stylesheet != get_option( 'stylesheet' ) && $stylesheet != get_option( 'template' ) ) {
    445             $url = add_query_arg( array(
    446                 'action'       => 'delete-selected',
    447                 'checked[]'    => $theme_key,
    448                 'theme_status' => $context,
    449                 'paged'        => $page,
    450                 's'            => $s,
    451             ), 'themes.php' );
     471            $url = add_query_arg(
     472                array(
     473                    'action'       => 'delete-selected',
     474                    'checked[]'    => $theme_key,
     475                    'theme_status' => $context,
     476                    'paged'        => $page,
     477                    's'            => $s,
     478                ), 'themes.php'
     479            );
    452480
    453481            /* translators: %s: theme name */
    454482            $aria_label = sprintf( _x( 'Delete %s', 'theme' ), $theme->display( 'Name' ) );
    455483
    456             $actions['delete'] = sprintf( '<a href="%s" class="delete" aria-label="%s">%s</a>',
     484            $actions['delete'] = sprintf(
     485                '<a href="%s" class="delete" aria-label="%s">%s</a>',
    457486                esc_url( wp_nonce_url( $url, 'bulk-themes' ) ),
    458487                esc_attr( $aria_label ),
     
    527556
    528557        $class = ! $allowed ? 'inactive' : 'active';
    529         if ( ! empty( $totals['upgrade'] ) && ! empty( $theme->update ) )
     558        if ( ! empty( $totals['upgrade'] ) && ! empty( $theme->update ) ) {
    530559            $class .= ' update';
     560        }
    531561
    532562        echo "<div class='theme-description'><p>" . $theme->display( 'Description' ) . "</p></div>
     
    536566        $theme_meta = array();
    537567
    538         if ( $theme->get('Version') ) {
    539             $theme_meta[] = sprintf( __( 'Version %s' ), $theme->display('Version') );
    540         }
    541         $theme_meta[] = sprintf( __( 'By %s' ), $theme->display('Author') );
    542 
    543         if ( $theme->get('ThemeURI') ) {
     568        if ( $theme->get( 'Version' ) ) {
     569            $theme_meta[] = sprintf( __( 'Version %s' ), $theme->display( 'Version' ) );
     570        }
     571        $theme_meta[] = sprintf( __( 'By %s' ), $theme->display( 'Author' ) );
     572
     573        if ( $theme->get( 'ThemeURI' ) ) {
    544574            /* translators: %s: theme name */
    545575            $aria_label = sprintf( __( 'Visit %s homepage' ), $theme->display( 'Name' ) );
    546576
    547             $theme_meta[] = sprintf( '<a href="%s" aria-label="%s">%s</a>',
     577            $theme_meta[] = sprintf(
     578                '<a href="%s" aria-label="%s">%s</a>',
    548579                $theme->display( 'ThemeURI' ),
    549580                esc_attr( $aria_label ),
     
    619650
    620651                case 'name':
    621 
    622652                    $active_theme_label = '';
    623653
     
    633663
    634664                        /* In case this is a child theme, label it properly */
    635                         if ( $stylesheet !== $template && $item->get_stylesheet() === $stylesheet) {
     665                        if ( $stylesheet !== $template && $item->get_stylesheet() === $stylesheet ) {
    636666                            $active_theme_label = ' &mdash; ' . __( 'Active Child Theme' );
    637667                        }
     
    642672                    $this->column_name( $item );
    643673
    644                     echo "</td>";
     674                    echo '</td>';
    645675                    break;
    646676
     
    658688                    $this->column_default( $item, $column_name );
    659689
    660                     echo "</td>";
     690                    echo '</td>';
    661691                    break;
    662692            }
     
    686716        }
    687717
    688         printf( '<tr class="%s" data-slug="%s">',
     718        printf(
     719            '<tr class="%s" data-slug="%s">',
    689720            esc_attr( $class ),
    690721            esc_attr( $stylesheet )
     
    693724        $this->single_row_columns( $theme );
    694725
    695         echo "</tr>";
    696 
    697         if ( $this->is_site_themes )
     726        echo '</tr>';
     727
     728        if ( $this->is_site_themes ) {
    698729            remove_action( "after_theme_row_$stylesheet", 'wp_theme_update_row' );
     730        }
    699731
    700732        /**
Note: See TracChangeset for help on using the changeset viewer.