diff options
| author | Tony Konzel <saxxonpike@gmail.com> | 2022-09-02 12:29:54 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-02 19:29:54 +0200 |
| commit | 052dfe27fc324f13f8d4cc2af802f57963d8fe7c (patch) | |
| tree | f282a88154b973dcbf21147bc23f777bf00c3645 | |
| parent | ea711efcc3508a45d131b7bae9ad7c214ee9801c (diff) | |
Core/Spells: Removed gathering failure chance from herbalism and mining nodes (#28227)
| -rw-r--r-- | src/server/game/Spells/Spell.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp index d475620a4c9..aa79387f047 100644 --- a/src/server/game/Spells/Spell.cpp +++ b/src/server/game/Spells/Spell.cpp @@ -5666,7 +5666,8 @@ SpellCastResult Spell::CheckCast(bool strict, uint32* param1 /*= nullptr*/, uint // chance for fail at lockpicking attempt // second check prevent fail at rechecks - if (skillId != SKILL_NONE && (!m_selfContainer || ((*m_selfContainer) != this))) + // herbalism and mining cannot fail as of patch 3.1.0 + if (skillId != SKILL_NONE && skillId != SKILL_HERBALISM && skillId != SKILL_MINING && (!m_selfContainer || ((*m_selfContainer) != this))) { bool canFailAtMax = skillId == SKILL_LOCKPICKING; |