From 8cd2c43afe043a85451d9e3e5aba71b8b7c6e0e9 Mon Sep 17 00:00:00 2001 From: Yohann Ferreira Date: Fri, 2 Dec 2011 23:51:07 +0100 Subject: Fixed the chr_inv_count function to handle equipment. the function can now count in the inventory and/or the player's equipment. I also fixed the script function and added a use case in the example map. + Fixes after Ablu's review. + 2nd fix after Ablu's review: Fix the inventory remove behaviour. Resolves: Mana-Mantis #288 Reviewed-by: Ablu --- src/scripting/lua.cpp | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) (limited to 'src/scripting/lua.cpp') diff --git a/src/scripting/lua.cpp b/src/scripting/lua.cpp index f681133..1465873 100644 --- a/src/scripting/lua.cpp +++ b/src/scripting/lua.cpp @@ -435,23 +435,28 @@ static int chr_inv_change(lua_State *s) } /** - * mana.chr_inv_count(Character*, int item_id...): int count... + * mana.chr_inv_count(Character*, bool in inventory, bool in equipment, + * int item_id...): int count... * Callback for counting items in inventory. + * The function can search in inventory and/or in the equipment. */ static int chr_inv_count(lua_State *s) { Character *q = getCharacter(s, 1); - if (!q) + if (!q || !lua_isboolean(s, 2) || !lua_isboolean(s, 3)) { raiseScriptError(s, "chr_inv_count called with incorrect parameters."); return 0; } - int nb_items = lua_gettop(s) - 1; - lua_checkstack(s, nb_items); + + bool inInventory = lua_toboolean(s, 2); + bool inEquipment = lua_toboolean(s, 3); + + int nb_items = lua_gettop(s) - 3; Inventory inv(q); int id, nb = 0; - for (int i = 2; i <= nb_items + 1; ++i) + for (int i = 4; i < nb_items + 4; ++i) { ItemClass *it; if (lua_isnumber(s, i)) @@ -474,7 +479,7 @@ static int chr_inv_count(lua_State *s) } else { - nb = inv.count(id); + nb = inv.count(id, inInventory, inEquipment); lua_pushinteger(s, nb); } } -- cgit