Strings should not have different translators comments?

[pavelevap]

See for example here: ​https://translate.wordpress.org/projects/wp/dev/admin/cs/default?filters[status]=either&filters[original_id]=3731&filters[translation_id]=139896

String Install %s have 3 different comments, but there should be only one? If special context is not needed (I am not sure, especially for importer), there should be one comment translators: %s: Importer, plugin or theme name?

Or maybe any general solution for GlotPress to unite these comments and display it better for translators? Or any special tool to automatically check for these differences?

[ocean90]

I've noticed that a while ago too, see ​https://wordpress.slack.com/archives/core-i18n/p1456785619000098.

Not sure how we can enhance that. @SergeyBiryukov has proposed to assign each comment to the file.

[SergeyBiryukov]

It looks like this in the .pot file:

#. translators: %s: attachment title
#. translators: %s: post title
#. translators: %s: taxonomy term name
#. translators: %s: post title

We could probably check if every line starts with translators: and parse it as a list in that case, removing duplicates in the process and displaying with proper formatting in GlotPress. This would be a custom plugin for translate.w.org.

P.S. I think questions in ticket summaries are perfectly fine :) Slack or not, they help brainstorming.

[pavelevap]

Ahh, I see, it was discussed before, but I searched only Trac and not Slack.

Is there any script to check all strings from .pot file which have more different translators comment?

There are two reasons for this ticket:

1) GlotPress UI - Yes, it would be fine to display better formatted translators comments.

2) In some cases different comments can be used as reminders of some context related problems, so we should check them regurarly and merge what is possible (sometimes only typo), I guess. For example linked string from summary is probably missing context. It can be Install Tumblr Importer (for plugin name), but also only Install Tumblr (for importer name). For importer would be maybe helpful some other string Install %s Importer?

[swissspidy]

For importer would be maybe helpful some other string Install %s Importer?

What about adding context to these Install %s strings?

[swissspidy]

FWIW, the wp i18n command detects strings with multiple different translator comments. It's possible to use that as part of the CI pipeline to flag these.