Changeset 37599 for trunk/tests/phpunit/tests/term/getTerms.php

Timestamp:

05/31/2016 12:53:27 PM (10 years ago)

Author:

boonebgorges

Message:

Ensure that get_terms() can accept querystring-style arguments.

Prior to [37572], arguments passed to get_terms() were passed immediately
through wp_parse_args(), which made it possible to pass arguments as a
querystring (hide_empty=0) rather than an array
(array( 'hide_empty' => false )). [37572] moved default argument parsing
into WP_Term_Query, while assuming that arguments passed to get_terms()
would be formatted as an array.

To provide compatibility, we now parse all args passed to get_terms() into
an array before processing.

See #35381.

File:

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

trunk/tests/phpunit/tests/term/getTerms.php

@@ -25 +25 @@
             'update_term_meta_cache' => false,
         ) );
+
+        $this->assertEqualSets( array( $term ), $found );
+    }
+
+    /**
+     * @ticket 35495
+     * @ticket 35381
+     */
+    public function test_legacy_params_as_query_string_should_be_properly_parsed() {
+        register_taxonomy( 'wptests_tax', 'post' );
+        $term = self::factory()->term->create( array( 'taxonomy' => 'wptests_tax' ) );
+
+        $found = get_terms( 'wptests_tax', 'hide_empty=0&fields=ids&update_term_meta_cache=0' );
+
+        $this->assertEqualSets( array( $term ), $found );
+    }
+
+    /**
+     * @ticket 35495
+     * @ticket 35381
+     */
+    public function test_new_params_as_query_string_should_be_properly_parsed() {
+        register_taxonomy( 'wptests_tax', 'post' );
+        $term = self::factory()->term->create( array( 'taxonomy' => 'wptests_tax' ) );
+
+        $found = get_terms( 'taxonomy=wptests_tax&hide_empty=0&fields=ids&update_term_meta_cache=0' );
 
         $this->assertEqualSets( array( $term ), $found );