[1.15.2] getBlockState does not updaate block for all players

[TallYate]

@Override
    public ActionResultType onItemUse(ItemUseContext context) {
        if(context.getWorld().getBlockState(context.getPos()).getBlock()==Blocks.BEDROCK) {
            context.getWorld().setBlockState(context.getPos(), BlockInit.busted_bedrock.getDefaultState());
            context.getItem().damageItem(10000, context.getPlayer(), player -> {});
        }
        return ActionResultType.PASS;
   }

This works fine, but with another player, they don't see the block change unless they left or right click it.

Not including a flag defaults to 3, which should update the block and send the change to clients.

[DavidM]

You need to do this on the server side. Use World#isRemote to check whether the sides (true for client, false for server).

[TallYate]

So this should work?
 

@Override
  public ActionResultType onItemUse(ItemUseContext context) {
  if(!context.getWorld().isRemote) {
    if(context.getWorld().getBlockState(context.getPos()).getBlock()==Blocks.BEDROCK) {
      context.getWorld().setBlockState(context.getPos(), BlockInit.busted_bedrock.getDefaultState(), 3);
      context.getItem().damageItem(10000, context.getPlayer(), player -> {});
    }
    return ActionResultType.PASS;
  }
  else
  {
    return ActionResultType.FAIL;
  }
}

 

[Novârch]

4 minutes ago, TallYate said:

So this should work?

Most likely, also why aren't you ever returning ActionResultType.SUCCESS?

[TallYate]

Mostly because I don't know what they do

[TallYate]

Oh thanks Novârch, changing it from ActionResultType.PASS to ActionResultType.SUCCESS fixed my problem!

Edited June 9, 2020 by TallYate
ActionResult to ActionResultType