[1.12.2] getMaterial and actual state

[Pyre540]

Hello,

I am extending vanilla redstone wire block that it has additional property - Color. The color value is stored in tileEntity since metadata is used to store redstone power value. Of course I override getActualState method to return state with color.

//getColor method retrieves color from tileEntity
@Override
public IBlockState getActualState(IBlockState state, IBlockAccess worldIn, BlockPos pos){
    state = super.getActualState(state, worldIn, pos);
    return state.withProperty(COLOR, getColor(worldIn, pos));
}

 

Blocks of different colors have different properties/behavior. One of them is that the blue colored block is not washed away by water. I achieved this by returning the appropriate material based on the state of the block.

@SuppressWarnings("deprecation")
@Override
public Material getMaterial(IBlockState state) {
    return state.getValue(COLOR) == WATERPROOF_COLOR ? ModMaterials.CIRCUITS_WATERPROOF : super.getMaterial(state);
}

 

The problem is that the passed state argument does not use getActualState method but World#getBlockState that only returns the metadata-based state, so after reloading the world this state contains the default color.

It is unfortunate that the getMaterial method does not take World and BlockPos as its parameters like for example getExplosionResistance method.

 

@Override
public float getExplosionResistance(World world, BlockPos pos, @Nullable Entity exploder, Explosion explosion) {
    return getColor(world, pos) == EXPLOSION_PROOF_COLOR ? EXPLOSION_PROOF_BLOCK_RESISTANCE : super.getExplosionResistance(world, pos, exploder, explosion);
}

 

Is there a way for the getMaterial method to use the "actual" state? Or is there, for example, any event that I can use to prevent block being removed by water?

[Animefan8888]

  On 10/19/2018 at 1:38 PM, Pyre540 said:

 

 Is there a way for the getMaterial method to use the "actual" state?

Expand  

Not exactly, what it looks like you will have to do is. In your TEs class override onLoad and change the blockstate there and tell the client it has changed. Or, and this might be the more elegant solution, maybe override getExtendedState and see if that does what you need to do.

[V0idWa1k3r]

  On 10/19/2018 at 2:17 PM, Animefan8888 said:

override getExtendedState

Expand  

getExtendedState is only used when rendering the block. It is only ever called in BlockRendererDispatcher#renderBlock and ForgeHooksClient.getDamageModel.

[Animefan8888]

  On 10/19/2018 at 2:26 PM, V0idWa1k3r said:

getExtendedState is only used when rendering the block. It is only ever called in BlockRendererDispatcher#renderBlock and ForgeHooksClient.getDamageModel.

Expand  

Ok, didn't know its call hierarchy, trying to help while I'm away from home.

[Pyre540]

  On 10/19/2018 at 2:17 PM, Animefan8888 said:

In your TEs class override onLoad and change the blockstate there and tell the client it has changed.

Expand  

This seems to do the trick. Thanks