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. git-svn-id: https://develop.svn.wordpress.org/trunk@40293 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
24b221bd34
commit
79cfdbd84c
|
@ -474,7 +474,10 @@ class WP_Term_Query {
|
||||||
$this->sql_clauses['where']['exclusions'] = preg_replace( '/^\s*AND\s*/', '', $exclusions );
|
$this->sql_clauses['where']['exclusions'] = preg_replace( '/^\s*AND\s*/', '', $exclusions );
|
||||||
}
|
}
|
||||||
|
|
||||||
if ( ! empty( $args['name'] ) ) {
|
if (
|
||||||
|
( ! empty( $args['name'] ) ) ||
|
||||||
|
( is_string( $args['name'] ) && 0 !== strlen( $args['name'] ) )
|
||||||
|
) {
|
||||||
$names = (array) $args['name'];
|
$names = (array) $args['name'];
|
||||||
foreach ( $names as &$_name ) {
|
foreach ( $names as &$_name ) {
|
||||||
// `sanitize_term_field()` returns slashed data.
|
// `sanitize_term_field()` returns slashed data.
|
||||||
|
@ -484,7 +487,10 @@ class WP_Term_Query {
|
||||||
$this->sql_clauses['where']['name'] = "t.name IN ('" . implode( "', '", array_map( 'esc_sql', $names ) ) . "')";
|
$this->sql_clauses['where']['name'] = "t.name IN ('" . implode( "', '", array_map( 'esc_sql', $names ) ) . "')";
|
||||||
}
|
}
|
||||||
|
|
||||||
if ( ! empty( $args['slug'] ) ) {
|
if (
|
||||||
|
( ! empty( $args['slug'] ) ) ||
|
||||||
|
( is_string( $args['slug'] ) && 0 !== strlen( $args['slug'] ) )
|
||||||
|
) {
|
||||||
if ( is_array( $args['slug'] ) ) {
|
if ( is_array( $args['slug'] ) ) {
|
||||||
$slug = array_map( 'sanitize_title', $args['slug'] );
|
$slug = array_map( 'sanitize_title', $args['slug'] );
|
||||||
$this->sql_clauses['where']['slug'] = "t.slug IN ('" . implode( "', '", $slug ) . "')";
|
$this->sql_clauses['where']['slug'] = "t.slug IN ('" . implode( "', '", $slug ) . "')";
|
||||||
|
|
|
@ -835,6 +835,15 @@ function get_term_by( $field, $value, $taxonomy = '', $output = OBJECT, $filter
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// 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 ) {
|
if ( 'id' === $field || 'term_id' === $field ) {
|
||||||
$term = get_term( (int) $value, $taxonomy, $output, $filter );
|
$term = get_term( (int) $value, $taxonomy, $output, $filter );
|
||||||
if ( is_wp_error( $term ) || null === $term ) {
|
if ( is_wp_error( $term ) || null === $term ) {
|
||||||
|
|
|
@ -205,4 +205,48 @@ class Tests_Term_GetTermBy extends WP_UnitTestCase {
|
||||||
|
|
||||||
$this->assertEquals( 0, $action->get_call_count() );
|
$this->assertEquals( 0, $action->get_call_count() );
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 21760
|
||||||
|
*/
|
||||||
|
public function test_get_term_by_name_with_string_0() {
|
||||||
|
register_taxonomy( 'wptests_tax', 'post', array( 'hierarchical' => true ) );
|
||||||
|
|
||||||
|
$term_id = $this->factory->term->create( array(
|
||||||
|
'name' => '0',
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$found = get_term_by( 'name', '0', 'wptests_tax' );
|
||||||
|
$this->assertSame( $term_id, $found->term_id );
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 21760
|
||||||
|
*/
|
||||||
|
public function test_get_term_by_slug_with_string_0() {
|
||||||
|
register_taxonomy( 'wptests_tax', 'post', array( 'hierarchical' => true ) );
|
||||||
|
|
||||||
|
$term_id = $this->factory->term->create( array(
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
'name' => '0',
|
||||||
|
'slug' => '0',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$found = get_term_by( 'slug', '0', 'wptests_tax' );
|
||||||
|
$this->assertSame( $term_id, $found->term_id );
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 21760
|
||||||
|
*/
|
||||||
|
public function test_get_term_by_with_empty_string() {
|
||||||
|
register_taxonomy( 'wptests_tax', 'post', array( 'hierarchical' => true ) );
|
||||||
|
|
||||||
|
$found_by_slug = get_term_by( 'slug', '', 'wptests_tax' );
|
||||||
|
$found_by_name = get_term_by( 'name', '', 'wptests_tax' );
|
||||||
|
|
||||||
|
$this->assertFalse( $found_by_slug );
|
||||||
|
$this->assertFalse( $found_by_name );
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -2223,6 +2223,57 @@ class Tests_Term_getTerms extends WP_UnitTestCase {
|
||||||
return 'foo';
|
return 'foo';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 21760
|
||||||
|
*/
|
||||||
|
public function test_with_term_slug_equal_to_string_0() {
|
||||||
|
register_taxonomy( 'wptests_tax', 'post', array( 'hierarchical' => true ) );
|
||||||
|
|
||||||
|
$term_id = self::factory()->term->create( array(
|
||||||
|
'name' => '0',
|
||||||
|
'slug' => '0',
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$found = get_terms( array(
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
'hide_empty' => 0,
|
||||||
|
'slug' => '0',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$this->assertEqualSets( array( $term_id ), wp_list_pluck( $found, 'term_id' ) );
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @ticket 21760
|
||||||
|
*/
|
||||||
|
public function test_with_multiple_term_slugs_one_equal_to_string_0() {
|
||||||
|
register_taxonomy( 'wptests_tax', 'post', array( 'hierarchical' => true ) );
|
||||||
|
|
||||||
|
$term_id1 = self::factory()->term->create( array(
|
||||||
|
'name' => '0',
|
||||||
|
'slug' => '0',
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$term_id2 = self::factory()->term->create( array(
|
||||||
|
'name' => 'Test',
|
||||||
|
'slug' => 'test',
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
) );
|
||||||
|
|
||||||
|
$found = get_terms( array(
|
||||||
|
'taxonomy' => 'wptests_tax',
|
||||||
|
'hide_empty' => 0,
|
||||||
|
'slug' => array(
|
||||||
|
'0',
|
||||||
|
'test',
|
||||||
|
),
|
||||||
|
) );
|
||||||
|
|
||||||
|
$this->assertEqualSets( array( $term_id1, $term_id2 ), wp_list_pluck( $found, 'term_id' ) );
|
||||||
|
}
|
||||||
|
|
||||||
protected function create_hierarchical_terms_and_posts() {
|
protected function create_hierarchical_terms_and_posts() {
|
||||||
$terms = array();
|
$terms = array();
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue