From f9b7f3fa9f1e18d5f63bf75d138d0355b6cfacda Mon Sep 17 00:00:00 2001 From: hondacrx Date: Sun, 11 Sep 2022 14:34:55 -0400 Subject: [PATCH] Core/Players: assign forbidden skills to a proper position to no longer interfere with legit skills that have been stored as first element. Port From (https://github.com/TrinityCore/TrinityCore/commit/2556d6d705c519fd461b9d9f10a4387d9beb199e) --- Source/Game/Entities/Player/Player.DB.cs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/Source/Game/Entities/Player/Player.DB.cs b/Source/Game/Entities/Player/Player.DB.cs index 8092f994b..2b97a11bf 100644 --- a/Source/Game/Entities/Player/Player.DB.cs +++ b/Source/Game/Entities/Player/Player.DB.cs @@ -366,9 +366,8 @@ namespace Game.Entities SkillRaceClassInfoRecord rcEntry = Global.DB2Mgr.GetSkillRaceClassInfo(skill, race, GetClass()); if (rcEntry == null) { - Log.outError(LogFilter.Player, "Character: {0}(GUID: {1} Race: {2} Class: {3}) has skill {4} not allowed for his race/class combination", - GetName(), GetGUID().ToString(), race, GetClass(), skill); - mSkillStatus.Add(skill, new SkillStatusData(0, SkillState.Deleted)); + Log.outError(LogFilter.Player, $"Player::_LoadSkills: Player '{GetName()}' ({GetGUID()}, Race: {race}, Class: {GetClass()}) has forbidden skill {skill} for his race/class combination"); + mSkillStatus.Add(skill, new SkillStatusData((uint)mSkillStatus.Count, SkillState.Deleted)); continue; }