Description of problem: In present version, pushType source implies copyTrans on, while pushType=both or trans implies copyTrans off. However, "[INFO] Copy previous translations: true" still appears even when pushType=trans. This message is misleading. Please either 1) Hide copyTrans status unless it is given in command line of setting files. or 2) Actually contact server and return whether server is going to do the copyTrans 1) should be a simpler fix, and "copyTrans is not specified" implies "use server default", thus no further change is needed even when server default changed. 2) should give the users the clear message that how server actually behave, but need to consider the behavior changes in different server versions. Version-Release number of selected component (if applicable): Zanata version 1.8.0-SNAPSHOT (20121019-1428) INFO] client API version: 1.8.0-SNAPSHOT, server API version: 1.8.0-SNAPSHOT How reproducible: Always Steps to Reproduce: 1. Assume a project and version is created 2. mvn -e zanata:push -Dzanata.pushType=both Actual results: "[INFO] Copy previous translations: true" still appears even when pushType=trans. Expected results: 1) Hide copyTrans status unless it is given in command line of setting files. or 2) Actually contact server and return whether server is going to do the copyTrans
> pushType source implies copyTrans on, while pushType=both or trans implies copyTrans off Not quite, but copyTrans only makes sense when pushing source strings. If the source is being pushed (ie pushType=both or pushType=source), copyTrans should be enabled. If only the translations are pushed, copyTrans is irrelevant. When pushType=trans, we should (a) not ask the server to perform copyTrans, and (b) the log message should say something like "Copy previous translations: disabled since pushType=trans". This will make the copyTrans behaviour similar to Zanata 1.7, which only requests copyTrans when pushing source documents (since copyTrans used to be an option of putTranslations, not a separate REST API feature).
Copy trans should only happen now when pushing source files (pushType = 'source' or 'both'). In case the copy Trans option is specified, but only translations are pushed, the client will disable copy trans and will inform the user in the initial confirmation message. See: https://github.com/zanata/zanata-client/commit/5079efb86b1fada007d7df22f5cf8856550b9c7e
VERIFIED with Zanata version 1.8.0-SNAPSHOT (20121023-0030) client API version: 1.8.0-SNAPSHOT, server API version: 1.8.0-SNAPSHOT
Fix released in Zanata 2.0.