From 3f33cac6eb6fd5240a8af62eb757487ec3decdb0 Mon Sep 17 00:00:00 2001 From: Florent Gluck <florent.gluck@hesge.ch> Date: Thu, 24 Nov 2022 11:24:33 +0100 Subject: [PATCH] minor changes --- src/cmdTemplate/templateCreate.go | 2 +- src/cmdUser/userAdd.go | 4 ++-- src/cmdUser/userDel.go | 4 ++-- src/cmdUser/userSetCaps.go | 4 ++-- src/cmdUser/userUpdatePwd.go | 2 +- src/cmdUser/userWhoami.go | 2 +- src/cmdVM/{vmSetAccess.go => vmAddAccess.go} | 14 +++++++------- src/cmdVM/vmCreate.go | 2 +- src/cmdVersion/version.go | 2 +- src/nexus-cli/nexus-cli.go | 2 +- src/nexush/nexush.go | 2 +- 11 files changed, 20 insertions(+), 20 deletions(-) rename src/cmdVM/{vmSetAccess.go => vmAddAccess.go} (91%) diff --git a/src/cmdTemplate/templateCreate.go b/src/cmdTemplate/templateCreate.go index c1deef0..cb18761 100644 --- a/src/cmdTemplate/templateCreate.go +++ b/src/cmdTemplate/templateCreate.go @@ -25,7 +25,7 @@ func (cmd *Create)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name+" ID name access") + u.PrintlnErr("USAGE: "+cmd.GetName()+" ID name access") u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") u.PrintlnErr("ID ID of the VM used to create the template.") u.PrintlnErr("name Name of the template to create.") diff --git a/src/cmdUser/userAdd.go b/src/cmdUser/userAdd.go index 7263ef8..cb2756e 100644 --- a/src/cmdUser/userAdd.go +++ b/src/cmdUser/userAdd.go @@ -29,8 +29,8 @@ func (cmd *Add)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name+" email firstname lastname password [capability ...]") - u.PrintlnErr(" "+cmd.Name+" file.csv") + u.PrintlnErr("USAGE: "+cmd.GetName()+" email firstname lastname password [capability ...]") + u.PrintlnErr(" "+cmd.GetName()+" file.csv") u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") const usage string = `file.csv 5-column CSV file defining each new user: column1 email diff --git a/src/cmdUser/userDel.go b/src/cmdUser/userDel.go index a759906..fc71614 100644 --- a/src/cmdUser/userDel.go +++ b/src/cmdUser/userDel.go @@ -28,8 +28,8 @@ func (cmd *Del)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name+" email [email ...]") - u.PrintlnErr(" "+cmd.Name+" file.csv") + u.PrintlnErr("USAGE: "+cmd.GetName()+" email [email ...]") + u.PrintlnErr(" "+cmd.GetName()+" file.csv") u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") u.PrintlnErr("file.csv 1-column CSV file containing emails of users to delete.") } diff --git a/src/cmdUser/userSetCaps.go b/src/cmdUser/userSetCaps.go index 4da91f1..cbd749e 100644 --- a/src/cmdUser/userSetCaps.go +++ b/src/cmdUser/userSetCaps.go @@ -29,8 +29,8 @@ func (cmd *SetCaps)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name+" email [capability ...]") - u.PrintlnErr(" "+cmd.Name+" file.csv") + u.PrintlnErr("USAGE: "+cmd.GetName()+" email [capability ...]") + u.PrintlnErr(" "+cmd.GetName()+" file.csv") u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") const usage string = `file.csv 2-column CSV file defining each user's capabilities: column1 email diff --git a/src/cmdUser/userUpdatePwd.go b/src/cmdUser/userUpdatePwd.go index dfd1c77..3c75d18 100644 --- a/src/cmdUser/userUpdatePwd.go +++ b/src/cmdUser/userUpdatePwd.go @@ -28,7 +28,7 @@ func (cmd *UpdatePwd)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name) + u.PrintlnErr("USAGE: "+cmd.GetName()) u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") } diff --git a/src/cmdUser/userWhoami.go b/src/cmdUser/userWhoami.go index 16143ed..9be1062 100644 --- a/src/cmdUser/userWhoami.go +++ b/src/cmdUser/userWhoami.go @@ -23,7 +23,7 @@ func (cmd *Whoami)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name) + u.PrintlnErr("USAGE: "+cmd.GetName()) u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") } diff --git a/src/cmdVM/vmSetAccess.go b/src/cmdVM/vmAddAccess.go similarity index 91% rename from src/cmdVM/vmSetAccess.go rename to src/cmdVM/vmAddAccess.go index f59bcb1..f4ba9fb 100644 --- a/src/cmdVM/vmSetAccess.go +++ b/src/cmdVM/vmAddAccess.go @@ -7,21 +7,21 @@ import ( g "nexus-client/globals" ) -type SetAccess struct { +type AddAccess struct { Name string } -func (cmd *SetAccess)GetName() string { +func (cmd *AddAccess)GetName() string { return cmd.Name } -func (cmd *SetAccess)GetDesc() []string { +func (cmd *AddAccess)GetDesc() []string { return []string{ "Adds a user's VM access in one or more VMs.", "Requires VM_SET_ACCESS user capability and VM_SET_ACCESS VM access capability."} } -func (cmd *SetAccess)PrintUsage() { +func (cmd *AddAccess)PrintUsage() { for _, desc := range cmd.GetDesc() { u.PrintlnErr(desc) } @@ -31,12 +31,12 @@ func (cmd *SetAccess)PrintUsage() { printRegexUsageDetails() } -func (cmd *SetAccess)Run(args []string) int { +func (cmd *AddAccess)Run(args []string) int { client := g.GetInstance().Client host := g.GetInstance().Host argc := len(args) - if argc < 1 { + if argc < 2 { cmd.PrintUsage() return 1 } @@ -88,7 +88,7 @@ func (cmd *SetAccess)Run(args []string) int { return statusCode } -func (cmd *SetAccess)parseArgs(args []string) (string, []string, []string, error) { +func (cmd *AddAccess)parseArgs(args []string) (string, []string, []string, error) { var patterns []string var capabilities []string emailFound := false diff --git a/src/cmdVM/vmCreate.go b/src/cmdVM/vmCreate.go index 80bd260..892e109 100644 --- a/src/cmdVM/vmCreate.go +++ b/src/cmdVM/vmCreate.go @@ -26,7 +26,7 @@ func (cmd *Create)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name+" name cpus ram nic template [count|file.csv]") + u.PrintlnErr("USAGE: "+cmd.GetName()+" name cpus ram nic template [count|file.csv]") u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") const usage string = `name Name of the VM to create. cpus Number of CPUs, between 1 and 16. diff --git a/src/cmdVersion/version.go b/src/cmdVersion/version.go index 15538c1..ce885ea 100644 --- a/src/cmdVersion/version.go +++ b/src/cmdVersion/version.go @@ -27,7 +27,7 @@ func (cmd *Version)PrintUsage() { u.PrintlnErr(desc) } u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") - u.PrintlnErr("USAGE: "+cmd.Name) + u.PrintlnErr("USAGE: "+cmd.GetName()) u.PrintlnErr("―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――") } diff --git a/src/nexus-cli/nexus-cli.go b/src/nexus-cli/nexus-cli.go index d3bc4e1..5e3bcb5 100644 --- a/src/nexus-cli/nexus-cli.go +++ b/src/nexus-cli/nexus-cli.go @@ -40,7 +40,7 @@ var cmdList = []cmd.Command { &cmdVM.Create{"vmcreate"}, &cmdVM.Edit{"vmedit"}, &cmdVM.Del{"vmdel"}, - &cmdVM.SetAccess{"vmaddaccess"}, + &cmdVM.AddAccess{"vmaddaccess"}, &cmdVM.DelAccess{"vmdelaccess"}, &cmdVM.ExportDir{"vmexportdir"}, &cmdVM.ImportDir{"vmimportdir"}, diff --git a/src/nexush/nexush.go b/src/nexush/nexush.go index 1b0ced3..a6f829d 100644 --- a/src/nexush/nexush.go +++ b/src/nexush/nexush.go @@ -48,7 +48,7 @@ var cmdList = []cmd.Command { &cmdVM.Create{"vmcreate"}, &cmdVM.Edit{"vmedit"}, &cmdVM.Del{"vmdel"}, - &cmdVM.SetAccess{"vmaddaccess"}, + &cmdVM.AddAccess{"vmaddaccess"}, &cmdVM.DelAccess{"vmdelaccess"}, &cmdVM.ExportDir{"vmexportdir"}, &cmdVM.ImportDir{"vmimportdir"}, -- GitLab