azerothcore-wotlk icon indicating copy to clipboard operation
azerothcore-wotlk copied to clipboard

fix(DB/Karazhan): implement Threat Drop mechanic on Mana Feeders

Open elthehablo opened this issue 1 year ago • 8 comments

Changes Proposed:

  • mana feeders should now drop threat every 2 second on all targets, then apply threat to a random target on threat list. this does not work the same way it does on retail, therefore it is a hack fix.
  • now use hate to zero aura obtained from sniff that doesn't seem to work

Issues Addressed:

  • Closes https://github.com/azerothcore/azerothcore-wotlk/issues/16269
  • Closes https://github.com/chromiecraft/chromiecraft/issues/5609

SOURCE:

  • see issue
  • used sniffs from Neifion to more accurately determine timers and actual spellIDs being used

Tests Performed:

  • tested ingame with 2 characters. seems to work but would preferably need testing with a full group to see if it works properly (multi-boxing?)

How to Test the Changes:

  1. go to menagerie and kara with multiple chars
  2. engage the Mana Feeders

Known Issues and TODO List:

  • [ ] proper timing for threat drop mechanic - needs sniffing
  • [ ] figure out whether threat is added to a random target on top of the threat drop - also needs sniffing

How to Test AzerothCore PRs

When a PR is ready to be tested, it will be marked as [WAITING TO BE TESTED].

You can help by testing PRs and writing your feedback here on the PR's page on GitHub. Follow the instructions here:

http://www.azerothcore.org/wiki/How-to-test-a-PR

REMEMBER: when testing a PR that changes something generic (i.e. a part of code that handles more than one specific thing), the tester should not only check that the PR does its job (e.g. fixing spell XXX) but especially check that the PR does not cause any regression (i.e. introducing new bugs).

For example: if a PR fixes spell X by changing a part of code that handles spells X, Y, and Z, we should not only test X, but we should test Y and Z as well.

elthehablo avatar May 15 '23 13:05 elthehablo

Gotta check sniffs for threat drop auras here (“hate to zero”)

Nyeriah avatar May 15 '23 14:05 Nyeriah

Gotta check sniffs for threat drop auras here (“hate to zero”)

I will add commits if someone does

elthehablo avatar May 15 '23 14:05 elthehablo

Gotta check sniffs for threat drop auras here (“hate to zero”)

image

elthehablo avatar May 18 '23 23:05 elthehablo

Seems to be always cast together with Astral Bite if I am not mistaken.

elthehablo avatar May 18 '23 23:05 elthehablo

lol I think I need someone who is better at reading sniffs to read those hieroglyphics and explain them to me in monkey terms

elthehablo avatar May 19 '23 00:05 elthehablo

Gonna take another look at this tomorrow

elthehablo avatar Jun 02 '23 21:06 elthehablo

Does anyone have any idea how "hate to zero" is supposed to work? The only ones in the db that seem to make use of it as well are the Splintered Skeletons near The Ravenian in Scholomance.

elthehablo avatar Jun 04 '23 20:06 elthehablo

They are periodic auras that reset threat whenever their periodic effect fires. Is there anything specific you'd like to know about it?

Nyeriah avatar Jun 04 '23 20:06 Nyeriah

status of this?

FrancescoBorzi avatar Jun 29 '23 14:06 FrancescoBorzi

status of this?

Needs testing in group. Also the hate to zero aura seems to do nothing, so the threat drop mechanic is implemented on the side

elthehablo avatar Jun 29 '23 14:06 elthehablo

Ideally someone who can decipher the sniffs better than I can should try to figure this out

elthehablo avatar Jun 29 '23 14:06 elthehablo

This is on hold until we start working on Karazhan in the upcoming weeks

Nyeriah avatar Jun 29 '23 15:06 Nyeriah

This is on hold until we start working on Karazhan in the upcoming weeks

good to know, updated the labels

FrancescoBorzi avatar Jun 29 '23 15:06 FrancescoBorzi

Status

Kitzunu avatar Sep 17 '23 19:09 Kitzunu

Status

I personally have not looked into the sniffs for this creature. Partially because my skills and tools in reading sniffs are limited

elthehablo avatar Sep 17 '23 20:09 elthehablo

@elthehablo bug Bench. He'll love it

Kitzunu avatar Sep 27 '23 19:09 Kitzunu

we technically don't even need SAI, add Astral Bite,Hate to Zero and magic immunity to template_addon

"Hate to zero" is just -100% threat (and to "Hate to half" is -50%) (procced on melee swing/ability with 100% chance, it's in spell_dbc table) https://github.com/cmangos/mangos-wotlk/commit/8f79da87e7d75a49910078796a150de4428681c7 (assigned scripts to the spell Ids are still valid to date in wotlk mangos db)

avarishd avatar Oct 23 '23 18:10 avarishd

status?

Kitzunu avatar Nov 12 '23 16:11 Kitzunu