From 8271686cb40e80d37e57a7c17249a87e8ea9fe68 Mon Sep 17 00:00:00 2001 From: Alexander Novotny Date: Fri, 12 Apr 2024 19:41:58 -0400 Subject: [PATCH] [PIP] Implement Shaun, Father of Synths (#12109) * Added method to set triggered abilities to optional * TokenCopy effect now copies permanentModifier * Implemented Shaun, Father of Synths * remove TODO * Made `setOptional` chainable --- .../src/mage/cards/s/ShaunFatherOfSynths.java | 81 +++++++++++++++++++ Mage.Sets/src/mage/sets/Fallout.java | 1 + .../java/mage/abilities/TriggeredAbility.java | 2 + .../mage/abilities/TriggeredAbilityImpl.java | 15 ++++ .../common/CreateTokenCopyTargetEffect.java | 3 +- 5 files changed, 101 insertions(+), 1 deletion(-) create mode 100644 Mage.Sets/src/mage/cards/s/ShaunFatherOfSynths.java diff --git a/Mage.Sets/src/mage/cards/s/ShaunFatherOfSynths.java b/Mage.Sets/src/mage/cards/s/ShaunFatherOfSynths.java new file mode 100644 index 00000000000..26fa9ecdd3b --- /dev/null +++ b/Mage.Sets/src/mage/cards/s/ShaunFatherOfSynths.java @@ -0,0 +1,81 @@ +package mage.cards.s; + +import java.util.UUID; + +import mage.MageInt; +import mage.abilities.TriggeredAbility; +import mage.abilities.common.AttacksWithCreaturesTriggeredAbility; +import mage.abilities.common.LeavesBattlefieldTriggeredAbility; +import mage.abilities.effects.Effect; +import mage.abilities.effects.common.CreateTokenCopyTargetEffect; +import mage.abilities.effects.common.ExileAllEffect; +import mage.cards.CardImpl; +import mage.cards.CardSetInfo; +import mage.constants.CardType; +import mage.constants.SubType; +import mage.constants.SuperType; +import mage.filter.common.FilterControlledCreaturePermanent; +import mage.filter.common.FilterControlledPermanent; +import mage.filter.predicate.mageobject.AnotherPredicate; +import mage.filter.predicate.permanent.AttackingPredicate; +import mage.filter.predicate.permanent.TokenPredicate; +import mage.target.TargetPermanent; + +/** + * + * @author alexander-novo + */ +public class ShaunFatherOfSynths extends CardImpl { + + private static final FilterControlledCreaturePermanent attackFilter = new FilterControlledCreaturePermanent( + "attacking legendary creature you control other than {this}"); + private static final FilterControlledPermanent exileFilter = new FilterControlledPermanent(SubType.SYNTH, + "Synth tokens you control"); + + static { + attackFilter.add(AnotherPredicate.instance); + attackFilter.add(AttackingPredicate.instance); + attackFilter.add(SuperType.LEGENDARY.getPredicate()); + + exileFilter.add(TokenPredicate.TRUE); + } + + public ShaunFatherOfSynths(UUID ownerId, CardSetInfo setInfo) { + super(ownerId, setInfo, new CardType[] { CardType.CREATURE }, "{3}{U}{R}"); + + this.supertype.add(SuperType.LEGENDARY); + this.subtype.add(SubType.HUMAN); + this.subtype.add(SubType.SCIENTIST); + this.power = new MageInt(3); + this.toughness = new MageInt(4); + + // Whenever you attack, you may create a tapped and attacking token that's a copy of target attacking legendary creature you control other than Shaun, except it's not legendary and it's a Synth artifact creature in addition to its other types. + Effect effect = new CreateTokenCopyTargetEffect( + null, null, + false, 1, true, true) + .setPermanentModifier((token) -> { + token.removeSuperType(SuperType.LEGENDARY); + token.addCardType(CardType.CREATURE); + token.addCardType(CardType.ARTIFACT); + token.addSubType(SubType.SYNTH); + }).setText( + "create a tapped and attacking token that's a copy of target attacking legendary creature you control other than Shaun, except it's not legendary and it's a Synth artifact creature in addition to its other types"); + TriggeredAbility ability = new AttacksWithCreaturesTriggeredAbility(effect, 1); + ability.addTarget(new TargetPermanent(attackFilter)); + ability.setOptional(); + this.addAbility(ability); + + // When Shaun leaves the battlefield, exile all Synth tokens you control. + this.addAbility(new LeavesBattlefieldTriggeredAbility(new ExileAllEffect(exileFilter), false)); + } + + private ShaunFatherOfSynths(final ShaunFatherOfSynths card) { + super(card); + } + + @Override + public ShaunFatherOfSynths copy() { + return new ShaunFatherOfSynths(this); + } + +} \ No newline at end of file diff --git a/Mage.Sets/src/mage/sets/Fallout.java b/Mage.Sets/src/mage/sets/Fallout.java index dad3bf0c8c3..96bc55ea63c 100644 --- a/Mage.Sets/src/mage/sets/Fallout.java +++ b/Mage.Sets/src/mage/sets/Fallout.java @@ -268,6 +268,7 @@ public final class Fallout extends ExpansionSet { cards.add(new SetCardInfo("Sentry Bot", 552, Rarity.RARE, mage.cards.s.SentryBot.class, NON_FULL_USE_VARIOUS)); cards.add(new SetCardInfo("Sentry Bot", 899, Rarity.RARE, mage.cards.s.SentryBot.class, NON_FULL_USE_VARIOUS)); cards.add(new SetCardInfo("Shadowblood Ridge", 288, Rarity.RARE, mage.cards.s.ShadowbloodRidge.class)); + cards.add(new SetCardInfo("Shaun, Father of Synths", 119, Rarity.RARE, mage.cards.s.ShaunFatherOfSynths.class)); cards.add(new SetCardInfo("Sheltered Thicket", 289, Rarity.RARE, mage.cards.s.ShelteredThicket.class)); cards.add(new SetCardInfo("Sierra, Nuka's Biggest Fan", 25, Rarity.RARE, mage.cards.s.SierraNukasBiggestFan.class, NON_FULL_USE_VARIOUS)); cards.add(new SetCardInfo("Sierra, Nuka's Biggest Fan", 372, Rarity.RARE, mage.cards.s.SierraNukasBiggestFan.class, NON_FULL_USE_VARIOUS)); diff --git a/Mage/src/main/java/mage/abilities/TriggeredAbility.java b/Mage/src/main/java/mage/abilities/TriggeredAbility.java index b9a074a477d..92d0f3dcdd7 100644 --- a/Mage/src/main/java/mage/abilities/TriggeredAbility.java +++ b/Mage/src/main/java/mage/abilities/TriggeredAbility.java @@ -46,6 +46,8 @@ public interface TriggeredAbility extends Ability { boolean isOptional(); + TriggeredAbility setOptional(); + boolean isLeavesTheBattlefieldTrigger(); void setLeavesTheBattlefieldTrigger(boolean leavesTheBattlefieldTrigger); diff --git a/Mage/src/main/java/mage/abilities/TriggeredAbilityImpl.java b/Mage/src/main/java/mage/abilities/TriggeredAbilityImpl.java index e81625ad36a..4e72bc0ece3 100644 --- a/Mage/src/main/java/mage/abilities/TriggeredAbilityImpl.java +++ b/Mage/src/main/java/mage/abilities/TriggeredAbilityImpl.java @@ -363,6 +363,21 @@ public abstract class TriggeredAbilityImpl extends AbilityImpl implements Trigge return optional; } + @Override + public TriggeredAbility setOptional() { + this.optional = true; + + if (getEffects().stream().filter( + effect -> effect instanceof DoIfCostPaid && (this.optional && ((DoIfCostPaid) effect).isOptional())) + .findAny().isPresent()) { + throw new IllegalArgumentException( + "DoIfCostPaid effect must have only one optional settings, but it have two (trigger + DoIfCostPaid): " + + this.getClass().getSimpleName()); + } + + return this; + } + @Override public TriggeredAbilityImpl setAbilityWord(AbilityWord abilityWord) { super.setAbilityWord(abilityWord); diff --git a/Mage/src/main/java/mage/abilities/effects/common/CreateTokenCopyTargetEffect.java b/Mage/src/main/java/mage/abilities/effects/common/CreateTokenCopyTargetEffect.java index 6e35a885845..d1c7fd59748 100644 --- a/Mage/src/main/java/mage/abilities/effects/common/CreateTokenCopyTargetEffect.java +++ b/Mage/src/main/java/mage/abilities/effects/common/CreateTokenCopyTargetEffect.java @@ -61,7 +61,7 @@ public class CreateTokenCopyTargetEffect extends OneShotEffect { private final int tokenPower; private final int tokenToughness; private boolean useLKI = false; - private PermanentModifier permanentModifier = null; // TODO: miss copy constructor? Make serializable? + private PermanentModifier permanentModifier = null; // TODO: These constructors are a mess. Copy effects need to be reworked altogether, hopefully clean it up then. @@ -154,6 +154,7 @@ public class CreateTokenCopyTargetEffect extends OneShotEffect { this.tokenPower = effect.tokenPower; this.tokenToughness = effect.tokenToughness; this.useLKI = effect.useLKI; + this.permanentModifier = effect.permanentModifier; } @Override