Hello. I have a problem with my gui screen. It crash game from time to time(but mostly it works correctly) without any obvoiuse reasons, when it tries to call "super.renderBackground".

java.lang.NullPointerException: Cannot read field "level" because "this.minecraft" is null
	at net.minecraft.client.gui.screens.Screen.renderBackground(Screen.java:442) ~[forge-1.19.2-43.1.47_mapped_official_1.19.2-recomp.jar%23183!/:?] 

May be some one know why it happens?

 

My screen: https://github.com/NightKosh/Sophisticated-wolves/blob/feature/1.19.2_update/src/main/java/sophisticated_wolves/gui/WolfFoodConfigScreen.java

"debug" and "latest" logs https://gist.github.com/NightKosh/e73c29bb2ea336b1d6336f64d92ec384

Edited November 10, 20222 yr by Kosh

Line 522: java.lang.IllegalStateException: Rendersystem called from wrong thread

Screens are client side only, so make sure you're opening the screen from the client

 

Edited November 10, 20222 yr by MistaOmega

10 minutes ago, MistaOmega said:

Line 522: java.lang.IllegalStateException: Rendersystem called from wrong thread

Screens are client side only, so make sure you're opening the screen from the client

 

So

Minecraft.getInstance().setScreen(....);

opens screen for both client and server side?

 

 

It looks like it solve my problem. Thank you!

Edited November 10, 20222 yr by Kosh

Yeah, I'd check to see what tick that you're referencing during this block:

 

else if (FoodUtils.isBone(stack)) {
                Minecraft.getInstance().setScreen(new WolfFoodConfigScreen(this));
                stack.shrink(1);
                return InteractionResult.SUCCESS;
            }

add a level.IsClientSide() check before setting that screen so that when the server side tick passes through that function, it is ignored