Bug fix: warning was shown when invoking jdenticon with command line …

…options `-v`/`--version` or `-h`/`--help`.
dmester · Jul 30, 2020 · 5c62467 · 5c62467
1 parent 76aa0ca
commit 5c62467
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/bin/jdenticon.js b/bin/jdenticon.js
@@ -71,9 +71,16 @@ function parseArgs(args) {
                 var alias = aliases[aliasIndex];
 
                 if (arg === alias) {
-                    var value = hasValue ? args[argIndex + 1] : true;
-                    if (argIndex + 1 >= args.length) {
-                        console.warn("WARN Missing value of argument " + alias);
+                    var value;
+
+                    if (hasValue) {
+                        if (argIndex + 1 < args.length) {
+                            value = args[argIndex + 1];
+                        } else {
+                            console.warn("WARN Missing value of argument " + alias);
+                        }
+                    } else {
+                        value = true;
                     }
 
                     args.splice(argIndex, hasValue ? 2 : 1);