Translations: Treat strings with unique comments as unique

[garrett-eclipse]

Hello,

This originally stems from GP, but as it just uses the source POT strings than would be an issue with the string extractor or possible the core as it might require context be added instead of segregating by unique comments.

Original GP - ​https://github.com/GlotPress/GlotPress-WP/issues/930

When a string with multiple instances has a comment supplied that's unique for each instance those instances should all be unique rows instead of a merged instance as those comments can provide context which may lead to alternate translations.
Some examples;
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357863&filters%5Btranslation_id%5D=38532994
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357860&filters%5Btranslation_id%5D=38532991
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357859&filters%5Btranslation_id%5D=38532990
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357858&filters%5Btranslation_id%5D=38532989
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357857&filters%5Btranslation_id%5D=38532988
​https://translate.wordpress.org/projects/wp/dev/admin/en-ca/default?filters%5Bstatus%5D=either&filters%5Boriginal_id%5D=2357855&filters%5Btranslation_id%5D=38533010
In most of the above cases the comment is to indicate what a param is and in the above it's different in some cases as in one it's about the page while the other is the term, etc.
Due to this would make sense that each would be a unique translation.

Let me know if this is more an issue of adding context via core code rather than updating the string extractor to take unique comments into account and distinguishing each unique instance as it's own string. If that's the case I'll open a core ticket to add context to strings.

Thanks

[garrett-eclipse]

String with two instances both which have unique comments

[yoavf]

What makes a string unique is the combination of its context and its text (or just singular text if it has a plural). If a string is different enough that it can have different translations, then a context should be used (with or without a comment).

If for the examples above any translator feels like a context should be added (because they would like to supply a different translation), then it warrants a core ticket to add a context to one or more instances of the string.

[tobifjellner]

I think we can close this ticket.
We do have this specific possibility, when needed, by using _x() and similar functions that allow for context to be specified.
But the comment in itself is NOT a part of the lookup and cannot be used to separate strings from each other.

[garrett-eclipse]

Thanks @yoavf and @tobifjellner that makes sense. I'll review the strings I thought should be segregated when I have some more time and open the appropriate tickets in core to introduce context. Appreciate the feedback