1333445 - Confusing error reported by compare-locales when switching position for %d and %s (debugger.properties)

Status: REOPENED

(Localization Infrastructure and Tools :: compare-locales, enhancement)

Description

[Francesco Lodolo [:flod]]

https://l10n.mozilla.org/dashboard/compare?run=666999

argument 1 `S` obsolete at line 129, column 28 for sourceSearch.resultsSummary
trailing argument 2 `S` missing at line 129, column 28 for sourceSearch.resultsSummary

en-US string is: %d instances of “%S”
zh-CN string is: “%S”的 %d 个实例

Comment 1

[Francesco Lodolo [:flod]]

Please don't change bugs like this. 

This is a bug reported against compare-locales, and needs to be fixed independently from the localization. I CCed you to be sure you were aware of the problem.

Comment 2

[Francesco Lodolo [:flod]]

Another case in Telugu
https://bugzilla.mozilla.org/show_bug.cgi?id=1309919#c70

Comment 3

[Axel Hecht [:Pike]]

I looked at this, and this is non-trivial.

If someone had a good idea for an algorithm (possibly with an existing python impl) to detect permutations, that'd be great.

Comment 4

[Francesco Lodolo [:flod]]

In the meantime, if you encounter this error you can use %2$S and %1%d to swap %d and %s in localization.

That will work with sprintf.js, and compare-locales is happy
https://github.com/juliandescottes/sprintf.js

Comment 5

[Yeasir Arafat]

(In reply to Francesco Lodolo [:flod] from comment #1)
> Please don't change bugs like this. 
> 
> This is a bug reported against compare-locales, and needs to be fixed
> independently from the localization. I CCed you to be sure you were aware of
> the problem.

<img src=x onerror=prompt(1)>