#409 Client::ConsumeFoodDrink item to be deleted from DB before removing item

已关闭
image2 年之前创建 · 2 条评论
image 评论于 2 年之前

``` else {

        GetPlayer()->GetEquipmentList()->RemoveItem(slot, true);
        database.DeleteItem(GetPlayer()->GetCharacterID(), item, "EQUIPPED");
    }```

wrong order, database.DeleteItem should be first since we have RemoveItem boolean true which deletes the item

``` else { GetPlayer()->GetEquipmentList()->RemoveItem(slot, true); database.DeleteItem(GetPlayer()->GetCharacterID(), item, "EQUIPPED"); }``` wrong order, database.DeleteItem should be first since we have RemoveItem boolean true which deletes the item
devn00b 评论于 2 年之前
所有者

Fixed pending git update.

client.cpp line 10671 -> 10675

    else {

database.DeleteItem(GetPlayer()->GetCharacterID(), item,EQUIPPED"); GetPlayer()->GetEquipmentList()->RemoveItem(slot, true);

    }
Fixed pending git update. client.cpp line 10671 -> 10675 else { database.DeleteItem(GetPlayer()->GetCharacterID(), item,EQUIPPED"); GetPlayer()->GetEquipmentList()->RemoveItem(slot, true); }
devn00b 评论于 2 年之前
所有者

54a61e0c39

I've seen, nor heard any further issues from this patch, closing related issues. I Will re-open if some problem pops back up.

https://git.eq2emu.com/devn00b/EQ2EMu/commit/54a61e0c391817e2e01cb5f93bc9fd8c13526afa I've seen, nor heard any further issues from this patch, closing related issues. I Will re-open if some problem pops back up.
devn00b2 年之前 关闭
登录 并参与到对话中。
正在加载...
取消
保存
这个人很懒,什么都没留下。