Opened 6 years ago
Closed 6 years ago
#17162 closed defect (fixed)
Japanese translation replaced by Korean
Reported by: | LuxuryCoop | Owned by: | team |
---|---|---|---|
Priority: | critical | Milestone: | 19.01 |
Component: | Core | Version: | |
Keywords: | i18n regression japan | Cc: | stoecker, simon04, GerdP |
Description
I uploaded a traslation(.po) file to Launchpad for Korean few weeks ago, but actually the file was imported in Japanese translation due to my mistake.
Then I had seen error messege that the import failed, so I think no problem was occoured.
However, I just knew there are successful some import.
Is it possible that Translation in Japanese revert to before version 14620? I can't repair the translation in Launchpad, and the faults is too many to revert one by one.
I'm sorry about my huge mistake.
Attachments (1)
Change History (25)
comment:1 by , 6 years ago
Cc: | added |
---|---|
Keywords: | i18n added |
comment:2 by , 6 years ago
It seems someone reuploaded the japanese translations already:
Uploaded by Yuki Shira on 2019-01-01 02:15:07 UTC
https://bazaar.launchpad.net/~openstreetmap/josm/josm_trans/revision/2501#josm/ja.po
comment:3 by , 6 years ago
"Yuki Shira" uploaded ja.po today at 02:15 UTC.
Sounds like a hotfix is necessary...
comment:4 by , 6 years ago
Translations look fine now, but all entries are marked "for review" with the korean translation being suggested. Is there a way to dismiss suggestions in batch? There's 4866 strings, too much to dismiss individually.
comment:5 by , 6 years ago
Keywords: | regression added |
---|---|
Milestone: | → 18.12 |
comment:6 by , 6 years ago
I have dismissed all Korean translation suggestion with using Tampermonkey.
But there are 300+ Korean translations still remain and they should be removed.
Use this regular expression to find Korean characters: [가-힣]
comment:7 by , 6 years ago
When you download the current ja.po, then you can replace the wrong strings with "" and upload again to remove them (or translate them instead :-).
You need a current downloaded file, as the header contains information about the launchpad time.
It's better, when one of you does this. For us Europeans it simply looks too similar...
comment:8 by , 6 years ago
Downloading current file, copying the header lines into a pre-error-file and uploading again can do a "reset". But if there have been changes inbetween, they will be lost in this case.
Most important header is the "X-Launchpad-Export-Date".
Historic files can be found here: https://code.launchpad.net/~openstreetmap/josm/josm_trans
comment:9 by , 6 years ago
At first, I replaced the wrong strings with "" and uploaded, but launchpad completely ignored empty strings :(
So we have replaced the remaining wrong strings manually.
I think you can release a hotfix.
comment:10 by , 6 years ago
At first, I replaced the wrong strings with "" and uploaded, but launchpad completely ignored empty strings :(
Was the "X-Launchpad-Export-Date" after the error date? Otherwise Launchpad assumes you uploaded an old file and thus ignores changes which reverse the last modifications.
I think you can release a hotfix.
It needs a bit until the changes are in BZR for the i18n tools.
comment:11 by , 6 years ago
As I did not know about "reset", I uploaded ja.po file with this header:
"X-Launchpad-Export-Date: 2019-01-01 00:25+0000\n"
I used Japanese translations in ja.po (revision 2480) which has this header:
"X-Launchpad-Export-Date: 2018-12-10 04:53+0000\n"
comment:12 by , 6 years ago
I will do an i18n update tomorrow to get latest changes from Launchpad. I'll let you check it to be sure the hotfix is correct before releasing it.
by , 6 years ago
comment:13 by , 6 years ago
Dirk, we should update the i18n script to detect this kind of mistake early. In this case we had the opportunity to detect the error by checking the value of following lines:
- Korean translation for josm => Japanese translation for josm
- Language-Team : Korean <ko@…>\n => Language-Team : Japanese <tr@…>\n
- Language: ko\n => Language: ja\n
Probably checking the "Language" line is pretty straightforward.
follow-up: 17 comment:16 by , 6 years ago
I have checked josm-snapshot-14663.jar and josm_josm-ja.po.
I have no problem with Japanese translations :)
But could you remove LuxuryCoop from Japanese translation contributors list? :P
follow-up: 18 comment:17 by , 6 years ago
But could you remove LuxuryCoop from Japanese translation contributors list? :P
No. That's automatic...
Even if it was a bad contribution, it was one :-)
comment:18 by , 6 years ago
Replying to stoecker:
But could you remove LuxuryCoop from Japanese translation contributors list? :P
No. That's automatic...
Even if it was a bad contribution, it was one :-)
On the same topic you and I are also "contributors" of almost every language. It doesn't mean we speak 30+ languages, just that we fixed a lot of (technical) translation errors :)
comment:20 by , 6 years ago
Cc: | added |
---|
Sorry, I don't even know enough German to order something different from beer or sausages... :)
So. If this issue is fixed we can release an hotfix now.
@Simon, Gerd: as there has been a lot of changes since a few days, do you feel comfortable with promoting current JOSM as new tested version, or do you need a few days to stabilize any work in progress? Maybe we could wait to fix the keyboard issue #13160 first.
comment:21 by , 6 years ago
comment:22 by , 6 years ago
comment:23 by , 6 years ago
Keywords: | japan added |
---|---|
Milestone: | 18.12 → 19.01 |
Summary: | My mistake for Japanese translation → Japanese translation replaced by Korean |
comment:24 by , 6 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Dirk, any idea?