[1.8] Different Light Levels Based on Metadata

[DankParrot]

I have a simple ore block with subblocks for different types of ore. I want one of the variants of it to glow, similarly to redstone ore. Would the best way to do this be to override getLightValue, or is there a  preferred way of doing such a thing?

[DankParrot]

Ok, thanks. Just wanted to check to make sure there were no problems that could occur from the way I'm doing it.

[jeffryfisher]

Just be aware that when you start overriding methods to start using metadata where vanilla has never used metadata before, you can run into bugs in vanilla code that only a mod could expose. For example (depending on Forge version), it is possible for some of your block's methods to be called in the midst of deletion after your block has been replaced by air at its coords. If vanilla never tried to get the blockstate at coords, but you do, then you blow up.

 

Therefore, when you start testing, be sure to delete your block to see if it handles deletion correctly. If you get a crash report in a method called against an air block, then your method will need to check that the block at coords is not BlockAir (or is still an instance of your block) and handle the condition gracefully.

 

[Dark32]

You try to find this function in Vanilla block:

public int getLightValue(IBlockAccess world, BlockPos pos)