WordPress.org

Make WordPress Core

Changeset 18857


Ignore:
Timestamp:
10/01/11 17:59:46 (3 years ago)
Author:
nacin
Message:

Move all new WP_Screen properties to private. We should introduce getters. Correct legacy (and ideally read-only) properties to @access public. see #18785.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-admin/includes/screen.php

    r18854 r18857  
    401401     * @since 3.3.0 
    402402     * @var string 
    403      * @access private 
     403     * @access public 
    404404     */ 
    405405    public $action = ''; 
     
    411411     * @since 3.3.0 
    412412     * @var string 
    413      * @access private 
     413     * @access public 
    414414     */ 
    415415    public $base; 
     
    420420     * @since 3.3.0 
    421421     * @var string 
    422      * @access private 
     422     * @access public 
    423423     */ 
    424424    public $id; 
     
    429429     * @since 3.3.0 
    430430     * @var bool 
    431      * @access private 
     431     * @access public 
    432432     */ 
    433433    public $is_network; 
     
    438438     * @since 3.3.0 
    439439     * @var bool 
    440      * @access private 
     440     * @access public 
    441441     */ 
    442442    public $is_user; 
     
    449449     * @since 3.3.0 
    450450     * @var string 
    451      * @access private 
     451     * @access public 
    452452     */ 
    453453    public $parent_base; 
     
    459459     * @since 3.3.0 
    460460     * @var string 
    461      * @access private 
     461     * @access public 
    462462     */ 
    463463    public $parent_file; 
     
    469469     * @since 3.3.0 
    470470     * @var string 
    471      * @access private 
     471     * @access public 
    472472     */ 
    473473    public $post_type; 
     
    478478     * @since 3.3.0 
    479479     * @var string 
    480      * @access private 
     480     * @access public 
    481481     */ 
    482482    public $taxonomy; 
     
    489489     * @access private 
    490490     */ 
    491     public $help_tabs = array(); 
     491    private $_help_tabs = array(); 
    492492 
    493493    /** 
     
    498498     * @access private 
    499499     */ 
    500     public $help_sidebar = ''; 
     500    private $_help_sidebar = ''; 
    501501 
    502502    /** 
     
    507507     * @access private 
    508508     */ 
    509     public $options = array(); 
     509    private $_options = array(); 
    510510 
    511511 
     
    622622     */ 
    623623    public function add_option( $option, $args = array() ) { 
    624         $this->options[ $option ] = $args; 
     624        $this->_options[ $option ] = $args; 
    625625    } 
    626626 
     
    655655            $args['id'] = sanitize_html_class( $args['title'] ); 
    656656 
    657         $this->help_tabs[] = $args; 
     657        $this->_help_tabs[] = $args; 
    658658    } 
    659659 
     
    667667     */ 
    668668    public function add_help_sidebar( $content ) { 
    669         $this->help_sidebar = $content; 
     669        $this->_help_sidebar = $content; 
    670670    } 
    671671 
     
    708708            <div id="contextual-help-wrap" class="hidden"> 
    709709                <ul class="contextual-help-tabs"> 
    710                     <?php foreach ( $this->help_tabs as $i => $tab ): 
     710                    <?php foreach ( $this->_help_tabs as $i => $tab ): 
    711711                        $link_id  = "tab-link-{$tab['id']}"; 
    712712                        $panel_id = "tab-panel-{$tab['id']}"; 
     
    723723 
    724724                <div class="contextual-help-sidebar"> 
    725                     <?php echo $this->help_sidebar; ?> 
     725                    <?php echo $this->_help_sidebar; ?> 
    726726                </div> 
    727727 
    728728                <div class="contextual-help-tabs-wrap"> 
    729                     <?php foreach ( $this->help_tabs as $i => $tab ): 
     729                    <?php foreach ( $this->_help_tabs as $i => $tab ): 
    730730                        $panel_id = "tab-panel-{$tab['id']}"; 
    731731                        $classes  = ( $i == 0 ) ? 'active' : ''; 
     
    764764 
    765765        // Check if there are per-page options. 
    766         $show_screen = $show_screen || isset( $this->options['per_page'] ); 
     766        $show_screen = $show_screen || isset( $this->_options['per_page'] ); 
    767767 
    768768        $this->_screen_settings = apply_filters( 'screen_settings', '', $this ); 
     
    778778            $show_screen = true; 
    779779 
    780         if ( ! empty( $this->options ) ) 
     780        if ( ! empty( $this->_options ) ) 
    781781            $show_screen = true; 
    782782 
     
    849849            add_screen_option( 'layout_columns', array('max' => $columns[ $this->id ] ) ); 
    850850 
    851         if ( ! isset( $this->options['layout_columns'] ) ) { 
     851        if ( ! isset( $this->_options['layout_columns'] ) ) { 
    852852            $screen_layout_columns = 0; 
    853853            return; 
     
    855855 
    856856        $screen_layout_columns = get_user_option("screen_layout_$this->id"); 
    857         $num = $this->options['layout_columns']['max']; 
     857        $num = $this->_options['layout_columns']['max']; 
    858858 
    859859        if ( ! $screen_layout_columns ) { 
    860             if ( isset( $this->options['layout_columns']['default'] ) ) 
    861                 $screen_layout_columns = $this->options['layout_columns']['default']; 
     860            if ( isset( $this->_options['layout_columns']['default'] ) ) 
     861                $screen_layout_columns = $this->_options['layout_columns']['default']; 
    862862            else 
    863863                $screen_layout_columns = 'auto'; 
     
    887887 
    888888    function render_per_page_options() { 
    889         if ( ! isset( $this->options['per_page'] ) ) 
     889        if ( ! isset( $this->_options['per_page'] ) ) 
    890890            return; 
    891891 
    892         $per_page_label = $this->options['per_page']['label']; 
    893  
    894         if ( empty( $this->options['per_page']['option'] ) ) { 
     892        $per_page_label = $this->_options['per_page']['label']; 
     893 
     894        if ( empty( $this->_options['per_page']['option'] ) ) { 
    895895            $option = str_replace( '-', '_', "{$this->id}_per_page" ); 
    896896        } else { 
    897             $option = $this->options['per_page']['option']; 
     897            $option = $this->_options['per_page']['option']; 
    898898        } 
    899899 
    900900        $per_page = (int) get_user_option( $option ); 
    901901        if ( empty( $per_page ) || $per_page < 1 ) { 
    902             if ( isset($this->options['per_page']['default']) ) 
    903                 $per_page = $this->options['per_page']['default']; 
     902            if ( isset($this->_options['per_page']['default']) ) 
     903                $per_page = $this->_options['per_page']['default']; 
    904904            else 
    905905                $per_page = 20; 
Note: See TracChangeset for help on using the changeset viewer.