Skip to content

isManaAbility for CardTraits, because of Triggered mana abilities #9583

@Hanmac

Description

@Hanmac

While looking at this part for #9532

if (ck.isRemoveNonMana()) {
// List only has nonMana
if (null == mana) {
list.removeIf(Predicate.not(SpellAbility::isManaAbility));
} else if (false == mana) {
list.clear();
}
} else if (ck.isRemoveAll()) {
list.clear();
}

and thinking on how to optimize it, like using a Predicate,
I noticed that it doesn't respect "Triggered mana abilities",

so the two cards that use it (maybe unify the property name?)

  • Town-Razer Tyrant with RemoveNonManaAbilities
  • Blood Sun with ExceptManaAbilities

Should, in the rare chance that something with a "Triggered mana ability" somehow got turned into a non-basic land,
have them keep these Triggers.

Metadata

Metadata

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions