Target: Fix Banish in immune to damage check. (#235)

Author: ileboii

playerbot/strategy/values/PossibleAttackTargetsValue.cpp

@@ -173,23 +173,25 @@ bool PossibleAttackTargetsValue::IsImmuneToDamage(Unit* target, Player* player)
 
     // Immune to damage
     PlayerbotAI* ai = player->GetPlayerbotAI();
-    if (ai)
+    if (!ai)
+        return false;
+
+    for (const Aura* aura : ai->GetAuras(target))
     {
-        for (const Aura* aura : ai->GetAuras(target))
-        {
-            if (!aura->IsPositive())
-                continue;
+        const SpellEntry* spellInfo = aura->GetSpellProto();
+        if (!spellInfo)
+            continue;
 
-            const SpellEntry* spellInfo = aura->GetSpellProto();
-            if (!spellInfo)
-                continue;
+        if (spellInfo->Mechanic == MECHANIC_BANISH)
+            return true;
 
-            if (spellInfo->Mechanic == MECHANIC_BANISH ||
-                spellInfo->Mechanic == MECHANIC_INVULNERABILITY ||
-                spellInfo->Mechanic == MECHANIC_IMMUNE_SHIELD)
-            {
-                return true;
-            }
+        if (!aura->IsPositive())
+            continue;
+
+        if (spellInfo->Mechanic == MECHANIC_INVULNERABILITY ||
+            spellInfo->Mechanic == MECHANIC_IMMUNE_SHIELD)
+        {
+            return true;
         }
     }