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

Update DestroyAllEffect.java

Browse Source
This commit is contained in:
Simisays
2023-02-04 12:10:22 +01:00
parent 08ca2cd08c
commit 04bcd5202e
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
forge-game/src/main/java/forge/game/ability/effects/DestroyAllEffect.java
View File

@@ -52,7 +52,6 @@ public class DestroyAllEffect extends SpellAbilityEffect {
final String desc = sa.getParamOrDefault("ValidDescription", ""); final String desc = sa.getParamOrDefault("ValidDescription", "");
Player targetPlayer = sa.getTargets().getFirstTargetedPlayer(); Player targetPlayer = sa.getTargets().getFirstTargetedPlayer();
String valid = sa.getParamOrDefault("ValidCards", ""); String valid = sa.getParamOrDefault("ValidCards", "");
// Ugh. If calculateAmount needs to be called with DestroyAll it _needs_ // Ugh. If calculateAmount needs to be called with DestroyAll it _needs_
@@ -79,7 +78,9 @@ public class DestroyAllEffect extends SpellAbilityEffect {
if (sa.hasParam("RememberAllObjects")) { if (sa.hasParam("RememberAllObjects")) {
card.addRemembered(list); card.addRemembered(list);
} }
if (list.isEmpty() && isOptional) {
return;
}
if (isOptional && !sa.getActivatingPlayer().getController().confirmAction(sa, null, Localizer.getInstance().getMessage("lblWouldYouLikeDestroy",(desc)), null)) { if (isOptional && !sa.getActivatingPlayer().getController().confirmAction(sa, null, Localizer.getInstance().getMessage("lblWouldYouLikeDestroy",(desc)), null)) {
return; return;