WordPress.org
Get WordPress

Make WordPress Core

Changeset 28493


Ignore:
Timestamp:
05/19/2014 01:16:16 AM (12 years ago)
Author:
wonderboymusic
Message:

Add access modifiers to methods and members of list table classes:

  • WP_List_Table is the base class that implements __get() and __call() for BC
  • Adds unit tests to confirm that subclasses properly inherit magic methods
  • Add modifiers to subclasses: WP_Links_List_Table, WP_Media_List_Table, WP_MS_Sites_List_Table, WP_MS_Themes_List_Table, WP_MS_Users_List_Table, WP_Plugin_Install_List_Table, WP_Plugins_List_Table, WP_Posts_List_Table, WP_Terms_List_Table, WP_Theme_Install_List_Table, WP_Themes_List_Table

See #27881, #22234.

Location:
trunk
Files:
14 edited

Legend:

Unmodified
Added
Removed

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

    r27029 r28493  
    1010class WP_Links_List_Table extends WP_List_Table {
    1111
    12         function __construct( $args = array() ) {
     12        public function __construct( $args = array() ) {
    1313                parent::__construct( array(
    1414                        'plural' => 'bookmarks',
    … …  
    1717        }
    1818
    19         function ajax_user_can() {
     19        public function ajax_user_can() {
    2020                return current_user_can( 'manage_links' );
    2121        }
    2222
    23         function prepare_items() {
     23        public function prepare_items() {
    2424                global $cat_id, $s, $orderby, $order;
    2525
    … …  
    4040        }
    4141
    42         function no_items() {
     42        public function no_items() {
    4343                _e( 'No links found.' );
    4444        }
    4545
    46         function get_bulk_actions() {
     46        protected function get_bulk_actions() {
    4747                $actions = array();
    4848                $actions['delete'] = __( 'Delete' );
    … …  
    5151        }
    5252
    53         function extra_tablenav( $which ) {
     53        protected function extra_tablenav( $which ) {
    5454                global $cat_id;
    5555
    … …  
    7676        }
    7777
    78         function get_columns() {
     78        protected function get_columns() {
    7979                return array(
    8080                        'cb'         => '<input type="checkbox" />',
    … …  
    8888        }
    8989
    90         function get_sortable_columns() {
     90        protected function get_sortable_columns() {
    9191                return array(
    9292                        'name'    => 'name',
    … …  
    9797        }
    9898
    99         function display_rows() {
     99        protected function display_rows() {
    100100                global $cat_id;
    101101

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

    r28389 r28493  
    1717         * @access protected
    1818         */
    19         var $items;
     19        protected $items;
    2020
    2121        /**
    … …  
    2626         * @access private
    2727         */
    28         var $_args;
     28        private $_args;
    2929
    3030        /**
    … …  
    3535         * @access private
    3636         */
    37         var $_pagination_args = array();
     37        private $_pagination_args = array();
    3838
    3939        /**
    … …  
    4444         * @access protected
    4545         */
    46         var $screen;
     46        protected $screen;
    4747
    4848        /**
    … …  
    5353         * @access private
    5454         */
    55         var $_actions;
     55        private $_actions;
    5656
    5757        /**
    … …  
    6262         * @access private
    6363         */
    64         var $_pagination;
     64        private $_pagination;
    6565
    6666        /**
    … …  
    7070         * @access protected
    7171         */
    72         function __construct( $args = array() ) {
     72        public function __construct( $args = array() ) {
    7373                $args = wp_parse_args( $args, array(
    7474                        'plural' => '',
    … …  
    9797
    9898        /**
     99         * Make private properties readable for backwards compatibility
     100         *
     101         * @since 4.0.0
     102         * @param string $name
     103         * @return mixed
     104         */
     105        public function __get( $name ) {
     106                return $this->$name;
     107        }
     108
     109        /**
     110         * Make private/protected methods readable for backwards compatibility
     111         *
     112         * @since 4.0.0
     113         * @param string $name
     114         * @param array $arguments
     115         * @return mixed
     116         */
     117        public function __call( $name, $arguments ) {
     118                return call_user_func_array( array( $this, $name ), $arguments );
     119        }
     120
     121        /**
    99122         * Checks the current user's permissions
    100123         * @uses wp_die()
    … …  
    104127         * @abstract
    105128         */
    106         function ajax_user_can() {
     129        public function ajax_user_can() {
    107130                die( 'function WP_List_Table::ajax_user_can() must be over-ridden in a sub-class.' );
    108131        }
    … …  
    116139         * @abstract
    117140         */
    118         function prepare_items() {
     141        public function prepare_items() {
    119142                die( 'function WP_List_Table::prepare_items() must be over-ridden in a sub-class.' );
    120143        }
    … …  
    126149         * @access protected
    127150         */
    128         function set_pagination_args( $args ) {
     151        protected function set_pagination_args( $args ) {
    129152                $args = wp_parse_args( $args, array(
    130153                        'total_items' => 0,
    … …  
    154177         * @return array
    155178         */
    156         function get_pagination_arg( $key ) {
     179        public function get_pagination_arg( $key ) {
    157180                if ( 'page' == $key )
    158181                        return $this->get_pagenum();
    … …  
    170193         * @return bool
    171194         */
    172         function has_items() {
     195        public function has_items() {
    173196                return !empty( $this->items );
    174197        }
    … …  
    180203         * @access public
    181204         */
    182         function no_items() {
     205        public function no_items() {
    183206                _e( 'No items found.' );
    184207        }
    … …  
    193216         * @param string $input_id The search input id
    194217         */
    195         function search_box( $text, $input_id ) {
     218        public function search_box( $text, $input_id ) {
    196219                if ( empty( $_REQUEST['s'] ) && !$this->has_items() )
    197220                        return;
    … …  
    225248         * @return array
    226249         */
    227         function get_views() {
     250        protected function get_views() {
    228251                return array();
    229252        }
    … …  
    235258         * @access public
    236259         */
    237         function views() {
     260        public function views() {
    238261                $views = $this->get_views();
    239262                /**
    … …  
    269292         * @return array
    270293         */
    271         function get_bulk_actions() {
     294        protected function get_bulk_actions() {
    272295                return array();
    273296        }
    … …  
    279302         * @access public
    280303         */
    281         function bulk_actions() {
     304        public function bulk_actions() {
    282305                if ( is_null( $this->_actions ) ) {
    283306                        $no_new_actions = $this->_actions = $this->get_bulk_actions();
    … …  
    327350         * @return string|bool The action name or False if no action was selected
    328351         */
    329         function current_action() {
     352        public function current_action() {
    330353                if ( isset( $_REQUEST['action'] ) && -1 != $_REQUEST['action'] )
    331354                        return $_REQUEST['action'];
    … …  
    347370         * @return string
    348371         */
    349         function row_actions( $actions, $always_visible = false ) {
     372        protected function row_actions( $actions, $always_visible = false ) {
    350373                $action_count = count( $actions );
    351374                $i = 0;
    … …  
    371394         * @access protected
    372395         */
    373         function months_dropdown( $post_type ) {
     396        protected function months_dropdown( $post_type ) {
    374397                global $wpdb, $wp_locale;
    375398
    … …  
    426449         * @access protected
    427450         */
    428         function view_switcher( $current_mode ) {
     451        protected function view_switcher( $current_mode ) {
    429452                $modes = array(
    430453                        'list'    => __( 'List View' ),
    … …  
    454477         * @param int $pending_comments
    455478         */
    456         function comments_bubble( $post_id, $pending_comments ) {
     479        protected function comments_bubble( $post_id, $pending_comments ) {
    457480                $pending_phrase = sprintf( __( '%s pending' ), number_format( $pending_comments ) );
    458481
    … …  
    474497         * @return int
    475498         */
    476         function get_pagenum() {
     499        protected function get_pagenum() {
    477500                $pagenum = isset( $_REQUEST['paged'] ) ? absint( $_REQUEST['paged'] ) : 0;
    478501
    … …  
    491514         * @return int
    492515         */
    493         function get_items_per_page( $option, $default = 20 ) {
     516        protected function get_items_per_page( $option, $default = 20 ) {
    494517                $per_page = (int) get_user_option( $option );
    495518                if ( empty( $per_page ) || $per_page < 1 )
    … …  
    517540         * @access protected
    518541         */
    519         function pagination( $which ) {
     542        protected function pagination( $which ) {
    520543                if ( empty( $this->_pagination_args ) ) {
    521544                        return;
    … …  
    612635         * @return array
    613636         */
    614         function get_columns() {
     637        protected function get_columns() {
    615638                die( 'function WP_List_Table::get_columns() must be over-ridden in a sub-class.' );
    616639        }
    … …  
    629652         * @return array
    630653         */
    631         function get_sortable_columns() {
     654        protected function get_sortable_columns() {
    632655                return array();
    633656        }
    … …  
    641664         * @return array
    642665         */
    643         function get_column_info() {
     666        protected function get_column_info() {
    644667                if ( isset( $this->_column_headers ) )
    645668                        return $this->_column_headers;
    … …  
    686709         * @return int
    687710         */
    688         function get_column_count() {
     711        public function get_column_count() {
    689712                list ( $columns, $hidden ) = $this->get_column_info();
    690713                $hidden = array_intersect( array_keys( $columns ), array_filter( $hidden ) );
    … …  
    700723         * @param bool $with_id Whether to set the id attribute or not
    701724         */
    702         function print_column_headers( $with_id = true ) {
     725        protected function print_column_headers( $with_id = true ) {
    703726                list( $columns, $hidden, $sortable ) = $this->get_column_info();
    704727
    … …  
    768791         * @access public
    769792         */
    770         function display() {
     793        public function display() {
    771794                $singular = $this->_args['singular'];
    772795
    … …  
    806829         * @return array
    807830         */
    808         function get_table_classes() {
     831        protected function get_table_classes() {
    809832                return array( 'widefat', 'fixed', $this->_args['plural'] );
    810833        }
    … …  
    816839         * @access protected
    817840         */
    818         function display_tablenav( $which ) {
     841        protected function display_tablenav( $which ) {
    819842                if ( 'top' == $which )
    820843                        wp_nonce_field( 'bulk-' . $this->_args['plural'] );
    … …  
    841864         * @access protected
    842865         */
    843         function extra_tablenav( $which ) {}
     866        protected function extra_tablenav( $which ) {}
    844867
    845868        /**
    … …  
    849872         * @access protected
    850873         */
    851         function display_rows_or_placeholder() {
     874        protected function display_rows_or_placeholder() {
    852875                if ( $this->has_items() ) {
    853876                        $this->display_rows();
    … …  
    865888         * @access protected
    866889         */
    867         function display_rows() {
     890        protected function display_rows() {
    868891                foreach ( $this->items as $item )
    869892                        $this->single_row( $item );
    … …  
    878901         * @param object $item The current item
    879902         */
    880         function single_row( $item ) {
     903        protected function single_row( $item ) {
    881904                static $row_class = '';
    882905                $row_class = ( $row_class == '' ? ' class="alternate"' : '' );
    … …  
    895918         * @param object $item The current item
    896919         */
    897         function single_row_columns( $item ) {
     920        protected function single_row_columns( $item ) {
    898921                list( $columns, $hidden ) = $this->get_column_info();
    899922
    … …  
    931954         * @access public
    932955         */
    933         function ajax_response() {
     956        public function ajax_response() {
    934957                $this->prepare_items();
    935958
    … …  
    964987         * @access private
    965988         */
    966         function _js_vars() {
     989        private function _js_vars() {
    967990                $args = array(
    968991                        'class'  => get_class( $this ),

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

    r28293 r28493  
    1010class WP_Media_List_Table extends WP_List_Table {
    1111
    12         function __construct( $args = array() ) {
     12        public function __construct( $args = array() ) {
    1313                $this->detached = isset( $_REQUEST['detached'] ) || isset( $_REQUEST['find_detached'] );
    1414
    … …  
    1919        }
    2020
    21         function ajax_user_can() {
     21        public function ajax_user_can() {
    2222                return current_user_can('upload_files');
    2323        }
    2424
    25         function prepare_items() {
     25        public function prepare_items() {
    2626                global $lost, $wp_query, $post_mime_types, $avail_post_mime_types;
    2727
    … …  
    4242        }
    4343
    44         function get_views() {
     44        protected function get_views() {
    4545                global $wpdb, $post_mime_types, $avail_post_mime_types;
    4646
    … …  
    7575        }
    7676
    77         function get_bulk_actions() {
     77        protected function get_bulk_actions() {
    7878                $actions = array();
    7979                $actions['delete'] = __( 'Delete Permanently' );
    … …  
    8484        }
    8585
    86         function extra_tablenav( $which ) {
     86        protected function extra_tablenav( $which ) {
    8787?>
    8888                <div class="alignleft actions">
    … …  
    105105        }
    106106
    107         function current_action() {
     107        public function current_action() {
    108108                if ( isset( $_REQUEST['find_detached'] ) )
    109109                        return 'find_detached';
    … …  
    118118        }
    119119
    120         function has_items() {
     120        public function has_items() {
    121121                return have_posts();
    122122        }
    123123
    124         function no_items() {
     124        public function no_items() {
    125125                _e( 'No media attachments found.' );
    126126        }
    127127
    128         function get_columns() {
     128        protected function get_columns() {
    129129                $posts_columns = array();
    130130                $posts_columns['cb'] = '<input type="checkbox" />';
    … …  
    181181        }
    182182
    183         function get_sortable_columns() {
     183        protected function get_sortable_columns() {
    184184                return array(
    185185                        'title'    => 'title',
    … …  
    191191        }
    192192
    193         function display_rows() {
     193        protected function display_rows() {
    194194                global $post;
    195195
    … …  
    422422        }
    423423
    424         function _get_row_actions( $post, $att_title ) {
     424        private function _get_row_actions( $post, $att_title ) {
    425425                $actions = array();
    426426

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

    r27029 r28493  
    1010class WP_MS_Sites_List_Table extends WP_List_Table {
    1111
    12         function __construct( $args = array() ) {
     12        public function __construct( $args = array() ) {
    1313                parent::__construct( array(
    1414                        'plural' => 'sites',
    … …  
    1717        }
    1818
    19         function ajax_user_can() {
     19        public function ajax_user_can() {
    2020                return current_user_can( 'manage_sites' );
    2121        }
    2222
    23         function prepare_items() {
     23        public function prepare_items() {
    2424                global $s, $mode, $wpdb;
    2525
    … …  
    121121        }
    122122
    123         function no_items() {
     123        public function no_items() {
    124124                _e( 'No sites found.' );
    125125        }
    126126
    127         function get_bulk_actions() {
     127        protected function get_bulk_actions() {
    128128                $actions = array();
    129129                if ( current_user_can( 'delete_sites' ) )
    … …  
    135135        }
    136136
    137         function pagination( $which ) {
     137        protected function pagination( $which ) {
    138138                global $mode;
    139139
    … …  
    144144        }
    145145
    146         function get_columns() {
     146        protected function get_columns() {
    147147                $blogname_columns = ( is_subdomain_install() ) ? __( 'Domain' ) : __( 'Path' );
    148148                $sites_columns = array(
    … …  
    170170        }
    171171
    172         function get_sortable_columns() {
     172        protected function get_sortable_columns() {
    173173                return array(
    174174                        'blogname'    => 'blogname',
    … …  
    178178        }
    179179
    180         function display_rows() {
     180        protected function display_rows() {
    181181                global $mode;
    182182

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

    r27090 r28493  
    1010class WP_MS_Themes_List_Table extends WP_List_Table {
    1111
    12         var $site_id;
    13         var $is_site_themes;
    14 
    15         function __construct( $args = array() ) {
     12        public $site_id;
     13        public $is_site_themes;
     14
     15        public function __construct( $args = array() ) {
    1616                global $status, $page;
    1717
    … …  
    3333        }
    3434
    35         function get_table_classes() {
     35        protected function get_table_classes() {
    3636                return array( 'widefat', 'plugins' );   // todo: remove and add CSS for .themes
    3737        }
    3838
    39         function ajax_user_can() {
     39        public function ajax_user_can() {
    4040                if ( $this->is_site_themes )
    4141                        return current_user_can( 'manage_sites' );
    … …  
    4444        }
    4545
    46         function prepare_items() {
     46        public function prepare_items() {
    4747                global $status, $totals, $page, $orderby, $order, $s;
    4848
    … …  
    132132        }
    133133
    134         function _search_callback( $theme ) {
     134        public function _search_callback( $theme ) {
    135135                static $term;
    136136                if ( is_null( $term ) )
    … …  
    153153
    154154        // Not used by any core columns.
    155         function _order_callback( $theme_a, $theme_b ) {
     155        public function _order_callback( $theme_a, $theme_b ) {
    156156                global $orderby, $order;
    157157
    … …  
    168168        }
    169169
    170         function no_items() {
     170        public function no_items() {
    171171                if ( ! $this->has_items )
    172172                        _e( 'No themes found.' );
    … …  
    175175        }
    176176
    177         function get_columns() {
     177        protected function get_columns() {
    178178                global $status;
    179179
    … …  
    185185        }
    186186
    187         function get_sortable_columns() {
     187        protected function get_sortable_columns() {
    188188                return array(
    189189                        'name'         => 'name',
    … …  
    191191        }
    192192
    193         function get_views() {
     193        protected function get_views() {
    194194                global $totals, $status;
    195195
    … …  
    234234        }
    235235
    236         function get_bulk_actions() {
     236        protected function get_bulk_actions() {
    237237                global $status;
    238238
    … …  
    251251        }
    252252
    253         function display_rows() {
     253        protected function display_rows() {
    254254                foreach ( $this->items as $theme )
    255255                        $this->single_row( $theme );
    256256        }
    257257
    258         function single_row( $theme ) {
     258        protected function single_row( $theme ) {
    259259                global $status, $page, $s, $totals;
    260260

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

    r28295 r28493  
    1010class WP_MS_Users_List_Table extends WP_List_Table {
    1111
    12         function ajax_user_can() {
     12        public function ajax_user_can() {
    1313                return current_user_can( 'manage_network_users' );
    1414        }
    1515
    16         function prepare_items() {
     16        public function prepare_items() {
    1717                global $usersearch, $role, $wpdb, $mode;
    1818
    … …  
    7070        }
    7171
    72         function get_bulk_actions() {
     72        protected function get_bulk_actions() {
    7373                $actions = array();
    7474                if ( current_user_can( 'delete_users' ) )
    … …  
    8080        }
    8181
    82         function no_items() {
     82        public function no_items() {
    8383                _e( 'No users found.' );
    8484        }
    8585
    86         function get_views() {
     86        protected function get_views() {
    8787                global $role;
    8888
    … …  
    100100        }
    101101
    102         function pagination( $which ) {
     102        protected function pagination( $which ) {
    103103                global $mode;
    104104
    … …  
    109109        }
    110110
    111         function get_columns() {
     111        protected function get_columns() {
    112112                $users_columns = array(
    113113                        'cb'         => '<input type="checkbox" />',
    … …  
    131131        }
    132132
    133         function get_sortable_columns() {
     133        protected function get_sortable_columns() {
    134134                return array(
    135135                        'username'   => 'login',
    … …  
    140140        }
    141141
    142         function display_rows() {
     142        protected function display_rows() {
    143143                global $mode;
    144144

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

    r28388 r28493  
    1010class WP_Plugin_Install_List_Table extends WP_List_Table {
    1111
    12         function ajax_user_can() {
     12        public function ajax_user_can() {
    1313                return current_user_can('install_plugins');
    1414        }
    1515
    16         function prepare_items() {
     16        public function prepare_items() {
    1717                include( ABSPATH . 'wp-admin/includes/plugin-install.php' );
    1818
    … …  
    134134        }
    135135
    136         function no_items() {
     136        public function no_items() {
    137137                _e( 'No plugins match your request.' );
    138138        }
    139139
    140         function get_views() {
     140        protected function get_views() {
    141141                global $tabs, $tab;
    142142
    … …  
    151151        }
    152152
    153         function display_tablenav( $which ) {
     153        protected function display_tablenav( $which ) {
    154154                if ( 'top' ==  $which ) { ?>
    155155                        <div class="tablenav top">
    … …  
    175175        }
    176176
    177         function get_table_classes() {
     177        protected function get_table_classes() {
    178178                return array( 'widefat', $this->_args['plural'] );
    179179        }
    180180
    181         function get_columns() {
     181        protected function get_columns() {
    182182                return array(
    183183                        'name'        => _x( 'Name', 'plugin name' ),
    … …  
    188188        }
    189189
    190         function display_rows() {
     190        protected function display_rows() {
    191191                $plugins_allowedtags = array(
    192192                        'a' => array( 'href' => array(),'title' => array(), 'target' => array() ),

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

    r27507 r28493  
    1010class WP_Plugins_List_Table extends WP_List_Table {
    1111
    12         function __construct( $args = array() ) {
     12        public function __construct( $args = array() ) {
    1313                global $status, $page;
    1414
    … …  
    2828        }
    2929
    30         function get_table_classes() {
     30        protected function get_table_classes() {
    3131                return array( 'widefat', $this->_args['plural'] );
    3232        }
    3333
    34         function ajax_user_can() {
     34        public function ajax_user_can() {
    3535                return current_user_can('activate_plugins');
    3636        }
    3737
    38         function prepare_items() {
     38        public function prepare_items() {
    3939                global $status, $plugins, $totals, $page, $orderby, $order, $s;
    4040
    … …  
    173173        }
    174174
    175         function _search_callback( $plugin ) {
     175        public function _search_callback( $plugin ) {
    176176                static $term;
    177177                if ( is_null( $term ) )
    … …  
    187187        }
    188188
    189         function _order_callback( $plugin_a, $plugin_b ) {
     189        public function _order_callback( $plugin_a, $plugin_b ) {
    190190                global $orderby, $order;
    191191
    … …  
    202202        }
    203203
    204         function no_items() {
     204        public function no_items() {
    205205                global $plugins;
    206206
    … …  
    211211        }
    212212
    213         function get_columns() {
     213        protected function get_columns() {
    214214                global $status;
    215215
    … …  
    221221        }
    222222
    223         function get_sortable_columns() {
     223        protected function get_sortable_columns() {
    224224                return array();
    225225        }
    226226
    227         function get_views() {
     227        protected function get_views() {
    228228                global $totals, $status;
    229229
    … …  
    269269        }
    270270
    271         function get_bulk_actions() {
     271        protected function get_bulk_actions() {
    272272                global $status;
    273273
    … …  
    290290        }
    291291
    292         function bulk_actions() {
     292        public function bulk_actions() {
    293293                global $status;
    294294
    … …  
    299299        }
    300300
    301         function extra_tablenav( $which ) {
     301        protected function extra_tablenav( $which ) {
    302302                global $status;
    303303
    … …  
    317317        }
    318318
    319         function current_action() {
     319        public function current_action() {
    320320                if ( isset($_POST['clear-recent-list']) )
    321321                        return 'clear-recent-list';
    … …  
    324324        }
    325325
    326         function display_rows() {
     326        protected function display_rows() {
    327327                global $status;
    328328
    … …  
    334334        }
    335335
    336         function single_row( $item ) {
     336        protected function single_row( $item ) {
    337337                global $status, $page, $s, $totals;
    338338

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

    r27964 r28493  
    1717         * @access protected
    1818         */
    19         var $hierarchical_display;
     19        protected $hierarchical_display;
    2020
    2121        /**
    … …  
    2626         * @access protected
    2727         */
    28         var $comment_pending_count;
     28        protected $comment_pending_count;
    2929
    3030        /**
    … …  
    3535         * @access private
    3636         */
    37         var $user_posts_count;
     37        private $user_posts_count;
    3838
    3939        /**
    … …  
    4444         * @access private
    4545         */
    46         var $sticky_posts_count = 0;
    47 
    48         function __construct( $args = array() ) {
     46        private $sticky_posts_count = 0;
     47
     48        public function __construct( $args = array() ) {
    4949                global $post_type_object, $wpdb;
    5050
    … …  
    7575        }
    7676
    77         function ajax_user_can() {
     77        public function ajax_user_can() {
    7878                return current_user_can( get_post_type_object( $this->screen->post_type )->cap->edit_posts );
    7979        }
    8080
    81         function prepare_items() {
     81        public function prepare_items() {
    8282                global $avail_post_stati, $wp_query, $per_page, $mode;
    8383
    … …  
    110110        }
    111111
    112         function has_items() {
     112        public function has_items() {
    113113                return have_posts();
    114114        }
    115115
    116         function no_items() {
     116        public function no_items() {
    117117                if ( isset( $_REQUEST['post_status'] ) && 'trash' == $_REQUEST['post_status'] )
    118118                        echo get_post_type_object( $this->screen->post_type )->labels->not_found_in_trash;
    … …  
    121121        }
    122122
    123         function get_views() {
     123        protected function get_views() {
    124124                global $locked_post_status, $avail_post_stati;
    125125
    … …  
    182182        }
    183183
    184         function get_bulk_actions() {
     184        protected function get_bulk_actions() {
    185185                $actions = array();
    186186
    … …  
    198198        }
    199199
    200         function extra_tablenav( $which ) {
     200        protected function extra_tablenav( $which ) {
    201201                global $cat;
    202202?>
    … …  
    240240        }
    241241
    242         function current_action() {
     242        public function current_action() {
    243243                if ( isset( $_REQUEST['delete_all'] ) || isset( $_REQUEST['delete_all2'] ) )
    244244                        return 'delete_all';
    … …  
    247247        }
    248248
    249         function pagination( $which ) {
     249        protected function pagination( $which ) {
    250250                global $mode;
    251251
    … …  
    256256        }
    257257
    258         function get_table_classes() {
     258        protected function get_table_classes() {
    259259                return array( 'widefat', 'fixed', is_post_type_hierarchical( $this->screen->post_type ) ? 'pages' : 'posts' );
    260260        }
    261261
    262         function get_columns() {
     262        protected function get_columns() {
    263263                $post_type = $this->screen->post_type;
    264264
    … …  
    346346        }
    347347
    348         function get_sortable_columns() {
     348        protected function get_sortable_columns() {
    349349                return array(
    350350                        'title'    => 'title',
    … …  
    355355        }
    356356
    357         function display_rows( $posts = array(), $level = 0 ) {
     357        protected function display_rows( $posts = array(), $level = 0 ) {
    358358                global $wp_query, $per_page;
    359359
    … …  
    370370        }
    371371
    372         function _display_rows( $posts, $level = 0 ) {
     372        private function _display_rows( $posts, $level = 0 ) {
    373373                global $mode;
    374374
    … …  
    385385        }
    386386
    387         function _display_rows_hierarchical( $pages, $pagenum = 1, $per_page = 20 ) {
     387        private function _display_rows_hierarchical( $pages, $pagenum = 1, $per_page = 20 ) {
    388388                global $wpdb;
    389389
    … …  
    477477         * @param int $per_page
    478478         */
    479         function _page_rows( &$children_pages, &$count, $parent, $level, $pagenum, $per_page ) {
     479        private function _page_rows( &$children_pages, &$count, $parent, $level, $pagenum, $per_page ) {
    480480
    481481                if ( ! isset( $children_pages[$parent] ) )
    … …  
    522522        }
    523523
    524         function single_row( $post, $level = 0 ) {
     524        protected function single_row( $post, $level = 0 ) {
    525525                global $mode;
    526526                static $alternate;
    … …  
    880880         * @since 3.1.0
    881881         */
    882         function inline_edit() {
     882        public function inline_edit() {
    883883                global $mode;
    884884

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

    r28390 r28493  
    1010class WP_Terms_List_Table extends WP_List_Table {
    1111
    12         var $callback_args;
    13 
    14         function __construct( $args = array() ) {
     12        public $callback_args;
     13
     14        public function __construct( $args = array() ) {
    1515                global $post_type, $taxonomy, $action, $tax;
    1616
    … …  
    3939        }
    4040
    41         function ajax_user_can() {
     41        public function ajax_user_can() {
    4242                return current_user_can( get_taxonomy( $this->screen->taxonomy )->cap->manage_terms );
    4343        }
    4444
    45         function prepare_items() {
     45        public function prepare_items() {
    4646                $tags_per_page = $this->get_items_per_page( 'edit_' . $this->screen->taxonomy . '_per_page' );
    4747
    … …  
    9898        }
    9999
    100         function has_items() {
     100        public function has_items() {
    101101                // todo: populate $this->items in prepare_items()
    102102                return true;
    103103        }
    104104
    105         function get_bulk_actions() {
     105        protected function get_bulk_actions() {
    106106                $actions = array();
    107107                $actions['delete'] = __( 'Delete' );
    … …  
    110110        }
    111111
    112         function current_action() {
     112        public function current_action() {
    113113                if ( isset( $_REQUEST['action'] ) && isset( $_REQUEST['delete_tags'] ) && ( 'delete' == $_REQUEST['action'] || 'delete' == $_REQUEST['action2'] ) )
    114114                        return 'bulk-delete';
    … …  
    117117        }
    118118
    119         function get_columns() {
     119        protected function get_columns() {
    120120                $columns = array(
    121121                        'cb'          => '<input type="checkbox" />',
    … …  
    135135        }
    136136
    137         function get_sortable_columns() {
     137        protected function get_sortable_columns() {
    138138                return array(
    139139                        'name'        => 'name',
    … …  
    145145        }
    146146
    147         function display_rows_or_placeholder() {
     147        protected function display_rows_or_placeholder() {
    148148                $taxonomy = $this->screen->taxonomy;
    149149
    … …  
    194194        }
    195195
    196         function _rows( $taxonomy, $terms, &$children, $start, $per_page, &$count, $parent = 0, $level = 0 ) {
     196        private function _rows( $taxonomy, $terms, &$children, $start, $per_page, &$count, $parent = 0, $level = 0 ) {
    197197
    198198                $end = $start + $per_page;
    … …  
    242242        }
    243243
    244         function single_row( $tag, $level = 0 ) {
     244        protected function single_row( $tag, $level = 0 ) {
    245245                global $taxonomy;
    246246                $tag = sanitize_term( $tag, $taxonomy );
    … …  
    256256        }
    257257
    258         function column_cb( $tag ) {
     258        public function column_cb( $tag ) {
    259259                $default_term = get_option( 'default_' . $this->screen->taxonomy );
    260260
    … …  
    266266        }
    267267
    268         function column_name( $tag ) {
     268        public function column_name( $tag ) {
    269269                $taxonomy = $this->screen->taxonomy;
    270270                $tax = get_taxonomy( $taxonomy );
    … …  
    340340        }
    341341
    342         function column_description( $tag ) {
     342        public function column_description( $tag ) {
    343343                return $tag->description;
    344344        }
    345345
    346         function column_slug( $tag ) {
     346        public function column_slug( $tag ) {
    347347                /** This filter is documented in wp-admin/edit-tag-form.php */
    348348                return apply_filters( 'editable_slug', $tag->slug );
    349349        }
    350350
    351         function column_posts( $tag ) {
     351        public function column_posts( $tag ) {
    352352                $count = number_format_i18n( $tag->count );
    353353
    … …  
    373373        }
    374374
    375         function column_links( $tag ) {
     375        public function column_links( $tag ) {
    376376                $count = number_format_i18n( $tag->count );
    377377                if ( $count )
    … …  
    380380        }
    381381
    382         function column_default( $tag, $column_name ) {
     382        public function column_default( $tag, $column_name ) {
    383383                /**
    384384                 * Filter the displayed columns in the terms list table.
    … …  
    401401         * @since 3.1.0
    402402         */
    403         function inline_edit() {
     403        public function inline_edit() {
    404404                $tax = get_taxonomy( $this->screen->taxonomy );
    405405

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

    r28287 r28493  
    1010class WP_Theme_Install_List_Table extends WP_Themes_List_Table {
    1111
    12         var $features = array();
    13 
    14         function ajax_user_can() {
     12        public $features = array();
     13
     14        public function ajax_user_can() {
    1515                return current_user_can( 'install_themes' );
    1616        }
    1717
    18         function prepare_items() {
     18        public function prepare_items() {
    1919                include( ABSPATH . 'wp-admin/includes/theme-install.php' );
    2020
    … …  
    141141        }
    142142
    143         function no_items() {
     143        public function no_items() {
    144144                _e( 'No themes match your request.' );
    145145        }
    146146
    147         function get_views() {
     147        protected function get_views() {
    148148                global $tabs, $tab;
    149149
    … …  
    158158        }
    159159
    160         function display() {
     160        public function display() {
    161161                wp_nonce_field( "fetch-list-" . get_class( $this ), '_ajax_fetch_list_nonce' );
    162162?>
    … …  
    184184        }
    185185
    186         function display_rows() {
     186        protected function display_rows() {
    187187                $themes = $this->items;
    188188                foreach ( $themes as $theme ) {
    … …  
    215215         *     public 'download_link' => string 'http://wordpress.org/themes/download/magazine-basic.1.1.zip'
    216216         */
    217         function single_row( $theme ) {
     217        protected function single_row( $theme ) {
    218218                global $themes_allowedtags;
    219219
    … …  
    295295         * Prints the wrapper for the theme installer.
    296296         */
    297         function theme_installer() {
     297        public function theme_installer() {
    298298                ?>
    299299                <div id="theme-installer" class="wp-full-overlay expanded">
    … …  
    324324         * @param object $theme - A WordPress.org Theme API object.
    325325         */
    326         function theme_installer_single( $theme ) {
     326        public function theme_installer_single( $theme ) {
    327327                ?>
    328328                <div id="theme-installer" class="wp-full-overlay single-theme">
    … …  
    342342         * @param object $theme - A WordPress.org Theme API object.
    343343         */
    344         function install_theme_info( $theme ) {
     344        public function install_theme_info( $theme ) {
    345345                global $themes_allowedtags;
    346346
    … …  
    409409         * @uses $type Global; type of search.
    410410         */
    411         function _js_vars( $extra_args = array() ) {
     411        private function _js_vars( $extra_args = array() ) {
    412412                global $tab, $type;
    413413                parent::_js_vars( compact( 'tab', 'type' ) );

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

    r27507 r28493  
    1111
    1212        protected $search_terms = array();
    13         var $features = array();
    14 
    15         function __construct( $args = array() ) {
     13        public $features = array();
     14
     15        public function __construct( $args = array() ) {
    1616                parent::__construct( array(
    1717                        'ajax' => true,
    … …  
    2020        }
    2121
    22         function ajax_user_can() {
     22        public function ajax_user_can() {
    2323                // Do not check edit_theme_options here. AJAX calls for available themes require switch_themes.
    2424                return current_user_can( 'switch_themes' );
    2525        }
    2626
    27         function prepare_items() {
     27        public function prepare_items() {
    2828                $themes = wp_get_themes( array( 'allowed' => true ) );
    2929
    … …  
    5858        }
    5959
    60         function no_items() {
     60        public function no_items() {
    6161                if ( $this->search_terms || $this->features ) {
    6262                        _e( 'No items found.' );
    … …  
    8686        }
    8787
    88         function tablenav( $which = 'top' ) {
     88        public function tablenav( $which = 'top' ) {
    8989                if ( $this->get_pagination_arg( 'total_pages' ) <= 1 )
    9090                        return;
    … …  
    9898        }
    9999
    100         function display() {
     100        public function display() {
    101101                wp_nonce_field( "fetch-list-" . get_class( $this ), '_ajax_fetch_list_nonce' );
    102102?>
    … …  
    111111        }
    112112
    113         function get_columns() {
     113        protected function get_columns() {
    114114                return array();
    115115        }
    116116
    117         function display_rows_or_placeholder() {
     117        protected function display_rows_or_placeholder() {
    118118                if ( $this->has_items() ) {
    119119                        $this->display_rows();
    … …  
    125125        }
    126126
    127         function display_rows() {
     127        protected function display_rows() {
    128128                $themes = $this->items;
    129129
    … …  
    209209        }
    210210
    211         function search_theme( $theme ) {
     211        public function search_theme( $theme ) {
    212212                // Search the features
    213213                foreach ( $this->features as $word ) {
    … …  
    250250         * @uses _pagination_args['total_pages']
    251251         */
    252         function _js_vars( $extra_args = array() ) {
     252        private function _js_vars( $extra_args = array() ) {
    253253                $search_string = isset( $_REQUEST['s'] ) ? esc_attr( wp_unslash( $_REQUEST['s'] ) ) : '';
    254254

  • trunk/tests/phpunit/includes/functions.php

    r28480 r28493  
    4646        private $foo = 'bar';
    4747
    48         function __get( $name ) {
     48        public function __get( $name ) {
    4949                return $this->$name;
    5050        }
     51
     52        public function __call( $name, $arguments ) {
     53                return call_user_func_array( array( $this, $name ), $arguments );
     54        }
     55
     56        private function callMe() {
     57                return 'maybe';
     58        }
    5159}
     60
     61class Basic_Subclass extends Basic_Object {}

  • trunk/tests/phpunit/tests/basic.php

    r28480 r28493  
    128128                $this->assertEquals( 'bar', $basic->foo );
    129129        }
     130
     131        function test_subclass_magic_getter() {
     132                $basic = new Basic_Subclass();
     133
     134                $this->assertEquals( 'bar', $basic->foo );
     135        }
     136
     137        function test_call_method() {
     138                $basic = new Basic_Object();
     139
     140                $this->assertEquals( 'maybe', $basic->callMe() );
     141        }
     142
     143        function test_subclass_call_method() {
     144                $basic = new Basic_Subclass();
     145
     146                $this->assertEquals( 'maybe', $basic->callMe() );
     147        }
    130148}
Note: See TracChangeset for help on using the changeset viewer.