search

minetest-mods / unified_inventory

An extensible inventory mod which allows searching crafting and browsing for recipes in the same dialogue.
Other
50 stars 38 forks source link

replacing one (empty) bag w/ another results in the inventory not being usable #208

Closed fluxionary closed 6 months ago

fluxionary commented 2 years ago

you can look inside the new bag, but you can't put anything inside it.

eshattow commented 2 years ago

I found that once a bag item is set to a bag slot i.e. Bag 1, clicking that bag item to grab it and click i.e. "Bag 1" button then placing within its own inventory the bag disappears completely. Not sure if that is an old bug or related to this.

SmallJoker commented 2 years ago

Marked as "Invalid" because this is a Minetest bug, or rather "undefined behaviour" of callback execution order. Waiting for the PR linked above to be merged.

SmallJoker commented 2 years ago

@eshattow That's a different issue. Fixed in 177debd.

SmallJoker commented 1 year ago

@fluxionary This seems to be fixed on my end. Could you please try to reproduce this?

fluxionary commented 1 year ago

@fluxionary This seems to be fixed on my end. Could you please try to reproduce this?

i still have the issue. i'm testing w/ the most recent unified_inventory (https://github.com/minetest-mods/unified_inventory/commit/d6d4bea819fe44ef447491834ef25a834a575bc2) and the most recent minetest (https://github.com/minetest/minetest/commit/531122ee8681d9de7b38879563da4cdcc71bfe44). i get the following warning in my console:

2023-06-20 11:04:32: WARNING[Main]: GUIInventoryList::draw(): The inventory list "bag3contents" @ "current_player" doesn't exist
fluxionary commented 6 months ago

the engine issue got fixed. testing with the most recent build, i can't replicate the issue.