Changeset 52809 for trunk/src/wp-includes/formatting.php

Timestamp:

02/28/2022 03:40:15 PM (3 years ago)

Author:

SergeyBiryukov

Message:

I18N: Add a $locale parameter for remove_accents().

This highlights the fact that remove_accents() is locale-aware and makes it easier to utilize the function with different locales without having to use switch_to_locale() or the locale filter.

Additionally, this commit relaxes the check for character replacements in German locales to include formal and informal variants of any de_* locale, even if WordPress does not have a native translation for some of them yet.

Props malthert, johnbillion, knutsp, ocean90, SergeyBiryukov.
Fixes #54415.

File:

• trunk/src/wp-includes/formatting.php (modified) (2 diffs)

trunk/src/wp-includes/formatting.php

@@ -1584 +1584 @@
  * @since 4.8.0 Added locale support for `bs_BA`.
  * @since 5.7.0 Added locale support for `de_AT`.
- *
- * @param string $string Text that might have accent characters
+ * @since 6.0.0 Added the `$locale` parameter.
+ *
+ * @param string $string Text that might have accent characters.
+ * @param string $locale Optional. The locale to use for accent removal. Some character
+ *                       replacements depend on the locale being used (e.g. 'de_DE').
+ *                       Defaults to the current locale.
  * @return string Filtered string with replaced "nice" characters.
  */
-function remove_accents( $string ) {
+function remove_accents( $string, $locale = '' ) {
     if ( ! preg_match( '/[\x80-\xff]/', $string ) ) {
         return $string;
@@ -1924 +1928 @@
 
         // Used for locale-specific rules.
-        $locale = get_locale();
-
-        if ( in_array( $locale, array( 'de_DE', 'de_DE_formal', 'de_CH', 'de_CH_informal', 'de_AT' ), true ) ) {
+        if ( empty( $locale ) ) {
+            $locale = get_locale();
+        }
+
+        /*
+         * German has various locales (de_DE, de_CH, de_AT, ...) with formal and informal variants.
+         * There is no 3-letter locale like 'def', so checking for 'de' instead of 'de_' is safe,
+         * since 'de' itself would be a valid locale too).
+         */
+        if ( str_starts_with( $locale, 'de' ) ) {
             $chars['Ä'] = 'Ae';
             $chars['ä'] = 'ae';