moot pointed out i made a boo boo in STANDARDABILITYGRANT thanks you! forgot to add the "suggest good" part of the condiation...as we dont act to give a creature the ability 2 times. (atleast until stacking abilities are added)

This commit is contained in:
omegablast2002@yahoo.com
2010-12-08 14:37:39 +00:00
parent 079d2dbd22
commit 682af98bf7
+1 -5
View File
@@ -446,10 +446,6 @@ int AIAction::getEfficiency()
{ {
efficiencyModifier /= p->game->hand->nb_cards; efficiencyModifier /= p->game->hand->nb_cards;
} }
// Z, please review. If this condition is hit, it's immediately wiped out later at line 464 (the don't give flying twice check),
// since both satisfy the check for (target->has(a->abilitygranted)).
// Is this if statement section obsolete?
if (suggestion == BAKA_EFFECT_BAD && p != target->controller() && target->has(a->abilitygranted) && p->isAI()) if (suggestion == BAKA_EFFECT_BAD && p != target->controller() && target->has(a->abilitygranted) && p->isAI())
{ {
efficiency += efficiencyModifier; efficiency += efficiencyModifier;
@@ -460,7 +456,7 @@ int AIAction::getEfficiency()
efficiency += efficiencyModifier; efficiency += efficiencyModifier;
} }
if (target->has(a->abilitygranted)) if (suggestion == BAKA_EFFECT_GOOD && target->has(a->abilitygranted))
{ {
//trying to avoid Ai giving ie:flying creatures ie:flying twice. //trying to avoid Ai giving ie:flying creatures ie:flying twice.
efficiency = 0; efficiency = 0;