sort valid flags
This commit is contained in:
@@ -17,6 +17,7 @@ package cmd
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"errors"
|
"errors"
|
||||||
|
"sort"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
||||||
@@ -56,6 +57,7 @@ func (t *leagueFlag) Set(v string) error {
|
|||||||
_, ok := leagueIDs[v]
|
_, ok := leagueIDs[v]
|
||||||
if !ok {
|
if !ok {
|
||||||
allowed := allowedFlags(leagueIDs)
|
allowed := allowedFlags(leagueIDs)
|
||||||
|
sort.Strings(allowed)
|
||||||
allowedList := strings.Join(allowed, ", ")
|
allowedList := strings.Join(allowed, ", ")
|
||||||
err = errors.New("league must be one of " + allowedList)
|
err = errors.New("league must be one of " + allowedList)
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
@@ -17,6 +17,7 @@ package cmd
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"errors"
|
"errors"
|
||||||
|
"sort"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
||||||
@@ -61,6 +62,7 @@ func (t *sportFlag) Set(v string) error {
|
|||||||
_, ok := sportIDs[v]
|
_, ok := sportIDs[v]
|
||||||
if !ok {
|
if !ok {
|
||||||
allowed := allowedFlags(sportIDs)
|
allowed := allowedFlags(sportIDs)
|
||||||
|
sort.Strings(allowed)
|
||||||
allowedList := strings.Join(allowed, ", ")
|
allowedList := strings.Join(allowed, ", ")
|
||||||
err = errors.New("sport must be one of " + allowedList)
|
err = errors.New("sport must be one of " + allowedList)
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
@@ -17,6 +17,7 @@ package cmd
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"errors"
|
"errors"
|
||||||
|
"sort"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
"scm.dairydemon.net/filifa/mlblive/cmd/internal/statsapi"
|
||||||
@@ -85,6 +86,7 @@ func (t *teamFlag) Set(v string) error {
|
|||||||
_, ok := teamIDs[v]
|
_, ok := teamIDs[v]
|
||||||
if !ok {
|
if !ok {
|
||||||
allowed := allowedFlags(teamIDs)
|
allowed := allowedFlags(teamIDs)
|
||||||
|
sort.Strings(allowed)
|
||||||
allowedList := strings.Join(allowed, ", ")
|
allowedList := strings.Join(allowed, ", ")
|
||||||
err = errors.New("team must be one of " + allowedList)
|
err = errors.New("team must be one of " + allowedList)
|
||||||
} else {
|
} else {
|
||||||
|
|||||||
Reference in New Issue
Block a user