[Minor] Rename cli flag folder to email-folder (#343)

rename cli flag mail to email
This commit is contained in:
Aarti Jivrajani 2022-07-19 10:21:30 -07:00 committed by GitHub
parent d00332f328
commit 7f60c00466
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -16,10 +16,10 @@ import (
// exchange bucket info from flags
var (
folder string
mail string
backupID string
user string
emailFolder string
email string
backupID string
user string
)
// called by restore.go to map parent subcommands to provider-specific handling.
@ -32,8 +32,8 @@ func addExchangeCommands(parent *cobra.Command) *cobra.Command {
switch parent.Use {
case restoreCommand:
c, fs = utils.AddCommand(parent, exchangeRestoreCmd)
fs.StringVar(&folder, "folder", "", "Name of the mail folder being restored")
fs.StringVar(&mail, "mail", "", "ID of the mail message being restored")
fs.StringVar(&emailFolder, "email-folder", "", "Name of the email folder being restored")
fs.StringVar(&email, "email", "", "ID of the email being restored")
fs.StringVar(&backupID, "backup", "", "ID of the backup to restore")
cobra.CheckErr(c.MarkFlagRequired("backup"))
fs.StringVar(&user, "user", "", "ID of the user whose exchange data will get restored")
@ -59,7 +59,7 @@ func restoreExchangeCmd(cmd *cobra.Command, args []string) error {
return nil
}
if err := validateRestoreFlags(user, folder, mail, backupID); err != nil {
if err := validateRestoreFlags(user, emailFolder, email, backupID); err != nil {
return errors.Wrap(err, "Missing required flags")
}
@ -86,7 +86,7 @@ func restoreExchangeCmd(cmd *cobra.Command, args []string) error {
}
defer utils.CloseRepo(ctx, r)
ro, err := r.NewRestore(ctx, backupID, exchangeRestoreSelectors(user, folder, mail))
ro, err := r.NewRestore(ctx, backupID, exchangeRestoreSelectors(user, emailFolder, email))
if err != nil {
return errors.Wrap(err, "Failed to initialize Exchange restore")
}
@ -127,11 +127,11 @@ func validateRestoreFlags(u, f, m, rpid string) error {
return errors.New("a restore point ID is requried")
}
lu, lf, lm := len(u), len(f), len(m)
if (lu == 0 || u == utils.Wildcard) && (lf+lm > 0) {
return errors.New("a specific --user must be provided if --folder or --mail is specified")
if (lu == 0 || u == "*") && (lf+lm > 0) {
return errors.New("a specific --user must be provided if --email-folder or --email is specified")
}
if (lf == 0 || f == utils.Wildcard) && lm > 0 {
return errors.New("a specific --folder must be provided if a --mail is specified")
if (lf == 0 || f == "*") && lm > 0 {
return errors.New("a specific --email-folder must be provided if a --email is specified")
}
return nil
}