Custom clientbound packet handling [1.19] [SOLVED]

[RInventor7]

Hi! I'm trying to send some custom network packets from server to client. The packets are sent but I have some problems handling them on client. To avoid arbitrary chuck generation I'm trying to use level#hasChunckAt(pos). It's deprected. Is this the correct thing to use or should I use something else? And it doesn't execute the code in client because I can get the ServerLevel but I need ClientLevel? How can I get it working? Thanks.

import java.util.function.Supplier;

import com.rinventor.transportmod.core.base.PTMDbg;

import net.minecraft.core.BlockPos;
import net.minecraft.network.FriendlyByteBuf;
import net.minecraft.world.level.Level;
import net.minecraftforge.api.distmarker.Dist;
import net.minecraftforge.fml.DistExecutor;
import net.minecraftforge.network.NetworkEvent;

public class InformationScreenSyncMessage {

	public final BlockPos pos;
	public final String data;

    public InformationScreenSyncMessage(BlockPos pos, String data) {
    	this.pos = pos;
    	this.data = data;
    }

    public InformationScreenSyncMessage(FriendlyByteBuf buffer) {
    	this(buffer.readBlockPos(), buffer.readUtf());
    }

    public void encode(FriendlyByteBuf buffer) {
    	buffer.writeBlockPos(this.pos);
    	buffer.writeUtf(this.data);
    }

    public static void handle(InformationScreenSyncMessage msg, Supplier<NetworkEvent.Context> ctx) {
        ctx.get().enqueueWork(() ->
            // Make sure it's only executed on the physical client
            DistExecutor.unsafeRunWhenOn(Dist.CLIENT, () -> () -> InformationScreenClient.handlePacket(msg, ctx))
        );
        ctx.get().setPacketHandled(true);
    }
    
    public class InformationScreenClient
    {
    	@SuppressWarnings("deprecation")
	public static void handlePacket(InformationScreenSyncMessage msg, Supplier<NetworkEvent.Context> ctx) {
    	    Level level = ctx.get().getSender().getLevel();
    	    BlockPos pos = msg.pos;
    	    if (level.hasChunkAt(pos)) {
    	    	if (level.getBlockEntity(pos) instanceof InformationScreenBlockEntity be) {
    			be.data = msg.data;
    		}
    	    }
    	}
    	
    }

}

 

Edited March 2, 2023 by RInventor7

[warjort]

level = Minecraft.getInstance().level

level.isLoaded(BlockPos)

Mojang have moved to "sections" instead of chunks.

It won't generate chunks on the client, it will crash if you access unloaded chunks.