fix(DB/Spell): Fix Omen of Clarity not proccing from utility spells#24783
Conversation
SpellTypeMask=3 blocked Gift of the Wild, Mark of the Wild and other magic-class utility spells from triggering Omen of Clarity. Revert to SpellTypeMask=0 to match TrinityCore. Triggered spells like Furor are already blocked by the triggered spell check in SpellAuras.cpp.
Reject SPELL_EFFECT_ENERGIZE spells in CheckProc since they are not real player casts and should not trigger Clearcasting.
|
Tested. This is how it works with this PR: Omen Procs
Did I miss something? Sorry about the order of spells; it's just the order I tested them in. |
|
Still missing the Predators swiftness + spell behavior right ? It should have a 50-60% chance to proc when a spell is used with it. Theory was it used predators swiftness duration ( 10 secs ) as the "cast time". This was also added to classic wotlk. ( 3.5/60 * 10 = 58.3%) |
Let me revisit this in another PR so I can merge this for now EDIT: @bonfax4 cast time it is already in the calculation thanks to @Tereneckla |
Not what I ment exactly. Its for instant casts with predators swiftness specifically, doesn't use cast time exactly, it just has a higher chance to proc. Its like @ 60%. |
…zerothcore#24783) Co-authored-by: blinkysc <[email protected]>
…zerothcore#24783) Co-authored-by: blinkysc <[email protected]>
Changes Proposed:
This PR proposes changes to:
Reverts
SpellTypeMaskfrom 3 to 0 for Omen of Clarity (16864), matching TrinityCore.SpellTypeMask=3blocked magic-class utility spells like Gift of the Wild and Mark of the Wild from triggering OoC.Regression from #24233.
AI-assisted Pull Requests
Important
While the use of AI tools when preparing pull requests is not prohibited, contributors must clearly disclose when such tools have been used and specify the model involved.
Contributors are also expected to fully understand the changes they are submitting and must be able to explain and justify those changes when requested by maintainers.
Issues Addressed:
SOURCE:
The changes have been validated through:
spell_procfor 16864 usesSpellTypeMask=0.Tests Performed:
This PR has been:
How to Test the Changes:
Known Issues and TODO List:
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.