Changeset 34809 for trunk/src/wp-includes/taxonomy-functions.php

Timestamp:

10/03/2015 08:24:09 PM (9 years ago)

Author:

boonebgorges

Message:

When creating terms, avoid false dupe checks due to accented characters.

wp_insert_term() doesn't allow the creation of a term when the term name
is the same as another term in the same hierarchy level of the same taxonomy.
Previously, this duplicate check used get_term_by( 'name' ), which uses the
database collation to determine sameness. But common collations do not
distinguish between accented and non-accented versions of a character. As a
result, it was impossible to create a term 'Foo' if a sibling term with an
accented character existed.

We address this problem by using get_terms() to do the duplicate check. This
query returns all potentially matching terms. We then do a stricter check
for equivalence in PHP, before determining whether one of the matches is
indeed a duplicate.

Props boonebgorges, tyxla, geza.miklo, mehulkaklotar.
Fixes #33864.

File:

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

trunk/src/wp-includes/taxonomy-functions.php

@@ -2517 +2517 @@
      * unless a unique slug has been explicitly provided.
      */
-    if ( $name_match = get_term_by( 'name', $name, $taxonomy ) ) {
+    $name_matches = get_terms( $taxonomy, array(
+        'name' => $name,
+        'hide_empty' => false,
+    ) );
+
+    /*
+     * The `name` match in `get_terms()` doesn't differentiate accented characters,
+     * so we do a stricter comparison here.
+     */
+    $name_match = null;
+    if ( $name_matches ) {
+        foreach ( $name_matches as $_match ) {
+            if ( strtolower( $name ) === strtolower( $_match->name ) ) {
+                $name_match = $_match;
+                break;
+            }
+        }
+    }
+
+    if ( $name_match ) {
         $slug_match = get_term_by( 'slug', $slug, $taxonomy );
         if ( ! $slug_provided || $name_match->slug === $slug || $slug_match ) {