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

Fixed an issue where if an ability had more than one instance of the same type of CostPart (such as that of Demon's Herald, which has three sacrifice costs), the AI was able to get away with paying the cost only once.

Browse Source
This commit is contained in:
rikimbo
2014-06-13 00:17:15 +00:00
parent 3c66861b48
commit 12de3043d8
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
forge-game/src/main/java/forge/game/cost/CostPayment.java
View File

@@ -157,8 +157,8 @@ public class CostPayment {
this.ability.setActivatingPlayer(decisionMaker.getPlayer());
}
Map<Class<? extends CostPart>, PaymentDecision> decisions = new HashMap<Class<? extends CostPart>, PaymentDecision>();
Map<CostPart, PaymentDecision> decisions = new HashMap<CostPart, PaymentDecision>();
// Set all of the decisions before attempting to pay anything
for (final CostPart part : this.cost.getCostParts()) {
PaymentDecision decision = part.accept(decisionMaker);
@@ -167,11 +167,11 @@ public class CostPayment {
if (decisionMaker.paysRightAfterDecision() && !part.payAsDecided(decisionMaker.getPlayer(), decision, ability))
return false;
decisions.put(part.getClass(), decision);
decisions.put(part, decision);
}
for (final CostPart part : this.cost.getCostParts()) {
if (!part.payAsDecided(decisionMaker.getPlayer(), decisions.get(part.getClass()), this.ability)) {
if (!part.payAsDecided(decisionMaker.getPlayer(), decisions.get(part), this.ability)) {
return false;
}
// abilities care what was used to pay for them