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

Timestamp:

06/23/2016 02:47:44 PM (5 years ago)

Author:

swissspidy

Message:

I18N: Enable unloading of text domains that have been loaded just in time.

[37415] removed the requirement to call load_plugin_textdomain() / load_theme_textdomain(). This caused unload_textdomain() to not work properly anymore in these cases.

With this change, unloaded text domains need to be explicitly loaded manually if you want to use them again.

Props swissspidy, ocean90.
Fixes #37113. See #34114.

File:

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

trunk/src/wp-includes/l10n.php

@@ -510 +510 @@
  * @since 1.5.0
  *
- * @global array $l10n
+ * @global array $l10n          An array of all currently loaded text domains.
+ * @global array $l10n_unloaded An array of all text domains that have been unloaded again.
  *
  * @param string $domain Text domain. Unique identifier for retrieving translated strings.
@@ -517 +518 @@
  */
 function load_textdomain( $domain, $mofile ) {
-    global $l10n;
+    global $l10n, $l10n_unloaded;
+
+    $l10n_unloaded = (array) $l10n_unloaded;
 
     /**
@@ -531 +534 @@
 
     if ( true == $plugin_override ) {
+        unset( $l10n_unloaded[ $domain ] );
+
         return true;
     }
@@ -562 +567 @@
         $mo->merge_with( $l10n[$domain] );
 
+    unset( $l10n_unloaded[ $domain ] );
+
     $l10n[$domain] = &$mo;
 
@@ -572 +579 @@
  * @since 3.0.0
  *
- * @global array $l10n
+ * @global array $l10n          An array of all currently loaded text domains.
+ * @global array $l10n_unloaded An array of all text domains that have been unloaded again.
  *
  * @param string $domain Text domain. Unique identifier for retrieving translated strings.
@@ -578 +586 @@
  */
 function unload_textdomain( $domain ) {
-    global $l10n;
+    global $l10n, $l10n_unloaded;
+
+    $l10n_unloaded = (array) $l10n_unloaded;
 
     /**
@@ -590 +600 @@
     $plugin_override = apply_filters( 'override_unload_textdomain', false, $domain );
 
-    if ( $plugin_override )
+    if ( $plugin_override ) {
+        $l10n_unloaded[ $domain ] = true;
+
         return true;
+    }
 
     /**
@@ -604 +617 @@
     if ( isset( $l10n[$domain] ) ) {
         unset( $l10n[$domain] );
+
+        $l10n_unloaded[ $domain ] = true;
+
         return true;
     }
@@ -794 +810 @@
  *
  * @see get_translations_for_domain()
+ * @global array $l10n_unloaded An array of all text domains that have been unloaded again.
  *
  * @param string $domain Text domain. Unique identifier for retrieving translated strings.
@@ -799 +816 @@
  */
 function _load_textdomain_just_in_time( $domain ) {
+    global $l10n_unloaded;
+
+    $l10n_unloaded = (array) $l10n_unloaded;
+
     static $cached_mofiles = null;
 
     // Short-circuit if domain is 'default' which is reserved for core.
-    if ( 'default' === $domain ) {
+    if ( 'default' === $domain || isset( $l10n_unloaded[ $domain ] ) ) {
         return false;
     }