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

Timestamp:

03/02/2012 10:09:26 PM (13 years ago)

Author:

nacin

Message:

Output themes and theme-install infinite scrolling args in JS, rather than parsing query strings. props DH-Shredder, helenyhou. Make WP_Theme_Install_List_Table extend WP_Themes_List_Table. Doesn't help much yet, but we should be able to dry things up further. see #19815.

File:

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

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

@@ -21 +21 @@
     function ajax_user_can() {
         // Do not check edit_theme_options here. AJAX calls for available themes require switch_themes.
-        return current_user_can('switch_themes');
+        return current_user_can( 'switch_themes' );
     }
 
@@ -53 +53 @@
             'total_items' => count( $themes ),
             'per_page' => $per_page,
+            'infinite_scroll' => true,
         ) );
     }
@@ -203 +204 @@
         return true;
     }
+
+    /**
+     * Send required variables to JavaScript land
+     *
+     * @since 3.4
+     * @access private
+     *
+     * @uses $this->features Array of all feature search terms.
+     * @uses get_pagenum()
+     * @uses _pagination_args['total_pages']
+     */
+     function _js_vars( $extra_args = array() ) {
+        $search_string = isset( $_REQUEST['s'] ) ? esc_attr( stripslashes( $_REQUEST['s'] ) ) : '';
+
+        $total_pages = 1;
+        if ( ! empty( $this->_pagination_args['total_pages'] ) )
+            $total_pages = $this->_pagination_args['total_pages'];
+
+        $args = array(
+            'search' => $search_string,
+            'features' => $this->features,
+            'paged' => $this->get_pagenum(),
+            'total_pages' => $total_pages,
+        );
+
+        if ( is_array( $extra_args ) )
+        $args = array_merge( $args, $extra_args );
+
+        printf( "<script type='text/javascript'>var theme_list_args = %s;</script>\n", json_encode( $args ) );
+        parent::_js_vars();
+    }
 }