WordPress.org

Make WordPress Core

Ticket #41293: 41293.3.patch

File 41293.3.patch, 2.3 KB (added by boonebgorges, 3 years ago)
  • src/wp-includes/taxonomy.php

    diff --git src/wp-includes/taxonomy.php src/wp-includes/taxonomy.php
    index f2db5d81df..6313641d14 100644
    function wp_get_object_terms($object_ids, $taxonomies, $args = array()) { 
    19121912
    19131913        // Taxonomies registered without an 'args' param are handled here.
    19141914        if ( ! empty( $taxonomies ) ) {
    1915                 $terms = array_merge( $terms, get_terms( $args ) );
     1915                $terms_from_remaining_taxonomies = get_terms( $args );
     1916
     1917                // Array keys should be preserved for values of $fields that use term_id for keys.
     1918                if ( ! empty( $args['fields'] ) && 0 === strpos( $args['fields'], 'id=>' ) ) {
     1919                        $terms = $terms + $terms_from_remaining_taxonomies;
     1920                } else {
     1921                        $terms = array_merge( $terms, $terms_from_remaining_taxonomies );
     1922                }
    19161923        }
    19171924
    19181925        /**
  • tests/phpunit/tests/term/query.php

    diff --git tests/phpunit/tests/term/query.php tests/phpunit/tests/term/query.php
    index 855db6fb28..3d3379dfc2 100644
    class Tests_Term_Query extends WP_UnitTestCase { 
    428428                $this->assertEquals( array( $term_ids[1], $term_ids[0], 1 ), wp_list_pluck( $terms, 'term_id' ) );
    429429        }
    430430
     431        /**
     432         * @ticket 41293
     433         */
     434        public function test_should_allow_same_args_with_the_get_terms() {
     435                register_post_type( 'wptests_pt' );
     436                register_taxonomy( 'wptests_tax', 'wptests_pt' );
     437                $t1 = self::factory()->term->create( array(
     438                        'taxonomy' => 'wptests_tax',
     439                        'name' => 'foo',
     440                        'slug' => 'bar',
     441                ) );
     442                $t2 = self::factory()->term->create( array(
     443                        'taxonomy' => 'wptests_tax',
     444                        'name' => 'bar',
     445                        'slug' => 'foo',
     446                ) );
     447
     448                $p = self::factory()->post->create( array(
     449                        'post_type' => 'wptests_pt',
     450                ) );
     451
     452                wp_set_object_terms( $p, array( $t1, $t2 ), 'wptests_tax' );
     453
     454                $expected = wp_get_post_terms( $p, 'wptests_tax', array(
     455                        'fields' => 'ids',
     456                ) );
     457
     458                $found1 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
     459                        'fields' => 'id=>parent',
     460                ) ) );
     461
     462                $found2 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
     463                        'fields' => 'id=>slug',
     464                ) ) );
     465
     466                $found3 = array_keys( wp_get_object_terms( $p, 'wptests_tax', array(
     467                        'fields' => 'id=>name',
     468                ) ) );
     469
     470                $this->assertSame( $expected, $found1 );
     471                $this->assertSame( $expected, $found2 );
     472                $this->assertSame( $expected, $found3 );
     473        }
     474
    431475        /**
    432476         * @ticket 41796
    433477         */