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-screen.php

    r41978 r42343  
    109109     * The taxonomy associated with the screen, if any.
    110110     * The 'edit-tags.php?taxonomy=category' screen has a taxonomy of 'category'.
     111     *
    111112     * @since 3.3.0
    112113     * @var string
    … …  
    130131    private $_help_sidebar = '';
    131132
    132     /**
     133    /**
    133134     * The accessible hidden headings and text associated with the screen, if any.
    134135     *
    … …  
    192193     *
    193194     * @param string|WP_Screen $hook_name Optional. The hook name (also known as the hook suffix) used to determine the screen.
    194      *                                    Defaults to the current $hook_suffix global.
     195     *                                    Defaults to the current $hook_suffix global.
    195196     * @return WP_Screen Screen object.
    196197     */
    … …  
    201202
    202203        $post_type = $taxonomy = null;
    203         $in_admin = false;
    204         $action = '';
    205 
    206         if ( $hook_name )
     204        $in_admin  = false;
     205        $action    = '';
     206
     207        if ( $hook_name ) {
    207208            $id = $hook_name;
    208         else
     209        } else {
    209210            $id = $GLOBALS['hook_suffix'];
     211        }
    210212
    211213        // For those pesky meta boxes.
    212214        if ( $hook_name && post_type_exists( $hook_name ) ) {
    213215            $post_type = $id;
    214             $id = 'post'; // changes later. ends up being $base.
     216            $id        = 'post'; // changes later. ends up being $base.
    215217        } else {
    216             if ( '.php' == substr( $id, -4 ) )
     218            if ( '.php' == substr( $id, -4 ) ) {
    217219                $id = substr( $id, 0, -4 );
     220            }
    218221
    219222            if ( 'post-new' == $id || 'link-add' == $id || 'media-new' == $id || 'user-new' == $id ) {
    220                 $id = substr( $id, 0, -4 );
     223                $id     = substr( $id, 0, -4 );
    221224                $action = 'add';
    222225            }
    … …  
    225228        if ( ! $post_type && $hook_name ) {
    226229            if ( '-network' == substr( $id, -8 ) ) {
    227                 $id = substr( $id, 0, -8 );
     230                $id       = substr( $id, 0, -8 );
    228231                $in_admin = 'network';
    229232            } elseif ( '-user' == substr( $id, -5 ) ) {
    230                 $id = substr( $id, 0, -5 );
     233                $id       = substr( $id, 0, -5 );
    231234                $in_admin = 'user';
    232235            }
    … …  
    236239                $maybe = substr( $id, 5 );
    237240                if ( taxonomy_exists( $maybe ) ) {
    238                     $id = 'edit-tags';
     241                    $id       = 'edit-tags';
    239242                    $taxonomy = $maybe;
    240243                } elseif ( post_type_exists( $maybe ) ) {
    241                     $id = 'edit';
     244                    $id        = 'edit';
    242245                    $post_type = $maybe;
    243246                }
    244247            }
    245248
    246             if ( ! $in_admin )
     249            if ( ! $in_admin ) {
    247250                $in_admin = 'site';
     251            }
    248252        } else {
    249             if ( defined( 'WP_NETWORK_ADMIN' ) && WP_NETWORK_ADMIN )
     253            if ( defined( 'WP_NETWORK_ADMIN' ) && WP_NETWORK_ADMIN ) {
    250254                $in_admin = 'network';
    251             elseif ( defined( 'WP_USER_ADMIN' ) && WP_USER_ADMIN )
     255            } elseif ( defined( 'WP_USER_ADMIN' ) && WP_USER_ADMIN ) {
    252256                $in_admin = 'user';
    253             else
     257            } else {
    254258                $in_admin = 'site';
    255         }
    256 
    257         if ( 'index' == $id )
     259            }
     260        }
     261
     262        if ( 'index' == $id ) {
    258263            $id = 'dashboard';
    259         elseif ( 'front' == $id )
     264        } elseif ( 'front' == $id ) {
    260265            $in_admin = false;
     266        }
    261267
    262268        $base = $id;
    … …  
    264270        // If this is the current screen, see if we can be more accurate for post types and taxonomies.
    265271        if ( ! $hook_name ) {
    266             if ( isset( $_REQUEST['post_type'] ) )
     272            if ( isset( $_REQUEST['post_type'] ) ) {
    267273                $post_type = post_type_exists( $_REQUEST['post_type'] ) ? $_REQUEST['post_type'] : false;
    268             if ( isset( $_REQUEST['taxonomy'] ) )
     274            }
     275            if ( isset( $_REQUEST['taxonomy'] ) ) {
    269276                $taxonomy = taxonomy_exists( $_REQUEST['taxonomy'] ) ? $_REQUEST['taxonomy'] : false;
     277            }
    270278
    271279            switch ( $base ) {
    272                 case 'post' :
    273                     if ( isset( $_GET['post'] ) )
     280                case 'post':
     281                    if ( isset( $_GET['post'] ) ) {
    274282                        $post_id = (int) $_GET['post'];
    275                     elseif ( isset( $_POST['post_ID'] ) )
     283                    } elseif ( isset( $_POST['post_ID'] ) ) {
    276284                        $post_id = (int) $_POST['post_ID'];
    277                     else
     285                    } else {
    278286                        $post_id = 0;
     287                    }
    279288
    280289                    if ( $post_id ) {
    281290                        $post = get_post( $post_id );
    282                         if ( $post )
     291                        if ( $post ) {
    283292                            $post_type = $post->post_type;
     293                        }
    284294                    }
    285295                    break;
    286                 case 'edit-tags' :
    287                 case 'term' :
    288                     if ( null === $post_type && is_object_in_taxonomy( 'post', $taxonomy ? $taxonomy : 'post_tag' ) )
     296                case 'edit-tags':
     297                case 'term':
     298                    if ( null === $post_type && is_object_in_taxonomy( 'post', $taxonomy ? $taxonomy : 'post_tag' ) ) {
    289299                        $post_type = 'post';
     300                    }
    290301                    break;
    291302                case 'upload':
    … …  
    296307
    297308        switch ( $base ) {
    298             case 'post' :
    299                 if ( null === $post_type )
     309            case 'post':
     310                if ( null === $post_type ) {
    300311                    $post_type = 'post';
     312                }
    301313                $id = $post_type;
    302314                break;
    303             case 'edit' :
    304                 if ( null === $post_type )
     315            case 'edit':
     316                if ( null === $post_type ) {
    305317                    $post_type = 'post';
     318                }
    306319                $id .= '-' . $post_type;
    307320                break;
    308             case 'edit-tags' :
    309             case 'term' :
    310                 if ( null === $taxonomy )
     321            case 'edit-tags':
     322            case 'term':
     323                if ( null === $taxonomy ) {
    311324                    $taxonomy = 'post_tag';
     325                }
    312326                // The edit-tags ID does not contain the post type. Look for it in the request.
    313327                if ( null === $post_type ) {
    314328                    $post_type = 'post';
    315                     if ( isset( $_REQUEST['post_type'] ) && post_type_exists( $_REQUEST['post_type'] ) )
     329                    if ( isset( $_REQUEST['post_type'] ) && post_type_exists( $_REQUEST['post_type'] ) ) {
    316330                        $post_type = $_REQUEST['post_type'];
     331                    }
    317332                }
    318333
    … …  
    331346        if ( isset( self::$_registry[ $id ] ) ) {
    332347            $screen = self::$_registry[ $id ];
    333             if ( $screen === get_current_screen() )
     348            if ( $screen === get_current_screen() ) {
    334349                return $screen;
     350            }
    335351        } else {
    336             $screen = new WP_Screen();
    337             $screen->id     = $id;
     352            $screen     = new WP_Screen();
     353            $screen->id = $id;
    338354        }
    339355
    … …  
    364380        global $current_screen, $taxnow, $typenow;
    365381        $current_screen = $this;
    366         $taxnow = $this->taxonomy;
    367         $typenow = $this->post_type;
     382        $taxnow         = $this->taxonomy;
     383        $typenow        = $this->post_type;
    368384
    369385        /**
    … …  
    394410     */
    395411    public function in_admin( $admin = null ) {
    396         if ( empty( $admin ) )
     412        if ( empty( $admin ) ) {
    397413            return (bool) $this->in_admin;
     414        }
    398415
    399416        return ( $admin == $this->in_admin );
    … …  
    423440     */
    424441    public function set_parentage( $parent_file ) {
    425         $this->parent_file = $parent_file;
     442        $this->parent_file         = $parent_file;
    426443        list( $this->parent_base ) = explode( '?', $parent_file );
    427         $this->parent_base = str_replace( '.php', '', $this->parent_base );
     444        $this->parent_base         = str_replace( '.php', '', $this->parent_base );
    428445    }
    429446
    … …  
    483500     */
    484501    public function get_option( $option, $key = false ) {
    485         if ( ! isset( $this->_options[ $option ] ) )
     502        if ( ! isset( $this->_options[ $option ] ) ) {
    486503            return null;
     504        }
    487505        if ( $key ) {
    488             if ( isset( $this->_options[ $option ][ $key ] ) )
     506            if ( isset( $this->_options[ $option ][ $key ] ) ) {
    489507                return $this->_options[ $option ][ $key ];
     508            }
    490509            return null;
    491510        }
    … …  
    534553     */
    535554    public function get_help_tab( $id ) {
    536         if ( ! isset( $this->_help_tabs[ $id ] ) )
     555        if ( ! isset( $this->_help_tabs[ $id ] ) ) {
    537556            return null;
     557        }
    538558        return $this->_help_tabs[ $id ];
    539559    }
    … …  
    564584            'priority' => 10,
    565585        );
    566         $args = wp_parse_args( $args, $defaults );
     586        $args     = wp_parse_args( $args, $defaults );
    567587
    568588        $args['id'] = sanitize_html_class( $args['id'] );
    569589
    570590        // Ensure we have an ID and title.
    571         if ( ! $args['id'] || ! $args['title'] )
     591        if ( ! $args['id'] || ! $args['title'] ) {
    572592            return;
     593        }
    573594
    574595        // Allows for overriding an existing tab with that ID.
    … …  
    636657    }
    637658
    638     /**
     659    /**
    639660     * Get the accessible hidden headings and text used in the screen.
    640661     *
    … …  
    686707            'heading_list'       => __( 'Items list' ),
    687708        );
    688         $content = wp_parse_args( $content, $defaults );
     709        $content  = wp_parse_args( $content, $defaults );
    689710
    690711        $this->_screen_reader_content = $content;
    … …  
    735756         * @param string    $screen_id Screen ID.
    736757         * @param WP_Screen $this      Current WP_Screen instance.
    737          *
    738758         */
    739759        $old_help = apply_filters( 'contextual_help', $old_help, $this->id, $this );
    … …  
    752772             */
    753773            $default_help = apply_filters( 'default_contextual_help', '' );
    754             if ( $default_help )
     774            if ( $default_help ) {
    755775                $old_help = '<p>' . $default_help . '</p>';
     776            }
    756777        }
    757778
    758779        if ( $old_help ) {
    759             $this->add_help_tab( array(
    760                 'id'      => 'old-contextual-help',
    761                 'title'   => __('Overview'),
    762                 'content' => $old_help,
    763             ) );
     780            $this->add_help_tab(
     781                array(
     782                    'id'      => 'old-contextual-help',
     783                    'title'   => __( 'Overview' ),
     784                    'content' => $old_help,
     785                )
     786            );
    764787        }
    765788
    … …  
    767790
    768791        $help_class = 'hidden';
    769         if ( ! $help_sidebar )
     792        if ( ! $help_sidebar ) {
    770793            $help_class .= ' no-sidebar';
     794        }
    771795
    772796        // Time to render!
    … …  
    774798        <div id="screen-meta" class="metabox-prefs">
    775799
    776             <div id="contextual-help-wrap" class="<?php echo esc_attr( $help_class ); ?>" tabindex="-1" aria-label="<?php esc_attr_e('Contextual Help Tab'); ?>">
     800            <div id="contextual-help-wrap" class="<?php echo esc_attr( $help_class ); ?>" tabindex="-1" aria-label="<?php esc_attr_e( 'Contextual Help Tab' ); ?>">
    777801                <div id="contextual-help-back"></div>
    778802                <div id="contextual-help-columns">
    … …  
    807831                        <?php
    808832                        $classes = 'help-tab-content active';
    809                         foreach ( $this->get_help_tabs() as $tab ):
     833                        foreach ( $this->get_help_tabs() as $tab ) :
    810834                            $panel_id = "tab-panel-{$tab['id']}";
    811835                            ?>
    … …  
    817841
    818842                                // If it exists, fire tab callback.
    819                                 if ( ! empty( $tab['callback'] ) )
     843                                if ( ! empty( $tab['callback'] ) ) {
    820844                                    call_user_func_array( $tab['callback'], array( $this, $tab ) );
     845                                }
    821846                                ?>
    822847                            </div>
    … …  
    845870        $columns = apply_filters( 'screen_layout_columns', array(), $this->id, $this );
    846871
    847         if ( ! empty( $columns ) && isset( $columns[ $this->id ] ) )
    848             $this->add_option( 'layout_columns', array('max' => $columns[ $this->id ] ) );
     872        if ( ! empty( $columns ) && isset( $columns[ $this->id ] ) ) {
     873            $this->add_option( 'layout_columns', array( 'max' => $columns[ $this->id ] ) );
     874        }
    849875
    850876        if ( $this->get_option( 'layout_columns' ) ) {
    851             $this->columns = (int) get_user_option("screen_layout_$this->id");
    852 
    853             if ( ! $this->columns && $this->get_option( 'layout_columns', 'default' ) )
     877            $this->columns = (int) get_user_option( "screen_layout_$this->id" );
     878
     879            if ( ! $this->columns && $this->get_option( 'layout_columns', 'default' ) ) {
    854880                $this->columns = $this->get_option( 'layout_columns', 'default' );
    855         }
    856         $GLOBALS[ 'screen_layout_columns' ] = $this->columns; // Set the global for back-compat.
     881            }
     882        }
     883        $GLOBALS['screen_layout_columns'] = $this->columns; // Set the global for back-compat.
    857884
    858885        // Add screen options
    859         if ( $this->show_screen_options() )
     886        if ( $this->show_screen_options() ) {
    860887            $this->render_screen_options();
     888        }
    861889        ?>
    862890        </div>
    863891        <?php
    864         if ( ! $this->get_help_tabs() && ! $this->show_screen_options() )
     892        if ( ! $this->get_help_tabs() && ! $this->show_screen_options() ) {
    865893            return;
     894        }
    866895        ?>
    867896        <div id="screen-meta-links">
    … …  
    870899            <button type="button" id="contextual-help-link" class="button show-settings" aria-controls="contextual-help-wrap" aria-expanded="false"><?php _e( 'Help' ); ?></button>
    871900            </div>
    872         <?php endif;
    873         if ( $this->show_screen_options() ) : ?>
     901        <?php
     902        endif;
     903if ( $this->show_screen_options() ) :
     904        ?>
    874905            <div id="screen-options-link-wrap" class="hide-if-no-js screen-meta-toggle">
    875906            <button type="button" id="show-settings-link" class="button show-settings" aria-controls="screen-options-wrap" aria-expanded="false"><?php _e( 'Screen Options' ); ?></button>
    … …  
    881912
    882913    /**
    883      *
    884914     * @global array $wp_meta_boxes
    885915     *
    … …  
    889919        global $wp_meta_boxes;
    890920
    891         if ( is_bool( $this->_show_screen_options ) )
     921        if ( is_bool( $this->_show_screen_options ) ) {
    892922            return $this->_show_screen_options;
     923        }
    893924
    894925        $columns = get_column_headers( $this );
    … …  
    898929        switch ( $this->base ) {
    899930            case 'widgets':
    900                 $nonce = wp_create_nonce( 'widgets-access' );
    901                 $this->_screen_settings = '<p><a id="access-on" href="widgets.php?widgets-access=on&_wpnonce=' . urlencode( $nonce ) . '">' . __('Enable accessibility mode') . '</a><a id="access-off" href="widgets.php?widgets-access=off&_wpnonce=' . urlencode( $nonce ) . '">' . __('Disable accessibility mode') . "</a></p>\n";
     931                $nonce                  = wp_create_nonce( 'widgets-access' );
     932                $this->_screen_settings = '<p><a id="access-on" href="widgets.php?widgets-access=on&_wpnonce=' . urlencode( $nonce ) . '">' . __( 'Enable accessibility mode' ) . '</a><a id="access-off" href="widgets.php?widgets-access=off&_wpnonce=' . urlencode( $nonce ) . '">' . __( 'Disable accessibility mode' ) . "</a></p>\n";
    902933                break;
    903             case 'post' :
    904                 $expand = '<fieldset class="editor-expand hidden"><legend>' . __( 'Additional settings' ) . '</legend><label for="editor-expand-toggle">';
    905                 $expand .= '<input type="checkbox" id="editor-expand-toggle"' . checked( get_user_setting( 'editor_expand', 'on' ), 'on', false ) . ' />';
    906                 $expand .= __( 'Enable full-height editor and distraction-free functionality.' ) . '</label></fieldset>';
     934            case 'post':
     935                $expand                 = '<fieldset class="editor-expand hidden"><legend>' . __( 'Additional settings' ) . '</legend><label for="editor-expand-toggle">';
     936                $expand                .= '<input type="checkbox" id="editor-expand-toggle"' . checked( get_user_setting( 'editor_expand', 'on' ), 'on', false ) . ' />';
     937                $expand                .= __( 'Enable full-height editor and distraction-free functionality.' ) . '</label></fieldset>';
    907938                $this->_screen_settings = $expand;
    908939                break;
    … …  
    925956        $this->_screen_settings = apply_filters( 'screen_settings', $this->_screen_settings, $this );
    926957
    927         if ( $this->_screen_settings || $this->_options )
     958        if ( $this->_screen_settings || $this->_options ) {
    928959            $show_screen = true;
     960        }
    929961
    930962        /**
    … …  
    951983     */
    952984    public function render_screen_options( $options = array() ) {
    953         $options = wp_parse_args( $options, array(
    954             'wrap' => true,
    955         ) );
     985        $options = wp_parse_args(
     986            $options, array(
     987                'wrap' => true,
     988            )
     989        );
    956990
    957991        $wrapper_start = $wrapper_end = $form_start = $form_end = '';
    … …  
    960994        if ( $options['wrap'] ) {
    961995            $wrapper_start = '<div id="screen-options-wrap" class="hidden" tabindex="-1" aria-label="' . esc_attr__( 'Screen Options Tab' ) . '">';
    962             $wrapper_end = '</div>';
     996            $wrapper_end   = '</div>';
    963997        }
    964998
    … …  
    9661000        if ( 'widgets' !== $this->base ) {
    9671001            $form_start = "\n<form id='adv-settings' method='post'>\n";
    968             $form_end = "\n" . wp_nonce_field( 'screen-options-nonce', 'screenoptionnonce', false, false ) . "\n</form>\n";
     1002            $form_end   = "\n" . wp_nonce_field( 'screen-options-nonce', 'screenoptionnonce', false, false ) . "\n</form>\n";
    9691003        }
    9701004
    … …  
    10151049            meta_box_prefs( $this );
    10161050
    1017             if ( 'dashboard' === $this->id && has_action( 'welcome_panel' ) && current_user_can( 'edit_theme_options' ) ) {
    1018                 if ( isset( $_GET['welcome'] ) ) {
    1019                     $welcome_checked = empty( $_GET['welcome'] ) ? 0 : 1;
    1020                     update_user_meta( get_current_user_id(), 'show_welcome_panel', $welcome_checked );
    1021                 } else {
    1022                     $welcome_checked = get_user_meta( get_current_user_id(), 'show_welcome_panel', true );
    1023                     if ( 2 == $welcome_checked && wp_get_current_user()->user_email != get_option( 'admin_email' ) ) {
    1024                         $welcome_checked = false;
    1025                     }
     1051        if ( 'dashboard' === $this->id && has_action( 'welcome_panel' ) && current_user_can( 'edit_theme_options' ) ) {
     1052            if ( isset( $_GET['welcome'] ) ) {
     1053                $welcome_checked = empty( $_GET['welcome'] ) ? 0 : 1;
     1054                update_user_meta( get_current_user_id(), 'show_welcome_panel', $welcome_checked );
     1055            } else {
     1056                $welcome_checked = get_user_meta( get_current_user_id(), 'show_welcome_panel', true );
     1057                if ( 2 == $welcome_checked && wp_get_current_user()->user_email != get_option( 'admin_email' ) ) {
     1058                    $welcome_checked = false;
    10261059                }
    1027                 echo '<label for="wp_welcome_panel-hide">';
    1028                 echo '<input type="checkbox" id="wp_welcome_panel-hide"' . checked( (bool) $welcome_checked, true, false ) . ' />';
    1029                 echo _x( 'Welcome', 'Welcome panel' ) . "</label>\n";
    1030             }
     1060            }
     1061            echo '<label for="wp_welcome_panel-hide">';
     1062            echo '<input type="checkbox" id="wp_welcome_panel-hide"' . checked( (bool) $welcome_checked, true, false ) . ' />';
     1063            echo _x( 'Welcome', 'Welcome panel' ) . "</label>\n";
     1064        }
    10311065        ?>
    10321066        </fieldset>
    … …  
    10931127
    10941128        $screen_layout_columns = $this->get_columns();
    1095         $num = $this->get_option( 'layout_columns', 'max' );
     1129        $num                   = $this->get_option( 'layout_columns', 'max' );
    10961130
    10971131        ?>
    10981132        <fieldset class='columns-prefs'>
    1099         <legend class="screen-layout"><?php _e( 'Layout' ); ?></legend><?php
    1100             for ( $i = 1; $i <= $num; ++$i ):
    1101                 ?>
    1102                 <label class="columns-prefs-<?php echo $i; ?>">
    1103                     <input type='radio' name='screen_columns' value='<?php echo esc_attr( $i ); ?>'
    1104                         <?php checked( $screen_layout_columns, $i ); ?> />
    1105                     <?php printf( _n( '%s column', '%s columns', $i ), number_format_i18n( $i ) ); ?>
     1133        <legend class="screen-layout"><?php _e( 'Layout' ); ?></legend>
     1134                                                <?php
     1135                                                for ( $i = 1; $i <= $num; ++$i ) :
     1136                                                    ?>
     1137                                                    <label class="columns-prefs-<?php echo $i; ?>">
     1138                <input type='radio' name='screen_columns' value='<?php echo esc_attr( $i ); ?>'
     1139                    <?php checked( $screen_layout_columns, $i ); ?> />
     1140                <?php printf( _n( '%s column', '%s columns', $i ), number_format_i18n( $i ) ); ?>
    11061141                </label>
    11071142                <?php
    1108             endfor; ?>
     1143            endfor;
     1144            ?>
    11091145        </fieldset>
    11101146        <?php
    … …  
    11901226        }
    11911227
    1192         $view_mode_post_types = get_post_types( array( 'hierarchical' => false, 'show_ui' => true ) );
     1228        $view_mode_post_types = get_post_types(
     1229            array(
     1230                'hierarchical' => false,
     1231                'show_ui'      => true,
     1232            )
     1233        );
    11931234
    11941235        /**
Note: See TracChangeset for help on using the changeset viewer.