Fix Sorcerous Squall (#12519)

wrong player chooses, is a "may" effect (implemented as "up to one")
This commit is contained in:
ssk97 2024-06-24 20:59:23 -07:00 committed by GitHub
parent f53954d56a
commit 3e171f373e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -71,17 +71,18 @@ class SorcerousSquallEffect extends OneShotEffect {
@Override @Override
public boolean apply(Game game, Ability source) { public boolean apply(Game game, Ability source) {
Player player = game.getPlayer(source.getFirstTarget()); Player player = game.getPlayer(source.getControllerId());
if (player == null) { if (player == null) {
return false; return false;
} }
FilterCard filter = new FilterInstantOrSorceryCard("instant or sorcery card from that player's graveyard"); FilterCard filter = new FilterInstantOrSorceryCard("instant or sorcery card from that player's graveyard");
filter.add(new OwnerIdPredicate(source.getFirstTarget())); filter.add(new OwnerIdPredicate(source.getFirstTarget()));
Target target = new TargetCardInGraveyard(1, 1, filter, true); Target target = new TargetCardInGraveyard(0, 1, filter, true);
player.choose(outcome, target, source, game); if (player.choose(outcome, target, source, game)) {
Effect effect = new MayCastTargetCardEffect(CastManaAdjustment.WITHOUT_PAYING_MANA_COST, true); Effect effect = new MayCastTargetCardEffect(CastManaAdjustment.WITHOUT_PAYING_MANA_COST, true);
effect.setTargetPointer(new FixedTarget(target.getFirstTarget(), game)); effect.setTargetPointer(new FixedTarget(target.getFirstTarget(), game));
effect.apply(game, source); effect.apply(game, source);
}
return true; return true;
} }
} }