Changeset 7956

Timestamp:

05/19/2008 09:35:18 PM (18 years ago)

Author:

ryan

Message:

Put checked categories at the top of the checlist. Props mdawaffe. fixes #7000 for trunk

Location:

trunk

Files:

• wp-admin/admin-ajax.php (modified) (2 diffs)
• wp-admin/edit-form-advanced.php (modified) (1 diff)
• wp-admin/includes/template.php (modified) (4 diffs)
• wp-admin/js/post.js (modified) (2 diffs)
• wp-includes/script-loader.php (modified) (1 diff)

trunk/wp-admin/admin-ajax.php

@@ -151 +151 @@
         $post_category = isset($_POST['post_category'])? (array) $_POST['post_category'] : array();
         $checked_categories = array_map( 'absint', (array) $post_category );
+        $popular_ids = isset( $_POST['popular_ids'] ) ?
+                        array_map( 'absint', explode( ',', $_POST['popular_ids'] ) ) :
+                        false;
 
         $x = new WP_Ajax_Response();
@@ -164 +167 @@
                 $category = get_category( $cat_id );
                 ob_start();
-                        wp_category_checklist( 0, $cat_id, $checked_categories );
+                        wp_category_checklist( 0, $cat_id, $checked_categories, $popular_ids );
                 $data = ob_get_contents();
                 ob_end_clean();

trunk/wp-admin/edit-form-advanced.php

@@ -261 +261 @@
 <div id="categories-all" class="ui-tabs-panel">
         <ul id="categorychecklist" class="list:category categorychecklist form-no-clear">
-                <?php wp_category_checklist($post->ID) ?>
+                <?php wp_category_checklist($post->ID, false, false, $popular_ids) ?>
         </ul>
 </div>

trunk/wp-admin/includes/template.php

@@ -149 +149 @@
 }
 
-function wp_category_checklist( $post_id = 0, $descendants_and_self = 0, $selected_cats = false ) {
+function wp_category_checklist( $post_id = 0, $descendants_and_self = 0, $selected_cats = false, $popular_cats = false ) {
         $walker = new Walker_Category_Checklist;
         $descendants_and_self = (int) $descendants_and_self;
@@ -155 +155 @@
         $args = array();
         
-        if ( $post_id )
+        if ( is_array( $selected_cats ) )
+                $args['selected_cats'] = $selected_cats;
+        elseif ( $post_id )
                 $args['selected_cats'] = wp_get_post_categories($post_id);
         else
                 $args['selected_cats'] = array();
-        if ( is_array( $selected_cats ) )
-                $args['selected_cats'] = $selected_cats;
-        $args['popular_cats'] = get_terms( 'category', array( 'fields' => 'ids', 'orderby' => 'count', 'order' => 'DESC', 'number' => 10, 'hierarchical' => false ) );
+
+        if ( is_array( $popular_cats ) )
+                $args['popular_cats'] = $popular_cats;
+        else
+                $args['popular_cats'] = get_terms( 'category', array( 'fields' => 'ids', 'orderby' => 'count', 'order' => 'DESC', 'number' => 10, 'hierarchical' => false ) );
+
         if ( $descendants_and_self ) {
                 $categories = get_categories( "child_of=$descendants_and_self&hierarchical=0&hide_empty=0" );
@@ -170 +175 @@
         }
 
-        $args = array($categories, 0, $args);
-        $output = call_user_func_array(array(&$walker, 'walk'), $args);
-
-        echo $output;
-}
-
-function wp_popular_terms_checklist( $taxonomy, $default = 0, $number = 10 ) {
+        // Post process $categories rather than adding an exclude to the get_terms() query to keep the query the same across all posts (for any query cache)
+        $checked_categories = array();
+        for ( $i = 0; isset($categories[$i]); $i++ ) {
+                if ( in_array($categories[$i]->term_id, $args['selected_cats']) ) {
+                        $checked_categories[] = $categories[$i];
+                        unset($categories[$i]);
+                }
+        }
+
+        // Put checked cats on top
+        echo call_user_func_array(array(&$walker, 'walk'), array($checked_categories, 0, $args));
+        // Then the rest of them
+        echo call_user_func_array(array(&$walker, 'walk'), array($categories, 0, $args));
+}
+
+function wp_popular_terms_checklist( $taxonomy, $default = 0, $number = 10, $echo = true ) {
         global $post_ID;
         if ( $post_ID )
@@ -187 +201 @@
         foreach ( (array) $categories as $category ) {
                 $popular_ids[] = $category->term_id;
+                if ( !$echo ) // hack for AJAX use
+                        continue;
                 $id = "popular-category-$category->term_id";
                 ?>

trunk/wp-admin/js/post.js

@@ -121 +121 @@
                 noSyncChecks = false;
         };
+        var popularCats = jQuery('#categorychecklist-pop :checkbox').map( function() { return parseInt(jQuery(this).val(), 10); } ).get().join(',');
         var catAddBefore = function( s ) {
-                s.data += '&' + jQuery( '#categorychecklist :checked' ).serialize();
+                s.data += '&popular_ids=' + popularCats + '&' + jQuery( '#categorychecklist :checked' ).serialize();
                 return s;
         };
@@ -152 +153 @@
         jQuery('#category-add-toggle').click( function() {
                 jQuery(this).parents('div:first').toggleClass( 'wp-hidden-children' );
-                categoryTabs.tabsClick( 1 );
+                // categoryTabs.tabs( 'select', '#categories-all' ); // this is broken (in the UI beta?)
+                categoryTabs.find( 'a[href="#categories-all"]' ).click();
                 jQuery('#newcat').focus();
                 return false;

trunk/wp-includes/script-loader.php

@@ -146 +146 @@
                                 'cancel' => __('Cancel'),
                         ) );
-                        $this->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20080422' );
+                        $this->add( 'post', '/wp-admin/js/post.js', array('suggest', 'jquery-ui-tabs', 'wp-lists', 'postbox', 'slug'), '20080519' );
                         $this->localize( 'post', 'postL10n', array(
                                 'tagsUsed' =>  __('Tags used on this post:'),