Skip to content

I18N: Use /* */ block style for translator comments #12274

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
itzmekhokan wants to merge 1 commit into
base: trunk
Choose a base branch
from
Open

I18N: Use /* */ block style for translator comments #12274

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/wp-admin/includes/class-wp-site-health.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2241,12 +2241,12 @@ public function get_test_rest_availability() {
'<p>%s</p><p>%s<br>%s</p>',
__( 'When testing the REST API, an error was encountered:' ),
sprintf(
// translators: %s: The REST API URL.
/* translators: %s: The REST API URL. */
__( 'REST API Endpoint: %s' ),
$url
),
sprintf(
// translators: 1: The WordPress error code. 2: The WordPress error message.
/* translators: 1: The WordPress error code. 2: The WordPress error message. */
__( 'REST API Response: (%1$s) %2$s' ),
$r->get_error_code(),
$r->get_error_message()
Expand All @@ -2261,12 +2261,12 @@ public function get_test_rest_availability() {
'<p>%s</p><p>%s<br>%s</p>',
__( 'When testing the REST API, an unexpected result was returned:' ),
sprintf(
// translators: %s: The REST API URL.
/* translators: %s: The REST API URL. */
__( 'REST API Endpoint: %s' ),
$url
),
sprintf(
// translators: 1: The WordPress error code. 2: The HTTP status code error message.
/* translators: 1: The WordPress error code. 2: The HTTP status code error message. */
__( 'REST API Response: (%1$s) %2$s' ),
wp_remote_retrieve_response_code( $r ),
wp_remote_retrieve_response_message( $r )
Expand Down
2 changes: 1 addition & 1 deletion src/wp-admin/site-health.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
$current_tab = $_GET['tab'] ?? '';

$title = sprintf(
// translators: %s: The currently displayed tab.
/* translators: %s: The currently displayed tab. */
__( 'Site Health - %s' ),
( isset( $tabs[ $current_tab ] ) ? esc_html( $tabs[ $current_tab ] ) : esc_html( reset( $tabs ) ) )
);
Expand Down
2 changes: 1 addition & 1 deletion src/wp-includes/abilities-api/class-wp-abilities-registry.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ public static function get_instance(): ?self {
_doing_it_wrong(
__METHOD__,
sprintf(
// translators: %s: init action.
/* translators: %s: init action. */
__( 'Ability API should not be initialized before the %s action has fired.' ),
'<code>init</code>'
),
Expand Down
2 changes: 1 addition & 1 deletion src/wp-includes/abilities-api/class-wp-ability-categories-registry.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ public static function get_instance(): ?self {
_doing_it_wrong(
__METHOD__,
sprintf(
// translators: %s: init action.
/* translators: %s: init action. */
__( 'Ability API should not be initialized before the %s action has fired.' ),
'<code>init</code>'
),
Expand Down
2 changes: 1 addition & 1 deletion src/wp-includes/class-wp-icons-registry.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ protected function register( $icon_name, $icon_properties ) {
_doing_it_wrong(
__METHOD__,
sprintf(
// translators: %s is the name of any user-provided key
/* translators: %s is the name of any user-provided key */

@mukeshpanchal27 mukeshpanchal27 Jun 23, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to add end period.

Suggested change
/* translators: %s is the name of any user-provided key */
/* translators: %s is the name of any user-provided key. */

__( 'Invalid icon property: "%s".' ),
$key
),
Expand Down
6 changes: 3 additions & 3 deletions src/wp-includes/fonts/class-wp-font-collection.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ private function load_from_json( $file_or_url ) {
$file = file_exists( $file_or_url ) ? wp_normalize_path( realpath( $file_or_url ) ) : false;

if ( ! $url && ! $file ) {
// translators: %s: File path or URL to font collection JSON file.
/* translators: %s: File path or URL to font collection JSON file. */
$message = __( 'Font collection JSON file is invalid or does not exist.' );
_doing_it_wrong( __METHOD__, $message, '6.5.0' );
return new WP_Error( 'font_collection_json_missing', $message );
Expand Down Expand Up @@ -184,7 +184,7 @@ private function load_from_url( $url ) {
return new WP_Error(
'font_collection_request_error',
sprintf(
// translators: %s: Font collection URL.
/* translators: %s: Font collection URL. */
__( 'Error fetching the font collection data from "%s".' ),
$url
)
Expand Down Expand Up @@ -224,7 +224,7 @@ private function sanitize_and_validate_data( $data, $required_properties = array
foreach ( $required_properties as $property ) {
if ( empty( $data[ $property ] ) ) {
$message = sprintf(
// translators: 1: Font collection slug, 2: Missing property name, e.g. "font_families".
/* translators: 1: Font collection slug, 2: Missing property name, e.g. "font_families". */
__( 'Font collection "%1$s" has missing or empty property: "%2$s".' ),
$this->slug,
$property
Expand Down
4 changes: 2 additions & 2 deletions src/wp-includes/html-api/class-wp-html-processor.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ private function create_fragment_at_current_node( string $html ) {
_doing_it_wrong(
__METHOD__,
sprintf(
// translators: %s: A tag name like INPUT or BR.
/* translators: %s: A tag name like INPUT or BR. */
__( 'The context element cannot be a void element, found "%s".' ),
$tag_name
),
Expand All @@ -515,7 +515,7 @@ private function create_fragment_at_current_node( string $html ) {
_doing_it_wrong(
__METHOD__,
sprintf(
// translators: %s: A tag name like IFRAME or TEXTAREA.
/* translators: %s: A tag name like IFRAME or TEXTAREA. */
__( 'The context element "%s" is not supported.' ),
$tag_name
),
Expand Down
2 changes: 1 addition & 1 deletion src/wp-includes/rest-api/endpoints/class-wp-rest-icons-controller.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ public function get_icon( $name ) {
return new WP_Error(
'rest_icon_not_found',
sprintf(
// translators: %s is the name of any user-provided name
/* translators: %s is the name of any user-provided name */
__( 'Icon not found: "%s".' ),
$name
),
Expand Down
Loading