WordPress.org
Get WordPress

Make WordPress Core

Changeset 48424


Ignore:
Timestamp:
07/10/2020 04:02:00 PM (6 years ago)
Author:
SergeyBiryukov
Message:

Administration: Bring some consistency to handling list table view modes.

  • Remove duplicate variables and DocBlocks.
  • Add missing description for the $mode global.
  • Use sentence case for "Compact view" and "Extended view" labels.

Follow-up to [48398], [48423].

Props afercia, Offereins, SergeyBiryukov.
See #49715.

Location:
trunk/src/wp-admin/includes
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/class-wp-comments-list-table.php

    r48415 r48424  
    8181         * @global int    $post_id
    8282         * @global string $comment_status
     83         * @global string $comment_type
    8384         * @global string $search
    84          * @global string $comment_type
    8585         */
    8686        public function prepare_items() {
    87                 global $post_id, $comment_status, $search, $comment_type;
     87                global $post_id, $comment_status, $comment_type, $search;
     88
     89                if ( ! empty( $_REQUEST['mode'] ) ) {
     90                        $mode = 'extended' === $_REQUEST['mode'] ? 'extended' : 'list';
     91                        set_user_setting( 'posts_list_mode', $mode );
     92                }
    8893
    8994                $comment_status = isset( $_REQUEST['comment_status'] ) ? $_REQUEST['comment_status'] : 'all';
    … …  
    123128                if ( $doing_ajax && isset( $_REQUEST['offset'] ) ) {
    124129                        $start += $_REQUEST['offset'];
    125                 }
    126 
    127                 if ( ! empty( $_REQUEST['mode'] ) ) {
    128                         $mode = 'extended' === $_REQUEST['mode'] ? 'extended' : 'list';
    129                         set_user_setting( 'posts_list_mode', $mode );
    130                 } else {
    131                         $mode = get_user_setting( 'posts_list_mode', 'list' );
    132130                }
    133131
    … …  
    760758
    761759                $always_visible = false;
    762                 $mode           = get_user_setting( 'posts_list_mode', 'list' );
     760
     761                $mode = get_user_setting( 'posts_list_mode', 'list' );
     762
    763763                if ( 'extended' === $mode ) {
    764764                        $always_visible = true;
    765765                }
    766766
    767                 $i    = 0;
    768767                $out .= '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
     768
     769                $i = 0;
     770
    769771                foreach ( $actions as $action => $link ) {
    770772                        ++$i;
    771                         ( ( ( 'approve' === $action || 'unapprove' === $action ) && 2 === $i ) || 1 === $i ) ? $sep = '' : $sep = ' | ';
    772 
    773                         // Reply and quickedit need a hide-if-no-js span when not added with ajax.
     773
     774                        if ( ( ( 'approve' === $action || 'unapprove' === $action ) && 2 === $i )
     775                                || 1 === $i
     776                        ) {
     777                                $sep = '';
     778                        } else {
     779                                $sep = ' | ';
     780                        }
     781
     782                        // Reply and quickedit need a hide-if-no-js span when not added with Ajax.
    774783                        if ( ( 'reply' === $action || 'quickedit' === $action ) && ! wp_doing_ajax() ) {
    775784                                $action .= ' hide-if-no-js';
    776                         } elseif ( ( 'untrash' === $action && 'trash' === $the_comment_status ) || ( 'unspam' === $action && 'spam' === $the_comment_status ) ) {
     785                        } elseif ( ( 'untrash' === $action && 'trash' === $the_comment_status )
     786                                || ( 'unspam' === $action && 'spam' === $the_comment_status )
     787                        ) {
    777788                                if ( '1' == get_comment_meta( $comment->comment_ID, '_wp_trash_meta_status', true ) ) {
    778789                                        $action .= ' approve';
    … …  
    784795                        $out .= "<span class='$action'>$sep$link</span>";
    785796                }
     797
    786798                $out .= '</div>';
    787799

  • trunk/src/wp-admin/includes/class-wp-list-table.php

    r48415 r48424  
    167167                if ( empty( $this->modes ) ) {
    168168                        $this->modes = array(
    169                                 'list'     => __( 'Compact View' ),
    170                                 'extended' => __( 'Extended View' ),
     169                                'list'     => __( 'Compact view' ),
     170                                'extended' => __( 'Extended view' ),
    171171                        );
    172172                }
    … …  
    518518        protected function row_actions( $actions, $always_visible = false ) {
    519519                $action_count = count( $actions );
    520                 $i            = 0;
    521520
    522521                if ( ! $action_count ) {
    … …  
    525524
    526525                $mode = get_user_setting( 'posts_list_mode', 'list' );
     526
    527527                if ( 'extended' === $mode ) {
    528528                        $always_visible = true;
    … …  
    530530
    531531                $out = '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
     532
     533                $i = 0;
     534
    532535                foreach ( $actions as $action => $link ) {
    533536                        ++$i;
    534                         ( $i == $action_count ) ? $sep = '' : $sep = ' | ';
    535                         $out                          .= "<span class='$action'>$link$sep</span>";
    536                 }
     537
     538                        $sep = ( $i < $action_count ) ? ' | ' : '';
     539
     540                        $out .= "<span class='$action'>$link$sep</span>";
     541                }
     542
    537543                $out .= '</div>';
    538544
    … …  
    12531259         */
    12541260        protected function get_table_classes() {
    1255                 $mode       = get_user_setting( 'posts_list_mode', 'list' );
    1256                 $mode_class = 'extended' === $mode ? 'table-view-extended' : 'table-view-list';
    1257                 $mode       = get_user_setting( 'posts_list_mode', 'list' );
    1258                 /**
    1259                  * Filters the current view mode.
    1260                  *
    1261                  * @since 5.5.0
    1262                  *
    1263                  * @param string $mode The current selected mode. Default value of
    1264                  *                     posts_list_mode user setting.
    1265                  */
     1261                $mode = get_user_setting( 'posts_list_mode', 'list' );
     1262
     1263                /** This filter is documented in wp-admin/includes/class-wp-screen.php */
    12661264                $mode = apply_filters( 'table_view_mode', $mode );
    12671265
    1268                 $mode_class = 'extended' === $mode ? 'table-view-extended' : 'table-view-' . $mode;
     1266                $mode_class = esc_attr( 'table-view-' . $mode );
    12691267
    12701268                return array( 'widefat', 'fixed', 'striped', $mode_class, $this->_args['plural'] );

  • trunk/src/wp-admin/includes/class-wp-media-list-table.php

    r48417 r48424  
    6262
    6363        /**
     64         * @global string   $mode                  List table view mode.
    6465         * @global WP_Query $wp_query              WordPress Query object.
    6566         * @global array    $post_mime_types
    6667         * @global array    $avail_post_mime_types
    67          * @global string   $mode
    6868         */
    6969        public function prepare_items() {
    70                 global $wp_query, $post_mime_types, $avail_post_mime_types, $mode;
     70                global $mode, $wp_query, $post_mime_types, $avail_post_mime_types;
     71
     72                $mode = empty( $_REQUEST['mode'] ) ? 'list' : $_REQUEST['mode'];
    7173
    7274                // Exclude attachments scheduled for deletion in the next two hours
    … …  
    9698
    9799                $this->is_trash = isset( $_REQUEST['attachment-filter'] ) && 'trash' === $_REQUEST['attachment-filter'];
    98 
    99                 $mode = empty( $_REQUEST['mode'] ) ? 'list' : $_REQUEST['mode'];
    100100
    101101                $this->set_pagination_args(

  • trunk/src/wp-admin/includes/class-wp-ms-sites-list-table.php

    r47219 r48424  
    6363         * @since 3.1.0
    6464         *
     65         * @global string $mode List table view mode.
    6566         * @global string $s
    66          * @global string $mode
    6767         * @global wpdb   $wpdb WordPress database abstraction object.
    6868         */
    6969        public function prepare_items() {
    70                 global $s, $mode, $wpdb;
     70                global $mode, $s, $wpdb;
    7171
    7272                if ( ! empty( $_REQUEST['mode'] ) ) {
    … …  
    577577
    578578        /**
    579          * @global string $mode
     579         * @global string $mode List table view mode.
    580580         */
    581581        public function display_rows() {

  • trunk/src/wp-admin/includes/class-wp-ms-users-list-table.php

    r47557 r48424  
    2525
    2626        /**
     27         * @global string $mode       List table view mode.
    2728         * @global string $usersearch
    2829         * @global string $role
    29          * @global string $mode
    3030         */
    3131        public function prepare_items() {
    32                 global $usersearch, $role, $mode;
     32                global $mode, $usersearch, $role;
     33
     34                if ( ! empty( $_REQUEST['mode'] ) ) {
     35                        $mode = 'excerpt' === $_REQUEST['mode'] ? 'excerpt' : 'list';
     36                        set_user_setting( 'network_users_list_mode', $mode );
     37                } else {
     38                        $mode = get_user_setting( 'network_users_list_mode', 'list' );
     39                }
    3340
    3441                $usersearch = isset( $_REQUEST['s'] ) ? wp_unslash( trim( $_REQUEST['s'] ) ) : '';
    … …  
    8289                if ( isset( $_REQUEST['order'] ) ) {
    8390                        $args['order'] = $_REQUEST['order'];
    84                 }
    85 
    86                 if ( ! empty( $_REQUEST['mode'] ) ) {
    87                         $mode = 'excerpt' === $_REQUEST['mode'] ? 'excerpt' : 'list';
    88                         set_user_setting( 'network_users_list_mode', $mode );
    89                 } else {
    90                         $mode = get_user_setting( 'network_users_list_mode', 'list' );
    9191                }
    9292
    … …  
    423423                        $actions = apply_filters( 'ms_user_list_site_actions', $actions, $val->userblog_id );
    424424
    425                         $i            = 0;
    426425                        $action_count = count( $actions );
     426
     427                        $i = 0;
     428
    427429                        foreach ( $actions as $action => $link ) {
    428430                                ++$i;
    429                                 $sep = ( $i == $action_count ) ? '' : ' | ';
     431
     432                                $sep = ( $i < $action_count ) ? ' | ' : '';
     433
    430434                                echo "<span class='$action'>$link$sep</span>";
    431435                        }
     436
    432437                        echo '</small></span><br/>';
    433438                }

  • trunk/src/wp-admin/includes/class-wp-posts-list-table.php

    r48415 r48424  
    136136
    137137        /**
     138         * @global string   $mode             List table view mode.
    138139         * @global array    $avail_post_stati
    139140         * @global WP_Query $wp_query         WordPress Query object.
    140141         * @global int      $per_page
    141          * @global string   $mode
    142142         */
    143143        public function prepare_items() {
    144                 global $avail_post_stati, $wp_query, $per_page, $mode;
     144                global $mode, $avail_post_stati, $wp_query, $per_page;
     145
     146                if ( ! empty( $_REQUEST['mode'] ) ) {
     147                        $mode = 'extended' === $_REQUEST['mode'] ? 'extended' : 'list';
     148                        set_user_setting( 'posts_list_mode', $mode );
     149                } else {
     150                        $mode = get_user_setting( 'posts_list_mode', 'list' );
     151                }
    145152
    146153                // Is going to call wp().
    … …  
    176183                                }
    177184                        }
    178                 }
    179 
    180                 if ( ! empty( $_REQUEST['mode'] ) ) {
    181                         $mode = 'extended' === $_REQUEST['mode'] ? 'extended' : 'list';
    182                         set_user_setting( 'posts_list_mode', $mode );
    183                 } else {
    184                         $mode = get_user_setting( 'posts_list_mode', 'list' );
    185185                }
    186186
    … …  
    596596
    597597        /**
     598         * @global string $mode List table view mode.
     599         *
    598600         * @return array
    599601         */
    600602        protected function get_table_classes() {
    601                 $mode       = get_user_setting( 'posts_list_mode', 'list' );
    602                 $mode_class = 'extended' === $mode ? 'table-view-extended' : 'table-view-list';
    603                 $mode       = get_user_setting( 'posts_list_mode', 'list' );
    604                 /**
    605                  * Filters the current view mode.
    606                  *
    607                  * @since 5.5.0
    608                  *
    609                  * @param string $mode The current selected mode. Default value of
    610                  *                     posts_list_mode user setting.
    611                  */
     603                global $mode;
     604
     605                /** This filter is documented in wp-admin/includes/class-wp-screen.php */
    612606                $mode = apply_filters( 'table_view_mode', $mode );
    613607
    614                 $mode_class = 'extended' === $mode ? 'table-view-extended' : 'table-view-' . $mode;
     608                $mode_class = esc_attr( 'table-view-' . $mode );
    615609
    616610                return array( 'widefat', 'fixed', 'striped', $mode_class, is_post_type_hierarchical( $this->screen->post_type ) ? 'pages' : 'posts' );

  • trunk/src/wp-admin/includes/dashboard.php

    r48191 r48424  
    757757
    758758                $i = 0;
     759
    759760                foreach ( $actions as $action => $link ) {
    760761                        ++$i;
    761762
    762                         if ( ( ( 'approve' === $action || 'unapprove' === $action ) && 2 === $i ) || 1 === $i ) {
     763                        if ( ( ( 'approve' === $action || 'unapprove' === $action ) && 2 === $i )
     764                                || 1 === $i
     765                        ) {
    763766                                $sep = '';
    764767                        } else {
Note: See TracChangeset for help on using the changeset viewer.