Changeset 28500 for trunk/src/wp-admin/includes/list-table.php

Timestamp:

05/19/2014 05:03:44 AM (11 years ago)

Author:

wonderboymusic

Message:

Fix some hackificator odds and ends in wp-admin:

• wp-activate.php and wp-admin/themes.php don't need the closing PHP tag
• Switch single quotes for HTML attribute values to double in a few places
• Convert include_once file.php syntax to include_once( 'file.php' )
• Add access modifiers to methods/members in: _WP_List_Table_Compat, Walker_Nav_Menu_Edit, Walker_Nav_Menu_Checklist, WP_Screen, Walker_Category_Checklist
• edit_user() doesn't need to import the $wpdb global
• wp_list_widgets() doesn't need to import the $sidebars_widgets global
• switch/endswitch syntax is not supported in Hack
• A <ul> in wp-admin/users.php is unclosed

See #27881.

File:

• trunk/src/wp-admin/includes/list-table.php (modified) (3 diffs)

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

@@ -85 +85 @@
  */
 class _WP_List_Table_Compat extends WP_List_Table {
-    var $_screen;
-    var $_columns;
+    public $_screen;
+    public $_columns;
 
-    function _WP_List_Table_Compat( $screen, $columns = array() ) {
+    public function _WP_List_Table_Compat( $screen, $columns = array() ) {
         if ( is_string( $screen ) )
             $screen = convert_to_screen( $screen );
@@ -100 +100 @@
     }
 
-    function get_column_info() {
+    public function get_column_info() {
         $columns = get_column_headers( $this->_screen );
         $hidden = get_hidden_columns( $this->_screen );
@@ -108 +108 @@
     }
 
-    function get_columns() {
+    public function get_columns() {
         return $this->_columns;
     }