Branch: refs/heads/master
Home: https://github.com/zanata/zanata-platform
Commit: 89a95219b404085a857304862366983c0d841c7d
https://github.com/zanata/zanata-platform/commit/89a95219b404085a857304862366983c0d841c7d
Author: Damian Jansen <djan...@redhat.com>
Date: 2018-01-11 (Thu, 11 Jan 2018)
Changed paths:
M
common/zanata-adapter-glossary/src/main/java/org/zanata/adapter/glossary/GlossaryCSVReader.java
M
common/zanata-adapter-glossary/src/test/java/org/zanata/adapter/glossary/GlossaryCSVReaderTest.java
M common/zanata-adapter-glossary/src/test/resources/glossary/translate1.csv
M common/zanata-adapter-glossary/src/test/resources/glossary/translate2.csv
M docs/user-guide/glossary/upload-glossaries.md
Log Message:
-----------
fix(ZNTA-2247): Remove magic numbers for glossary csv columns (#592)
* fix(ZNTA-2247): Remove magic numbers for glossary csv columns
Still expects column 0 to be the source locale, but checks that the
PoS and Description columns are named. This prevents garbage in
said fields.
This change also prevents repeatedly setting these fields, only
doing so once per row.
* fix(ZNTA-2247): Allow several synonyms for data fields
Fix invalid test, contradicting the specified behaviour.
* fix(ZNTA-2247): Add docs and improve code as per review
_______________________________________________
zanata-commits mailing list
zanata-commits@redhat.com
https://www.redhat.com/mailman/listinfo/zanata-commits
