Minetest-WorldEditAdditions icon indicating copy to clipboard operation
Minetest-WorldEditAdditions copied to clipboard

//replacemix shelf bug

Open VorTechnix opened this issue 4 years ago • 7 comments

Bug applies to default:bookshelf, xdecor:multishelf and vessels:shelf. When placed using //replacemix their inventories cannot be opened/accessed.

VorTechnix avatar Oct 08 '21 18:10 VorTechnix

Upon further investigation this is an upstream issue as well as the WorldEdit command //set (and i assume the rest of the placing commands) have the same problem. When we find a solution for this issue we should open an issue/pull request in the WorldEdit repo to apply our solution to the commands there.

VorTechnix avatar Oct 08 '21 18:10 VorTechnix

Chests seem to have the same problem. (But I can't be sure since I have pipeworks which overrides chests).

VorTechnix avatar Oct 08 '21 19:10 VorTechnix

Hmmm, weird. Is this also the case if such items are placed on the ground?

It may also be there's an on_place (or whatever it's called) that needs to be called that isn't when we use Voxel Manipulators.

sbrl avatar Oct 08 '21 23:10 sbrl

I tried both in air and on the ground. Like you I have a suspicion that there is some sort of on_place weirdness going on. If that is the cause it would not be practical to add node filtering and alternate handling to all our commands. So I propose we make something like //fixstor(age) to handle the issue.

Note: I checked the code yesterday and items like chests and shelves are registered using a function called register_storage not register_node.

VorTechnix avatar Oct 10 '21 01:10 VorTechnix

Hmm yeah. It's probably more general than just storage though - something like //setplayer perhaps that uses minetest.set_node()?

sbrl avatar Oct 10 '21 11:10 sbrl

Perhaps this is an ownership issue (which would make this issue overlap with #69). It could be that when storage is placed owner is set but only shown/enforced if protected=true.

VorTechnix avatar Oct 10 '21 14:10 VorTechnix

Only way to know for sure is to experiment I guess!

sbrl avatar Oct 10 '21 15:10 sbrl