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

Timestamp:

02/16/2016 09:14:45 PM (9 years ago)

Author:

ocean90

Message:

i18n: Prevent is_textdomain_loaded() from returning true even if there are no translations for the domain.

In get_translations_for_domain() don't fill the global $l10n with NOOP_Translations instances, return a NOOP_Translations instance instead.

Props nacin, jrf.
Fixes #21319.

File:

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

trunk/src/wp-includes/l10n.php

@@ -794 +794 @@
 function get_translations_for_domain( $domain ) {
     global $l10n;
-    if ( !isset( $l10n[$domain] ) ) {
-        $l10n[$domain] = new NOOP_Translations;
-    }
-    return $l10n[$domain];
+    if ( isset( $l10n[ $domain ] ) ) {
+        return $l10n[ $domain ];
+    }
+
+    static $noop_translations = null;
+    if ( null === $noop_translations ) {
+        $noop_translations = new NOOP_Translations;
+    }
+
+    return $noop_translations;
 }
 
@@ -812 +818 @@
 function is_textdomain_loaded( $domain ) {
     global $l10n;
-    return isset( $l10n[$domain] );
+    return isset( $l10n[ $domain ] );
 }