From cf0b7532097e7a59d59ea71a89ba39f8a4ab62ad Mon Sep 17 00:00:00 2001 From: Koala Yeung Date: Sat, 6 May 2017 10:05:38 +0800 Subject: Streamline javascript translation by improving translationRunner (#2808) * package.json: Add "build:*" targets * Improve react-intl-translations-manager workflow. * Added "build:production" to build production bundle. * Added "build:development" to build development bundle. * Fix json translation files * Run `yarn manage:translations` to fix translation files. * Fix `pl.json` for syntax error. * translationRunner: auto detect existing languages * Auto detect existing rfc5646 language tag in *.json filenames in `app/javascript/mastodon/locale` folder. No need to manually define every new language in the languages array here. * translationRunner: add more functionality * Allow script user to specify language code to check. * Added available language check. * Added --force flag to force creation of unexists language. * Added --help flag and help messages. * gitignore: ignore npm-debug.log * Fix webpack error if NODE_ENV is not defined Default to use 'development' in config/webpack/configuration.js --- config/webpack/translationRunner.js | 106 ++++++++++++++++++++++++++---------- 1 file changed, 78 insertions(+), 28 deletions(-) (limited to 'config/webpack/translationRunner.js') diff --git a/config/webpack/translationRunner.js b/config/webpack/translationRunner.js index c636170b9..937c2edd0 100644 --- a/config/webpack/translationRunner.js +++ b/config/webpack/translationRunner.js @@ -1,34 +1,84 @@ +/*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 translationsDirectory = 'app/javascript/mastodon/locales'; +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) => { + if (fn.match(localeFn)) { + acc.push(fn.replace(localeFn, '$1')); + } + return acc; +}, []); + +// 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} +`); + process.exit(0); +} + +// 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); + } +} manageTranslations({ messagesDirectory: 'build/messages', - translationsDirectory: 'app/javascript/mastodon/locales/', + translationsDirectory, detectDuplicateIds: false, singleMessagesFile: true, - languages: [ - 'ar', - 'en', - 'de', - 'es', - 'fa', - 'hr', - 'hu', - 'io', - 'it', - 'fr', - 'nl', - 'no', - 'oc', - 'pt', - 'pt-BR', - 'uk', - 'fi', - 'eo', - 'ru', - 'ja', - 'zh-HK', - 'zh-CN', - 'bg', - 'id', - ], -}) + languages, +}); -- cgit