Block does not Light

[zalasz]

Hi, recently i started with minecraft mod development and i was trying to lit my block but for some reason it does not seem to work. I might have forgot something but im pretty sure it was suppose to work with the piece of code below right? I know that for some of you this might seem a dumb post but i really would appreciate if someone could help me. Thanks in advance.

public class AerialLight extends Block {

    public static final BooleanProperty LIT = BlockStateProperties.LIT;

    public AerialLight(Properties properties){
        super(properties.sound(SoundType.WOOL).noCollission());
        this.registerDefaultState(this.stateDefinition.any().setValue(LIT, Boolean.valueOf(true)));
    }

    public void createBlockStateDefinition(StateDefinition.Builder<Block, BlockState> p_48928_) {
        p_48928_.add(LIT);
    }
}

[zalasz]

  On 12/17/2021 at 9:11 AM, diesieben07 said:

• Use @Override when overriding methods.
• You added a block state property to your block. It can now store whether it should be lit or not. You have not told the game to actually make your block light up. Use the lightLevel method of the Properties class.

Expand  

I thought LIT would actually light my block but apparently is just a way to verify its state my bad haha, but if lightlevel is defined in the constructor how would i then turn it off with use()? Becaus right now i can change my block state with LIT but not the lightlevel. 

[Draco18s]

That boolean property could be called MADE_OF_GOLD for all the relevance it has to its name.

[zalasz]

  On 12/17/2021 at 4:43 PM, Draco18s said:

That boolean property could be called MADE_OF_GOLD for all the relevance it has to its name.

Expand  

Got it, that was really confusing me haha but how do i update my block lightlevel?

[zalasz]

  On 12/17/2021 at 6:08 PM, diesieben07 said:

 

Expand  

But that would be in my block class constructor. I was wondering if i could update the lightLevel after defining it in the constructor for example:

public AerialLight(Properties properties){
        super(properties.sound(SoundType.WOOL).noCollission().lightLevel((p_60954_) -> {
            return 15;
        }));
        this.registerDefaultState(this.stateDefinition.any().setValue(LIT, Boolean.valueOf(true)));
}

public InteractionResult use(BlockState state, Level level, BlockPos pos, Player player, InteractionHand hand, BlockHitResult hit){
	//Update the lightLevel here for 0
	return InteractionResult.SUCCESS;
}

I don't know if I explained myself well😅 but is it possible?

[zalasz]

  On 12/17/2021 at 6:53 PM, diesieben07 said:

lightLevel takes a ToIntFunction as its parameter. You can return different results based on the BlockState.

Expand  

I have tried that before but my problem is that because my class constructor will always be called first i don't know how would i be returning different values with the blockstate from use(). I have tried the following, but obviously it would never work

public static final BooleanProperty LIT = BlockStateProperties.LIT;
private static int LightUp = 15;    

public AerialLight(Properties properties){
        super(properties.sound(SoundType.WOOL).noCollission().lightLevel((p_60954_) -> {
            return LightUp; 
        }));
        this.registerDefaultState(this.stateDefinition.any().setValue(LIT, Boolean.valueOf(true)));
    }

@Override
  public InteractionResult use(BlockState state, Level level, BlockPos pos, Player player, InteractionHand hand, BlockHitResult hit){
        if(state.getValue(LIT)){
            level.setBlock(pos, state.setValue(LIT, Boolean.valueOf(false)), 3);
            LightUp = 0;
            return InteractionResult.SUCCESS;
        }else{
            LightUp = 15;
            level.setBlock(pos, state.setValue(LIT, Boolean.valueOf(true)), 3);
            return InteractionResult.SUCCESS;
        }
    }

I even tried with a verification inside lightLevel as it is a function

[zalasz]

  On 12/17/2021 at 8:57 PM, diesieben07 said:

You have to check the LIT property inside the lambda passed to lightLevel.

Expand  

Bruh, nevermind i am just dumb because when i did the verification for some reason my brain completely erased the fact that "p_60954_" was a parameter for the lambda expression being the reason why nothing was working. Well i appreciate alot your patience, really, thank you.