Add error handling for remove command
This commit is contained in:
parent
dcbd275c18
commit
b342a5df75
@ -71,10 +71,21 @@ func removeCards(cards []string, count int64) error {
|
|||||||
|
|
||||||
// Loop over different cards
|
// Loop over different cards
|
||||||
for _, card := range cards {
|
for _, card := range cards {
|
||||||
|
|
||||||
|
if !strings.Contains(card, "/") {
|
||||||
|
l.Errorf("Invalid card format %s. Needs to be set/collector number i.e. \"usg/13\"", card)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
// Extract collector number and set name from input & remove leading zeros
|
// Extract collector number and set name from input & remove leading zeros
|
||||||
collectorNumber := strings.TrimLeft(strings.Split(card, "/")[1], "0")
|
collectorNumber := strings.TrimLeft(strings.Split(card, "/")[1], "0")
|
||||||
setName := strings.Split(card, "/")[0]
|
setName := strings.Split(card, "/")[0]
|
||||||
|
|
||||||
|
if collectorNumber == "" {
|
||||||
|
l.Errorf("Invalid card format %s. Needs to be set/collector number i.e. \"usg/13\"", card)
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
// Fetch card from scryfall
|
// Fetch card from scryfall
|
||||||
c, err := findCardByCollectorNumber(coll, setName, collectorNumber)
|
c, err := findCardByCollectorNumber(coll, setName, collectorNumber)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user