Changeset 49236 for trunk/tests/phpunit/tests/l10n/localeSwitcher.php

Timestamp:

10/20/2020 04:03:58 PM (5 years ago)

Author:

ocean90

Message:

I18N: Introduce WP_Textdomain_Registry to store text domains and their language directory paths.

Previously, when using switch_to_locale() all current loaded text domains were unloaded and added to the $l10n_unloaded global. This prevented the just-in-time loading for text domains after a switch. The just-in-time loading was also only possible if the translations were stored in WP_LANG_DIR. Both issues have been fixed.

• Adds WP_Textdomain_Registry to keep track of the language directory paths for all plugins and themes.
• Updates all load_*_textdomain() functions to store the path in WP_Textdomain_Registry.
• Adds $reloadable parameter to unload_textdomain() to define whether a text domain can be loaded just-in-time again. This is used by WP_Locale_Switcher::load_translations().
• Extends _load_textdomain_just_in_time() to also support text domains of plugins and themes with custom language directories.
• Fixes the incorrect test_plugin_translation_after_switching_locale_twice() test which should have catch this issue earlier.
• Adds a new test plugin/theme to test the loading of translations with a custom language directory.
• Deprecates the now unused and private _get_path_to_translation() and _get_path_to_translation_from_lang_dir() functions.

Props yoavf, swissspidy, dd32, ocean90.
See #26511.
Fixes #39210.

File:

• trunk/tests/phpunit/tests/l10n/localeSwitcher.php (modified) (2 diffs)

trunk/tests/phpunit/tests/l10n/localeSwitcher.php

@@ -23 +23 @@
         $this->previous_locale = '';
 
-        unset( $GLOBALS['l10n'] );
-        unset( $GLOBALS['l10n_unloaded'] );
-        _get_path_to_translation( null, true );
+        unset( $GLOBALS['l10n'], $GLOBALS['l10n_unloaded'] );
+
+        /** @var WP_Textdomain_Registry $wp_textdomain_registry */
+        global $wp_textdomain_registry;
+
+        $wp_textdomain_registry->reset();
     }
 
     public function tearDown() {
-        unset( $GLOBALS['l10n'] );
-        unset( $GLOBALS['l10n_unloaded'] );
-        _get_path_to_translation( null, true );
+        unset( $GLOBALS['l10n'], $GLOBALS['l10n_unloaded'] );
+
+        /** @var WP_Textdomain_Registry $wp_textdomain_registry */
+        global $wp_textdomain_registry;
+
+        $wp_textdomain_registry->reset();
 
         parent::tearDown();
@@ -389 +395 @@
     }
 
+    /**
+     * @ticket 39210
+     */
+    public function test_switch_reloads_plugin_translations_outside_wp_lang_dir() {
+        global $wp_locale_switcher, $wp_textdomain_registry;
+
+        $locale_switcher = clone $wp_locale_switcher;
+
+        $wp_locale_switcher = new WP_Locale_Switcher();
+        $wp_locale_switcher->init();
+
+        require_once DIR_TESTDATA . '/plugins/custom-internationalized-plugin/custom-internationalized-plugin.php';
+
+        $this->assertSame( WP_PLUGIN_DIR . '/custom-internationalized-plugin/languages/', $wp_textdomain_registry->get( 'custom-internationalized-plugin' ) );
+
+        $expected = custom_i18n_plugin_test();
+        $this->assertSame( 'This is a dummy plugin', $expected );
+
+        switch_to_locale( 'es_ES' );
+        switch_to_locale( 'de_DE' );
+
+        $expected = custom_i18n_plugin_test();
+        $this->assertSame( 'Das ist ein Dummy Plugin', $expected );
+
+        restore_previous_locale();
+
+        $expected = custom_i18n_plugin_test();
+        $this->assertSame( 'Este es un plugin dummy', $expected );
+
+        restore_current_locale();
+
+        $wp_locale_switcher = $locale_switcher;
+    }
+
+    /**
+     * @ticket 39210
+     */
+    public function test_switch_reloads_theme_translations_outside_wp_lang_dir() {
+        global $wp_locale_switcher, $wp_textdomain_registry;
+
+        $locale_switcher = clone $wp_locale_switcher;
+
+        $wp_locale_switcher = new WP_Locale_Switcher();
+        $wp_locale_switcher->init();
+
+        switch_theme( 'custom-internationalized-theme' );
+
+        require_once get_stylesheet_directory() . '/functions.php';
+
+        $this->assertSame( get_template_directory() . '/languages/', $wp_textdomain_registry->get( 'custom-internationalized-theme' ) );
+
+        $expected = custom_i18n_theme_test();
+
+        $this->assertSame( 'This is a dummy theme', $expected );
+
+        switch_to_locale( 'es_ES' );
+        switch_to_locale( 'de_DE' );
+
+        $expected = custom_i18n_theme_test();
+        $this->assertSame( 'Das ist ein Dummy Theme', $expected );
+
+        restore_previous_locale();
+
+        $expected = custom_i18n_theme_test();
+        $this->assertSame( 'Este es un tema dummy', $expected );
+
+        restore_current_locale();
+
+        $wp_locale_switcher = $locale_switcher;
+    }
+
     public function filter_locale() {
         return 'es_ES';