Changeset 16182 for trunk/wp-admin/includes/class-wp-list-table.php

Timestamp:

11/04/2010 03:00:55 PM (14 years ago)

Author:

scribu

Message:

Default to get_current_screen() in list table classes. See #14579

File:

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

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

@@ -77 +77 @@
      * @access protected
      */
-    function WP_List_Table( $args ) {
+    function WP_List_Table( $args = array() ) {
         $args = wp_parse_args( $args, array(
-            'screen' => '',
+            'screen' => get_current_screen(),
             'plural' => '',
             'singular' => '',
@@ -633 +633 @@
      */
     function get_table_classes() {
-        extract( $this->_args );
-
-        return array( 'widefat', 'fixed', $plural );
+        return array( 'widefat', 'fixed', $this->_args['plural'] );
     }
 
@@ -808 +806 @@
      */
     function _js_vars() {
-        extract( $this->_args );
-
-        $class = get_class( $this );
-
-        printf( "<script type='text/javascript'>list_args = %s;</script>\n",
-            json_encode( compact( 'screen', 'class' ) )
+        $args = array(
+            'class' => get_class( $this ),
+            'screen' => $this->_screen
         );
+
+        printf( "<script type='text/javascript'>list_args = %s;</script>\n", json_encode( $args ) );
     }
 }