-
Notifications
You must be signed in to change notification settings - Fork 783
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
[DSC] Add Zimone's Hypothesis #13181
base: master
Are you sure you want to change the base?
Conversation
Not sure why the build failed on the check, it did work on my computer and was able to test it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Pull Request contains the changes to implement Zimone's Hypothesis, as well as a new predicate to filter creatures by odd/even power
|
||
// You may put a +1/+1 counter on a creature. | ||
this.getSpellAbility().addEffect(new AddCountersTargetEffect(CounterType.P1P1.createInstance())); | ||
this.getSpellAbility().addTarget(new TargetCreaturePermanent(0, 1, StaticFilters.FILTER_PERMANENT_CREATURE, true)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately, this construction doesn't work as you might hope. Despite the not-target param, it's still asking for a target on cast. Since it's not targeting, that must happen only on resolution.
What I typically see for this is a custom effect - see Titan of Industry as a reference and add a player.chooseUse
to account for the "you may"
public final class ZimonesHypothesis extends CardImpl { | ||
|
||
public ZimonesHypothesis(UUID ownerId, CardSetInfo setInfo) { | ||
super(ownerId, setInfo, new CardType[]{CardType.SORCERY}, "{3}{U}{U}"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the reason for the verify failure - this card is an Instant, not a Sorcery
Implements Zimone's Hypothesis. First time implementing a new card, but tested and it worked for me. Based most of the code off of the pre-existing Extinction Event Implementation. Additionally created new predicate for checking parity on creature power; currently only used on this card but generic enough that it could be useful in the future.