Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Oracle: skip cards with a star in the collectors' number #3706

Merged
merged 2 commits into from
May 5, 2019

Conversation

ctrlaltca
Copy link
Contributor

Related Ticket(s)

Short roundup of the initial problem

WAR contains alternate printings of cards that are only available in a specific language.
These localized printings are currently being shown instead of the english ones.
All of these printings contains a star character ★ in their collectors' number.

What will change with this Pull Request?

Oracle will skip importing all hr cards with a ★ in the collectors' number.

@ZeldaZach
Copy link
Member

Can also skip isAlternative=true cards. The MTGJSON weekly re-build (which is happening now) added this back in for WAR

Copy link
Member

@ebbit1q ebbit1q left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we need this. People are mad.

@ctrlaltca ctrlaltca merged commit 8f5bfd1 into Cockatrice:master May 5, 2019
@ctrlaltca ctrlaltca deleted the skip_star branch May 5, 2019 20:38
ebbit1q added a commit to ebbit1q/Cockatrice that referenced this pull request Mar 1, 2020
this will correct Cockatrice#3706 and Cockatrice#3715 being a bit overzealous in removing
cards and thus fix scheme cards being removed in entirety
fix Cockatrice#3845
note that this causes a lot more cards to be added that are in promo
sets, if these promo sets should prove to be problematic they should be
disabled somehow as having them as an option is still nice.
ZeldaZach pushed a commit that referenced this pull request Mar 17, 2020
* ignore stars and promos but only sometimes

this will correct #3706 and #3715 being a bit overzealous in removing
cards and thus fix scheme cards being removed in entirety
fix #3845
note that this causes a lot more cards to be added that are in promo
sets, if these promo sets should prove to be problematic they should be
disabled somehow as having them as an option is still nice.

* remove debug lines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

War of the Spark (WAR) planeswalkers only load the Japanese alternative art image
4 participants