Ticket #29848: 29848.patch

src/wp-includes/taxonomy.php

@@ -2455 +2455 @@
                         // The alias isn't in a group, so let's create a new one and firstly add the alias term to it.
                         $term_group = $wpdb->get_var("SELECT MAX(term_group) FROM $wpdb->terms") + 1;
 
-                        /**
-                         * Fires immediately before the given terms are edited.
-                         *
-                         * @since 2.9.0
-                         *
-                         * @param int    $term_id  Term ID.
-                         * @param string $taxonomy Taxonomy slug.
-                         */
-                        do_action( 'edit_terms', $alias->term_id, $taxonomy );
-                        $wpdb->update($wpdb->terms, compact('term_group'), array('term_id' => $alias->term_id) );
-
-                        /**
-                         * Fires immediately after the given terms are edited.
-                         *
-                         * @since 2.9.0
-                         *
-                         * @param int    $term_id  Term ID
-                         * @param string $taxonomy Taxonomy slug.
-                         */
-                        do_action( 'edited_terms', $alias->term_id, $taxonomy );
+                        wp_update_term( $alias->term_id, $taxonomy, array(
+                                'term_group' => $term_group,
+                        ) );
                 }
         }
 
@@ -3004 +2987 @@
                         return new WP_Error('duplicate_term_slug', sprintf(__('The slug “%s” is already in use by another term'), $slug));
         }
 
-        /** This action is documented in wp-includes/taxonomy.php */
+        /**
+         * Fires immediately before the given terms are edited.
+         *
+         * @since 2.9.0
+         *
+         * @param int    $term_id  Term ID.
+         * @param string $taxonomy Taxonomy slug.
+         */
         do_action( 'edit_terms', $term_id, $taxonomy );
         $wpdb->update($wpdb->terms, compact( 'name', 'slug', 'term_group' ), compact( 'term_id' ) );
         if ( empty($slug) ) {
@@ -3012 +3002 @@
                 $wpdb->update( $wpdb->terms, compact( 'slug' ), compact( 'term_id' ) );
         }
 
-        /** This action is documented in wp-includes/taxonomy.php */
+        /**
+         * Fires immediately after the given terms are edited.
+         *
+         * @since 2.9.0
+         *
+         * @param int    $term_id  Term ID
+         * @param string $taxonomy Taxonomy slug.
+         */
         do_action( 'edited_terms', $term_id, $taxonomy );
 
         $tt_id = $wpdb->get_var( $wpdb->prepare( "SELECT tt.term_taxonomy_id FROM $wpdb->term_taxonomy AS tt INNER JOIN $wpdb->terms AS t ON tt.term_id = t.term_id WHERE tt.taxonomy = %s AND t.term_id = %d", $taxonomy, $term_id) );

tests/phpunit/tests/term.php

@@ -244 +244 @@
                 $this->assertEquals( 0, term_exists(NULL) );
         }
 
+        public function test_wp_insert_term_alias_of_no_term_group() {
+                register_taxonomy( 'wptests_tax', 'post' );
+                $original_term_id = $this->factory->term->create( array(
+                        'taxonomy' => 'wptests_tax',
+                ) );
+                $original_term = get_term( $original_term_id, 'wptests_tax' );
+
+                $created_term_ids = wp_insert_term( 'Foo', 'wptests_tax', array(
+                        'alias_of' => $original_term->slug,
+                ) );
+                $created_term = get_term( $created_term_ids['term_id'], 'wptests_tax' );
+
+                $updated_original_term = get_term( $original_term->term_id, 'wptests_tax' );
+
+                $term = get_term( $created_term_ids['term_id'], 'wptests_tax' );
+                _unregister_taxonomy( 'wptests_tax' );
+
+                $this->assertSame( 0, $original_term->term_group );
+                $this->assertNotEmpty( $created_term->term_group );
+                $this->assertSame( $created_term->term_group, $updated_original_term->term_group );
+        }
+
         public function test_wp_insert_term_duplicate_name_slug_non_hierarchical() {
                 register_taxonomy( 'foo', 'post', array() );