[9867][patch] Potions

[zhenya]

в общем как я понял проблема возникает когда пьют бутылочки макросом и можно выпить > 1. ( кд бутылочек выставляется в spell::finish, так что есть потенциальная возможность выпить 2 бутылочку до того как каст 1ой дошел до spell::finish ).
фикс
в Player.h после

        void SetLastPotionId(uint32 item_id) { m_lastPotionId = item_id; }

добавляем

        uint32 GetLastPotionId() { return m_lastPotionId; }

далее в
Spell.cpp в после

SpellCastResult Spell::CheckCast(bool strict)
...
        else
            return SPELL_FAILED_NOT_READY;
    }

добавляем

    // cooldown for potions set in spell::finish so we can use 2 potion before sendcooldownevent
    // after sendcooldown in spell::finish m_lastPotionId sets to 0
    if (m_caster->GetTypeId() == TYPEID_PLAYER && ((Player*)m_caster)->GetLastPotionId() != 0)
        return SPELL_FAILED_NOT_READY;

возможно это хак не знаю. просьба потестить. возможно эту проверку надо сделать в
void WorldSession::HandleUseItemOpcode(WorldPacket& recvPacket)

    if (pUser->isInCombat())
    {
        for(int i = 0; i < MAX_ITEM_PROTO_SPELLS; ++i)
        {
            if (SpellEntry const *spellInfo = sSpellStore.LookupEntry(proto->Spells[i].SpellId))
            {
                if (IsNonCombatSpell(spellInfo))
                {
                    pUser->SendEquipError(EQUIP_ERR_NOT_IN_COMBAT,pItem,NULL);
                    return;
                }
            }
        }
    }

меняем на

    if (pUser->isInCombat())
    {
        for(int i = 0; i < MAX_ITEM_PROTO_SPELLS; ++i)
        {
            if (SpellEntry const *spellInfo = sSpellStore.LookupEntry(proto->Spells[i].SpellId))
            {
                if (IsNonCombatSpell(spellInfo))
                {
                    pUser->SendEquipError(EQUIP_ERR_NOT_IN_COMBAT,pItem,NULL);
                    return;
                }
            }
        }
        // todo may be find better error.
        if (pItem->IsPotion() && pUser->GetLastPotionId())
        {
            pUser->SendEquipError(EQUIP_ERR_OBJECT_IS_BUSY,pItem,NULL);
            return;
        }
            
    }