Function wp_remove_surrounding_empty_script_tags should use sprintf with placeholder

[swissspidy]

The function wp_remove_surrounding_empty_script_tags() currently uses the following function call:

__( 'Function wp_remove_surrounding_empty_script_tags() used incorrectly in PHP.' )

wp_remove_surrounding_empty_script_tags() should not be translatable.

This should be changed to use sprintf with a placeholder and an accompanying translator comment.

Something like this:

sprintf(
        /* translators: %s: wp_remove_surrounding_empty_script_tags() */
        __( 'Function %s used incorrectly in PHP.' ),
        'wp_remove_surrounding_empty_script_tags()'
);

I use sprint to prevent to translate "wp_remove_surrounding_empty_script_tags".

The detailed explanation is as follows, quoted from the track.

The function wp_remove_surrounding_empty_script_tags() currently uses the following function call:

( 'Function wp_remove_surrounding_empty_script_tags() used incorrectly in PHP.' )
wp_remove_surrounding_empty_script_tags() should not be translatable.

Trac ticket: https://core.trac.wordpress.org/ticket/60590

[kadamwhite]

I have tested the patch from @naoki0h at WordCamp Kansai Contributor Day and it works as expected. After discussion with @kirasong I'm marking this for commit, since we're before the string freeze.

[kadamwhite]

In 57699:

Script Loader: Inject wp_remove_surrounding_empty_script_tags function name in returned error string using sprintf.

The name of this function should not be editable by the translator in the internationalized return error string.

Props naoki0h, swissspidy, kirasong.
Fixes #60590.

[kadamwhite]

(Note: after rereading the trac workflow keyword guide it seems the i18n-change keyword is intended for use after string freeze, so apologies in advance if this was not the right context to use that. Leaving it in place out of an abundance of caution.)

Committed in https://core.trac.wordpress.org/changeset/57699 :tada: