Progressive Mode can "miss" a couple of craft recipes about craftguide HOT 5 CLOSED

Maybe also check on certain events, like actually crafting the item. ;-)

I think polling the inventory all the time is not a pretty solution, I suppose there are still ways for the mod to “miss” items. But it might be OK for a workaround.

Will test later.

What we would actually need would be an engine call to detect when the contents of the player inventory changed. This would make things much simpler.

from craftguide.

Maybe also check on certain events, like actually crafting the item

0956e86

from craftguide.

Progressive mode currently only updates the "discovered" recipes whenever the filter is applied (when the craftguide is opened). It does this by adding undiscovered items in the player's inventory to the discovered list. The planks are never discovered because you don't open the craftguide with them in your inventory.

It would be better to "discover" an item the moment it appears in a player's inventory, but there is no Minetest callback for this. The current system could be improved by discovering items on_dignode, but this won't cover all cases so it would still be necessary to periodically check the player's inventory for undiscovered items. Doing this check when the guide is opened makes no sense because a player could find all the items in the game and put them in a chest, then open the guide with an empty inv, and they will have discovered nothing.

from craftguide.

We could check the current player inventory inside minetest.register_globalstep, and if an item is not inside data.inv_items: update the progressive items list according to that.

from craftguide.

96c3047
Can you test please?

from craftguide.