<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" /><style type="text/css"><!--
#msg dl { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fc0 solid; padding: 6px; }
#msg ul, pre { overflow: auto; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<title>[16195] trunk/wp-admin/includes: Rename property _screen to screen.</title>
</head>
<body>
<div id="msg">
<dl>
<dt>Revision</dt> <dd><a href="http://trac.wordpress.org/changeset/16195">16195</a></dd>
<dt>Author</dt> <dd>scribu</dd>
<dt>Date</dt> <dd>2010-11-04 21:02:13 +0000 (Thu, 04 Nov 2010)</dd>
</dl>
<h3>Log Message</h3>
<pre>Rename property _screen to screen. See <a href="http://trac.wordpress.org/ticket/14579">#14579</a></pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkwpadminincludesclasswplisttablephp">trunk/wp-admin/includes/class-wp-list-table.php</a></li>
<li><a href="#trunkwpadminincludesclasswpmedialisttablephp">trunk/wp-admin/includes/class-wp-media-list-table.php</a></li>
<li><a href="#trunkwpadminincludesclasswppluginslisttablephp">trunk/wp-admin/includes/class-wp-plugins-list-table.php</a></li>
<li><a href="#trunkwpadminincludesclasswppostslisttablephp">trunk/wp-admin/includes/class-wp-posts-list-table.php</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkwpadminincludesclasswplisttablephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/class-wp-list-table.php (16194 => 16195)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/class-wp-list-table.php 2010-11-04 20:49:30 UTC (rev 16194)
+++ trunk/wp-admin/includes/class-wp-list-table.php 2010-11-04 21:02:13 UTC (rev 16195)
</span><span class="lines">@@ -48,9 +48,9 @@
</span><span class="cx"> *
</span><span class="cx"> * @since 3.1.0
</span><span class="cx"> * @var object
</span><del>- * @access private
</del><ins>+ * @access protected
</ins><span class="cx"> */
</span><del>- var $_screen;
</del><ins>+ var $screen;
</ins><span class="cx">
</span><span class="cx"> /**
</span><span class="cx"> * Cached bulk actions
</span><span class="lines">@@ -84,15 +84,15 @@
</span><span class="cx"> 'ajax' => true
</span><span class="cx"> ) );
</span><span class="cx">
</span><del>- $this->_screen = $args['screen'];
</del><ins>+ $this->screen = $args['screen'];
</ins><span class="cx">
</span><del>- if ( is_string( $this->_screen ) )
- $this->_screen = convert_to_screen( $this->_screen );
</del><ins>+ if ( is_string( $this->screen ) )
+ $this->screen = convert_to_screen( $this->screen );
</ins><span class="cx">
</span><del>- add_filter( 'manage_' . $this->_screen->id . '_columns', array( &$this, 'get_columns' ), 0 );
</del><ins>+ add_filter( 'manage_' . $this->screen->id . '_columns', array( &$this, 'get_columns' ), 0 );
</ins><span class="cx">
</span><span class="cx"> if ( !$args['plural'] )
</span><del>- $args['plural'] = $this->_screen->base;
</del><ins>+ $args['plural'] = $this->screen->base;
</ins><span class="cx">
</span><span class="cx"> $this->_args = $args;
</span><span class="cx">
</span><span class="lines">@@ -204,7 +204,7 @@
</span><span class="cx"> */
</span><span class="cx"> function views() {
</span><span class="cx"> $views = $this->get_views();
</span><del>- $views = apply_filters( 'views_' . $this->_screen->base, $views );
</del><ins>+ $views = apply_filters( 'views_' . $this->screen->base, $views );
</ins><span class="cx">
</span><span class="cx"> if ( empty( $views ) )
</span><span class="cx"> return;
</span><span class="lines">@@ -237,7 +237,7 @@
</span><span class="cx">
</span><span class="cx"> if ( is_null( $this->_actions ) ) {
</span><span class="cx"> $this->_actions = $this->get_bulk_actions();
</span><del>- $this->_actions = apply_filters( 'bulk_actions-' . $this->_screen->base, $this->_actions );
</del><ins>+ $this->_actions = apply_filters( 'bulk_actions-' . $this->screen->base, $this->_actions );
</ins><span class="cx"> $two = '';
</span><span class="cx"> }
</span><span class="cx"> else {
</span><span class="lines">@@ -535,9 +535,9 @@
</span><span class="cx"> */
</span><span class="cx"> function get_column_info() {
</span><span class="cx"> if ( !isset( $this->_column_headers ) ) {
</span><del>- $columns = get_column_headers( $this->_screen );
- $hidden = get_hidden_columns( $this->_screen );
- $sortable = apply_filters( 'manage_' . $this->_screen->id . '_sortable_columns', $this->get_sortable_columns() );
</del><ins>+ $columns = get_column_headers( $this->screen );
+ $hidden = get_hidden_columns( $this->screen );
+ $sortable = apply_filters( 'manage_' . $this->screen->id . '_sortable_columns', $this->get_sortable_columns() );
</ins><span class="cx">
</span><span class="cx"> $this->_column_headers = array( $columns, $hidden, $sortable );
</span><span class="cx"> }
</span><span class="lines">@@ -554,7 +554,7 @@
</span><span class="cx"> * @param bool $with_id Whether to set the id attribute or not
</span><span class="cx"> */
</span><span class="cx"> function print_column_headers( $with_id = true ) {
</span><del>- $screen = $this->_screen;
</del><ins>+ $screen = $this->screen;
</ins><span class="cx">
</span><span class="cx"> list( $columns, $hidden, $sortable ) = $this->get_column_info();
</span><span class="cx">
</span><span class="lines">@@ -807,7 +807,7 @@
</span><span class="cx"> function _js_vars() {
</span><span class="cx"> $args = array(
</span><span class="cx"> 'class' => get_class( $this ),
</span><del>- 'screen' => $this->_screen
</del><ins>+ 'screen' => $this->screen
</ins><span class="cx"> );
</span><span class="cx">
</span><span class="cx"> printf( "<script type='text/javascript'>list_args = %s;</script>\n", json_encode( $args ) );
</span></span></pre></div>
<a id="trunkwpadminincludesclasswpmedialisttablephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/class-wp-media-list-table.php (16194 => 16195)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/class-wp-media-list-table.php 2010-11-04 20:49:30 UTC (rev 16194)
+++ trunk/wp-admin/includes/class-wp-media-list-table.php 2010-11-04 21:02:13 UTC (rev 16195)
</span><span class="lines">@@ -151,13 +151,13 @@
</span><span class="cx"> $posts_columns['author'] = __( 'Author' );
</span><span class="cx"> //$posts_columns['tags'] = _x( 'Tags', 'column name' );
</span><span class="cx"> /* translators: column name */
</span><del>- if ( 'upload' == $this->_screen->id ) {
</del><ins>+ if ( 'upload' == $this->screen->id ) {
</ins><span class="cx"> $posts_columns['parent'] = _x( 'Attached to', 'column name' );
</span><span class="cx"> $posts_columns['comments'] = '<div class="vers"><img alt="Comments" src="' . esc_url( admin_url( 'images/comment-grey-bubble.png' ) ) . '" /></div>';
</span><span class="cx"> }
</span><span class="cx"> /* translators: column name */
</span><span class="cx"> $posts_columns['date'] = _x( 'Date', 'column name' );
</span><del>- $posts_columns = apply_filters( 'manage_media_columns', $posts_columns, 'upload' != $this->_screen->id );
</del><ins>+ $posts_columns = apply_filters( 'manage_media_columns', $posts_columns, 'upload' != $this->screen->id );
</ins><span class="cx">
</span><span class="cx"> return $posts_columns;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkwpadminincludesclasswppluginslisttablephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/class-wp-plugins-list-table.php (16194 => 16195)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/class-wp-plugins-list-table.php 2010-11-04 20:49:30 UTC (rev 16194)
+++ trunk/wp-admin/includes/class-wp-plugins-list-table.php 2010-11-04 21:02:13 UTC (rev 16195)
</span><span class="lines">@@ -57,7 +57,7 @@
</span><span class="cx"> 'dropins' => array()
</span><span class="cx"> );
</span><span class="cx">
</span><del>- if ( ! is_multisite() || ( $this->_screen->is_network && current_user_can('manage_network_plugins') ) ) {
</del><ins>+ if ( ! is_multisite() || ( $this->screen->is_network && current_user_can('manage_network_plugins') ) ) {
</ins><span class="cx"> if ( apply_filters( 'show_advanced_plugins', true, 'mustuse' ) )
</span><span class="cx"> $plugins['mustuse'] = get_mu_plugins();
</span><span class="cx"> if ( apply_filters( 'show_advanced_plugins', true, 'dropins' ) )
</span><span class="lines">@@ -85,16 +85,16 @@
</span><span class="cx">
</span><span class="cx"> foreach ( (array) $plugins['all'] as $plugin_file => $plugin_data ) {
</span><span class="cx"> // Filter into individual sections
</span><del>- if ( is_plugin_active_for_network($plugin_file) && !$this->_screen->is_network ) {
</del><ins>+ if ( is_plugin_active_for_network($plugin_file) && !$this->screen->is_network ) {
</ins><span class="cx"> unset( $plugins['all'][ $plugin_file ] );
</span><span class="cx"> continue;
</span><span class="cx"> } elseif ( is_multisite() && is_network_only_plugin( $plugin_file ) && !current_user_can( 'manage_network_plugins' ) ) {
</span><span class="cx"> $plugins['network'][ $plugin_file ] = $plugin_data;
</span><del>- } elseif ( ( !$this->_screen->is_network && is_plugin_active( $plugin_file ) )
- || ( $this->_screen->is_network && is_plugin_active_for_network( $plugin_file ) ) ) {
</del><ins>+ } elseif ( ( !$this->screen->is_network && is_plugin_active( $plugin_file ) )
+ || ( $this->screen->is_network && is_plugin_active_for_network( $plugin_file ) ) ) {
</ins><span class="cx"> $plugins['active'][ $plugin_file ] = $plugin_data;
</span><span class="cx"> } else {
</span><del>- if ( !$this->_screen->is_network && isset( $recently_activated[ $plugin_file ] ) ) // Was the plugin recently activated?
</del><ins>+ if ( !$this->screen->is_network && isset( $recently_activated[ $plugin_file ] ) ) // Was the plugin recently activated?
</ins><span class="cx"> $plugins['recently_activated'][ $plugin_file ] = $plugin_data;
</span><span class="cx"> $plugins['inactive'][ $plugin_file ] = $plugin_data;
</span><span class="cx"> }
</span><span class="lines">@@ -128,7 +128,7 @@
</span><span class="cx"> uasort( $this->items, array( &$this, '_order_callback' ) );
</span><span class="cx"> }
</span><span class="cx">
</span><del>- $plugins_per_page = $this->get_items_per_page( str_replace( '-', '_', "{$this->_screen->id}_per_page" ) );
</del><ins>+ $plugins_per_page = $this->get_items_per_page( str_replace( '-', '_', "{$this->screen->id}_per_page" ) );
</ins><span class="cx">
</span><span class="cx"> $start = ( $page - 1 ) * $plugins_per_page;
</span><span class="cx">
</span><span class="lines">@@ -254,14 +254,14 @@
</span><span class="cx"> $actions = array();
</span><span class="cx">
</span><span class="cx"> if ( 'active' != $status ) {
</span><del>- $action = $this->_screen->is_network ? 'network-activate-selected' : 'activate-selected';
</del><ins>+ $action = $this->screen->is_network ? 'network-activate-selected' : 'activate-selected';
</ins><span class="cx"> $actions[ $action ] = __( 'Activate' );
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if ( 'inactive' != $status && 'recent' != $status )
</span><span class="cx"> $actions['deactivate-selected'] = __( 'Deactivate' );
</span><span class="cx">
</span><del>- if ( !is_multisite() || $this->_screen->is_network ) {
</del><ins>+ if ( !is_multisite() || $this->screen->is_network ) {
</ins><span class="cx"> if ( current_user_can( 'update_plugins' ) )
</span><span class="cx"> $actions['update-selected'] = __( 'Update' );
</span><span class="cx"> if ( current_user_can( 'delete_plugins' ) && ( 'active' != $status ) )
</span><span class="lines">@@ -313,11 +313,11 @@
</span><span class="cx"> );
</span><span class="cx">
</span><span class="cx"> if ( 'mustuse' == $context ) {
</span><del>- if ( is_multisite() && !$this->_screen->is_network )
</del><ins>+ if ( is_multisite() && !$this->screen->is_network )
</ins><span class="cx"> continue;
</span><span class="cx"> $is_active = true;
</span><span class="cx"> } elseif ( 'dropins' == $context ) {
</span><del>- if ( is_multisite() && !$this->_screen->is_network )
</del><ins>+ if ( is_multisite() && !$this->screen->is_network )
</ins><span class="cx"> continue;
</span><span class="cx"> $dropins = _get_dropins();
</span><span class="cx"> $plugin_name = $plugin_file;
</span><span class="lines">@@ -337,15 +337,15 @@
</span><span class="cx"> $description .= '<p>' . $plugin_data['Description'] . '</p>';
</span><span class="cx"> } else {
</span><span class="cx"> $is_active_for_network = is_plugin_active_for_network($plugin_file);
</span><del>- if ( $this->_screen->is_network )
</del><ins>+ if ( $this->screen->is_network )
</ins><span class="cx"> $is_active = $is_active_for_network;
</span><span class="cx"> else
</span><span class="cx"> $is_active = is_plugin_active( $plugin_file );
</span><span class="cx">
</span><del>- if ( $is_active_for_network && !is_super_admin() && !$this->_screen->is_network )
</del><ins>+ if ( $is_active_for_network && !is_super_admin() && !$this->screen->is_network )
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- if ( $this->_screen->is_network ) {
</del><ins>+ if ( $this->screen->is_network ) {
</ins><span class="cx"> if ( $is_active_for_network ) {
</span><span class="cx"> if ( current_user_can( 'manage_network_plugins' ) )
</span><span class="cx"> $actions['network_deactivate'] = '<a href="' . wp_nonce_url('plugins.php?action=deactivate&amp;networkwide=1&amp;plugin=' . $plugin_file . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'deactivate-plugin_' . $plugin_file) . '" title="' . __('Deactivate this plugin') . '">' . __('Network Deactivate') . '</a>';
</span><span class="lines">@@ -359,7 +359,7 @@
</span><span class="cx"> if ( $is_active ) {
</span><span class="cx"> $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>';
</span><span class="cx"> } else {
</span><del>- if ( is_network_only_plugin( $plugin_file ) && !$this->_screen->is_network )
</del><ins>+ if ( is_network_only_plugin( $plugin_file ) && !$this->screen->is_network )
</ins><span class="cx"> continue;
</span><span class="cx">
</span><span class="cx"> $actions['activate'] = '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;plugin=' . $plugin_file . '&amp;plugin_status=' . $context . '&amp;paged=' . $page . '&amp;s=' . $s, 'activate-plugin_' . $plugin_file) . '" title="' . __('Activate this plugin') . '" class="edit">' . __('Activate') . '</a>';
</span><span class="lines">@@ -367,7 +367,7 @@
</span><span class="cx"> if ( ! is_multisite() && current_user_can('delete_plugins') )
</span><span class="cx"> $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>';
</span><span class="cx"> } // end if $is_active
</span><del>- } // end if $this->_screen->is_network
</del><ins>+ } // end if $this->screen->is_network
</ins><span class="cx">
</span><span class="cx"> if ( current_user_can('edit_plugins') && is_writable(WP_PLUGIN_DIR . '/' . $plugin_file) )
</span><span class="cx"> $actions['edit'] = '<a href="plugin-editor.php?file=' . $plugin_file . '" title="' . __('Open this file in the Plugin Editor') . '" class="edit">' . __('Edit') . '</a>';
</span></span></pre></div>
<a id="trunkwpadminincludesclasswppostslisttablephp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/includes/class-wp-posts-list-table.php (16194 => 16195)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/includes/class-wp-posts-list-table.php 2010-11-04 20:49:30 UTC (rev 16194)
+++ trunk/wp-admin/includes/class-wp-posts-list-table.php 2010-11-04 21:02:13 UTC (rev 16195)
</span><span class="lines">@@ -256,7 +256,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> function get_columns() {
</span><del>- $screen = $this->_screen;
</del><ins>+ $screen = $this->screen;
</ins><span class="cx">
</span><span class="cx"> if ( empty( $screen ) )
</span><span class="cx"> $post_type = 'post';
</span><span class="lines">@@ -675,7 +675,7 @@
</span><span class="cx"> function inline_edit() {
</span><span class="cx"> global $mode;
</span><span class="cx">
</span><del>- $screen = $this->_screen;
</del><ins>+ $screen = $this->screen;
</ins><span class="cx">
</span><span class="cx"> $post = get_default_post_to_edit( $screen->post_type );
</span><span class="cx"> $post_type_object = get_post_type_object( $screen->post_type );
</span></span></pre>
</div>
</div>
</body>
</html>