MrTigreroux / TigerReports

Spigot plugin to manage player reports.
16 stars 17 forks source link

ConfigUtils.getInfoMessage (String;String;)String; maybe called before the configuration loads #161

Closed zhusun890 closed 4 months ago

zhusun890 commented 4 months ago

When the plugin enables, it invokes MenuRawItem.init ()V (at TigerReports.java, line 84) and this checks the server version using VersionUtils.getMinecraftVersion ()I when getMinecraftVersion ()I caught an NumberFormatException, it calls ConfigUtils.getInfoMessage (String;String;)String; But currently, all configurations wasn't loaded, because they're going to be loaded when TigerReports.load ()V was called, but it would've called after the invocation MenuRawItem.init ()V, so there might be an NullPointerException being thrown when the plugin enables sometimes.

MrTigreroux commented 4 months ago

Hello, thank you for reporting this issue, I just fixed it in the new version 5.3.3. Do not hesitate if you find other issues.