Exported .ts files are corrupted (Missing Context)

Description

When exporting a translated .ts file for Qt Linguist, it seems that the generated file has several issues

  • wrong setting of "language" ->always "en_US"

  • missing context -> that corrupts the .ts file

  • translation type always remains in type "unfinished"

Attached is a source example and a downloaded translation file in German.
Both tested with Web UI and CLI

Environment

None

Activity

Show:
Benjamin Demarteau
June 19, 2018, 11:38 AM

I'm experiencing the same thing, it makes the .ts support mostly useless (people can translate, but translations are unusable). Is this a configuration problem or a zanata bug?

Damian Jansen
June 21, 2018, 11:17 AM

translation type always remains in type "unfinished"

This was fixed in db3327e5c915b3555604175cb029fa96ff1e199f (https://github.com/zanata/zanata-platform/pull/834, master) but not yet released.

missing context -> that corrupts the .ts file

This is a bug in Zanata, on a missing writer.handleEvent(event) in the TSAdapter.java.

wrong setting of "language" ->always "en_US"

This seems to be the more difficult problem to solve, as the locale property provided by TSFilter is read-only. I'll investigate this further.

Thanks for the bug report, and apologies for the inconvenience this has caused.

Benjamin Demarteau
June 22, 2018, 7:08 AM

thanks for the rapid reaction, do you have an ETA on 4.5.0?

Damian Jansen
June 25, 2018, 1:47 AM

we're tagged a final for 4.5, and are heading into deployment stages, so if all goes well a "personally pessimistic ETA" would be two weeks. This will contain the fix for the missing contexts.

The "unfinished" issue is located in master, which will be 4.6 and we don't have a definite release date for that one yet. The locale issue is still being worked on - the Okapi filter property seems to be read-only and I have a somewhat "unpleasant" workaround that will need some tightening.

Benjamin Demarteau
June 27, 2018, 10:59 AM

I deployed this and I can confirm this fixes the context problem. Thanks for the quick release!

Ready for Release

Assignee

Damian Jansen

Reporter

Sebastian Schretter

Labels

None

Tested Version/s

None

Sprint

None

Fix versions

Affects versions

Priority

High
Configure