Merge pull request #50 from yumaoka/rfe-49-cli-updlang

Added update-bundle command to gp-cli

Author: yumaoka

gp-cli.md

@@ -139,6 +139,17 @@ are the translation target languages.
 java -jar gp-cli.jar create -b MyNewBundle -l en,fr,de -j mycreds.json
 ```
 
+#### update (update-bundle)
+
+Updates an existing bundle's configuration.
+
+The following example sets French (fr), German (de) and Italian (it)
+as target languages and the translation instruction note "These are
+Java MessageFormat ...".
+```
+java -jar gp-cli.jar update -b MyBundle -l fr,de,it -n "These are Java MessageFormat pattern strings"
+```
+
 #### import
 
 Imports resource data to a bundle. This command takes an input language.

gp-cli/src/main/java/com/ibm/g11n/pipeline/tools/cli/GPCmd.java

@@ -1,5 +1,5 @@
 /*  
- * Copyright IBM Corp. 2015,2016
+ * Copyright IBM Corp. 2015,2016,2017
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -46,6 +46,7 @@ public static void main(String[] args) {
         jc.addCommand("list-bundle", new ListBundlesCmd(), "list");
         jc.addCommand("show-bundle", new ShowBundleCmd(), "show");
         jc.addCommand("create-bundle", new CreateBundleCmd(), "create");
+        jc.addCommand("update-bundle", new UpdateBundleCmd(), "update");
         jc.addCommand("delete-bundle", new DeleteBundleCmd(), "delete");
         jc.addCommand("copy-bundle", new CopyBundleCmd(), "copy");
         jc.addCommand("copy-all-bundles", new CopyAllBundlesCmd());

gp-cli/src/main/java/com/ibm/g11n/pipeline/tools/cli/ShowBundleCmd.java

@@ -1,5 +1,5 @@
 /*  
- * Copyright IBM Corp. 2015,2016
+ * Copyright IBM Corp. 2015,2016,2017
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,6 +16,7 @@
 package com.ibm.g11n.pipeline.tools.cli;
 
 import java.util.Date;
+import java.util.List;
 import java.util.Set;
 
 import com.beust.jcommander.Parameters;
@@ -36,6 +37,7 @@ final class ShowBundleCmd extends BundleCmd {
     static class BundleDataJson {
         String sourceLanguage;
         Set<String> targetLanguages;
+        List<String> notes;
         boolean readOnly;
         String updatedBy;
         Date updatedAt;
@@ -50,6 +52,7 @@ protected void _execute() {
 
             outJson.sourceLanguage = bundleData.getSourceLanguage();
             outJson.targetLanguages = bundleData.getTargetLanguages();
+            outJson.notes = bundleData.getNotes();
             outJson.readOnly = bundleData.isReadOnly();
             outJson.updatedBy = bundleData.getUpdatedBy();
             outJson.updatedAt = bundleData.getUpdatedAt();

gp-cli/src/main/java/com/ibm/g11n/pipeline/tools/cli/UpdateBundleCmd.java

@@ -0,0 +1,119 @@
+/*  
+ * Copyright IBM Corp. 2017
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package com.ibm.g11n.pipeline.tools.cli;
+
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+
+import com.beust.jcommander.Parameter;
+import com.beust.jcommander.Parameters;
+import com.ibm.g11n.pipeline.client.BundleDataChangeSet;
+import com.ibm.g11n.pipeline.client.ServiceException;
+
+/**
+ * Updates an existing translation bundle.
+ * 
+ * @author Yoshito Umaoka
+ */
+@Parameters(commandDescription = "Updates an existing translation bundle.")
+public class UpdateBundleCmd extends BundleCmd {
+
+    @Parameter(
+            names = {"-l", "--languages"},
+            description = "List of bundle's target language ID(s) separted by comma. "
+                        + "Empty list \"\" will remove all existing target languages.")
+    private String languageIdsListStr;
+
+    @Parameter(
+            names = {"-n", "--note"},
+            description = "Translation instruction note. "
+                        + "Empty note \"\" will remove existing instruction note.")
+    private String note;
+
+    @Parameter(
+            names = {"-r", "--readOnly"},
+            description = "true to set the bundle read only.")
+    private String readOnlyStr;
+
+    @Override
+    protected void _execute() {
+        Set<String> trgLangs = null;
+        if (languageIdsListStr != null) {
+            String[] langs = languageIdsListStr.split(",");
+            trgLangs = new HashSet<>(langs.length);
+            for (String lang : langs) {
+                if (lang.isEmpty()) {
+                    continue;
+                }
+                trgLangs.add(lang);
+            }
+        }
+
+        List<String> notes = null;
+        if (note != null) {
+            if (note.isEmpty()) {
+                // Empty note will delete the existing note
+                notes = Collections.emptyList();
+            } else {
+                notes = Collections.singletonList(note);
+            }
+        }
+
+        Boolean readOnly = null;
+        if (readOnlyStr != null) {
+            if (readOnlyStr.equalsIgnoreCase("true")) {
+                readOnly = Boolean.TRUE;
+            } else if (readOnlyStr.equalsIgnoreCase("false")) {
+                readOnly = Boolean.FALSE;
+            } else {
+                System.out.println("Bad -r (--readOnly) argument value: " + readOnlyStr
+                        + ". The command argument will be ignored.");
+            }
+        }
+
+        if (languageIdsListStr == null && notes == null && readOnlyStr == null) {
+            System.out.println("Nothing to update.");
+            return;
+        }
+
+        BundleDataChangeSet changes = new BundleDataChangeSet();
+        changes
+            .setTargetLanguages(trgLangs)
+            .setNotes(notes)
+            .setReadOnly(readOnly);
+
+        try {
+            getClient().updateBundle(bundleId, changes);
+        } catch (ServiceException e) {
+            throw new RuntimeException(e);
+        }
+
+        System.out.println("Bundle " + bundleId + " was successfully updated.");
+        if (trgLangs != null) {
+            String newTrgLangs = trgLangs.isEmpty() ? "<removed>" : trgLangs.toString();
+            System.out.println("- Target languages: " + newTrgLangs);
+        }
+        if (notes != null) {
+            String newNote = notes.isEmpty() ? "<removed>" : notes.get(0);
+            System.out.println("- Translation instruction note: " + newNote);
+        }
+        if (readOnly != null) {
+            System.out.println("- Read only: " + readOnly);
+        }
+    }
+}