Change &#160 to   in network-settings.dtd

Somehow all occurrences of the entity   (non-breaking space) were changed to &#160 (moved); in Tor Launcher's network-settings.dtd files.

I am not sure where things went wrong, but at this point the strings on Transifex are bad. They were correct prior to commit 7ffed96e51cdb4207980810ece44c58d2d25fda6

We could:

1. Remove the non-breaking space characters (which arguably makes the text a little harder to read).

2. Fix the files in Tor Launcher and hope the problem does not recur.

3. Fix the strings via transifex.com and hope the problem does not recur.

Mike, do you have any theories about how this happened and how we can avoid it in the future (and what should we do right now)?

added component::applications/tor launcher owner::phoul priority::medium resolution::fixed status::closed type::defect labels

These appear to have been introduced introduced when we first solicited feedback from tor-dev for the settings strings:

commit 8c55331b3662c91dc468313f661cd9269c60fa72
Author: Kathy Brade <brade@pearlcrescent.com>
Date:   Tue May 7 17:37:07 2013 -0400

If we can fix these strings in transifex such that it does not decide to invalidate them, that is probably the best approach. I am not sure how to do that, though.

I think mttp and/or phoul are in contact with the transifex people. Perhaps they can offer some suggestion.

Trac:
Cc: mikeperry, brade to mikeperry, brade, mttp, phoul

I will get in contact with Transifex and look into this issue. Will update the ticket once I hear back.

Note: I took over for Runa as translation coordinator 5 months ago, please direct Transifex-related issues at me.

Trac:
Owner: brade to brade,phoul
Status: new to assigned

Trac:
Owner: brade,phoul to phoul

After looking a bit closer, it appears that the source .dtd also has this issue. https://gitweb.torproject.org/tor-launcher.git/blob_plain/refs/heads/master:/src/chrome/locale/en/network-settings.dtd

If this gets fixed, I will trigger an update in Transifex and the issue should resolve.

We fixed the DTD files in the tor-launcher repository:

https://gitweb.torproject.org/tor-launcher.git/commit/bc1edfcd37df551c13e8750d384eeb05b91699ff

My fear is that these strings will be broken again after they make the round trip through the Transifex process... we will see.

Trac:
Summary: &#160 (moved) in network-settings.dtd to Change &#160 (moved) to   in network-settings.dtd

As long as the source file (https://gitweb.torproject.org/tor-launcher.git/blob_plain/refs/heads/master:/src/chrome/locale/en/network-settings.dtd) remains accurate, Transifex should not cause us any issues.

It appears that there is indeed an issue on Transifex's side. While the updated strings appear in the editor, the actual translations we pull down (or if we pull down their copy of our source file), the "&#160 (moved)" issue remains.

I have contacted OpenITP, and will update this ticket once I hear back.

This should now be resolved:

https://gitweb.torproject.org/translation.git/commit/ec682268449116a319524a936578dd35675a02bf & https://gitweb.torproject.org/translation.git/commit/3ffd1e23a63f38ace655f7c3333cd954dd76bdb8

Trac:
Resolution: N/A to fixed
Status: assigned to closed

We are close now, but some of the files within translations.git still have &#160 (moved); in them:

af-ZA/network-settings.dtd
am-ET/network-settings.dtd
bg-BG/network-settings.dtd
bn-BD/network-settings.dtd
ca-ES/network-settings.dtd
cs-CZ/network-settings.dtd
el-GR/network-settings.dtd
es-NI/network-settings.dtd
si/network-settings.dtd
sq-AL/network-settings.dtd
tr/network-settings.dtd
zh-CN.GB2312/network-settings.dtd
zh-TW/network-settings.dtd
zh/network-settings.dtd

I am not sure what happened, and I don't know how things were fixed on the Transifex side, but maybe someone edited strings in the above files on Transifex recently and that somehow "locked in" the bad &#160 (moved); sequences? That is just a guess though.

Trac:
Status: closed to reopened
Resolution: fixed to N/A

These are languages that have been removed from Transifex, but have not yet been removed from our Git. I will be working on resolving this over the weekend, please see #11754 (moved).

Replying to phoul:

These are languages that have been removed from Transifex, but have not yet been removed from our Git. I will be working on resolving this over the weekend, please see #11754 (moved).

Sorry; I forgot about that. If it is helpful to you, brade or myself could remove the odl locales from tor-launcher.git (we would just need a list of locales).

I have fixed the issue for the tr & zh_TW teams, as these are teams that actually exist and had mistranslated  . This fix should appear in git in the next couple hours. As for the other teams, I will be removing them from git tomorrow.

My current list is: af-ZA am-ET bg-BG bn-BD ca-ES cs-CZ el-GR es-NI sq-AL zh-CN.GB2312 zh

These teams have now been removed from the tor-launcher-network-settings branch: https://gitweb.torproject.org/translation.git/commit/74c09ab260a1b425425e40ba27e92c53d444c665

Trac:
Status: reopened to closed
Resolution: N/A to fixed

Thanks for removing these locales. We removed them from tor-launcher.git.

In double-checking that all of the   entities were correct, we found one file that still has &#160 (moved); https://gitweb.torproject.org/translation.git/blob/refs/heads/tor-launcher-network-settings:/si/network-settings.dtd

What needs to happen at Transifex to fix this?

Replying to mcs:

Thanks for removing these locales. We removed them from tor-launcher.git.

In double-checking that all of the   entities were correct, we found one file that still has &#160 (moved); https://gitweb.torproject.org/translation.git/blob/refs/heads/tor-launcher-network-settings:/si/network-settings.dtd

What needs to happen at Transifex to fix this?

When Mike Perry updated translations recently, we picked up this file (brade and I just fixed it manually in Tor Launcher). This problem will continue to recur until it is fixed at Transifex and/or in translation.git.

phoul -- Can you please look at this problem? Thanks!

This is another team that no longer exists on Transifex, I have removed it from the tor-launcher-network-settings branch.

Sorry; I am reopening this bug. I am seeing this problem again in: pt_BR/network-settings.dtd

How exactly is Transifex fixing this problem? Is it per file or ?

Trac:
Status: closed to reopened
Resolution: fixed to N/A

No, the fix was global. The problem here is that certain teams are attempting to "translate"  , and are doing so incorrectly. In this case, " " had become "&#160 (moved)." in many of the strings, which caused the issue to reappear.

I have fixed this issue with the pt_BR team. One thing that might be useful here is adding a TRANSLATORS comment to the strings that include " " that specifically says not to translate it.

Trac:
Status: reopened to closed
Resolution: N/A to fixed

closed

mentioned in issue #11754 (moved)

moved to tpo/applications/tor-launcher#11699 (closed)

mentioned in issue tpo/applications/tor-launcher#11754 (closed)