external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-20 04:38:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

When a card like Explosive Vegetation asks you to search for multiple cards, only reveal the cards in the library once.

Browse Source
This commit is contained in:
Myrd
2015-12-30 20:01:57 +00:00
parent 2f963ebc9e
commit a7d99b7126
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
forge-game/src/main/java/forge/game/ability/effects/ChangeZoneEffect.java
View File

@@ -791,9 +791,11 @@ public class ChangeZoneEffect extends SpellAbilityEffect {
}
}
// If we're choosing multiple cards, only need to show the reveal dialog the first time through.
boolean shouldReveal = (i == 0);
Card c = null;
if (sa.hasParam("AtRandom")) {
if (delayedReveal != null) {
if (shouldReveal && delayedReveal != null) {
decider.getController().reveal(delayedReveal.getCards(), delayedReveal.getZone(), delayedReveal.getOwner(), delayedReveal.getMessagePrefix());
}
c = Aggregates.random(fetchList);
@@ -806,7 +808,7 @@ public class ChangeZoneEffect extends SpellAbilityEffect {
if (changeNum > 1) { //indicate progress if multiple cards being chosen
title += " (" + (i + 1) + " / " + changeNum + ")";
}
c = decider.getController().chooseSingleCardForZoneChange(destination, origin, sa, fetchList, delayedReveal, title, !sa.hasParam("Mandatory"), decider);
c = decider.getController().chooseSingleCardForZoneChange(destination, origin, sa, fetchList, shouldReveal ? delayedReveal : null, title, !sa.hasParam("Mandatory"), decider);
}
if (c == null) {