Fix incompatibility of ItemStack.to_table() introduced by stack meta

This commit is contained in:
rubenwardy 2017-02-05 18:15:46 +00:00
parent 8bc6a303b4
commit 0680c47d6c
2 changed files with 28 additions and 15 deletions

@ -848,6 +848,21 @@ ItemStack read_item(lua_State* L, int index,Server* srv)
istack.metadata.setString("", value); istack.metadata.setString("", value);
} }
lua_getfield(L, index, "meta");
fieldstable = lua_gettop(L);
if (lua_istable(L, fieldstable)) {
lua_pushnil(L);
while (lua_next(L, fieldstable) != 0) {
// key at index -2 and value at index -1
std::string key = lua_tostring(L, -2);
size_t value_len;
const char *value_cs = lua_tolstring(L, -1, &value_len);
std::string value(value_cs, value_len);
istack.metadata.setString(name, value);
lua_pop(L, 1); // removes value, keeps key for next iteration
}
}
return istack; return istack;
} else { } else {
throw LuaError("Expecting itemstack, itemstring, table or nil"); throw LuaError("Expecting itemstack, itemstring, table or nil");

@ -225,11 +225,10 @@ int LuaItemStack::l_to_table(lua_State *L)
lua_pushinteger(L, item.wear); lua_pushinteger(L, item.wear);
lua_setfield(L, -2, "wear"); lua_setfield(L, -2, "wear");
if (item.metadata.size() == 1 && item.metadata.contains("")) { const std::string &metadata_str = item.metadata.getString("");
const std::string &value = item.metadata.getString(""); lua_pushlstring(L, metadata_str.c_str(), metadata_str.size());
lua_pushlstring(L, value.c_str(), value.size());
lua_setfield(L, -2, "metadata"); lua_setfield(L, -2, "metadata");
} else {
lua_newtable(L); lua_newtable(L);
const StringMap &fields = item.metadata.getStrings(); const StringMap &fields = item.metadata.getStrings();
for (StringMap::const_iterator it = fields.begin(); for (StringMap::const_iterator it = fields.begin();
@ -240,8 +239,7 @@ int LuaItemStack::l_to_table(lua_State *L)
lua_pushlstring(L, value.c_str(), value.size()); lua_pushlstring(L, value.c_str(), value.size());
lua_settable(L, -3); lua_settable(L, -3);
} }
lua_setfield(L, -2, "metadata"); lua_setfield(L, -2, "meta");
}
} }
return 1; return 1;
} }