[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;
        }
            
    }

[Anti]

Тоже самое патчем, по идее будет работать.

[zhenya]

варианты были или, а не вместе
в общем я думаю более правильно решение проблемы в void WorldSession::HandleUseItemOpcode(WorldPacket& recvPacket)

[Anti]

я видимо немного тупанул
Счяс переделаемс.

Там у меня был баг, что при юзаньи банки потом невозможно кастить. Фиксед.
Работает, но блин эрор меседж "вы больше не можете пить" - лихо! Плюсую
И кстати, я думаю все простят если не выводить еррор меседж.

        if (pItem->IsPotion() && pUser->GetLastPotionId())
            return;

Код от этого выигрывает и по сути дела нету никакой нужды что-то выводить, тк вывод для не частого действия, длящегося доли секунды, не нужен.

[Vladimir]

В [9867]. Спасибо. Специальное Anti за подготовку в виде diff-файла.