[1.16.5] world.setBlock with ModBlock

[grvt]

I'm attempting to replace grass blocks in a world with a ModBlock I called `scorched_earth`. However, `world.setBlock()` requires a block ID. I've looked for some way to get an ID for the block but can't seem to find any workarounds? Am I just doing it wrong?

Example (incomplete) code:

private void scorch(World worldIn, BlockPos pos, Block block) {
     BlockState state = worldIn.getBlockState(pos);
     if(block.isBurning(state, worldIn, pos)) {
         if(block.is(Blocks.GRASS_BLOCK)) worldIn.setBlock(pos, state, 0, ModBlocks.SCORCHED_EARTH.get());
     }
}

Edited October 17, 2021 by grvt

[grvt]

If I'm understanding Intellij right it wants an int? It might just be me reading it wrong though.

 

If I am reading it wrong, what actually is that argument for?

[Luis_ST]

learn basic java, since Block is not and int

you need to pass in the BlockPos (as first parm), the BlockState (as second parm), the place flag (as second parm), you could use 3,
the fourth parm is not needed since there is a method with only these 3 parm

Edited October 17, 2021 by Luis_ST

[grvt]

3 minutes ago, Luis_ST said:

learn basic java, since Block is not and int

I know. My question was what I should be putting there, because I didn't understand the method properly.

2 minutes ago, diesieben07 said:

The 1st parameter is the position to place at. The 2nd parameter is the block state to place. The 3rd parameter are the update flags (Constants.BlockFlags). The 4th parameter is recursion depth.

You really should not be using this overload. Use setBlockAndUpdate(BlockPos, BlockState) for most cases. If you need to set the flags, use setBlock(BlockPos, BlockState, int).

Thank you that makes a lot more sense.

[Draco18s]

Really what happened was that this parameter you were passing:

worldIn.setBlock(pos, state, ... );

was this:

BlockState state = worldIn.getBlockState(pos);

when it should have been this:

ModBlocks.SCORCHED_EARTH.get().getDefaultState()