From 63626b1267e77ad792f426b4e0a8e4efac4b2c95 Mon Sep 17 00:00:00 2001 From: Erona Date: Mon, 29 Oct 2018 22:19:39 +0800 Subject: refactor(bin): eliminate `var` and use template string refactor string things Signed-off-by: Erona --- bin/manage_users | 31 ++++++++++++++++--------------- 1 file changed, 16 insertions(+), 15 deletions(-) (limited to 'bin') diff --git a/bin/manage_users b/bin/manage_users index 30b4632d..f748e342 100755 --- a/bin/manage_users +++ b/bin/manage_users @@ -24,43 +24,44 @@ Usage: bin/manage_users [--pass password] (--add | --del) user-email // Using an async function to be able to use await inside async function createUser(argv) { - var existing_user = await models.User.findOne({where: {email: argv["add"]}}); + const existing_user = await models.User.findOne({where: {email: argv["add"]}}); // Cannot create already-existing users if(existing_user != undefined) { - console.log("User with e-mail "+existing_user.email+" already exists! Aborting ..."); + console.log(`User with e-mail ${existing_user.email} already exists! Aborting ...`); process.exit(1); } // Find whether we use cmdline or prompt password + let pass; if(argv["pass"] == undefined) { - var pass = readline.question("Password for "+argv["add"]+":", {hideEchoBack: true}); + pass = readline.question(`Password for ${argv["add"]}:`, {hideEchoBack: true}); } else { console.log("Using password from commandline..."); - var pass = "" + argv["pass"]; + pass = "" + argv["pass"]; } // Lets try to create, and check success - var ref = await models.User.create({email: argv["add"], password: pass}); + const ref = await models.User.create({email: argv["add"], password: pass}); if(ref == undefined) { - console.log("Could not create user with email "+argv["add"]); + console.log(`Could not create user with email ${argv["add"]}`); process.exit(1); } else - console.log("Created user with email "+argv["add"]); + console.log(`Created user with email ${argv["add"]}`); } // Using an async function to be able to use await inside async function deleteUser(argv) { // Cannot delete non-existing users - var existing_user = await models.User.findOne({where: {email: argv["del"]}}); - if(existing_user == undefined) { - console.log("User with e-mail "+argv["del"]+" does not exist, cannot delete"); + const existing_user = await models.User.findOne({where: {email: argv["del"]}}); + if(existing_user === undefined) { + console.log(`User with e-mail ${argv["del"]} does not exist, cannot delete`); process.exit(1); } // Sadly .destroy() does not return any success value with all // backends. See sequelize #4124 await existing_user.destroy(); - console.log("Deleted user "+argv["del"]+" ..."); + console.log(`Deleted user ${argv["del"]} ...`); } var options = { @@ -69,11 +70,11 @@ var options = { }; // Perform commandline-parsing -var argv = minimist(process.argv.slice(2)); +const argv = minimist(process.argv.slice(2)); -var keys = Object.keys(options); -var opts = keys.filter((key) => argv[key] !== undefined); -var action = opts[0]; +const keys = Object.keys(options); +const opts = keys.filter((key) => argv[key] !== undefined); +const action = opts[0]; // Check for options missing if (opts.length === 0) { -- cgit v1.2.3