Skip to content

Update to 1.21.8#7

Open
Minecradt wants to merge 5 commits intoTogAr2:masterfrom
Minecradt:master
Open

Update to 1.21.8#7
Minecradt wants to merge 5 commits intoTogAr2:masterfrom
Minecradt:master

Conversation

@Minecradt
Copy link

I did these things:

  • Update the depend version in gradle
  • Fix the placement to use the correct arguments.
  • Update this whole thing to 1.21.8

@Minecradt
Copy link
Author

updated the tests/Main.java!

} else {
event.getInstance().placeBlock(new BlockHandler.Placement(
Block.WATER, event.getInstance(), event.getBlockPosition().relative(event.getBlockFace())));
Block.WATER, event.getBlock(), event.getInstance(), event.getBlockPosition().relative(event.getBlockFace())));
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am pretty sure event.getBlock() does not return the right block. You should take the current block at the placement position (this returns the block at the interaction position)

} else if (event.getPlayer().getItemInHand(event.getHand()).material() == Material.LAVA_BUCKET) {
event.getInstance().placeBlock(new BlockHandler.Placement(
Block.LAVA, event.getInstance(), event.getBlockPosition().relative(event.getBlockFace())));
Block.LAVA, event.getBlock(), event.getInstance(), event.getBlockPosition().relative(event.getBlockFace())));
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same applies here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants