Skip to content

Commit 2841102

Browse files
committed
download: add mutual exclusive flag check for exercise and uuid
1 parent 20a0ed8 commit 2841102

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

cmd/download.go

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -64,32 +64,32 @@ func runDownload(cfg config.Config, flags *pflag.FlagSet, args []string) error {
6464
if err != nil {
6565
return err
6666
}
67-
if uuid == "" && slug == "" {
67+
if uuid != "" && slug != "" || uuid == slug {
6868
return errors.New("need an --exercise name or a solution --uuid")
6969
}
7070

71-
param := "latest"
72-
if uuid != "" {
73-
param = uuid
74-
}
75-
url := fmt.Sprintf("%s/solutions/%s", usrCfg.GetString("apibaseurl"), param)
76-
77-
client, err := api.NewClient(usrCfg.GetString("token"), usrCfg.GetString("apibaseurl"))
71+
track, err := flags.GetString("track")
7872
if err != nil {
7973
return err
8074
}
8175

82-
req, err := client.NewRequest("GET", url, nil)
76+
team, err := flags.GetString("team")
8377
if err != nil {
8478
return err
8579
}
8680

87-
track, err := flags.GetString("track")
81+
param := "latest"
82+
if uuid != "" {
83+
param = uuid
84+
}
85+
url := fmt.Sprintf("%s/solutions/%s", usrCfg.GetString("apibaseurl"), param)
86+
87+
client, err := api.NewClient(usrCfg.GetString("token"), usrCfg.GetString("apibaseurl"))
8888
if err != nil {
8989
return err
9090
}
9191

92-
team, err := flags.GetString("team")
92+
req, err := client.NewRequest("GET", url, nil)
9393
if err != nil {
9494
return err
9595
}

0 commit comments

Comments
 (0)