aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTony Konzel <saxxonpike@gmail.com>2022-09-02 12:29:54 -0500
committerGitHub <noreply@github.com>2022-09-02 19:29:54 +0200
commit052dfe27fc324f13f8d4cc2af802f57963d8fe7c (patch)
treef282a88154b973dcbf21147bc23f777bf00c3645 /src
parentea711efcc3508a45d131b7bae9ad7c214ee9801c (diff)
Core/Spells: Removed gathering failure chance from herbalism and mining nodes (#28227)
Diffstat (limited to 'src')
-rw-r--r--src/server/game/Spells/Spell.cpp3
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;