[patch] Echantments reqs

[zhenya]

В общем в некоторых профессиях есть вещи, которые можно наложить на итем и получать бонусы, а потом просто отменить профессию и иметь бонус от этих вещей

void WorldSession::HandleUnlearnSkillOpcode(WorldPacket & recv_data)
{
    uint32 skill_id;
    recv_data >> skill_id;
    GetPlayer()->SetSkill(skill_id, 0, 0);
}

далее

...
            // clear skill fields
            SetUInt32Value(PLAYER_SKILL_INDEX(itr->second.pos),0);
            SetUInt32Value(PLAYER_SKILL_VALUE_INDEX(itr->second.pos),0);
            SetUInt32Value(PLAYER_SKILL_BONUS_INDEX(itr->second.pos),0);

            // mark as deleted or simply remove from map if not saved yet
            if(itr->second.uState != SKILL_NEW)
                itr->second.uState = SKILL_DELETED;
            else
                mSkillStatus.erase(itr);

            // remove all spells that related to this skill
            for (uint32 j=0; j<sSkillLineAbilityStore.GetNumRows(); ++j)
                if(SkillLineAbilityEntry const *pAbility = sSkillLineAbilityStore.LookupEntry(j))
                    if (pAbility->skillId==id)
                        removeSpell(sSpellMgr.GetFirstSpellInChain(pAbility->spellId));
...

считаю что необходимо при удалении профы делать

_RemoveAllItemMods();
_ApplyAllItemMods();

+ считаю необходимым сделать обработку

struct SpellItemEnchantmentEntry
{
.....
    //uint32      requiredSkill;                       // 35       m_requiredSkillID
    //uint32      requiredSkillValue;                // 36       m_requiredSkillRank
....
};

также сделал чтобы энчанты требовали Skill и Value и Level персонажа.
также добавил sSkillLineAbilityStore с ключем skill_id для более удобного доступа к скилам вместо просмотра всего дбц.

[zergtmn]

Если на вещи уже есть инчант, но скилла не хватает, при поднятии скилла до нужного уровня инчант должен заработать. В UpdateCraftSkill или UpdateSkillPro тоже надо добавить пару

_RemoveAllItemMods();
_ApplyAllItemMods();

[zhenya]

Сообщение от zergtmn

Если на вещи уже есть инчант, но скилла не хватает, при поднятии скилла до нужного уровня инчант должен заработать. В UpdateCraftSkill или UpdateSkillPro тоже надо добавить пару

_RemoveAllItemMods();
_ApplyAllItemMods();

очень странно почему там повышается скил не через SetSkill.
мб лучше поменять SetUInt32Value(valueIndex,MAKE_SKILL_VALUE(new_val ue,MaxValue));
на SetSkill(SkillId,new_value,MaxValue);?