Changeset 55865 for trunk/src/wp-includes/l10n.php

Timestamp:

05/29/2023 08:34:43 AM (3 years ago)

Author:

swissspidy

Message:

I18N: Improve _load_textdomain_just_in_time() logic when there are no translation files.

Fixes a performance issue where the JIT logic is invoked for every translation call if the there are no translations in the current locale. With this change, the information is cached by adding Noop_Translations instances to the global $l10n array. This way, get_translations_for_domain() returns earlier, thus avoiding subsequent _load_textdomain_just_in_time() calls.

Props swissspidy, johnbillion, ocean90.
Fixes #58321.

File:

• trunk/src/wp-includes/l10n.php (modified) (3 diffs)

trunk/src/wp-includes/l10n.php

@@ -835 +835 @@
 
         if ( isset( $l10n[ $domain ] ) ) {
+                if ( $l10n[ $domain ] instanceof NOOP_Translations ) {
+                        unset( $l10n[ $domain ] );
+
+                        return false;
+                }
+
                 unset( $l10n[ $domain ] );
 
@@ -1308 +1314 @@
         }
 
+        $l10n[ $domain ] = &$noop_translations;
+
         return $noop_translations;
 }
@@ -1323 +1331 @@
 function is_textdomain_loaded( $domain ) {
         global $l10n;
-        return isset( $l10n[ $domain ] );
+        return isset( $l10n[ $domain ] ) && ! $l10n[ $domain ] instanceof NOOP_Translations;
 }