amadornes / MCMultiPart

A universal multipart API for Modern Minecraft
Other
67 stars 23 forks source link

[MCMP 2.0] After a reload of the world, the parts no longer have an MCMPWorldWrapper as their world #79

Closed 2xsaiko closed 7 years ago

2xsaiko commented 7 years ago

This causes some stuff that relies on the world returning the part state and not the multipart container's state to fail. I'm debugging it at the moment, trying to fix it.

2xsaiko commented 7 years ago

This patch fixes the issue, but I am not sure if there are other problems caused by this (couldn't upload it as .patch file, GitHub is really choosy) 79.patch.txt

2xsaiko commented 7 years ago

More background: My multiparts store their facing in the block state and this gets checked often in the tile entity. Now when I reload the world, the world no longer is a MCMPWorldWrapper that would return the correct block state, but a WorldServer that of course returns the BlockMultipartContainer's state