Hate modifiers patch

[blmille2]

Wow, sorry about that -- that'd would produce an insane amount of hate.

Here's the update:

Index: zone/attack.cpp
===================================================================
--- zone/attack.cpp	(revision 1603)
+++ zone/attack.cpp	(working copy)
@@ -1253,6 +1253,7 @@
 			other->MeleeMitigation(this, damage, min_hit);
 			ApplyMeleeDamageBonus(skillinuse, damage);
 			TryCriticalHit(other, skillinuse, damage);
+			hate += damage * (this->spellbonuses.hatemod + this->itembonuses.hatemod + this->aabonuses.hatemod)/100;
 			mlog(COMBAT__DAMAGE, "Final damage after all reductions: %d", damage);
 		}

[trevius]

Are hate modifiers not working without this patch? I thought they were already functional.

[blmille2]

I think they are getting calculated, but I followed the hit code and found no reference to it. This should allow warrior epic 2.0 and enchanter hate buffs to add hate.
Greenmachine noticed that the warrior epic 2.0 wasn't working, which prompted investigation. Note this is for effects and not procs.

Thanks!

[ghostriderdoom]

Originally Posted by blmille2

I think they are getting calculated, but I followed the hit code and found no reference to it. This should allow warrior epic 2.0 and enchanter hate buffs to add hate.
Greenmachine noticed that the warrior epic 2.0 wasn't working, which prompted investigation. Note this is for effects and not procs.

Thanks!

Epic 1.5 to be clear actually. And the focus effect it has just never felt like it did anything. I have had the warrior epic 1.5 for a year now. I didn't notice any change in aggro from getting the item.

[Caryatis]

Seems like this wouldn't accurately reflect agro from live as there is no additional agro for crits, however with the code structured like that, on crits your hate would be doubled. Also Im pretty sure that agro on live is based on weapon dmg, not actual dmg, which is why war weapons from recent expansions have less dmg/dly with relatively high elemental dmg tacked on(The Keen Edge of Discord vs Skulltutor). This keeps the war agro high but their dps in check, however that is not entirely relevant to your change and seems to be in the code already.

That said, it should be a relatively easy effect to test(feel doesnt really cut it, especially with 15%), use an item like Raex's Chestplate of Destruction and check the hatelist entries with and without it.

[blmille2]

I'm sorry, I didn't realize that crits didn't count toward additional hate from spells, effects, etc.

Please move the effective line up one, right above crits.

Thanks!

[EqTitan]

Originally Posted by Caryatis

Seems like this wouldn't accurately reflect agro from live as there is no additional agro for crits, however with the code structured like that, on crits your hate would be doubled. Also Im pretty sure that agro on live is based on weapon dmg, not actual dmg, which is why war weapons from recent expansions have less dmg/dly with relatively high elemental dmg tacked on(The Keen Edge of Discord vs Skulltutor). This keeps the war agro high but their dps in check, however that is not entirely relevant to your change and seems to be in the code already.

That said, it should be a relatively easy effect to test(feel doesnt really cut it, especially with 15%), use an item like Raex's Chestplate of Destruction and check the hatelist entries with and without it.

If the code is in then its in, thats the real question, is there code in for this exact kind of spell effect and how can we test besides taking a weopon or item with the multiplier on it completely off as it will still be a FEELING . Because as you stated its only 15%

[trevius]

Originally Posted by EqTitan

If the code is in then its in, thats the real question, is there code in for this exact kind of spell effect and how can we test besides taking a weopon or item with the multiplier on it completely off as it will still be a FEELING . Because as you stated its only 15%

Originally Posted by Caryatis

check the hatelist entries with and without it.

Just make a script to check hate list entries.

[blmille2]

Okay, so I assume you mean a quest script for the EVENT_COMBAT event?
Also, I'm having a hard time finding the variable that corresponds to the hate for a character.
Thanks!