Changeset 61223 for trunk/tests/phpunit/tests/dependencies/scripts.php

Timestamp:

11/12/2025 11:34:56 AM (10 months ago)

Author:

jonsurrell

Message:

Script Loader: Omit sourceURL from translation data when not printed.

The data returned from WP_Scripts::print_translations() when $display is false may be used in unpredictable ways that are incompatible with sourceURL comments. Omit the sourceURL comment in this case.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10505.

Follow-up to [60719].

Props jonsurrell, ralucastn, westonruter, peterwilsoncc.
See #63887.

File:

• trunk/tests/phpunit/tests/dependencies/scripts.php (modified) (1 diff)

trunk/tests/phpunit/tests/dependencies/scripts.php

@@ -4076 +4076 @@
                 $this->assertEqualHTML( $expected, $print_scripts );
         }
+
+        /**
+         * Ensure that `::print_translations()` does not include the sourceURL comment when `$display` is false.
+         *
+         * @ticket 63887
+         * @covers ::print_translations
+         */
+        public function test_print_translations_no_display_no_sourceurl() {
+                global $wp_scripts;
+                $this->add_html5_script_theme_support();
+
+                wp_register_script( 'wp-i18n', '/wp-includes/js/dist/wp-i18n.js', array(), null );
+                wp_enqueue_script( 'test-example', '/wp-includes/js/script.js', array(), null );
+                wp_set_script_translations( 'test-example', 'default', DIR_TESTDATA . '/languages' );
+
+                $translations_script_data = $wp_scripts->print_translations( 'test-example', false );
+                $this->assertStringNotContainsStringIgnoringCase( 'sourceURL=', $translations_script_data );
+        }
 }