Changeset 41038 for branches/4.8/tests/phpunit/tests/term/wpGetObjectTerms.php

Timestamp:

07/13/2017 01:42:44 PM (9 years ago)

Author:

boonebgorges

Message:

Taxonomy: Avoid duplicates when querying for terms in taxonomies registered with $args parameter.

[40514] introduced a bug that caused term queries to return some duplicates
when the $taxonomies array contained only taxonomies that were originally
registered with an $args array. We fix this bug by ensuring that
recursive get_terms() queries stop when all queried $taxonomies have
already been referenced.

Ports [41037] to the 4.8 branch.

Props bor0, atanasangelovdev.
Fixes #41010.

Location:

branches/4.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/term/wpGetObjectTerms.php (modified) (1 diff)

branches/4.8/tests/phpunit/tests/term/wpGetObjectTerms.php

@@ -733 +733 @@
                 $this->assertEquals( array( $t1, $t3, $t2 ), $found );
         }
+
+        /**
+         * @ticket 41010
+         */
+        public function test_duplicate_terms_should_not_be_returned_when_passed_multiple_taxonomies_registered_with_args_array() {
+                $taxonomy1 = 'wptests_tax';
+                $taxonomy2 = 'wptests_tax_2';
+
+                // Any non-empty 'args' array triggers the bug.
+                $taxonomy_arguments = array(
+                        'args' => array( 0 ),
+                );
+
+                register_taxonomy( $taxonomy1, 'post', $taxonomy_arguments );
+                register_taxonomy( $taxonomy2, 'post', $taxonomy_arguments );
+
+                $post_id = self::factory()->post->create();
+                $term_1_id = self::factory()->term->create( array(
+                        'taxonomy' => $taxonomy1,
+                ) );
+                $term_2_id = self::factory()->term->create( array(
+                        'taxonomy' => $taxonomy2,
+                ) );
+
+                wp_set_object_terms( $post_id, $term_1_id, $taxonomy1 );
+                wp_set_object_terms( $post_id, $term_2_id, $taxonomy2 );
+
+                $expected = array( $term_1_id, $term_2_id );
+
+                $actual = wp_get_object_terms( $post_id, array( $taxonomy1, $taxonomy2 ), array(
+                        'orderby' => 'term_id',
+                        'fields' => 'ids',
+                ) );
+
+                $this->assertEqualSets( $expected, $actual );
+        }
 }