[1.14.4] Replacing a Block Texture

[AzenoX]

Hey guys, I want to change a block's texture when a player click on it.

My goal is to create an altar with the possiblity to place the item on it.

 

There is my code:

	@SubscribeEvent
	public void event(PlayerInteractEvent e) {
		BlockPos bp = e.getPos();
		BlockState bs = e.getWorld().getBlockState(bp);
		Block b = bs.getBlock();
		PlayerEntity p = e.getPlayer();
		
			
		if((b == BlockList.altar) && (p.getHeldItemMainhand().getItem() == ItemList.amulet)) {
			Dimensions.getLogger().info("Trigger On");
			Block.replaceBlock(BlockList.altar.getDefaultState(), BlockList.altar_2.getDefaultState(), e.getWorld(), bp, 0);
			p.setHeldItem(e.getHand(), new ItemStack(Items.AIR, 1));
		}
			
		else if((b == BlockList.altar_2) && (p.getHeldItemMainhand().getItem() == Items.AIR)) {
			Dimensions.getLogger().info("Trigger Off");
			Block.replaceBlock(BlockList.altar_2.getDefaultState(), BlockList.altar.getDefaultState(), e.getWorld(), bp, 0);
			p.addItemStackToInventory(new ItemStack(ItemList.amulet, 1));
		}
			
	}

 

I have the debug message in the console and our inventory is modifying correctly  but the block does not replace.

 

Do someone know how to do that ?

Thank you.

[AzenoX]

13 minutes ago, diesieben07 said:

• Subscribing to just PlayerInteractEvent makes no sense . Choose one of the sub-events.
• Make sure to check that you are on the server (World#isRemote is false).
• You need to specify the correct block placement flags, you can't just use 0. See Constants.BlockFlags.

I'm a java beginner and I don't understand Interfaces or Abstraction very well. Where I can see childs event of PlayerInteractEvent ?

 

I changed flags and that works:

	@SubscribeEvent
	public void event(PlayerInteractEvent e) {
		BlockPos bp = e.getPos();
		BlockState bs = e.getWorld().getBlockState(bp);
		Block b = bs.getBlock();
		PlayerEntity p = e.getPlayer();
		
		if(!e.getWorld().isRemote) {
				
				if((b == BlockList.altar) && (p.getHeldItemMainhand().getItem() == ItemList.amulet)) {
					Dimensions.getLogger().info("Trigger On");
					Block.replaceBlock(BlockList.altar.getDefaultState(), BlockList.altar_2.getDefaultState(), e.getWorld(), bp, Constants.BlockFlags.DEFAULT_AND_RERENDER);
					p.setHeldItem(e.getHand(), new ItemStack(Items.AIR, 1));
				}
				
				else if((b == BlockList.altar_2) && (p.getHeldItemMainhand().getItem() == Items.AIR)) {
					Dimensions.getLogger().info("Trigger Off");
					Block.replaceBlock(BlockList.altar_2.getDefaultState(), BlockList.altar.getDefaultState(), e.getWorld(), bp, Constants.BlockFlags.DEFAULT_AND_RERENDER);
					p.addItemStackToInventory(new ItemStack(ItemList.amulet, 1));
				}
		}
	}

 

Thank you a lot !

[Draco18s]

44 minutes ago, AzenoX said:

I'm a java beginner and I don't understand Interfaces or Abstraction very well.

Interfaces are code-objects that describe what an object has or can do.

 

Consider this example:
Knobs expose a method called "twist": interacting with a knob involves twisting.

 - Dimmer switches are Knobs

Toggles expose a method called "change state": interacting with a toggle involves changing its state from ON to OFF or vice versa.

 - Light switches are Toggles

Buttons expose a method called "engage": interacting with a button involves changing its state from OFF to ON, and it automatically changes back when you stop.

 - Keyboard keys are Buttons

 

A bathroom fan timer implements both Knob and Button in the sense that you interact to twist it, but it automatically turns back OFF after a duration because you've stopped interacting with it.

 

Abstract classes are a code-object that exists halfway between a typical class and a typical interface, providing some default functionality, but leaving some up to its implementors.

 

None of that is relevant to events, however. What D7 is referring to is that the PlayerInteractEvent is that it has subclasses that are more specific. Always use the most specific event for what you are doing.