chris-wjn / ImprovedDamageFramework

Other
7 stars 5 forks source link

A NullPointerException occured when checked the tooltip of some weapons #18

Closed AlanightCreeper closed 11 months ago

AlanightCreeper commented 11 months ago

Mod Version: 0.2.1

Description: After testing for many times, I found that all the weapons which could cause NullPointerException, have one thing in common: I added the Crit Damage Attribute to these weapons in the config.

Critical part of log: Description: Rendering screen java.lang.NullPointerException: Cannot invoke "net.cwjn.idf.util.Color.getColor()" because "color" is null at net.cwjn.idf.util.Util.withColor(Util.java:86) ~[idf-0.2.1-1.19.2.jar%23177!/:0.2.1-1.19.2] {re:mixin,re:classloading} at net.cwjn.idf.event.ClientEventsForgeBus.writeDamageTooltip(ClientEventsForgeBus.java:498) ~[idf-0.2.1-1.19.2.jar%23177!/:0.2.1-1.19.2] {re:classloading} at net.cwjn.idf.event.ClientEventsForgeBus.onItemTooltip(ClientEventsForgeBus.java:289) ~[idf-0.2.1-1.19.2.jar%23177!/:0.2.1-1.19.2] {re:classloading} at net.cwjn.idf.event.__ClientEventsForgeBus_onItemTooltip_ItemTooltipEvent.invoke(.dynamic) ~[idf-0.2.1-1.19.2.jar%23177!/:0.2.1-1.19.2] {re:classloading,pl:eventbus:B} at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73) ~[eventbus-6.0.3.jar%2385!/:?] {} at net.minecraftforge.eventbus.EventBus.post(EventBus.java:315) ~[eventbus-6.0.3.jar%2385!/:?] {} at net.minecraftforge.eventbus.EventBus.post(EventBus.java:296) ~[eventbus-6.0.3.jar%2385!/:?] {} at net.minecraftforge.event.ForgeEventFactory.onItemTooltip(ForgeEventFactory.java:259) ~[forge-1.19.2-43.2.23-universal.jar%23192!/:?] {re:mixin,re:classloading} at net.minecraft.world.item.ItemStack.m41651(ItemStack.java:782) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,xf:fml:forge:itemstack,re:classloading,xf:fml:forge:itemstack,pl:mixin:APP:idf.mixin.json:MixinItemStack,pl:mixin:A} at net.minecraft.client.gui.screens.Screen.m96555(Screen.java:193) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:computing_frames,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,pl:mixin:APP:patchouli_xplat.mixins.json:client.AccessorScreen,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.gui.screens.Screen.m6057(Screen.java:170) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:computing_frames,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,pl:mixin:APP:patchouli_xplat.mixins.json:client.AccessorScreen,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.gui.screens.inventory.CreativeModeInventoryScreen.m6057(CreativeModeInventoryScreen.java:671) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:classloading,xf:fml:jecharacters:jecharacters-gen44,xf:fml:jecharacters:jecharacters-gen45,pl:runtimedistcleaner:A} at net.minecraft.client.gui.screens.inventory.AbstractContainerScreen.m7025(AbstractContainerScreen.java:173) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:runtimedistcleaner:A,re:classloading,pl:runtimedistcleaner:A} at net.minecraft.client.gui.screens.inventory.CreativeModeInventoryScreen.m6305(CreativeModeInventoryScreen.java:636) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:classloading,xf:fml:jecharacters:jecharacters-gen44,xf:fml:jecharacters:jecharacters-gen45,pl:runtimedistcleaner:A} at net.minecraftforge.client.ForgeHooksClient.drawScreenInternal(ForgeHooksClient.java:441) ~[forge-1.19.2-43.2.23-universal.jar%23192!/:?] {re:classloading,re:mixin} at net.minecraftforge.client.ForgeHooksClient.drawScreen(ForgeHooksClient.java:434) ~[forge-1.19.2-43.2.23-universal.jar%23192!/:?] {re:classloading,re:mixin} at net.minecraft.client.renderer.GameRenderer.m109093(GameRenderer.java:896) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,pl:mixin:APP:tslatentitystatus.mixins.json:client.GameRendererMixin,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.Minecraft.m91383(Minecraft.java:1115) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,pl:mixin:APP:tslatentitystatus.mixins.json:client.MinecraftMixin,pl:mixin:APP:architectury.mixins.json:MixinMinecraft,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.Minecraft.m91374(Minecraft.java:700) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:mixin,pl:accesstransformer:B,pl:runtimedistcleaner:A,re:classloading,pl:accesstransformer:B,pl:mixin:APP:tslatentitystatus.mixins.json:client.MinecraftMixin,pl:mixin:APP:architectury.mixins.json:MixinMinecraft,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.main.Main.m239872(Main.java:212) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:classloading,re:mixin,pl:runtimedistcleaner:A,pl:mixin:A,pl:runtimedistcleaner:A} at net.minecraft.client.main.Main.main(Main.java:51) ~[client-1.19.2-20220805.130853-srg.jar%23187!/:?] {re:classloading,re:mixin,pl:runtimedistcleaner:A,pl:mixin:A,pl:runtimedistcleaner:A} at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?] {} at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[?:?] {} at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?] {} at java.lang.reflect.Method.invoke(Method.java:568) ~[?:?] {} at net.minecraftforge.fml.loading.targets.CommonClientLaunchHandler.lambda$launchService$0(CommonClientLaunchHandler.java:27) ~[fmlloader-1.19.2-43.2.23.jar%23101!/:?] {} at cpw.mods.modlauncher.LaunchServiceHandlerDecorator.launch(LaunchServiceHandlerDecorator.java:30) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:53) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:71) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.Launcher.run(Launcher.java:106) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.Launcher.main(Launcher.java:77) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.BootstrapLaunchConsumer.accept(BootstrapLaunchConsumer.java:26) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.modlauncher.BootstrapLaunchConsumer.accept(BootstrapLaunchConsumer.java:23) [modlauncher-10.0.8.jar%2388!/:?] {} at cpw.mods.bootstraplauncher.BootstrapLauncher.main(BootstrapLauncher.java:141) [bootstraplauncher-1.1.2.jar:?] {}

chris-wjn commented 11 months ago

fixed in latest version