Simplify the handling of EXCLUDE_LANG_CODES (PR 11213 follow-up)

This commit is contained in:
Jonas Jenwald 2019-10-26 13:49:37 +02:00
parent 4a5a4328f4
commit 8852991c60

View File

@ -46,6 +46,13 @@ function downloadLanguageCodes() {
response.on('end', function() {
content = content.trim(); // Remove any leading/trailing white-space.
var langCodes = normalizeText(content).split('\n');
// Remove all locales that we don't want to download below.
for (var langCode of [DEFAULT_LOCALE, ...EXCLUDE_LANG_CODES]) {
var i = langCodes.indexOf(langCode);
if (i > -1) {
langCodes.splice(i, 1);
}
}
resolve(langCodes);
});
} else {
@ -107,8 +114,7 @@ async function downloadL10n(root, callback) {
var langCodes = await downloadLanguageCodes();
for (var langCode of langCodes) {
if (!langCode || langCode === DEFAULT_LOCALE ||
EXCLUDE_LANG_CODES.includes(langCode)) {
if (!langCode) {
continue;
}
await downloadLanguageFiles(root, langCode);
@ -119,7 +125,7 @@ async function downloadL10n(root, callback) {
var dirPath = path.join(root, entry), stat = fs.lstatSync(dirPath);
if (stat.isDirectory() && entry !== DEFAULT_LOCALE &&
(!langCodes.includes(entry) || EXCLUDE_LANG_CODES.includes(entry))) {
!langCodes.includes(entry)) {
removeCodes.push(entry);
}
}