From 4a263a429c9476bf126698a4c6f75c2eec4a131f Mon Sep 17 00:00:00 2001 From: Eric Mann Date: Thu, 12 Oct 2017 11:04:07 -0700 Subject: [PATCH] Simplify the CLI delete command. Props @gstro --- cmd/e3db/main.go | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/cmd/e3db/main.go b/cmd/e3db/main.go index 71481fc..3383ec3 100644 --- a/cmd/e3db/main.go +++ b/cmd/e3db/main.go @@ -342,22 +342,26 @@ func cmdReadFile(cmd *cli.Cmd) { } func cmdDelete(cmd *cli.Cmd) { - recordIDs := cmd.Strings(cli.StringsArg{ + recordID := cmd.String(cli.StringArg{ Name: "RECORD_ID", - Desc: "record IDs to delete", - Value: nil, + Desc: "record ID to delete", + Value: "", + HideValue: true, + }) + + version := cmd.String(cli.StringArg{ + Name: "VERSION", + Desc: "version ID of the record to delete", + Value: "", HideValue: true, }) - cmd.Spec = "RECORD_ID..." cmd.Action = func() { client := options.getClient() - for _, recordID := range *recordIDs { - err := client.Delete(context.Background(), recordID, "") - if err != nil { - dieErr(err) - } + err := client.Delete(context.Background(), *recordID, *version) + if err != nil { + dieErr(err) } } }