ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

(Hopefully) fixing Phallen's choice PR such that it doesn't ruin any cards with a choice that happens to be a cost

Browse Source
This commit is contained in:
Christian 'ketura' McCarty
2024-12-02 00:26:18 -06:00
parent 0a494f3ffb
commit a32ed8eaf2
1 changed files with 20 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
gemp-lotr/gemp-lotr-logic/src/main/java/com/gempukku/lotro/cards/build/field/effect/appender/Choice.java
View File

@@ -53,18 +53,32 @@ public class Choice implements EffectAppenderProducer {
textIndex++;
}
if (playableEffectAppenders.size() == 0) {
actionContext.setValueToMemory(memorize, "");
return null;
}
if (playableEffectAppenders.size() == 1) {
SubAction subAction = new SubAction(action);
playableEffectAppenders.get(0).appendEffect(cost, subAction, delegateActionContext);
playableEffectAppenders.getFirst().appendEffect(cost, subAction, delegateActionContext);
actionContext.setValueToMemory(memorize, textArray[0]);
return new StackActionEffect(subAction);
}
if (playableEffectAppenders.isEmpty()) {
actionContext.setValueToMemory(memorize, "");
// Costs that are a choice should not let the player pick freely, else it effectively results in there
// being no cost at all.
if(cost)
return null;
// If we are an effect however, then we will let "no valid choices" default to the player choosing
// which fork to go down; this should eliminate situations like Here Lies Balin doing nothing
// if there is a single 1-vitality Orc on the field
textIndex = 0;
for (var possibleEffectAppender : possibleEffectAppenders) {
playableEffectAppenders.add(possibleEffectAppender);
effectTexts.add(GameUtils.substituteText(textArray[textIndex], actionContext));
textIndex++;
}
}
SubAction subAction = new SubAction(action);
subAction.appendCost(
new PlayoutDecisionEffect(choosingPlayer,