Fix pin and secret arguments requirements

Author: replydev

src/args.rs

@@ -137,23 +137,23 @@ pub fn get_matches() -> ArgMatches {
                         .long("issuer")
                         .help("OTP Code issuer")
                         .num_args(1)
-                        .required_unless_present_any(["label", "algorithm", "digits", "counter"]),
+                        .required_unless_present_any(["label", "algorithm", "digits", "counter", "pin", "change-secret"]),
                 )
                 .arg(
                     Arg::new("label")
                         .short('l')
                         .long("label")
                         .help("OTP Code label")
                         .num_args(1)
-                        .required_unless_present_any(["issuer", "algorithm", "digits", "counter"]),
+                        .required_unless_present_any(["issuer", "algorithm", "digits", "counter", "pin", "change-secret"]),
                 )
                 .arg(
                     Arg::new("algorithm")
                         .short('a')
                         .long("algorithm")
                         .help("OTP Code algorithm")
                         .num_args(1)
-                        .required_unless_present_any(["label", "issuer", "digits", "counter"])
+                        .required_unless_present_any(["label", "issuer", "digits", "counter", "pin", "change-secret"])
                         .value_parser(["SHA1", "SHA256", "SHA512"]),
                 )
                 .arg(
@@ -163,7 +163,7 @@ pub fn get_matches() -> ArgMatches {
                         .help("OTP Code digits")
                         .num_args(1)
                         .value_parser(value_parser!(u64))
-                        .required_unless_present_any(["label", "algorithm", "issuer", "counter"]),
+                        .required_unless_present_any(["label", "algorithm", "issuer", "counter", "pin", "change-secret"]),
                 )
                 .arg(
                     Arg::new("period")
@@ -172,7 +172,7 @@ pub fn get_matches() -> ArgMatches {
                         .help("OTP Code period")
                         .num_args(1)
                         .value_parser(value_parser!(u64))
-                        .required_unless_present_any(["label", "algorithm", "issuer", "counter"]),
+                        .required_unless_present_any(["label", "algorithm", "issuer", "counter", "pin", "change-secret"]),
                 )
                 .arg(
                     Arg::new("counter")
@@ -181,21 +181,23 @@ pub fn get_matches() -> ArgMatches {
                         .help("HOTP code counter (only for HOTP codes)")
                         .num_args(1)
                         .value_parser(value_parser!(u64))
-                        .required_unless_present_any(["label", "algorithm", "issuer", "digits"]),
+                        .required_unless_present_any(["label", "algorithm", "issuer", "digits", "pin", "change-secret"]),
                 )
                 .arg(
                     Arg::new("pin")
                         .short('p')
                         .long("pin")
                         .help("Code pin (for Yandex and MOTP)")
                         .num_args(1)
+                        .required_unless_present_any(["label", "algorithm", "issuer", "digits", "counter", "change-secret"]),
                 )
                 .arg(
                     Arg::new("change-secret")
                         .short('k')
                         .long("change-secret")
                         .help("Change the OTP code secret")
-                        .action(ArgAction::SetTrue),
+                        .action(ArgAction::SetTrue)
+                        .required_unless_present_any(["label", "algorithm", "issuer", "digits", "counter", "pin"]),
                 ),
         )
         .subcommand(