ramou / ChessMod

Adding Pippin Barr's Chesses to minecraft.
MIT License
3 stars 3 forks source link

Linking boards needs to reset state on the second board #166

Closed ramou closed 2 months ago

ramou commented 2 months ago

Currently, if the state on both boards is the same (ish) it's probably ok. However, you can link boards with different state and then a move will run moveSafely for both, and only work for one, which is annoying. We need a setter that is public for QuantumChessBoardEntity that takes care of the cloning process, call it quantumImprint and pass it a QuantumChessBoardEntity and then we don't have to worry about what else we need to clone down the road for other boards :D

riri404 commented 2 months ago

Will have to ask for more instrctions for this issue. So far I created a skeleton for the quantumImprint() but if I understand well, we will need to deep cope the state before linking. I will push now what I added. Also I was reading and taking notes (especially NBT) on your new addition to the code and I added some comments to help me understand better.