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

Merge branch '1358-choosegenericeffect-atrandom-param-update-needed' into 'master'

Browse Source 
Resolve "ChooseGenericEffect AtRandom param update needed"

Closes #1358

See merge request core-developers/forge!2693
This commit is contained in:
Hans Mackowiak
2020-04-11 20:48:04 +00:00
parent b4cc8421c5 a0ea1f1d3f
commit cb7d879abb
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
forge-game/src/main/java/forge/game/ability/effects/ChooseGenericEffect.java
View File

@@ -67,8 +67,9 @@ public class ChooseGenericEffect extends SpellAbilityEffect {
SpellAbility chosenSA = null;
if (sa.hasParam("AtRandom")) {
int idxChosen = MyRandom.getRandom().nextInt(abilities.size());
chosenSA = abilities.get(idxChosen);
if (!abilities.isEmpty()) {
chosenSA = abilities.get(MyRandom.getRandom().nextInt(abilities.size()));
}
} else {
chosenSA = p.getController().chooseSingleSpellForEffect(abilities, sa, Localizer.getInstance().getMessage("lblChooseOne"),
ImmutableMap.of());
@@ -90,7 +91,7 @@ public class ChooseGenericEffect extends SpellAbilityEffect {
if (fallback != null) {
p.getGame().fireEvent(new GameEventCardModeChosen(p, host.getName(), fallback.getDescription(), sa.hasParam("ShowChoice")));
AbilityUtils.resolve(fallback);
} else {
} else if (!sa.hasParam("AtRandom")) {
System.err.println("Warning: all Unless costs were unpayable for " + host.getName() +", but it had no FallbackAbility defined. Doing nothing (this is most likely incorrect behavior).");
}
}