Upstream/WordPress
1
0
Fork 0
mirror of https://github.com/WordPress/WordPress.git synced 2025-02-02 05:31:25 +01:00
Code Issues Projects Releases Wiki Activity

Add filters to wp_unique_term_slug().

Browse Source 
This changeset adds two new filters:
* 'wp_unique_term_slug_is_bad_slug' lets developers control whether a test slug needs to be made unique, before the queries required to build a suffix are performed.
* 'wp_unique_term_slug' filters the output of the function.

These changes introduce parity with the filters in `wp_unique_post_slug()`.

Props coffee2code, bolo1988, boonebgorges.
Fixes #20783.
Built from https://develop.svn.wordpress.org/trunk@32837


git-svn-id: http://core.svn.wordpress.org/trunk@32808 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Boone Gorges 2015-06-18 13:20:26 +00:00
parent ecec2b61cf
commit 6105fca015
2 changed files with 49 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
wp-includes/taxonomy.php
View File

@ -3324,33 +3324,35 @@ function wp_remove_object_terms( $object_id, $terms, $taxonomy ) {
* @global wpdb $wpdb WordPress database abstraction object. * @global wpdb $wpdb WordPress database abstraction object.
* *
* @param string $slug The string that will be tried for a unique slug. * @param string $slug The string that will be tried for a unique slug.
* @param object $term The term object that the $slug will belong too. * @param object $term The term object that the `$slug` will belong to.
* @return string Will return a true unique slug. * @return string Will return a true unique slug.
*/ */
function wp_unique_term_slug( $slug, $term ) { function wp_unique_term_slug( $slug, $term ) {
global $wpdb; global $wpdb;
if ( ! term_exists( $slug ) ) $needs_suffix = true;
return $slug; $original_slug = $slug;
// As of 4.1, duplicate slugs are allowed as long as they're in different taxonomies. // As of 4.1, duplicate slugs are allowed as long as they're in different taxonomies.
if ( get_option( 'db_version' ) >= 30133 && ! get_term_by( 'slug', $slug, $term->taxonomy ) ) { if ( ! term_exists( $slug ) || get_option( 'db_version' ) >= 30133 && ! get_term_by( 'slug', $slug, $term->taxonomy ) ) {
return $slug; $needs_suffix = false;
} }
/* /*
* If the taxonomy supports hierarchy and the term has a parent, make the slug unique * If the taxonomy supports hierarchy and the term has a parent, make the slug unique
* by incorporating parent slugs. * by incorporating parent slugs.
*/ */
if ( is_taxonomy_hierarchical($term->taxonomy) && !empty($term->parent) ) { $parent_suffix = '';
if ( $needs_suffix && is_taxonomy_hierarchical( $term->taxonomy ) && ! empty( $term->parent ) ) {
$the_parent = $term->parent; $the_parent = $term->parent;
while ( ! empty($the_parent) ) { while ( ! empty($the_parent) ) {
$parent_term = get_term($the_parent, $term->taxonomy); $parent_term = get_term($the_parent, $term->taxonomy);
if ( is_wp_error($parent_term) || empty($parent_term) ) if ( is_wp_error($parent_term) || empty($parent_term) )
break; break;
$slug .= '-' . $parent_term->slug; $parent_suffix .= '-' . $parent_term->slug;
if ( ! term_exists( $slug ) ) if ( ! term_exists( $slug . $parent_suffix ) ) {
return $slug; break;
}
if ( empty($parent_term->parent) ) if ( empty($parent_term->parent) )
break; break;
@ -3359,6 +3361,19 @@ function wp_unique_term_slug( $slug, $term ) {
} }
// If we didn't get a unique slug, try appending a number to make it unique. // If we didn't get a unique slug, try appending a number to make it unique.
/**
* Filter whether the proposed unique term slug is bad.
*
* @since 4.3.0
*
* @param bool $needs_suffix Whether the slug needs to be made unique with a suffix.
* @param string $slug The slug.
* @param object $term Term object.
*/
if ( apply_filters( 'wp_unique_term_slug_is_bad_slug', $needs_suffix, $slug, $term ) ) {
if ( $parent_suffix ) {
$slug .= $parent_suffix;
} else {
if ( ! empty( $term->term_id ) ) if ( ! empty( $term->term_id ) )
$query = $wpdb->prepare( "SELECT slug FROM $wpdb->terms WHERE slug = %s AND term_id != %d", $slug, $term->term_id ); $query = $wpdb->prepare( "SELECT slug FROM $wpdb->terms WHERE slug = %s AND term_id != %d", $slug, $term->term_id );
else else
@ -3373,8 +3388,19 @@ function wp_unique_term_slug( $slug, $term ) {
} while ( $slug_check ); } while ( $slug_check );
$slug = $alt_slug; $slug = $alt_slug;
} }
}
}
return $slug; /**
* Filter the unique term slug.
*
* @since 4.3.0
*
* @param string $slug Unique term slug.
* @param object $term Term object.
* @param string $original_slug Slug originally passed to the function for testing.
*/
return apply_filters( 'wp_unique_term_slug', $slug, $term, $original_slug );
} }
/** /**

2
wp-includes/version.php
View File

@ -4,7 +4,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '4.3-alpha-32836'; $wp_version = '4.3-alpha-32837';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.