From 052dfe27fc324f13f8d4cc2af802f57963d8fe7c Mon Sep 17 00:00:00 2001 From: Tony Konzel Date: Fri, 2 Sep 2022 12:29:54 -0500 Subject: Core/Spells: Removed gathering failure chance from herbalism and mining nodes (#28227) --- src/server/game/Spells/Spell.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src') 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; -- cgit v1.2.3