807383 - Check for corrupted translation is giving false negatives

Status: RESOLVED FIXED

(www.mozilla.org :: Bedrock, defect)

Description

[Anthony Ricaud (:rik)]

Our check for corrupted translations assumes that the substituted strings will come in the same order which is not always the case.

See for this string:
English
Mozilla has always contributed to Web standards, going back as far as the start of the project, and we will continue to do that moving forward. We <a href="%(cofound_url)s" rel="external">co-founded</a> the <a href="%(whatwg_url)s" rel="external">WHAT-WG</a> to kick off <a href="%(html5_url)s" rel="external">HTML5</a>. We are a leaders in <a href="%(javascript_url)s" rel="external">JavaScript standardization</a>. We have some of the top <a href="%(dbaron_url)s" rel="external">CSS</a> and <a href="%(ocallahan_url)s" rel="external">layout</a> experts in the world.
[(u'%(cofound_url)s', u'cofound_url'),
 (u'%(whatwg_url)s', u'whatwg_url'),
 (u'%(html5_url)s', u'html5_url'),
 (u'%(javascript_url)s', u'javascript_url'),
 (u'%(dbaron_url)s', u'dbaron_url'),
 (u'%(ocallahan_url)s', u'ocallahan_url')]


German
Mozilla hat schon immer – seit dem Start des Projekts – zu Web-Standards beigetragen und wird dies weiter vorantreiben. Wir haben das <a href="%(whatwg_url)s" rel="external">WHAT-WG</a> <a href="%(cofound_url)s" rel="external">mitgegründet</a>, um <a href="%(html5_url)s" rel="external">HTML5</a> in Gang zu bringen. Wir sind führend bei der <a href="%(javascript_url)s" rel="external">Standardisierung von JavaScript</a>. Wir haben einige der besten Experten der Welt zu den Themen <a href="%(dbaron_url)s" rel="external">CSS</a> und <a href="%(ocallahan_url)s" rel="external">Layout</a>.
[(u'%(whatwg_url)s', u'whatwg_url'),
 (u'%(cofound_url)s', u'cofound_url'),
 (u'%(html5_url)s', u'html5_url'),
 (u'%(javascript_url)s', u'javascript_url'),
 (u'%(dbaron_url)s', u'dbaron_url'),
 (u'%(ocallahan_url)s', u'ocallahan_url')]

I'll take the opportunity to clarify the error message a bit.

Comment 1

[[github robot]]

Commit pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/02c535fc64da0097021de6f88f57b816480ac63d
Compare independently of the order

Also improves the error message received
fix bug 807383

Comment 2

[Anthony Ricaud (:rik)]

This has been pushed to production.