Yes, I saw this too.
A sanity check build into #setskill and #setskillall would do the trick.
Something like:
Code:
if (new_skill_value > GetMaxPossibleRank(skillid) {
Message(13, "ERROR: Value %d to high for %d ! Setting it to %d", new_skill_value , GetSkillName(tradeskill) , GetMaxPossibleRank(skillid);
new_skill_value = GetMaxPossibleRank(skillid);
}
The variables are only for example, I didnt look in the code.