Context Navigation

← Previous Change
Next Change →

taxonomy.php

Timestamp:

03/16/2017 02:03:53 AM (9 years ago)

Author:

boonebgorges

Message:

Improve querying for terms with falsey names and slugs.

Prior to [38677], get_term_by() would always return false if
an empty string were passed as the queried 'name' or 'slug'. The
refactor to use get_terms() broke this behavior; inappropriately
imprecise empty() checks caused the 'name' or 'slug' clause to be
discarded altogether when fetching terms, resulting in an incorrect
term being returned from the function.

We fix the regression by special-casing truly empty values passed
to get_term_by(), and ensuring that WP_Term_Query is properly
able to handle 0 and '0' term queries.

Props sstoqnov.
Fixes #21760.

File:

: 1 edited

trunk/src/wp-includes/taxonomy.php (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-includes/taxonomy.php

-              r40292
+              r40293
+        }
+        // No need to perform a query for empty 'slug' or 'name'.
+        if ( 'slug' === $field || 'name' === $field ) {
+                $value = (string) $value;
+                if ( 0 === strlen( $value ) ) {
+                        return false;
+                }
+        }
         if ( 'id' === $field || 'term_id' === $field ) {
                 $term = get_term( (int) $value, $taxonomy, $output, $filter );

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 40293 for trunk/src/wp-includes/taxonomy.php

Legend:

trunk/src/wp-includes/taxonomy.php

Download in other formats: