Changeset 37599

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.

Location:

trunk

Files:

• src/wp-includes/taxonomy.php (modified) (1 diff)
• tests/phpunit/tests/term/getTerms.php (modified) (1 diff)

trunk/src/wp-includes/taxonomy.php

@@ -1184 +1184 @@
          * (b) the first parameter shares no keys with the default array (ie, it's a list of taxonomies)
          */
-        $key_intersect  = array_intersect_key( $term_query->query_var_defaults, (array) $args );
+        $_args = wp_parse_args( $args );
+        $key_intersect  = array_intersect_key( $term_query->query_var_defaults, (array) $_args );
         $do_legacy_args = $deprecated || empty( $key_intersect );
 
         if ( $do_legacy_args ) {
                 $taxonomies = (array) $args;
-                $args = $deprecated;
+                $args = wp_parse_args( $deprecated );
                 $args['taxonomy'] = $taxonomies;
-        } elseif ( isset( $args['taxonomy'] ) && null !== $args['taxonomy'] ) {
-                $args['taxonomy'] = (array) $args['taxonomy'];
+        } else {
+                $args = wp_parse_args( $args );
+                if ( isset( $args['taxonomy'] ) && null !== $args['taxonomy'] ) {
+                        $args['taxonomy'] = (array) $args['taxonomy'];
+                }
         }
 

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 );