Skip to content

[fix] Correct a pair of "when you discard one or more X cards" triggered abilities #14550

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.

Sign up for GitHub

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

Open
muz wants to merge 2 commits into
base: master
Choose a base branch
from
+17 −16
Open

[fix] Correct a pair of "when you discard one or more X cards" triggered abilities #14550

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d871ee9
[fix] Correct Urza, Powerstone Prodigy to correctly trigger on discar…
muz Feb 25, 2026
d4b7ca6
[fix] Correct Mishra, Excavation Prodigy to correctly trigger on disc…
muz Feb 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions Mage.Sets/src/mage/cards/m/MishraExcavationProdigy.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
import mage.abilities.costs.common.DiscardCardCost;
import mage.abilities.costs.common.TapSourceCost;
import mage.abilities.costs.mana.GenericManaCost;
import mage.abilities.effects.common.DiscardCardControllerTriggeredAbility;
import mage.abilities.effects.common.DiscardOneOrMoreCardsTriggeredAbility;
import mage.abilities.effects.common.DrawCardSourceControllerEffect;
import mage.abilities.effects.mana.BasicManaEffect;
import mage.abilities.keyword.HasteAbility;
Expand All @@ -16,19 +16,16 @@
import mage.constants.CardType;
import mage.constants.SubType;
import mage.constants.SuperType;
import mage.filter.FilterCard;
import mage.filter.common.FilterArtifactCard;

import mage.constants.Zone;
import mage.filter.StaticFilters;
import java.util.UUID;

/**
* @author TheElk801
*/
public final class MishraExcavationProdigy extends CardImpl {

private static final FilterCard filter = new FilterArtifactCard("one or more artifact cards");

public MishraExcavationProdigy(UUID ownerId, CardSetInfo setInfo) {
public MishraExcavationProdigy(UUID ownerId, CardSetInfo setInfo) {
Copy link
Copy Markdown
Contributor

@xenohedron xenohedron Jun 1, 2026

Choose a reason for hiding this comment

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

wrong indentation?

super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{2}{R}");

this.supertype.add(SuperType.LEGENDARY);
Expand All @@ -47,8 +44,11 @@ public MishraExcavationProdigy(UUID ownerId, CardSetInfo setInfo) {
this.addAbility(ability);

// Whenever you discard one or more artifact cards, add {R}{R}. This ability triggers only once each turn.
this.addAbility(new DiscardCardControllerTriggeredAbility(
new BasicManaEffect(Mana.RedMana(2)), false, filter
this.addAbility(new DiscardOneOrMoreCardsTriggeredAbility(
Zone.BATTLEFIELD,
new BasicManaEffect(Mana.RedMana(2)),
false,
StaticFilters.FILTER_CARD_ARTIFACTS
).setTriggersLimitEachTurn(1));
}

Expand Down
15 changes: 8 additions & 7 deletions Mage.Sets/src/mage/cards/u/UrzaPowerstoneProdigy.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,16 @@
import mage.abilities.costs.common.TapSourceCost;
import mage.abilities.costs.mana.GenericManaCost;
import mage.abilities.effects.common.CreateTokenEffect;
import mage.abilities.effects.common.DiscardCardControllerTriggeredAbility;
import mage.abilities.effects.common.DiscardOneOrMoreCardsTriggeredAbility;
import mage.abilities.effects.common.DrawDiscardControllerEffect;
import mage.abilities.keyword.VigilanceAbility;
import mage.cards.CardImpl;
import mage.cards.CardSetInfo;
import mage.constants.CardType;
import mage.constants.SubType;
import mage.constants.SuperType;
import mage.filter.FilterCard;
import mage.filter.common.FilterArtifactCard;
import mage.constants.Zone;
import mage.filter.StaticFilters;
import mage.game.permanent.token.PowerstoneToken;

import java.util.UUID;
Expand All @@ -25,8 +25,6 @@
*/
public final class UrzaPowerstoneProdigy extends CardImpl {

private static final FilterCard filter = new FilterArtifactCard("one or more artifact cards");

public UrzaPowerstoneProdigy(UUID ownerId, CardSetInfo setInfo) {
super(ownerId, setInfo, new CardType[]{CardType.CREATURE}, "{2}{U}");

Expand All @@ -47,8 +45,11 @@ public UrzaPowerstoneProdigy(UUID ownerId, CardSetInfo setInfo) {
this.addAbility(ability);

// Whenever you discard one or more artifact cards, create a tapped Powerstone token. This ability triggers only once each turn.
this.addAbility(new DiscardCardControllerTriggeredAbility(
new CreateTokenEffect(new PowerstoneToken(), 1, true), false, filter
this.addAbility(new DiscardOneOrMoreCardsTriggeredAbility(
Zone.BATTLEFIELD,
new CreateTokenEffect(new PowerstoneToken(), 1, true),
false,
StaticFilters.FILTER_CARD_ARTIFACTS
).setTriggersLimitEachTurn(1));
}

Expand Down
Loading