Changeset 40513 for trunk/tests/phpunit/tests/term/query.php

Timestamp:

04/21/2017 07:13:51 PM (7 years ago)

Author:

boonebgorges

Message:

Restore support for taxonomy 'args' override when querying object terms.

[7520] introduced an undocumented feature whereby developers could
register a custom taxonomy with an 'args' parameter, consisting of
an array of config params that, when present, override corresponding
params in the $args array passed to wp_get_object_terms() when
using that function to query for terms in the specified taxonomy.

The wp_get_object_terms() refactor in [38667] failed to respect
this secret covenant, and the current changeset atones for the
transgression.

Props danielbachhuber.
Fixes #40496.

File:

• trunk/tests/phpunit/tests/term/query.php (modified) (1 diff)

trunk/tests/phpunit/tests/term/query.php

@@ -382 +382 @@
         $this->assertEquals( 1, $count );
     }
+
+    /**
+     * @ticket 40496
+     */
+    public function test_get_the_terms_should_respect_taxonomy_orderby() {
+        register_taxonomy( 'wptests_tax', 'post', array(
+            'sort' => true,
+            'args' => array(
+                'orderby' => 'term_order',
+            ),
+        ) );
+        $term_ids = self::factory()->term->create_many( 2, array(
+            'taxonomy' => 'wptests_tax',
+        ) );
+        $post_id = self::factory()->post->create();
+        wp_set_object_terms( $post_id, array( $term_ids[0], $term_ids[1] ), 'wptests_tax' );
+        $terms = get_the_terms( $post_id, 'wptests_tax' );
+        $this->assertEquals( array( $term_ids[0], $term_ids[1] ), wp_list_pluck( $terms, 'term_id' ) );
+        // Flip the order
+        wp_set_object_terms( $post_id, array( $term_ids[1], $term_ids[0] ), 'wptests_tax' );
+        $terms = get_the_terms( $post_id, 'wptests_tax' );
+        $this->assertEquals( array( $term_ids[1], $term_ids[0] ), wp_list_pluck( $terms, 'term_id' ) );
+    }
+
+    /**
+     * @ticket 40496
+     */
+    public function test_wp_get_object_terms_should_respect_taxonomy_orderby() {
+        register_taxonomy( 'wptests_tax', 'post', array(
+            'sort' => true,
+            'args' => array(
+                'orderby' => 'term_order',
+            ),
+        ) );
+        $term_ids = self::factory()->term->create_many( 2, array(
+            'taxonomy' => 'wptests_tax',
+        ) );
+        $post_id = self::factory()->post->create();
+        wp_set_object_terms( $post_id, array( $term_ids[0], $term_ids[1] ), 'wptests_tax' );
+        $terms = wp_get_object_terms( $post_id, array( 'category', 'wptests_tax' ) );
+        $this->assertEquals( array( $term_ids[0], $term_ids[1], 1 ), wp_list_pluck( $terms, 'term_id' ) );
+        // Flip the order
+        wp_set_object_terms( $post_id, array( $term_ids[1], $term_ids[0] ), 'wptests_tax' );
+        $terms = wp_get_object_terms( $post_id, array( 'category', 'wptests_tax' ) );
+        $this->assertEquals( array( $term_ids[1], $term_ids[0], 1 ), wp_list_pluck( $terms, 'term_id' ) );
+    }
 }