mtgjson / mtgsqlive

MTGJSON build scripts to generate alternative data formats
https://mtgjson.com
MIT License
54 stars 32 forks source link

sets.csv has empty sets #42

Closed ancestral closed 4 years ago

ancestral commented 4 years ago

MTGJSON Version: 4.6.1

Description of Bug:

sets.csv has sets with 0 baseSetSize.

omfgitsmark commented 4 years ago

I don't think this is a mtgsqlive issue since that's how it is in JSON as well. sets_baseSetSize_0

Maybe add a check for baseSetSize = 0 and if so set it to totalSetSize?

ZeldaZach commented 4 years ago

baseSetSize indicates how many cards are "expected" to be in a set, based on a card's numbering "XXX/YYY". If there is no YYY, then I can't establish an amount and mark it as 0.