TylerS1066
TylerS1066
> * [ ] **NEW** Subcraft Move This would be best left for a separate PR. I believe there are many assumptions that Subcrafts never move, so we'll have to...
Correct me if I'm wrong, but don't placeholders require a dependency on placeholder API? Base Movecraft has the goal of only depending on Spigot to allow extremely fast updates and...
With the merging of #674, the material makeup of a craft is cached in `Craft#getDataTag(Craft.MATERIALS)`. While we unfortunately don't cache fly and move blocks, that would be a good addition...
This likely is the anti double-chest merging protection kicking in and preventing the craft from moving in a way which could create a double chest (or a double barrel).
In a similar vein, forbidden blocks should also not stop movement of sinking crafts.
I ran a test and this does not fully work yet. It's better than before (without any update packets), but not fully working.
> What do you mean with not fully working? like sometimes it works sometimes it doesn't? Correct, it worked some of the time but not all. I didn't try it...
Rather than wait for a click, feedback could happen on placement.
Put a comment in here once you've tested the PR and I'll take a look.
I thought that NBT data for blocks was part of the `BlockState`, in which case it should be moved with it? If not, we'd need to do an analysis of...