From 99b9a0e5dea08d15a316f2556c2d6503b29e5d78 Mon Sep 17 00:00:00 2001
From: Koala Yeung <koalay@gmail.com>
Date: Sat, 6 May 2017 23:05:13 +0800
Subject: [PATCH] translateionRunner: improve minimalist options (#2835)

* translateionRunner: improve minimalist options

* Properly added flags.
* Added alias: -h, -f

* translationRunner: improve logic and flow

* Show all error messages instead of validate availability / name
  format separately.

* translationRunner: check messageDirectory existance

* translationRunner: changed throw string to Error

* translationRunner: use short cut for boolean
---
 config/webpack/translationRunner.js | 146 +++++++++++++++++-----------
 1 file changed, 90 insertions(+), 56 deletions(-)

diff --git a/config/webpack/translationRunner.js b/config/webpack/translationRunner.js
index 937c2edd0..ed80ae2bf 100644
--- a/config/webpack/translationRunner.js
+++ b/config/webpack/translationRunner.js
@@ -1,9 +1,82 @@
 /*eslint no-console: "off"*/
 const manageTranslations = require('react-intl-translations-manager').default;
-const argv = require('minimist')(process.argv.slice(2));
 const fs = require('fs');
 
+const testRFC5626 = function (reRFC5646) {
+  return function (language) {
+    if (!language.match(reRFC5646)) {
+      throw new Error('Not RFC5626 name');
+    }
+  }
+}
+
+const testAvailability = function (availableLanguages) {
+  return function (language) {
+    if ((argv.force !== true) && availableLanguages.indexOf(language) < 0) {
+      throw new Error('Not an available language');
+    }
+  }
+}
+
+const validateLanguages = function (languages, validators) {
+  let invalidLanguages = languages.reduce((acc, language) => {
+    try {
+      for (let validator of validators) {
+        validator(language);
+      }
+    } catch (error) {
+      acc.push({
+        language,
+        error,
+      });
+    }
+    return acc;
+  }, []);
+
+  if (invalidLanguages.length > 0) {
+    console.log(`\nError: Specified invalid LANGUAGES:`);
+    for (let {language, error} of invalidLanguages) {
+      console.error(`* ${language}: ${error}`);
+    }
+    console.log(`\nUse yarn "manage:translations -- --help" for usage information\n`);
+    process.exit(1);
+  }
+}
+
+const printHelpMessages = function () {
+  console.log(
+`Usage: yarn manage:translations -- [OPTIONS] [LANGUAGES]
+
+Manage javascript translation files in mastodon. Generates and update
+translations in translationsDirectory: ${translationsDirectory}
+
+OPTIONS
+  -h,--help    show this message
+  -f,--force   force using the provided languages. create files if not exists.
+               default: false
+
+LANGUAGES
+The RFC5646 language tag for the language you want to test or fix. If you want
+to input multiple languages, separate them with space.
+
+Available languages:
+${availableLanguages}
+`);
+}
+
+// parse arguments
+const argv = require('minimist')(process.argv.slice(2), {
+  'boolean': [
+    'force',
+    'help'
+  ],
+  'alias': {
+    'f': 'force',
+    'h': 'help',
+  },
+});
 const translationsDirectory = 'app/javascript/mastodon/locales';
+const messagesDirectory = 'build/messages';
 const localeFn = /^([a-z]{2,3}(|\-[A-Z]+))\.json$/;
 const reRFC5646 = /^[a-z]{2,3}(|\-[A-Z]+)$/;
 const availableLanguages = fs.readdirSync(`${process.cwd()}/${translationsDirectory}`).reduce((acc, fn) => {
@@ -14,69 +87,30 @@ const availableLanguages = fs.readdirSync(`${process.cwd()}/${translationsDirect
 }, []);
 
 // print help message
-if (argv.help !== undefined) {
-  console.log(
-`Usage: yarn manage:translations -- [OPTIONS] [LANGUAGES]
-
-Manage javascript translation files in mastodon. Generates and update
-translations in translationsDirectory: ${translationsDirectory}
-
-OPTIONS
-  --help    show this message
-  --force   force using the provided languages. create files if not exists.
-            default: false
-
-LANGUAGES
-The RFC5646 language tag for the language you want to test or fix. If you want
-to input multiple languages, separate them with space.
-
-Available languages:
-${availableLanguages}
-`);
+if (argv.help) {
+  printHelpMessages();
   process.exit(0);
 }
 
+// check if message directory exists
+if (!fs.existsSync(`${process.cwd()}/${messagesDirectory}`)) {
+  console.error(`\nError: messageDirectory not exists\n(${process.cwd()}/${messagesDirectory})\n`);
+  console.error(`Try to run "yarn build:development" first`);
+  process.exit(1);
+}
+
 // determine the languages list
 const languages = (argv._.length === 0) ? availableLanguages : argv._;
 
-// check if the languages provided are RFC5626 compliant
-(function() {
-  let invalidLanguages = languages.reduce((acc, language) => {
-    if (!language.match(reRFC5646)) {
-      acc.push(language);
-    }
-    return acc;
-  }, []);
-  if (invalidLanguages.length > 0) {
-    console.log(`Error:`);
-    for (let language of invalidLanguages) {
-      console.error(`* Not RFC5626 name: ${language}`);
-    }
-    console.log(`\nUse yarn "manage:translations -- --help" for usage information\n`);
-    process.exit(1);
-  }
-})();
-
-// make sure the language exists. Unless force to create locale file.
-if (argv.force !== true) {
-  let invalidLanguages = languages.reduce((acc, language) => {
-    if (availableLanguages.indexOf(language) < 0) {
-      acc.push(language);
-    }
-    return acc;
-  }, []);
-  if (invalidLanguages.length > 0) {
-    console.log(`Error:`);
-    for (let language of invalidLanguages) {
-      console.error(`* Language not available: ${language}`);
-    }
-    console.log(`\nIf you want to force creating the language(s) above, please add the --force option.\n`);
-    process.exit(1);
-  }
-}
+// validate languages
+validateLanguages(languages, [
+  testRFC5626(reRFC5646),
+  testAvailability(availableLanguages),
+]);
 
+// manage translations
 manageTranslations({
-  messagesDirectory: 'build/messages',
+  messagesDirectory,
   translationsDirectory,
   detectDuplicateIds: false,
   singleMessagesFile: true,