wrathful gladiator's libram of fortitude

[VENOM]

mangos rev 10786

on wowhead its has diffrent stat

Your crusader strike ability also grants you 204 attack power for 10 sec.

http://www.wowhead.com/spell=60636

Yes it is working. but it also grants 204 attack power to the opponent. and agro mobs too.

it gives you 204 attack power and also it gives 204 attack power to the enemy you ar fighting with

[Den]

Касается не только паладина http://www.wowhead.com/spell=60555
Исправляет срабатывание прока на жертву...

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 66d8c7d..c39708b 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -6431,6 +6431,9 @@ bool Spell::CheckTarget( Unit* target, SpellEffectIndex eff )
             return false;
     }
 
+   if (m_spellInfo->Id == 60555 && m_spellInfo->EffectImplicitTargetA[eff] != TARGET_SELF)
+        return false;
+
     // Check targets for LOS visibility (except spells without range limitations )
     switch(m_spellInfo->Effect[eff])
     {

add
Такая же беда и с Chains of Ice, срабатывает ещё и на себя.

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index c77a070..6ba2d33 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -6430,7 +6430,11 @@ bool Spell::CheckTarget( Unit* target, SpellEffectIndex eff )
         if(((Player*)target)->isGameMaster() && !IsPositiveSpell(m_spellInfo->Id))
             return false;
     }
-
+
+    // prevent proc Chains of Ice on self
+    if (m_spellInfo->Id == 45524 && m_spellInfo->EffectImplicitTargetA[eff] == TARGET_SELF)
+        return false;
+
     // Check targets for LOS visibility (except spells without range limitations )
     switch(m_spellInfo->Effect[eff])
     {

[Warlord123]

Таких заклинаний не так мало.
60544 Savage Aggression
60547 Hateful Aggression
60549 Deadly Aggression
60551 Furious Gladiator's Libram of Fortitude
60553 Relentless Aggression
60555 LK Arena 6 Aggression
60565 Savage Magic
60566 Hateful Magic
60567 Deadly Magic
60568 Furious Gladiator's Idol of Steadfastness
60569 Relentless Survival
60570 LK Arena 6 Magic

[Den]

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index c77a070..9c9dd00 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -6430,7 +6430,33 @@ bool Spell::CheckTarget( Unit* target, SpellEffectIndex eff )
         if(((Player*)target)->isGameMaster() && !IsPositiveSpell(m_spellInfo->Id))
             return false;
     }
-		
+	 
+    // check proc target
+    switch(m_spellInfo->Id)
+    {
+        // on self
+        case 60544:    // Savage Aggression
+        case 60547:    // Hateful Aggression
+        case 60549:    // Deadly Aggression
+        case 60551:    // Furious Gladiator's Libram of Fortitude
+	 case 60553:    // Relentless Aggression
+	 case 60555:    // LK Arena 6 Aggression
+        case 60565:    // Savage Magic
+        case 60566:    // Hateful Magic
+        case 60567:    // Deadly Magic
+        case 60568:    // Furious Gladiator's Idol of Steadfastness
+        case 60569:    // Relentless Survival
+        case 60570:    // LK Arena 6 Magic
+            if(m_spellInfo->EffectImplicitTargetA[eff] != TARGET_SELF)
+                return false;
+	      break;
+         // on victim
+	  case 45524:    // Chains of Ice
+             if(m_spellInfo->EffectImplicitTargetA[eff] == TARGET_SELF)
+                 return false;
+	  break;
+    }
+	
     // Check targets for LOS visibility (except spells without range limitations )
     switch(m_spellInfo->Effect[eff])
     {

[virusav]

У этих спеллов AttributesEx4: 0x00100000 (SPELL_ATTR_EX4_UNK20).
ADD: как оказалось, не у всех, значит, с атрибутом промах.

[Den]

Правильнее будет вот так ?

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index c77a070..75beb49 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -6430,7 +6430,15 @@ bool Spell::CheckTarget( Unit* target, SpellEffectIndex eff )
         if(((Player*)target)->isGameMaster() && !IsPositiveSpell(m_spellInfo->Id))
             return false;
     }
-		
+	 
+    // only proc spell on self
+    if (m_spellInfo->SpellIconID == 1962 && (m_spellInfo->AttributesEx4 & SPELL_ATTR_EX4_UNK20) ||      // spell: 60544, 60547, 60549, 60551, 60553, 60555
+	 (m_spellInfo->SpellIconID == 1961 && m_spellInfo->SpellVisual[0] == 5400))                      // spell: 60565, 60566, 60567, 60568, 60569, 60570
+    {
+        if(m_spellInfo->EffectImplicitTargetA[eff] != TARGET_SELF)
+            return false;
+    }
+	
     // Check targets for LOS visibility (except spells without range limitations )
     switch(m_spellInfo->Effect[eff])
     {

[rsa]

Костыли это все... Систему выбоки таргетов надо сильно чинить. Лично мне непонятно зачем при паре TARGET_SELF, NO_TARGET заливать поле целей по типу TARGET_SELECT.

[virusav]

m_spellInfo->SpellIconID == 1962 && m_spellInfo->AttributesEx4 & SPELL_ATTR_EX4_UNK20
Effect 1: Id 6 (SPELL_EFFECT_APPLY_AURA), NO_TARGET, NO_TARGET:
60544
60547
60549
60551
60553
60555
60766 -?

m_spellInfo->SpellIconID == 1961 && m_spellInfo->SpellVisual[0] == 5400
43742 - ?
43749 - ?
48838 - ?
Effect 1: Id 6 (SPELL_EFFECT_APPLY_AURA), NO_TARGET, NO_TARGET:
60565
60566
60567
60568
60569
60570

SpellIconID=180
SpellVisual=12328
45524
66020

По указанным в посте 6 выборкам получается больше спеллов, чем в списке в предыдущих постах.

Интересное сочетание NO_TARGET, NO_TARGET.

Если не ошибаюсь, то условие написано неверно:

if (m_spellInfo->SpellIconID == 1962 && (m_spellInfo->AttributesEx4 & SPELL_ATTR_EX4_UNK20) ||      // spell: 60544, 60547, 60549, 60551, 60553, 60555
+	 (m_spellInfo->SpellIconID == 1961 && m_spellInfo->SpellVisual[0] == 5400))                      // spell: 60565, 60566, 60567, 60568, 60569, 60570
+    {

Видимо, должно быть:

if ((m_spellInfo->SpellIconID == 1962 && m_spellInfo->AttributesEx4 & SPELL_ATTR_EX4_UNK20) ||      // spell: 60544, 60547, 60549, 60551, 60553, 60555
+	 (m_spellInfo->SpellIconID == 1961 && m_spellInfo->SpellVisual[0] == 5400))                      // spell: 60565, 60566, 60567, 60568, 60569, 60570
+    {

[rsa]

Попробуйте так:

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 6b9fdd0..da497c1 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2845,6 +2845,10 @@ void Spell::SetTargetMap(SpellEffectIndex effIndex, uint32 targetMode, UnitList&
                     {
                         case SPELL_AURA_ADD_FLAT_MODIFIER:  // some spell mods auras have 0 target modes instead expected TARGET_SELF(1) (and present for other ranks for same spell for example)
                         case SPELL_AURA_ADD_PCT_MODIFIER:
+                        case SPELL_AURA_MOD_RANGED_ATTACK_POWER:
+                        case SPELL_AURA_MOD_ATTACK_POWER:
+                        case SPELL_AURA_MOD_HEALING_DONE:
+                        case SPELL_AURA_MOD_DAMAGE_DONE:
                             targetUnitMap.push_back(m_caster);
                             break;
                         default:                            // apply to target in other case

тоже конечно костыль но хоть не такой кривой как приведенные выше...

[Den]

А это случаем не затронет, проки спелов которые должны срабатывать, не только на себя но и на всю пати/рейд ?

[rsa]

Я потому и написал "попробуйте". других пар с таргетом 0,0 действующих на врага вроде нет. а вот пары 18,0 или 0,18 подпадающие под это же условие могут быть (мне просто неизвестны). Если они найдутся - придется для пары 0,0 делать собственное case, не более того.