Changeset 4156

Timestamp:

09/01/2006 08:38:24 PM (20 years ago)

Author:

ryan

Message:

Use wp_dropdown_categories(). Fix padding. Don't allow a cat to be it's own parent. Props westi. fixes #3088

Location:

trunk

Files:

• wp-admin/admin-db.php (modified) (1 diff)
• wp-admin/edit-category-form.php (modified) (1 diff)
• wp-includes/classes.php (modified) (1 diff)

trunk/wp-admin/admin-db.php

@@ -109 +109 @@
 
     $category_parent = (int) $category_parent;
-    if ( empty($category_parent) || !get_category( $category_parent ) )
+    if ( empty($category_parent) || !get_category( $category_parent ) || $category_parent == $cat_ID )
         $category_parent = 0;
 

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

@@ -33 +33 @@
             <th scope="row" valign="top"><label for="category_parent"><?php _e('Category parent:') ?></label></th>
             <td>        
-            <select name='category_parent' id='category_parent'>
-      <option value='0' <?php if (!$category->category_parent) echo " selected='selected'"; ?>><?php _e('None') ?></option>
-      <?php wp_dropdown_cats($category->cat_ID, $category->category_parent); ?>
-      </select></td>
+                <?php wp_dropdown_categories('hide_empty=0&name=category_parent&selected=' . $category->category_parent . '&hierarchical=1&show_option_none=' . __('None')); ?>
+            </td>
         </tr>
         <tr>

trunk/wp-includes/classes.php

@@ -697 +697 @@
             $output .= ' selected="selected"';
         $output .= '>';
-        $output .= $cat_name;
+        $output .= $pad.$cat_name;
         if ( $args['show_count'] )
             $output .= '  ('. $category->category_count .')';