WordPress.org
Get WordPress

Make WordPress Core

Changeset 16235


Ignore:
Timestamp:
11/08/2010 03:43:44 PM (16 years ago)
Author:
scribu
Message:

Use get_current_screen() in list table classes. Fixes #15338

Location:
trunk/wp-admin/includes
Files:
6 edited

Legend:

Unmodified
Added
Removed

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

    r16205 r16235  
    3131
    3232                parent::WP_List_Table( array(
    33                         'screen' => 'edit-comments',
    3433                        'plural' => 'comments'
    3534                ) );

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

    r16226 r16235  
    7979        function WP_List_Table( $args = array() ) {
    8080                $args = wp_parse_args( $args, array(
    81                         'screen' => get_current_screen(),
    8281                        'plural' => '',
    8382                        'singular' => '',
    … …  
    8584                ) );
    8685
    87                 $this->screen = $args['screen'];
    88 
    89                 if ( is_string( $this->screen ) )
    90                         $this->screen = convert_to_screen( $this->screen );
    91 
    92                 add_filter( 'manage_' . $this->screen->id . '_columns', array( &$this, 'get_columns' ), 0 );
     86                $screen = get_current_screen();
     87
     88                add_filter( "manage_{$screen->id}_columns", array( &$this, 'get_columns' ), 0 );
    9389
    9490                if ( !$args['plural'] )
    95                         $args['plural'] = $this->screen->base;
     91                        $args['plural'] = $screen->base;
    9692
    9793                $this->_args = $args;
    … …  
    204200         */
    205201        function views() {
     202                $screen = get_current_screen();
     203
    206204                $views = $this->get_views();
    207                 $views = apply_filters( 'views_' . $this->screen->id, $views );
     205                $views = apply_filters( 'views_' . $screen->id, $views );
    208206
    209207                if ( empty( $views ) )
    … …  
    238236         */
    239237        function bulk_actions() {
     238                $screen = get_current_screen();
    240239
    241240                if ( is_null( $this->_actions ) ) {
    242241                        $this->_actions = $this->get_bulk_actions();
    243                         $this->_actions = apply_filters( 'bulk_actions-' . $this->screen->id, $this->_actions );
     242                        $this->_actions = apply_filters( 'bulk_actions-' . $screen->id, $this->_actions );
    244243                        $two = '';
    245244                }
    … …  
    539538        function get_column_info() {
    540539                if ( !isset( $this->_column_headers ) ) {
    541                         $columns = get_column_headers( $this->screen );
    542                         $hidden = get_hidden_columns( $this->screen );
    543                         $sortable = apply_filters( 'manage_' . $this->screen->id . '_sortable_columns', $this->get_sortable_columns() );
     540                        $screen = get_current_screen();
     541
     542                        $columns = get_column_headers( $screen );
     543                        $hidden = get_hidden_columns( $screen );
     544                        $sortable = apply_filters( "manage_{$screen->id}_sortable_columns", $this->get_sortable_columns() );
    544545
    545546                        $this->_column_headers = array( $columns, $hidden, $sortable );
    … …  
    558559         */
    559560        function print_column_headers( $with_id = true ) {
    560                 $screen = $this->screen;
     561                $screen = get_current_screen();
    561562
    562563                list( $columns, $hidden, $sortable ) = $this->get_column_info();
    … …  
    811812                $args = array(
    812813                        'class' => get_class( $this ),
    813                         'screen' => $this->screen
     814                        'screen' => get_current_screen()
    814815                );
    815816

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

    r16205 r16235  
    5858                );
    5959
    60                 if ( ! is_multisite() || ( $this->screen->is_network && current_user_can('manage_network_plugins') ) ) {
     60                $screen = get_current_screen();
     61
     62                if ( ! is_multisite() || ( $screen->is_network && current_user_can('manage_network_plugins') ) ) {
    6163                        if ( apply_filters( 'show_advanced_plugins', true, 'mustuse' ) )
    6264                                $plugins['mustuse'] = get_mu_plugins();
    … …  
    8688                foreach ( (array) $plugins['all'] as $plugin_file => $plugin_data ) {
    8789                        // Filter into individual sections
    88                         if ( is_plugin_active_for_network($plugin_file) && !$this->screen->is_network ) {
     90                        if ( is_plugin_active_for_network($plugin_file) && !$screen->is_network ) {
    8991                                unset( $plugins['all'][ $plugin_file ] );
    9092                                continue;
    9193                        } elseif ( is_multisite() && is_network_only_plugin( $plugin_file ) && !current_user_can( 'manage_network_plugins' ) ) {
    9294                                $plugins['network'][ $plugin_file ] = $plugin_data;
    93                         } elseif ( ( !$this->screen->is_network && is_plugin_active( $plugin_file ) )
    94                                 || ( $this->screen->is_network && is_plugin_active_for_network( $plugin_file ) ) ) {
     95                        } elseif ( ( !$screen->is_network && is_plugin_active( $plugin_file ) )
     96                                || ( $screen->is_network && is_plugin_active_for_network( $plugin_file ) ) ) {
    9597                                $plugins['active'][ $plugin_file ] = $plugin_data;
    9698                        } else {
    97                                 if ( !$this->screen->is_network && isset( $recently_activated[ $plugin_file ] ) ) // Was the plugin recently activated?
     99                                if ( !$screen->is_network && isset( $recently_activated[ $plugin_file ] ) ) // Was the plugin recently activated?
    98100                                        $plugins['recently_activated'][ $plugin_file ] = $plugin_data;
    99101                                $plugins['inactive'][ $plugin_file ] = $plugin_data;
    … …  
    129131                }
    130132
    131                 $plugins_per_page = $this->get_items_per_page( str_replace( '-', '_', "{$this->screen->id}_per_page" ) );
     133                $plugins_per_page = $this->get_items_per_page( str_replace( '-', '_', $screen->id . '_per_page' ) );
    132134
    133135                $start = ( $page - 1 ) * $plugins_per_page;
    … …  
    254256                $actions = array();
    255257
     258                $screen = get_current_screen();
     259
    256260                if ( 'active' != $status ) {
    257                         $action = $this->screen->is_network ? 'network-activate-selected' : 'activate-selected';
     261                        $action = $screen->is_network ? 'network-activate-selected' : 'activate-selected';
    258262                        $actions[ $action ] = __( 'Activate' );
    259263                }
    … …  
    262266                        $actions['deactivate-selected'] = __( 'Deactivate' );
    263267
    264                 if ( !is_multisite() || $this->screen->is_network ) {
     268                if ( !is_multisite() || $screen->is_network ) {
    265269                        if ( current_user_can( 'update_plugins' ) )
    266270                                $actions['update-selected'] = __( 'Update' );
    … …  
    302306
    303307                $context = $status;
     308
     309                $screen = get_current_screen();
    304310
    305311                foreach ( $this->items as $plugin_file => $plugin_data ) {
    … …  
    314320
    315321                        if ( 'mustuse' == $context ) {
    316                                 if ( is_multisite() && !$this->screen->is_network )
     322                                if ( is_multisite() && !$screen->is_network )
    317323                                        continue;
    318324                                $is_active = true;
    319325                        } elseif ( 'dropins' == $context ) {
    320                                 if ( is_multisite() && !$this->screen->is_network )
     326                                if ( is_multisite() && !$screen->is_network )
    321327                                        continue;
    322328                                $dropins = _get_dropins();
    … …  
    338344                        } else {
    339345                                $is_active_for_network = is_plugin_active_for_network($plugin_file);
    340                                 if ( $this->screen->is_network )
     346                                if ( $screen->is_network )
    341347                                        $is_active = $is_active_for_network;
    342348                                else
    343349                                        $is_active = is_plugin_active( $plugin_file );
    344350
    345                                 if ( $is_active_for_network && !is_super_admin() && !$this->screen->is_network )
     351                                if ( $is_active_for_network && !is_super_admin() && !$screen->is_network )
    346352                                        continue;
    347353
    348                                 if ( $this->screen->is_network ) {
     354                                if ( $screen->is_network ) {
    349355                                        if ( $is_active_for_network ) {
    350356                                                if ( current_user_can( 'manage_network_plugins' ) )
    … …  
    360366                                                $actions['deactivate'] = '<a href="' . wp_nonce_url('plugins.php?action=deactivate&amp;plugin=' . $plugin_file . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'deactivate-plugin_' . $plugin_file) . '" title="' . __('Deactivate this plugin') . '">' . __('Deactivate') . '</a>';
    361367                                        } else {
    362                                                 if ( is_network_only_plugin( $plugin_file ) && !$this->screen->is_network )
     368                                                if ( is_network_only_plugin( $plugin_file ) && !$screen->is_network )
    363369                                                        continue;
    364370
    … …  
    368374                                                        $actions['delete'] = '<a href="' . wp_nonce_url('plugins.php?action=delete-selected&amp;checked[]=' . $plugin_file . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'bulk-plugins') . '" title="' . __('Delete this plugin') . '" class="delete">' . __('Delete') . '</a>';
    369375                                        } // end if $is_active
    370                                  } // end if $this->screen->is_network
     376                                 } // end if $screen->is_network
    371377
    372378                                if ( current_user_can('edit_plugins') && is_writable(WP_PLUGIN_DIR . '/' . $plugin_file) )

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

    r16212 r16235  
    257257
    258258        function get_columns() {
    259                 $screen = $this->screen;
     259                $screen = get_current_screen();
    260260
    261261                if ( empty( $screen ) )
    … …  
    682682                global $mode;
    683683
    684                 $screen = $this->screen;
     684                $screen = get_current_screen();
    685685
    686686                $post = get_default_post_to_edit( $screen->post_type );

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

    r16214 r16235  
    291291
    292292        function column_default( $tag, $column_name ) {
    293                 return apply_filters( "manage_{$this->screen->taxonomy}_custom_column", '', $column_name, $tag->term_id );
     293                $screen = get_current_screen();
     294
     295                return apply_filters( "manage_{$screen->taxonomy}_custom_column", '', $column_name, $tag->term_id );
    294296        }
    295297

  • trunk/wp-admin/includes/deprecated.php

    r16182 r16235  
    704704        _deprecated_function( __FUNCTION__, '3.1', 'WP_List_Table' );
    705705
    706         global $wp_list_table;
    707 
    708         $wp_list_table = new _WP_List_Table_Compat($screen);
    709         $wp_list_table->_columns = $columns;
     706        $wp_list_table = new _WP_List_Table_Compat($screen, $columns);
    710707}
    711708
    … …  
    720717        _deprecated_function( __FUNCTION__, '3.1', 'WP_List_Table' );
    721718
    722         global $wp_list_table;
    723         if ( !is_a($wp_list_table, 'WP_List_Table') )
    724                 $wp_list_table = new _WP_List_Table_Compat($screen);
     719        $wp_list_table = new _WP_List_Table_Compat($screen);
    725720
    726721        $wp_list_table->print_column_headers($id);
    … …  
    729724// Helper class to be used only by deprecated functions
    730725class _WP_List_Table_Compat extends WP_List_Table {
    731 
    732         var $_columns = array();
    733 
    734         function _WP_List_Table_Compat( $screen ) {
    735                 parent::WP_List_Table( array(
    736                         'screen' => $screen,
    737                         'ajax' => false
    738                 ) );
     726        var $_screen;
     727        var $_columns;
     728
     729        function _WP_List_Table_Compat( $screen, $columns = array() ) {
     730                if ( is_string( $screen ) )
     731                        $screen = convert_to_screen( $screen );
     732
     733                $this->_screen = $screen;
     734
     735                if ( !empty( $columns ) ) {
     736                        $this->_columns = $columns;
     737                        add_filter( 'manage_' . $screen->id . '_columns', array( &$this, 'get_columns' ), 0 );
     738                }
     739        }
     740
     741        function get_column_info() {
     742                $columns = get_column_headers( $this->_screen );
     743                $hidden = get_hidden_columns( $this->_screen );
     744                $sortable = array();
     745
     746                return array( $columns, $hidden, $sortable );
    739747        }
    740748
Note: See TracChangeset for help on using the changeset viewer.