[10878][patch]Properly update achievements at .maxskill command use

[lightman]

What does the patch do? What feature it adds? - makes possible complete Master of arms achievement, when .maxskill command increases four skills to 400, and makes possible complete achievement Did somebody order a knuckle sandwich? when .maxskill increases unnarmed skill to 400.

Bug report thread? - none that I know of.

For which revision is the patch created for? - 10875

Who has been writing the patch? - myself (lightman)

patch:

diff --git a/src/game/Player.cpp b/src/game/Player.cpp
index 5582c85..757364c 100644
--- a/src/game/Player.cpp
+++ b/src/game/Player.cpp
@@ -5706,6 +5706,7 @@ void Player::UpdateSkillsToMaxSkillsForLevel()
        if(max > 1)
        {
            SetUInt32Value(valueIndex,MAKE_SKILL_VALUE(max,max));
+            GetAchievementMgr().UpdateAchievementCriteria(ACHIEVEMENT_CRITERIA_TYPE_REACH_SKILL_LEVEL, pskill);
            if(itr->second.uState != SKILL_NEW)
                itr->second.uState = SKILL_CHANGED;
        }

[VladimirMangos]

Exist another case when this need do also (maximize skill at level update by config option setting)

In [10878] with this addition. Thank you