Bug 260086 - DTP 1.6.2 Translation for 6 languages in group 1
Summary: DTP 1.6.2 Translation for 6 languages in group 1
Status: RESOLVED FIXED
Alias: None
Product: Babel
Classification: Technology
Component: Server (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P1 major (vote)
Target Milestone: ---   Edit
Assignee: Babel server inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-06 14:19 EST by Yasuo Doshiro CLA
Modified: 2009-03-23 14:22 EDT (History)
2 users (show)

See Also:


Attachments
DTP 1.6.2 Translation (1.81 MB, application/octet-stream)
2009-01-06 14:19 EST, Yasuo Doshiro CLA
no flags Details
Updated zip file. (1.81 MB, application/octet-stream)
2009-03-09 16:32 EDT, Yasuo Doshiro CLA
no flags Details
Korean in S.Chinese Datatools files (149.01 KB, image/x-png)
2009-03-16 17:39 EDT, Kit Lo CLA
no flags Details
DTP translation problem (55.51 KB, image/x-png)
2009-03-16 21:33 EDT, Kit Lo CLA
no flags Details
sqleditor-default-templates (10.07 KB, application/x-zip-compressed)
2009-03-16 21:37 EDT, Kit Lo CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yasuo Doshiro CLA 2009-01-06 14:19:26 EST
Created attachment 121678 [details]
DTP 1.6.2 Translation

The attached zip file contains DTP 1.6.2 translated *.properties files for
de_DE, es_ES, fr_FR, ja_JP, ko_KR, and zh_CN.

Please integrete this into Babel Server.

Thank you,
Doshiro.
Comment 1 Antoine Toulmé CLA 2009-01-06 15:52:46 EST
Can you please give the same information I asked you to give for 260013 ?

Additionally, please state whether the translations were made by professionals and tested in context.

Thanks!
Comment 2 Antoine Toulmé CLA 2009-01-10 02:23:18 EST
Opened IPZilla CQ:

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=2926
Comment 3 Antoine Toulmé CLA 2009-02-27 04:01:08 EST
Hi, the review came back with those comments:

Several plugin.properties files contained in the attachment contain the
following header which cannot be used in Eclipse:

# ===================================================================
# Licensed Materials - Property of IBM
# com.ibm.db.models.sql.query.edit
# (C) Copyright IBM Corporation 2005. All Rights Reserved.
#
# Note to U.S. Government Users Restricted Rights:
#   Use, duplication or disclosure restricted 
#   by GSA ADP Schedule Contract with IBM Corp.
# ===================================================================

The pertinent directories are:

de_DE/eclipse/plugins/org.eclipse.datatools.modelbase.sql.query.edit
es_ES/eclipse/plugins/org.eclipse.datatools.modelbase.sql.query.edit
fr_FR/eclipse/plugins/org.eclipse.datatools.modelbase.sql.query.edit
ko_KR/eclipse/plugins/org.eclipse.datatools.modelbase.sql.query.edit
zh_CN/eclipse/plugins/org.eclipse.datatools.modelbase.sql.query.edit

I've just checked the repo and the original file appears to have been corrected
in June/08; therefore, these translations files must be re-addressed.
Comment 4 Antoine Toulmé CLA 2009-03-09 05:40:54 EDT
Doshiro-san,

please take action regarding this contribution. Please remove the invalid headers identified in the previous comment, and submit your translations again, or it won't be possible to check those translations in.
Comment 5 Yasuo Doshiro CLA 2009-03-09 16:32:51 EDT
Created attachment 128098 [details]
Updated zip file.

Fixed comments in org.eclipse.datatools.modelbase.sql.query.edit/plugin.properties files.
Comment 6 Antoine Toulmé CLA 2009-03-09 17:33:30 EDT
OK, sent the zip to the IPZilla for IP clearance. Thanks!
Comment 7 Antoine Toulmé CLA 2009-03-10 16:53:46 EDT
OK, this passed the IP review. Working on it.
Comment 8 Antoine Toulmé CLA 2009-03-10 20:34:34 EDT
Committed to staging.

The symbols don't show ok on the recent translations page but they look ok in the translation UI:
http://babel.eclipse.org/staging/translate.php?project=datatools&version=1.6&file=org.eclipse.datatools.connectivity/plugins/org.eclipse.datatools.connectivity.sqm.core/src/org/eclipse/datatools/connectivity/sqm/internal/core/rte/fe/GenericDdlGeneration.properties&string=GENERATE_PK_CONSTRAINTS_DES

Please let me know if you think something is wrong with the translations provided.

Doshiro-san, were those translations made by experts and tested in context ?

Comment 9 Antoine Toulmé CLA 2009-03-16 16:56:28 EDT
Those translations were contributed to the live server.

You can see them through this URL:
http://babel.eclipse.org/babel/recent.php?userid=18673

Thank you very much for this contribution!
Comment 10 Kit Lo CLA 2009-03-16 17:39:19 EDT
Created attachment 129005 [details]
Korean in S.Chinese Datatools files

I just noticed a problem. I found some Korean characters in S.Chinese Datatools files.

I spot checked a few files from Doshiro-san's contributions. It looks like they are in S.Chinese.

Antoine, could you check the import script?
Comment 11 Kit Lo CLA 2009-03-16 17:40:12 EDT
reopen bug
Comment 12 Antoine Toulmé CLA 2009-03-16 17:49:16 EDT
Check the import script .... ?

The translations sat on the staging server for a week and everybody was happy with them. They have been contributed to the live server. You will need Denis to revert the translations intelligently, or I can delete all those contributed by Doshiro-san.

What's the problem ? What should I do ?
Comment 13 Kit Lo CLA 2009-03-16 21:33:05 EDT
Created attachment 129022 [details]
DTP translation problem

I think I found the problem. We expect all files under the root locale folder should contain translations for the same language.

However, in each locale folder, for this particular plugin, there is a folder containing the translations in all 6 languages. See attached screen capture. Depending on the order we import the files, we may import a copy of the file in another language.

I searched the translation zip. I only see one instance of such directory structure. We just missed this out of the ~1000 files in the zip. It's not easy to catch.
Comment 14 Kit Lo CLA 2009-03-16 21:37:22 EDT
Created attachment 129023 [details]
sqleditor-default-templates

I've reorganized these sqleditor-default-templates files, removing all extra files in each locale folder, leaving only the German template in the German folder, the French template in the Fench folder, etc.

Re-importing this attached zip and overwrite the current translations in the DB should fix the problem. Let's try that.
Comment 15 Antoine Toulmé CLA 2009-03-17 15:48:59 EDT
So the invalid translations are the Chinese ones only ?

So I can remove the translations by Doshiro-san, on DTP, in Chinese, and redo the import.

What do you think ?

I'll paste the query here.

Comment 16 Kit Lo CLA 2009-03-17 15:57:48 EDT
It's happening to all 6 languages because they all have the "nl" subdirectory in the contribution zip.
Comment 17 Antoine Toulmé CLA 2009-03-17 16:02:45 EDT
Aw, that's bad. I think I'd better go by using the key. I'll delete all the records with those keys translated by Doshiro-san.
Comment 18 Antoine Toulmé CLA 2009-03-20 14:34:01 EDT
OK so the query to get the invalid translations:

select * from translations where string_id IN (SELECT string_id from strings where name="Templates.if" or name="Templates.ifelse" or name="Templates.elseif" or name="Templates.else" or name="Templates.author" or name="Templates.while_condition" or name="Templates.simple.select" or name="Templates.begin.end" or name="Templates.case.expr" or name="Templates.case.search" or name="Templates.event" or name="Templates.function" or name="Templates.sp" or name="Templates.trigger.asa" or name="Templates.trigger" or name="Templates.trigger.column" or name="Templates.cursor" or name="Templates.delete" or name="Templates.insert" or name="Templates.open.cursor" or name="Templates.update" or name="Templates." or name="Templates." or name="Templates." or name="Templates.create.table" or name="Templates.select.into") and userid = 18673 and is_active and language_id <> 9;

Just have to replace select * by delete from.. let's do it.
Comment 19 Denis Roy CLA 2009-03-20 14:36:33 EDT
(In reply to comment #18)
> Just have to replace select * by delete from.. let's do it.

Then we need to run the translations healer, because any previous translations will still be marked inactive. 

I believe the healer is in classes/export.
Comment 20 Antoine Toulmé CLA 2009-03-20 14:38:13 EDT
If we run the import script next, do we need to do that ?

I deleted the strings just now on staging, I am about to import them. I will check how things go and will let you know if we need the healer script.
Comment 21 Denis Roy CLA 2009-03-20 14:43:39 EDT
No, don't need to run that if you're reimporting.  Thanks
Comment 22 Antoine Toulmé CLA 2009-03-20 15:19:47 EDT
I did all the cleanup on staging and live. Things went fine.

Kit, can you please check everything went fine and the right values are present ?
Comment 23 Kit Lo CLA 2009-03-21 18:29:22 EDT
I checked the translations of sqleditor default-templates in all 6 languages. The all look fine now. Thanks Antoine for fixing this!

I'm saving the file link to Babel server for easier reference later:
http://babel.eclipse.org/babel/translate.php?project=datatools&version=1.6&file=org.eclipse.datatools.sqltools/plugins/org.eclipse.datatools.sqltools.sqleditor/templates/default-templates.properties
Comment 24 Yasuo Doshiro CLA 2009-03-23 14:22:50 EDT
Antoine, Kit,

Thank you very much!

Doshiro.