Ticket #41293: 41293.3.patch

src/wp-includes/taxonomy.php

@@ -1912 +1912 @@
 
         // Taxonomies registered without an 'args' param are handled here.
         if ( ! empty( $taxonomies ) ) {
-                $terms = array_merge( $terms, get_terms( $args ) );
+                $terms_from_remaining_taxonomies = get_terms( $args );
+
+                // Array keys should be preserved for values of $fields that use term_id for keys.
+                if ( ! empty( $args['fields'] ) && 0 === strpos( $args['fields'], 'id=>' ) ) {
+                        $terms = $terms + $terms_from_remaining_taxonomies;
+                } else {
+                        $terms = array_merge( $terms, $terms_from_remaining_taxonomies );
+                }
         }
 
         /**

tests/phpunit/tests/term/query.php

@@ -428 +428 @@
                 $this->assertEquals( array( $term_ids[1], $term_ids[0], 1 ), wp_list_pluck( $terms, 'term_id' ) );
         }
 
+        /**
+         * @ticket 41293
+         */
+        public function test_should_allow_same_args_with_the_get_terms() {
+                register_post_type( 'wptests_pt' );
+                register_taxonomy( 'wptests_tax', 'wptests_pt' );
+                $t1 = self::factory()->term->create( array(
+                        'taxonomy' => 'wptests_tax',
+                        'name' => 'foo',
+                        'slug' => 'bar',
+                ) );
+                $t2 = self::factory()->term->create( array(
+                        'taxonomy' => 'wptests_tax',
+                        'name' => 'bar',
+                        'slug' => 'foo',
+                ) );
+
+                $p = self::factory()->post->create( array(
+                        'post_type' => 'wptests_pt',
+                ) );
+
+                wp_set_object_terms( $p, array( $t1, $t2 ), 'wptests_tax' );
+
+                $expected = wp_get_post_terms( $p, 'wptests_tax', array(
+                        'fields' => 'ids',
+                ) );
+
+                $found1 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
+                        'fields' => 'id=>parent',
+                ) ) );
+
+                $found2 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
+                        'fields' => 'id=>slug',
+                ) ) );
+
+                $found3 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
+                        'fields' => 'id=>name',
+                ) ) );
+
+                $this->assertSame( $expected, $found1 );
+                $this->assertSame( $expected, $found2 );
+                $this->assertSame( $expected, $found3 );
+        }
+
         /**
          * @ticket 41796
          */