Ticket #22400: 22400.taxonomy.php.diff

wp-admin/includes/taxonomy.php

@@ -90 +90 @@
  */
 function wp_insert_category($catarr, $wp_error = false) {
         $cat_defaults = array('cat_ID' => 0, 'taxonomy' => 'category', 'cat_name' => '', 'category_description' => '', 'category_nicename' => '', 'category_parent' => '');
-        $catarr = wp_parse_args($catarr, $cat_defaults);
-        extract($catarr, EXTR_SKIP);
+        $catarr = wp_parse_args( $catarr, $cat_defaults );
 
-        if ( trim( $cat_name ) == '' ) {
+        if ( trim( $catarr['cat_name'] ) == '' ) {
                 if ( ! $wp_error )
                         return 0;
                 else
                         return new WP_Error( 'cat_name', __('You did not enter a category name.') );
         }
 
-        $cat_ID = (int) $cat_ID;
+        $catarr['cat_ID'] = (int) $catarr['cat_ID'];
 
         // Are we updating or creating?
-        if ( !empty ($cat_ID) )
+        if ( !empty ( $catarr['cat_ID'] ) )
                 $update = true;
         else
                 $update = false;
 
-        $name = $cat_name;
-        $description = $category_description;
-        $slug = $category_nicename;
-        $parent = $category_parent;
+        $name = $catarr['cat_name'];
+        $description = $catarr['category_description'];
+        $slug = $catarr['category_nicename'];
+        $parent = $catarr['category_parent'];
 
         $parent = (int) $parent;
         if ( $parent < 0 )
                 $parent = 0;
 
-        if ( empty( $parent ) || ! term_exists( $parent, $taxonomy ) || ( $cat_ID && term_is_ancestor_of( $cat_ID, $parent, $taxonomy ) ) )
+        if ( empty( $parent ) || ! term_exists( $parent, $taxonomy ) || ( $catarr['cat_ID'] && term_is_ancestor_of( $catarr['cat_ID'], $parent, $catarr['taxonomy'] ) ) )
                 $parent = 0;
 
         $args = compact('name', 'slug', 'parent', 'description');
 
         if ( $update )
-                $cat_ID = wp_update_term($cat_ID, $taxonomy, $args);
+                $catarr['cat_ID'] = wp_update_term( $catarr['cat_ID'], $catarr['taxonomy'], $args);
         else
-                $cat_ID = wp_insert_term($cat_name, $taxonomy, $args);
+                $catarr['cat_ID'] = wp_insert_term( $catarr['cat_name'], $catarr['taxonomy'], $args);
 
-        if ( is_wp_error($cat_ID) ) {
+        if ( is_wp_error( $catarr['cat_ID'] ) ) {
                 if ( $wp_error )
-                        return $cat_ID;
+                        return $catarr['cat_ID'];
                 else
                         return 0;
         }
-
-        return $cat_ID['term_id'];
+        return $catarr['cat_ID']['term_id'];
 }
 
 /**