Jump to content

[1.12] Preventing block breaking on a server


AntiRix

Recommended Posts

Hi,

 

I'm using the following code to prevent the player breaking carrots which aren't fully-grown. It works fine on singleplayer, but not at all on multiplayer, in fact on multiplayer the event isn't being fired at all. Why could this be?

 

@SubscribeEvent
    public void onBlockBreak(BreakEvent event)
    {
        IBlockState state = event.getState();
        net.minecraft.block.Block block = state.getBlock();
        
        for (Object o : state.getProperties().entrySet())
        {
            Map.Entry e = (Map.Entry)o;
            
            if (e.getKey() instanceof PropertyInteger)
            {
                PropertyInteger prop = (PropertyInteger)e.getKey();

                if (prop.getName().equals("age"))
                {
                    int age = state.getValue(prop);
                    if (age < 7) event.setCanceled(true);
                }
            }
        }
}
Edited by AntiRix
Link to comment
Share on other sites

I've adjusted the code, but now it seems the block doesn't even have any properties. 

 

@SubscribeEvent
	public void onBlockBreak(PlayerInteractEvent.LeftClickBlock event)
	{
		IBlockState state = mc.theWorld.getBlockState(event.getPos());
		
		mc.thePlayer.addChatMessage(new TextComponentString("onBlockBreak"));
		for (Object o : state.getProperties().entrySet())
		{
			Map.Entry e = (Map.Entry)o;
			
			if (e.getKey() instanceof PropertyInteger)
			{
				PropertyInteger prop = (PropertyInteger)e.getKey();
				mc.thePlayer.addChatMessage(new TextComponentString(prop.getName() + ": " + state.getValue(prop)));
				if (prop.getName().equals("age"))
				{
					int age = state.getValue(prop);
					
					if (age < 7)
					{
						event.setUseBlock(Result.DENY);
						event.setUseItem(Result.DENY);
						event.setCanceled(true);
					}
					
					return;
				}
			}
		}
}

 

Edited by AntiRix
Link to comment
Share on other sites

If I have the following at the beginning of the method, nothing happens.

 

if (!mc.theWorld.isRemote) return;

 

If I have the following, properties are retrieved but it still doesn't cancel the breaking, even though age < 7 is true:

 

if (mc.theWorld.isRemote) return;

 

This makes no sense considering what you just said

Edited by AntiRix
Link to comment
Share on other sites

  • Guest locked this topic
Guest
This topic is now closed to further replies.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • They were already updated, and just to double check I even did a cleanup and fresh update from that same page. I'm quite sure drivers are not the problem here. 
    • i tried downloading the drivers but it says no AMD graphics hardware has been detected    
    • Update your AMD/ATI drivers - get the drivers from their website - do not update via system  
    • As the title says i keep on crashing on forge 1.20.1 even without any mods downloaded, i have the latest drivers (nvidia) and vanilla minecraft works perfectly fine for me logs: https://pastebin.com/5UR01yG9
    • Hello everyone, I'm making this post to seek help for my modded block, It's a special block called FrozenBlock supposed to take the place of an old block, then after a set amount of ticks, it's supposed to revert its Block State, Entity, data... to the old block like this :  The problem I have is that the system breaks when handling multi blocks (I tried some fix but none of them worked) :  The bug I have identified is that the function "setOldBlockFields" in the item's "setFrozenBlock" function gets called once for the 1st block of multiblock getting frozen (as it should), but gets called a second time BEFORE creating the first FrozenBlock with the data of the 1st block, hence giving the same data to the two FrozenBlock :   Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=head] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@73681674 BlockEntityData : id:"minecraft:bed",x:3,y:-60,z:-6} Old Block Fields set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=3, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} Frozen Block Entity set BlockState : Block{minecraft:black_bed}[facing=east,occupied=false,part=foot] BlockPos{x=2, y=-60, z=-6} BlockEntity : net.minecraft.world.level.block.entity.BedBlockEntity@6d1aa3da BlockEntityData : {id:"minecraft:bed",x:2,y:-60,z:-6} here is the code inside my custom "freeze" item :    @Override     public @NotNull InteractionResult useOn(@NotNull UseOnContext pContext) {         if (!pContext.getLevel().isClientSide() && pContext.getHand() == InteractionHand.MAIN_HAND) {             BlockPos blockPos = pContext.getClickedPos();             BlockPos secondBlockPos = getMultiblockPos(blockPos, pContext.getLevel().getBlockState(blockPos));             if (secondBlockPos != null) {                 createFrozenBlock(pContext, secondBlockPos);             }             createFrozenBlock(pContext, blockPos);             return InteractionResult.SUCCESS;         }         return super.useOn(pContext);     }     public static void createFrozenBlock(UseOnContext pContext, BlockPos blockPos) {         BlockState oldState = pContext.getLevel().getBlockState(blockPos);         BlockEntity oldBlockEntity = oldState.hasBlockEntity() ? pContext.getLevel().getBlockEntity(blockPos) : null;         CompoundTag oldBlockEntityData = oldState.hasBlockEntity() ? oldBlockEntity.serializeNBT() : null;         if (oldBlockEntity != null) {             pContext.getLevel().removeBlockEntity(blockPos);         }         BlockState FrozenBlock = setFrozenBlock(oldState, oldBlockEntity, oldBlockEntityData);         pContext.getLevel().setBlockAndUpdate(blockPos, FrozenBlock);     }     public static BlockState setFrozenBlock(BlockState blockState, @Nullable BlockEntity blockEntity, @Nullable CompoundTag blockEntityData) {         BlockState FrozenBlock = BlockRegister.FROZEN_BLOCK.get().defaultBlockState();         ((FrozenBlock) FrozenBlock.getBlock()).setOldBlockFields(blockState, blockEntity, blockEntityData);         return FrozenBlock;     }  
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.