Carpet

Carpet

2M Downloads

[Bug] movableBlockEntities can delete items in a chest if the piston is unloaded at the wrong moment

manyrandomthings opened this issue ยท 5 comments

commented

Steps to recreate:

Put a chest/barrel/furnace/etc with items in front of a piston
Run /tick freeze
Power the piston
Run /tick step 2
Save and quit
Load world
Items in the chest are missing

commented

This appears to be fixed for 1.17+ since carpet 1.4.39

commented

even with setDefaults set - so something with loading is messed up.

commented

to specify - doesn't happen with /tick step 1 - the piston is saved and loaded. With tick step 2 BE is saved, but not loaded back up (loading code 'fromTag' doesn't even execute).

commented

Have you tried reproducing with /tick step 3 and/or /tick step 4? What results do you get?

commented

welp, it takes a piston 3 ticks to push the block to I guess nothing special. 1 and 2 are the 'movement' ticks. While 1 uses both fromTag and toTag and works correctly, with 2 tileentity is not being saved with toTag. Haven't look deeper - not really have time right for these things.