Rarity query #210
Rarity query #210
Conversation
| - name: owner | ||
| in: query | ||
| required: false | ||
| type: string | ||
| - name: cardType | ||
| - name: statuses |
There was a problem hiding this comment.
why plural? a card can only have 1 status right?
There was a problem hiding this comment.
yes, but dont u query a list of cards?
There was a problem hiding this comment.
Eh yes, but you can only specify 1 status? So what you say is like "Give me all cards with status "permanent".
Example from real life:
You want to buy 4 apples, the apples are plural, then the nice lady in the marketplace asks you "Which sort of apples do you want?", then you yell "BOSKOP". The lady does not ask "Which sorts of apples do you want?"
| @@ -14,27 +14,83 @@ var _ = strconv.Itoa(0) | |||
|
|
|||
| func CmdQCards() *cobra.Command { | |||
| cmd := &cobra.Command{ | |||
| Use: "q-cards [owner] [status] [card-type] [classes] [sort-by] [name-contains] [keywords-contains] [notes-contains] [startercards-only] [balance-achors-only]", | |||
| Use: "q-cards [owner] [statuses] [card-types] [classes] [rarities] [sort-by] [name-contains] [keywords-contains] [notes-contains] [startercards-only] [balance-achors-only]", | |||
There was a problem hiding this comment.
same question for card-type? A card can only have 1 type, so why is it plural?
| default: | ||
| states = []types.Status{types.Status(types.Status_value[req.Status.String()])} | ||
| } | ||
| fmt.Printf("---> Hier: %#v\n", req) |
There was a problem hiding this comment.
debug print important for production?
| if bool(cardobjClass.Culture) != strings.Contains(req.Classes, "Culture") { | ||
| return false | ||
| } | ||
| fmt.Printf("%v, %v\n", cardobjClass, req.Classes) |
No description provided.