Changeset 57925 for trunk/tests/phpunit/tests/l10n/loadTextdomain.php

Timestamp:

04/04/2024 01:54:12 PM (5 months ago)

Author:

swissspidy

Message:

I18N: Bail early if an invalid text domain is passed to load_textdomain() et al.

Some plugins pass invalid values such as null instead of a string, which has never been supported by WordPress (no translations are loaded) and was technically undefined behavior. With the introduction of the new l10n library in #59656, which has stricter type hints, this could end up causing warnings or even fatal errors.

This change adds a deliberate short-circuit to load_textdomain() & co. to better handle such a case and document that it is not supported.

Props verygoode, swissspidy.
Fixes #60888.

File:

• trunk/tests/phpunit/tests/l10n/loadTextdomain.php (modified) (1 diff)

trunk/tests/phpunit/tests/l10n/loadTextdomain.php

@@ -317 +317 @@
         $this->assertSame( 0, $override_load_textdomain_callback->get_call_count(), 'Expected override_load_textdomain not to be called.' );
     }
+
+    /**
+     * @ticket 60888
+     * @covers ::load_plugin_textdomain
+     */
+    public function test_load_plugin_textdomain_invalid_domain() {
+        $this->assertFalse( load_plugin_textdomain( null ) );
+    }
+
+    /**
+     * @ticket 60888
+     * @covers ::load_muplugin_textdomain
+     */
+    public function test_load_muplugin_textdomain_invalid_domain() {
+        $this->assertFalse( load_muplugin_textdomain( null ) );
+    }
+
+    /**
+     * @ticket 60888
+     * @covers ::load_theme_textdomain
+     */
+    public function test_load_theme_textdomain_invalid_domain() {
+        $this->assertFalse( load_theme_textdomain( null ) );
+    }
+
+    /**
+     * @ticket 60888
+     * @covers ::load_textdomain
+     */
+    public function test_load_textdomain_invalid_domain() {
+        $this->assertFalse( load_textdomain( null, DIR_TESTDATA . '/pomo/thisfiledoesnotexist.mo' ) );
+    }
 }