Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DK] Anti-Magic Shell #2353

Closed
karolisb opened this issue Jul 21, 2011 · 19 comments
Closed

[DK] Anti-Magic Shell #2353

karolisb opened this issue Jul 21, 2011 · 19 comments
Assignees

Comments

@karolisb
Copy link

http://www.wowhead.com/spell=48707
Anti-Magic Shell is a little broken now:
It should defend DK from druid's Cyclone, from shaman's Hex and rogue's Poison application.

Some proofs:
http://www.warcraftmovies.com/movieview.php?id=137567
25:45 and 27:17 it shows, how DK AMS'ed shaman's Hex

Same movie, 28:50 it shows, how rogue gets near DK, DK uses AMS and no poisons gets applied, only runic power gained

@tobmaps
Copy link
Contributor

tobmaps commented Jul 21, 2011

you can try this https://gist.github.com/1096911 . Mostly based on info from here http://www.wowhead.com/spell=48707#comments:id=1049350 . One of examples why it's based on mechanics - http://www.wowhead.com/spell=66935 (triggers 66862 on normal difficulty and 67681 on heroic) , it should give you immune to aura apply, but damage still should be there (as you can see there is no Dispel types in triggered spell)

But i anyway unsure about my patch because i need to know for example if mage's Fireball should apply debuff or no when you have active Anti-Magic Shield.

And also i don't know if it should prevent aura apply of "ID - 72848 Arctic Breath" or no

@jambonfrite
Copy link

8310722 :)

@tobmaps
Copy link
Contributor

tobmaps commented Jul 26, 2011

8310722 :)

and? This commit will not fix your problem

@jambonfrite
Copy link

:(
this worked well on my server so it should work for karolisb i think ;)

@karolisb
Copy link
Author

karolisb commented Aug 9, 2011

http://www.youtube.com/watch?v=RIsabfXpRNM - 2:00 it shows how mage casts Scorch on AMS and something gets immuned, Living Bomb explodes. Maybe it is related to Fireball somehow. Haven't played for mage so I don't know.

Moreover AMS tooltip says "Prevents application of harmful magical effects". Maybe that Fireball's debuff is an magical effect.

@ghost
Copy link

ghost commented Aug 11, 2011

so...? this (8310722) fix are working or not?

@tobmaps
Copy link
Contributor

tobmaps commented Aug 11, 2011

aios, it will do nothing with Hex and poisons

@ghost
Copy link

ghost commented Aug 11, 2011

but do with druid's cyclone yep?

@Warpten
Copy link
Member

Warpten commented Aug 12, 2011

I personally did

diff --git a/src/server/game/Entities/Unit/Unit.cpp b/src/server/game/Entities/Unit/Unit.cpp
index 755449a..30ceaf2 100755
--- a/src/server/game/Entities/Unit/Unit.cpp
+++ b/src/server/game/Entities/Unit/Unit.cpp
@@ -11659,9 +11660,9 @@ bool Unit::IsImmunedToSpellEffect(SpellInfo const* spellInfo, uint32 index) cons
         // Check for immune to application of harmful magical effects
         AuraEffectList const& immuneAuraApply = GetAuraEffectsByType(SPELL_AURA_MOD_IMMUNE_AURA_APPLY_SCHOOL);
         for (AuraEffectList::const_iterator iter = immuneAuraApply.begin(); iter != immuneAuraApply.end(); ++iter)
-            if (spellInfo->Dispel == DISPEL_MAGIC &&                                      // Magic debuff
-                ((*iter)->GetMiscValue() & spellInfo->GetSchoolMask()) &&  // Check school
-                !spellInfo->IsPositiveEffect(index))                                  // Harmful
+            if ((spellInfo->Dispel == DISPEL_MAGIC || spellInfo->Dispel == DISPEL_CURSE)    // Magic or curse debuff
+                && ((*iter)->GetMiscValue() & spellInfo->GetSchoolMask())                   // Check school
+                && !spellInfo->IsPositiveEffect(index))                                     // Harmful
                 return true;
     }
 

Working with Hex, not tested with other stuff. Most probably hack like because i had no info when i did that.

@ghost
Copy link

ghost commented Aug 12, 2011

Thanks _ i merged two pathches)

@Warpten
Copy link
Member

Warpten commented Aug 12, 2011

I woud personally trust more tobmap's patch. My diff may break some stuff, but i didn't check for that.

@ghost
Copy link

ghost commented Aug 12, 2011

i can't find tobmabs patch... do you have it?

@Warpten
Copy link
Member

Warpten commented Aug 12, 2011

First answer on this issue, first link.

Want binoculars? (sry didnt mean giggles, fr-en translation brain typo :D)

@ghost
Copy link

ghost commented Aug 12, 2011

fuck - i don't read this message - very sorry - and thank you...

@SignFinder
Copy link
Contributor

Need fix AM-shell and cyclon for newest TC revision

@MagicCG
Copy link

MagicCG commented Oct 25, 2011

Well, it's quite a simple fix really.

DELETE FROM spell_linked_spell WHERE spell_trigger IN (48707);
INSERT INTO spell_linked_spell (spell_trigger, spell_effect, type, comment) VALUES
('48707', '-33786', '2', 'AMS Immune to Cyclone'),
('48707', '-51514', '2', 'AMS Immune to Hex');

@Warpten
Copy link
Member

Warpten commented Oct 25, 2011

Also, some people argue you can,t be dispelled under AMS. Any idea?

@MagicCG
Copy link

MagicCG commented Oct 25, 2011

Well

Surrounds the Death Knight in an Anti-Magic Shell, absorbing 75% of the damage dealt by harmful spells (up to a maximum of 50% of the Death Knight's health) and preventing application of harmful magical effects. Lasts 5 sec.

harmful magical effects. <-- Being the key words, i would think not.

@ghost ghost assigned QAston Jan 24, 2012
@Subv
Copy link
Contributor

Subv commented Apr 14, 2012

#2408

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants