Changeset 55865 for trunk/tests/phpunit/tests/l10n/loadTextdomainJustInTime.php

Timestamp:

05/29/2023 08:34:43 AM (2 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/tests/phpunit/tests/l10n/loadTextdomainJustInTime.php (modified) (1 diff)

trunk/tests/phpunit/tests/l10n/loadTextdomainJustInTime.php

@@ -123 +123 @@
 
     /**
+     * @ticket 58321
+     *
+     * @covers ::get_translations_for_domain
+     */
+    public function test_get_translations_for_domain_get_locale_is_called_only_once() {
+        $filter_locale = new MockAction();
+        add_filter( 'locale', array( $filter_locale, 'filter' ) );
+
+        get_translations_for_domain( 'internationalized-plugin' );
+        get_translations_for_domain( 'internationalized-plugin' );
+        get_translations_for_domain( 'internationalized-plugin' );
+        $translations = get_translations_for_domain( 'internationalized-plugin' );
+
+        remove_filter( 'locale', array( $filter_locale, 'filter' ) );
+
+        $this->assertSame( 1, $filter_locale->get_call_count() );
+        $this->assertInstanceOf( 'NOOP_Translations', $translations );
+        $this->assertFalse( is_textdomain_loaded( 'internationalized-plugin' ) );
+    }
+
+    /**
      * @ticket 37113
      *