WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (8 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-plugins-list-table.php

    r42228 r42343  
    3333        global $status, $page;
    3434
    35         parent::__construct( array(
    36             'plural' => 'plugins',
    37             'screen' => isset( $args['screen'] ) ? $args['screen'] : null,
    38         ) );
     35        parent::__construct(
     36            array(
     37                'plural' => 'plugins',
     38                'screen' => isset( $args['screen'] ) ? $args['screen'] : null,
     39            )
     40        );
    3941
    4042        $status = 'all';
    41         if ( isset( $_REQUEST['plugin_status'] ) && in_array( $_REQUEST['plugin_status'], array( 'active', 'inactive', 'recently_activated', 'upgrade', 'mustuse', 'dropins', 'search' ) ) )
     43        if ( isset( $_REQUEST['plugin_status'] ) && in_array( $_REQUEST['plugin_status'], array( 'active', 'inactive', 'recently_activated', 'upgrade', 'mustuse', 'dropins', 'search' ) ) ) {
    4244            $status = $_REQUEST['plugin_status'];
    43 
    44         if ( isset($_REQUEST['s']) )
    45             $_SERVER['REQUEST_URI'] = add_query_arg('s', wp_unslash($_REQUEST['s']) );
     45        }
     46
     47        if ( isset( $_REQUEST['s'] ) ) {
     48            $_SERVER['REQUEST_URI'] = add_query_arg( 's', wp_unslash( $_REQUEST['s'] ) );
     49        }
    4650
    4751        $page = $this->get_pagenum();
    … …  
    5963     */
    6064    public function ajax_user_can() {
    61         return current_user_can('activate_plugins');
    62     }
    63 
    64     /**
    65      *
     65        return current_user_can( 'activate_plugins' );
     66    }
     67
     68    /**
    6669     * @global string $status
    6770     * @global array  $plugins
    … …  
    123126
    124127            /** This action is documented in wp-admin/includes/class-wp-plugins-list-table.php */
    125             if ( apply_filters( 'show_advanced_plugins', true, 'dropins' ) )
     128            if ( apply_filters( 'show_advanced_plugins', true, 'dropins' ) ) {
    126129                $plugins['dropins'] = get_dropins();
     130            }
    127131
    128132            if ( current_user_can( 'update_plugins' ) ) {
    … …  
    131135                    if ( isset( $current->response[ $plugin_file ] ) ) {
    132136                        $plugins['all'][ $plugin_file ]['update'] = true;
    133                         $plugins['upgrade'][ $plugin_file ] = $plugins['all'][ $plugin_file ];
     137                        $plugins['upgrade'][ $plugin_file ]       = $plugins['all'][ $plugin_file ];
    134138                    }
    135139                }
    … …  
    165169        foreach ( $recently_activated as $key => $time ) {
    166170            if ( $time + WEEK_IN_SECONDS < time() ) {
    167                 unset( $recently_activated[$key] );
     171                unset( $recently_activated[ $key ] );
    168172            }
    169173        }
    … …  
    185189                    $plugins['upgrade'][ $plugin_file ] = $plugin_data = array_merge( (array) $plugin_info->response[ $plugin_file ], $plugin_data );
    186190                }
    187 
    188191            } elseif ( isset( $plugin_info->no_update[ $plugin_file ] ) ) {
    189192                $plugins['all'][ $plugin_file ] = $plugin_data = array_merge( (array) $plugin_info->no_update[ $plugin_file ], $plugin_data );
    … …  
    227230
    228231        if ( strlen( $s ) ) {
    229             $status = 'search';
     232            $status            = 'search';
    230233            $plugins['search'] = array_filter( $plugins['all'], array( $this, '_search_callback' ) );
    231234        }
    232235
    233236        $totals = array();
    234         foreach ( $plugins as $type => $list )
     237        foreach ( $plugins as $type => $list ) {
    235238            $totals[ $type ] = count( $list );
    236 
    237         if ( empty( $plugins[ $status ] ) && !in_array( $status, array( 'all', 'search' ) ) )
     239        }
     240
     241        if ( empty( $plugins[ $status ] ) && ! in_array( $status, array( 'all', 'search' ) ) ) {
    238242            $status = 'all';
     243        }
    239244
    240245        $this->items = array();
    241246        foreach ( $plugins[ $status ] as $plugin_file => $plugin_data ) {
    242247            // Translate, Don't Apply Markup, Sanitize HTML
    243             $this->items[$plugin_file] = _get_plugin_data_markup_translate( $plugin_file, $plugin_data, false, true );
     248            $this->items[ $plugin_file ] = _get_plugin_data_markup_translate( $plugin_file, $plugin_data, false, true );
    244249        }
    245250
    … …  
    251256        }
    252257
    253         wp_localize_script( 'updates', '_wpUpdatesItemCounts', array(
    254             'plugins' => $js_plugins,
    255             'totals'  => wp_get_update_data(),
    256         ) );
     258        wp_localize_script(
     259            'updates', '_wpUpdatesItemCounts', array(
     260                'plugins' => $js_plugins,
     261                'totals'  => wp_get_update_data(),
     262            )
     263        );
    257264
    258265        if ( ! $orderby ) {
    … …  
    270277        $start = ( $page - 1 ) * $plugins_per_page;
    271278
    272         if ( $total_this_page > $plugins_per_page )
     279        if ( $total_this_page > $plugins_per_page ) {
    273280            $this->items = array_slice( $this->items, $start, $plugins_per_page );
    274 
    275         $this->set_pagination_args( array(
    276             'total_items' => $total_this_page,
    277             'per_page' => $plugins_per_page,
    278         ) );
     281        }
     282
     283        $this->set_pagination_args(
     284            array(
     285                'total_items' => $total_this_page,
     286                'per_page'    => $plugins_per_page,
     287            )
     288        );
    279289    }
    280290
    … …  
    307317        global $orderby, $order;
    308318
    309         $a = $plugin_a[$orderby];
    310         $b = $plugin_b[$orderby];
    311 
    312         if ( $a == $b )
     319        $a = $plugin_a[ $orderby ];
     320        $b = $plugin_b[ $orderby ];
     321
     322        if ( $a == $b ) {
    313323            return 0;
     324        }
    314325
    315326        if ( 'DESC' === $order ) {
    … …  
    321332
    322333    /**
    323      *
    324334     * @global array $plugins
    325335     */
    … …  
    336346                echo ' <a href="' . esc_url( admin_url( 'plugin-install.php?tab=search&s=' . urlencode( $s ) ) ) . '">' . __( 'Search for plugins in the WordPress Plugin Directory.' ) . '</a>';
    337347            }
    338         } elseif ( ! empty( $plugins['all'] ) )
     348        } elseif ( ! empty( $plugins['all'] ) ) {
    339349            _e( 'No plugins found.' );
    340         else
     350        } else {
    341351            _e( 'You do not appear to have any plugins available at this time.' );
     352        }
    342353    }
    343354
    … …  
    373384
    374385    /**
    375      *
    376386     * @global string $status
    377387     * @return array
    … …  
    381391
    382392        return array(
    383             'cb'          => !in_array( $status, array( 'mustuse', 'dropins' ) ) ? '<input type="checkbox" />' : '',
     393            'cb'          => ! in_array( $status, array( 'mustuse', 'dropins' ) ) ? '<input type="checkbox" />' : '',
    384394            'name'        => __( 'Plugin' ),
    385395            'description' => __( 'Description' ),
    … …  
    395405
    396406    /**
    397      *
    398407     * @global array $totals
    399408     * @global string $status
    … …  
    405414        $status_links = array();
    406415        foreach ( $totals as $type => $count ) {
    407             if ( !$count )
     416            if ( ! $count ) {
    408417                continue;
     418            }
    409419
    410420            switch ( $type ) {
    … …  
    433443
    434444            if ( 'search' !== $type ) {
    435                 $status_links[$type] = sprintf( "<a href='%s'%s>%s</a>",
    436                     add_query_arg('plugin_status', $type, 'plugins.php'),
     445                $status_links[ $type ] = sprintf(
     446                    "<a href='%s'%s>%s</a>",
     447                    add_query_arg( 'plugin_status', $type, 'plugins.php' ),
    437448                    ( $type === $status ) ? ' class="current" aria-current="page"' : '',
    438449                    sprintf( $text, number_format_i18n( $count ) )
    439                     );
     450                );
    440451            }
    441452        }
    … …  
    445456
    446457    /**
    447      *
    448458     * @global string $status
    449459     * @return array
    … …  
    454464        $actions = array();
    455465
    456         if ( 'active' != $status )
     466        if ( 'active' != $status ) {
    457467            $actions['activate-selected'] = $this->screen->in_admin( 'network' ) ? __( 'Network Activate' ) : __( 'Activate' );
    458 
    459         if ( 'inactive' != $status && 'recent' != $status )
     468        }
     469
     470        if ( 'inactive' != $status && 'recent' != $status ) {
    460471            $actions['deactivate-selected'] = $this->screen->in_admin( 'network' ) ? __( 'Network Deactivate' ) : __( 'Deactivate' );
    461 
    462         if ( !is_multisite() || $this->screen->in_admin( 'network' ) ) {
    463             if ( current_user_can( 'update_plugins' ) )
     472        }
     473
     474        if ( ! is_multisite() || $this->screen->in_admin( 'network' ) ) {
     475            if ( current_user_can( 'update_plugins' ) ) {
    464476                $actions['update-selected'] = __( 'Update' );
    465             if ( current_user_can( 'delete_plugins' ) && ( 'active' != $status ) )
     477            }
     478            if ( current_user_can( 'delete_plugins' ) && ( 'active' != $status ) ) {
    466479                $actions['delete-selected'] = __( 'Delete' );
     480            }
    467481        }
    468482
    … …  
    477491        global $status;
    478492
    479         if ( in_array( $status, array( 'mustuse', 'dropins' ) ) )
     493        if ( in_array( $status, array( 'mustuse', 'dropins' ) ) ) {
    480494            return;
     495        }
    481496
    482497        parent::bulk_actions( $which );
    … …  
    490505        global $status;
    491506
    492         if ( ! in_array($status, array('recently_activated', 'mustuse', 'dropins') ) )
     507        if ( ! in_array( $status, array( 'recently_activated', 'mustuse', 'dropins' ) ) ) {
    493508            return;
     509        }
    494510
    495511        echo '<div class="alignleft actions">';
    … …  
    499515        } elseif ( 'top' === $which && 'mustuse' === $status ) {
    500516            /* translators: %s: mu-plugins directory name */
    501             echo '<p>' . sprintf( __( 'Files in the %s directory are executed automatically.' ),
     517            echo '<p>' . sprintf(
     518                __( 'Files in the %s directory are executed automatically.' ),
    502519                '<code>' . str_replace( ABSPATH, '/', WPMU_PLUGIN_DIR ) . '</code>'
    503520            ) . '</p>';
    504521        } elseif ( 'top' === $which && 'dropins' === $status ) {
    505522            /* translators: %s: wp-content directory name */
    506             echo '<p>' . sprintf( __( 'Drop-ins are advanced plugins in the %s directory that replace WordPress functionality when present.' ),
     523            echo '<p>' . sprintf(
     524                __( 'Drop-ins are advanced plugins in the %s directory that replace WordPress functionality when present.' ),
    507525                '<code>' . str_replace( ABSPATH, '', WP_CONTENT_DIR ) . '</code>'
    508526            ) . '</p>';
    … …  
    515533     */
    516534    public function current_action() {
    517         if ( isset($_POST['clear-recent-list']) )
     535        if ( isset( $_POST['clear-recent-list'] ) ) {
    518536            return 'clear-recent-list';
     537        }
    519538
    520539        return parent::current_action();
    … …  
    522541
    523542    /**
    524      *
    525543     * @global string $status
    526544     */
    … …  
    528546        global $status;
    529547
    530         if ( is_multisite() && ! $this->screen->in_admin( 'network' ) && in_array( $status, array( 'mustuse', 'dropins' ) ) )
     548        if ( is_multisite() && ! $this->screen->in_admin( 'network' ) && in_array( $status, array( 'mustuse', 'dropins' ) ) ) {
    531549            return;
    532 
    533         foreach ( $this->items as $plugin_file => $plugin_data )
     550        }
     551
     552        foreach ( $this->items as $plugin_file => $plugin_data ) {
    534553            $this->single_row( array( $plugin_file, $plugin_data ) );
     554        }
    535555    }
    536556
    … …  
    547567
    548568        list( $plugin_file, $plugin_data ) = $item;
    549         $context = $status;
    550         $screen = $this->screen;
     569        $context                           = $status;
     570        $screen                            = $this->screen;
    551571
    552572        // Pre-order.
    553573        $actions = array(
    554574            'deactivate' => '',
    555             'activate' => '',
    556             'details' => '',
    557             'delete' => '',
     575            'activate'   => '',
     576            'details'    => '',
     577            'delete'     => '',
    558578        );
    559579
    560580        // Do not restrict by default
    561581        $restrict_network_active = false;
    562         $restrict_network_only = false;
     582        $restrict_network_only   = false;
    563583
    564584        if ( 'mustuse' === $context ) {
    565585            $is_active = true;
    566586        } elseif ( 'dropins' === $context ) {
    567             $dropins = _get_dropins();
     587            $dropins     = _get_dropins();
    568588            $plugin_name = $plugin_file;
    569             if ( $plugin_file != $plugin_data['Name'] )
     589            if ( $plugin_file != $plugin_data['Name'] ) {
    570590                $plugin_name .= '<br/>' . $plugin_data['Name'];
     591            }
    571592            if ( true === ( $dropins[ $plugin_file ][1] ) ) { // Doesn't require a constant
    572                 $is_active = true;
     593                $is_active   = true;
    573594                $description = '<p><strong>' . $dropins[ $plugin_file ][0] . '</strong></p>';
    574595            } elseif ( defined( $dropins[ $plugin_file ][1] ) && constant( $dropins[ $plugin_file ][1] ) ) { // Constant is true
    575                 $is_active = true;
     596                $is_active   = true;
    576597                $description = '<p><strong>' . $dropins[ $plugin_file ][0] . '</strong></p>';
    577598            } else {
    578                 $is_active = false;
     599                $is_active   = false;
    579600                $description = '<p><strong>' . $dropins[ $plugin_file ][0] . ' <span class="error-message">' . __( 'Inactive:' ) . '</span></strong> ' .
    580601                    /* translators: 1: drop-in constant name, 2: wp-config.php */
    581                     sprintf( __( 'Requires %1$s in %2$s file.' ),
     602                    sprintf(
     603                        __( 'Requires %1$s in %2$s file.' ),
    582604                        "<code>define('" . $dropins[ $plugin_file ][1] . "', true);</code>",
    583605                        '<code>wp-config.php</code>'
    584606                    ) . '</p>';
    585607            }
    586             if ( $plugin_data['Description'] )
     608            if ( $plugin_data['Description'] ) {
    587609                $description .= '<p>' . $plugin_data['Description'] . '</p>';
     610            }
    588611        } else {
    589612            if ( $screen->in_admin( 'network' ) ) {
    590613                $is_active = is_plugin_active_for_network( $plugin_file );
    591614            } else {
    592                 $is_active = is_plugin_active( $plugin_file );
     615                $is_active               = is_plugin_active( $plugin_file );
    593616                $restrict_network_active = ( is_multisite() && is_plugin_active_for_network( $plugin_file ) );
    594                 $restrict_network_only = ( is_multisite() && is_network_only_plugin( $plugin_file ) && ! $is_active );
     617                $restrict_network_only   = ( is_multisite() && is_network_only_plugin( $plugin_file ) && ! $is_active );
    595618            }
    596619
    … …  
    600623                        /* translators: %s: plugin name */
    601624                        $actions['deactivate'] = '<a href="' . wp_nonce_url( 'plugins.php?action=deactivate&amp;plugin=' . urlencode( $plugin_file ) . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'deactivate-plugin_' . $plugin_file ) . '" aria-label="' . esc_attr( sprintf( _x( 'Network Deactivate %s', 'plugin' ), $plugin_data['Name'] ) ) . '">' . __( 'Network Deactivate' ) . '</a>';
    602                         }
     625                    }
    603626                } else {
    604627                    if ( current_user_can( 'manage_network_plugins' ) ) {
    … …  
    636659                    }
    637660                } // end if $is_active
    638 
    639661            } // end if $screen->in_admin( 'network' )
    640 
    641662        } // end if $context
    642663
    … …  
    716737        }
    717738
    718         $class = $is_active ? 'active' : 'inactive';
    719         $checkbox_id =  "checkbox_" . md5($plugin_data['Name']);
     739        $class       = $is_active ? 'active' : 'inactive';
     740        $checkbox_id = 'checkbox_' . md5( $plugin_data['Name'] );
    720741        if ( $restrict_network_active || $restrict_network_only || in_array( $status, array( 'mustuse', 'dropins' ) ) ) {
    721742            $checkbox = '';
    722743        } else {
    723             $checkbox = "<label class='screen-reader-text' for='" . $checkbox_id . "' >" . sprintf( __( 'Select %s' ), $plugin_data['Name'] ) . "</label>"
     744            $checkbox = "<label class='screen-reader-text' for='" . $checkbox_id . "' >" . sprintf( __( 'Select %s' ), $plugin_data['Name'] ) . '</label>'
    724745                . "<input type='checkbox' name='checked[]' value='" . esc_attr( $plugin_file ) . "' id='" . $checkbox_id . "' />";
    725746        }
    … …  
    729750        }
    730751
    731         if ( ! empty( $totals['upgrade'] ) && ! empty( $plugin_data['update'] ) )
     752        if ( ! empty( $totals['upgrade'] ) && ! empty( $plugin_data['update'] ) ) {
    732753            $class .= ' update';
     754        }
    733755
    734756        $plugin_slug = isset( $plugin_data['slug'] ) ? $plugin_data['slug'] : sanitize_title( $plugin_name );
    735         printf( '<tr class="%s" data-slug="%s" data-plugin="%s">',
     757        printf(
     758            '<tr class="%s" data-slug="%s" data-plugin="%s">',
    736759            esc_attr( $class ),
    737760            esc_attr( $plugin_slug ),
    … …  
    754777                    echo "<td class='plugin-title column-primary'><strong>$plugin_name</strong>";
    755778                    echo $this->row_actions( $actions, true );
    756                     echo "</td>";
     779                    echo '</td>';
    757780                    break;
    758781                case 'description':
    … …  
    764787
    765788                    $plugin_meta = array();
    766                     if ( !empty( $plugin_data['Version'] ) )
     789                    if ( ! empty( $plugin_data['Version'] ) ) {
    767790                        $plugin_meta[] = sprintf( __( 'Version %s' ), $plugin_data['Version'] );
    768                     if ( !empty( $plugin_data['Author'] ) ) {
     791                    }
     792                    if ( ! empty( $plugin_data['Author'] ) ) {
    769793                        $author = $plugin_data['Author'];
    770                         if ( !empty( $plugin_data['AuthorURI'] ) )
     794                        if ( ! empty( $plugin_data['AuthorURI'] ) ) {
    771795                            $author = '<a href="' . $plugin_data['AuthorURI'] . '">' . $plugin_data['Author'] . '</a>';
     796                        }
    772797                        $plugin_meta[] = sprintf( __( 'By %s' ), $author );
    773798                    }
    … …  
    775800                    // Details link using API info, if available
    776801                    if ( isset( $plugin_data['slug'] ) && current_user_can( 'install_plugins' ) ) {
    777                         $plugin_meta[] = sprintf( '<a href="%s" class="thickbox open-plugin-details-modal" aria-label="%s" data-title="%s">%s</a>',
    778                             esc_url( network_admin_url( 'plugin-install.php?tab=plugin-information&plugin=' . $plugin_data['slug'] .
    779                                 '&TB_iframe=true&width=600&height=550' ) ),
     802                        $plugin_meta[] = sprintf(
     803                            '<a href="%s" class="thickbox open-plugin-details-modal" aria-label="%s" data-title="%s">%s</a>',
     804                            esc_url(
     805                                network_admin_url(
     806                                    'plugin-install.php?tab=plugin-information&plugin=' . $plugin_data['slug'] .
     807                                    '&TB_iframe=true&width=600&height=550'
     808                                )
     809                            ),
    780810                            esc_attr( sprintf( __( 'More information about %s' ), $plugin_name ) ),
    781811                            esc_attr( $plugin_name ),
    … …  
    783813                        );
    784814                    } elseif ( ! empty( $plugin_data['PluginURI'] ) ) {
    785                         $plugin_meta[] = sprintf( '<a href="%s">%s</a>',
     815                        $plugin_meta[] = sprintf(
     816                            '<a href="%s">%s</a>',
    786817                            esc_url( $plugin_data['PluginURI'] ),
    787818                            __( 'Visit plugin site' )
    … …  
    806837                    echo implode( ' | ', $plugin_meta );
    807838
    808                     echo "</div></td>";
     839                    echo '</div></td>';
    809840                    break;
    810841                default:
    … …  
    824855                    do_action( 'manage_plugins_custom_column', $column_name, $plugin_file, $plugin_data );
    825856
    826                     echo "</td>";
    827             }
    828         }
    829 
    830         echo "</tr>";
     857                    echo '</td>';
     858            }
     859        }
     860
     861        echo '</tr>';
    831862
    832863        /**
Note: See TracChangeset for help on using the changeset viewer.