From 8a3d565c8e13cf30c48958b9dcc8142feee39322 Mon Sep 17 00:00:00 2001
From: gfyoung <gfyoung17@gmail.com>
Date: Sun, 19 Aug 2018 00:16:05 -0700
Subject: [PATCH] Don't use arguments keyword in gettext script

Causing SAST build to fail unnecessarily.

Follow-up to !19378.
---
 changelogs/unreleased/arguments-keyword-sast.yml |  5 +++++
 scripts/frontend/extract_gettext_all.js          | 10 +++++-----
 2 files changed, 10 insertions(+), 5 deletions(-)
 create mode 100644 changelogs/unreleased/arguments-keyword-sast.yml

diff --git a/changelogs/unreleased/arguments-keyword-sast.yml b/changelogs/unreleased/arguments-keyword-sast.yml
new file mode 100644
index 00000000000..2ecbc5e8174
--- /dev/null
+++ b/changelogs/unreleased/arguments-keyword-sast.yml
@@ -0,0 +1,5 @@
+---
+title: Don't use arguments keyword in gettext script
+merge_request: 21296
+author: gfyoung
+type: fixed
diff --git a/scripts/frontend/extract_gettext_all.js b/scripts/frontend/extract_gettext_all.js
index af8cc4c3341..725522a3540 100644
--- a/scripts/frontend/extract_gettext_all.js
+++ b/scripts/frontend/extract_gettext_all.js
@@ -7,7 +7,7 @@ const {
 } = require('gettext-extractor-vue');
 const ensureSingleLine = require('../../app/assets/javascripts/locale/ensure_single_line.js');
 
-const arguments = argumentsParser
+const args = argumentsParser
   .option('-f, --file <file>', 'Extract message from one single file')
   .option('-a, --all', 'Extract message from all js/vue files')
   .parse(process.argv);
@@ -61,12 +61,12 @@ function printJson() {
   console.log(JSON.stringify(messages));
 }
 
-if (arguments.file) {
-  vueParser.parseFile(arguments.file).then(() => printJson());
-} else if (arguments.all) {
+if (args.file) {
+  vueParser.parseFile(args.file).then(() => printJson());
+} else if (args.all) {
   vueParser.parseFilesGlob('{ee/app,app}/assets/javascripts/**/*.{js,vue}').then(() => printJson());
 } else {
   console.warn('ERROR: Please use the script correctly:');
-  arguments.outputHelp();
+  args.outputHelp();
   process.exit(1);
 }