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

ReplaceDamage: fixed "Prevent$ True" count for Prevention$ Effects too.

Browse Source 
make AI use it in damageIfUnblocked
This commit is contained in:
Hanmac
2017-05-27 13:52:23 +00:00
parent 4ef6d1c5c0
commit bbaf9078bd
2 changed files with 14 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
forge-ai/src/main/java/forge/ai/ComputerUtilCombat.java
View File

@@ -196,12 +196,8 @@ public class ComputerUtilCombat {
}
if (!attacked.getGame().getStaticEffects().getGlobalRuleChange(GlobalRuleChange.noPrevention)) {
// ask ReplacementDamage directly -- FIXME: this actually doesn't work right now, so it's temporarily reverted to the old form (see below).
//if (ComputerUtilCombat.isCombatDamagePrevented(attacker, attacked, damage)) {
if (attacker.hasKeyword("Prevent all damage that would be dealt by CARDNAME.")
|| attacker.hasKeyword("Prevent all damage that would be dealt to and by CARDNAME.")
|| attacker.hasKeyword("Prevent all combat damage that would be dealt by CARDNAME.")
|| attacker.hasKeyword("Prevent all combat damage that would be dealt to and dealt by CARDNAME.")) {
// ask ReplacementDamage directly
if (ComputerUtilCombat.isCombatDamagePrevented(attacker, attacked, damage)) {
return 0;
}
}

24
forge-game/src/main/java/forge/game/replacement/ReplaceDamage.java
View File

@@ -50,25 +50,25 @@ public class ReplaceDamage extends ReplacementEffect {
if (!runParams.get("Event").equals("DamageDone")) {
return false;
}
if (!(runParams.containsKey("Prevention") == getMapParams().containsKey("PreventionEffect"))) {
if (!(runParams.containsKey("Prevention") == (hasParam("PreventionEffect") || hasParam("Prevent")))) {
return false;
}
if (getMapParams().containsKey("ValidSource")) {
String validSource = getMapParams().get("ValidSource");
if (hasParam("ValidSource")) {
String validSource = getParam("ValidSource");
validSource = AbilityUtils.applyAbilityTextChangeEffects(validSource, this);
if (!matchesValid(runParams.get("DamageSource"), validSource.split(","), getHostCard())) {
return false;
}
}
if (getMapParams().containsKey("ValidTarget")) {
String validTarget = getMapParams().get("ValidTarget");
if (hasParam("ValidTarget")) {
String validTarget = getParam("ValidTarget");
validTarget = AbilityUtils.applyAbilityTextChangeEffects(validTarget, this);
if (!matchesValid(runParams.get("Affected"), validTarget.split(","), getHostCard())) {
return false;
}
}
if (getMapParams().containsKey("DamageAmount")) {
String full = getMapParams().get("DamageAmount");
if (hasParam("DamageAmount")) {
String full = getParam("DamageAmount");
String operator = full.substring(0, 2);
String operand = full.substring(2);
int intoperand = 0;
@@ -82,8 +82,8 @@ public class ReplaceDamage extends ReplacementEffect {
return false;
}
}
if (getMapParams().containsKey("IsCombat")) {
if (getMapParams().get("IsCombat").equals("True")) {
if (hasParam("IsCombat")) {
if (getParam("IsCombat").equals("True")) {
if (!((Boolean) runParams.get("IsCombat"))) {
return false;
}
@@ -93,13 +93,13 @@ public class ReplaceDamage extends ReplacementEffect {
}
}
}
if (getMapParams().containsKey("IsEquipping") && !getHostCard().isEquipping()) {
if (hasParam("IsEquipping") && !getHostCard().isEquipping()) {
return false;
}
// check for DamageRedirection, the Thing where the damage is redirected to must be a creature or planeswalker or a player
if (getMapParams().containsKey("DamageTarget")) {
String def = getMapParams().get("DamageTarget");
if (hasParam("DamageTarget")) {
String def = getParam("DamageTarget");
for (Player p : AbilityUtils.getDefinedPlayers(hostCard, def, null)) {
if (!p.getGame().getPlayers().contains(p)) {