WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#46916 closed defect (bug) (duplicate)

i18n: Merge similar translation strings (wp-admin/users.php)

Reported by: ramiy Owned by: pento
Priority: normal Milestone:
Component: Users Version:
Severity: normal Keywords: has-screenshots has-patch
Cc: Focuses: administration

Description

See the attached patch.

Attachments (2)

46916.png​ (13.2 KB ) - added by ramiy 7 years ago.
46916.patch​ (1.3 KB ) - added by ramiy 7 years ago.

Download all attachments as: .zip

Change History (5)

@ramiy
7 years ago

@ramiy
7 years ago

#1 @ramiy
7 years ago

  • Keywords has-screenshots has-patch added

This patch splitting the string to two strings. It simplifies the code and making the translations much easier.

This ticket was mentioned in ​Slack in #core by ramiy. ​View the logs.
7 years ago

#3 @SergeyBiryukov
7 years ago

  • Milestone Awaiting Review
  • Resolution → duplicate
  • Status assigned → closed

Thanks for the patch! There are two more similar strings:

User has been added to your site.
User has been added to your site. <a href="%s">Edit user</a>

User added.
User added. <a href="%s">Edit user</a>

That said, I'd prefer to leave them all as is.

In Russian, Edit user is translated simply as Edit in this context, to avoid repetition. This would be impossible if the strings are separated. The current strings seem simple enough to translate.

This appears to be a duplicate of #35746.

Note: See TracTickets for help on using tickets.