Solved All my getConfig() wrapper methods return null

Like the title says, no matter what I do, all my getConfig() wrappers return null.

I'm building against Bukkit 1.3.2 R-1.1 BTW

For example, I have this YAML file

Code:yaml

 
items:
pumpkin:
enabled: true
healFoodAmount: 1
healHealthAmount: 0
itemAmount: 10
potionEffects:
- regen
potionStrength:
regen: 1
potionDuration:
regen: 1
messageOnEat: 'You nibbled on some seeds'
messageInsufficient: 'Too..little..seeds..must..satisfy..cravings!'
messageUnable: 'Doesnt seem like you are able to nibble on any seeds'
 

and this is an example of my getter.

Code:java

 
public String getUnableMessage(String type){
String unableMessage = plugin.getConfig().getString("items."+type+".messageUnable");
return unableMessage;
}
 

and I'm using this code to debug

Code:java

 
 
try
{
getLogger().info(config.getUnableMessage("pumpkin"));
}
catch (Exception e)
{
// TODO Auto-generated catch block
getConfig().set("items.pumpkin.messageUnable", "[DEBUG] NOT ENOUGH MESSAGE GOES HERE");
saveConfig();
e.printStackTrace();
}
 

Now, this throws an NPE when using the getUnableMessage() method, but when I use the normal bukkit getConfig().getString(), it works fine. I don't see anything wrong with my wrapper methods.. Someone help please?

Full source of my plugin here
https://github.com/ron975/PumpkiNibble/tree/debug

 

entire stack trace will be helpful for you to determine what is causing the exception

 

im gonna go out on a limb and take a random guess.

Your type string in the config is lowercase, are you using something like, getUnableMessage(itemStack.getType().name());?

If so, try setting it to lowercase, name().toLowerCase()

 

I'm manually passing the string to the method, i.e.
getUnableMessage("pumpkin");

Sagacious_Zed
As for full stack trace, if I put
getLogger().info(config.getUnableMessage("pumpkin"));
in my onEnable()

Code:java

 
8:10:08 PM [SEVERE] Error occurred while enabling PumpkiNibble v1.0 (Is it up to date?)
java.lang.NullPointerException
at net.mystia.PumpkiNibble.PumpkiNibbleMain.onEnable(PumpkiNibbleMain.java:84)
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:217)
at org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoader.java:374)
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:381)
at org.bukkit.craftbukkit.CraftServer.loadPlugin(CraftServer.java:270)
at org.bukkit.craftbukkit.CraftServer.enablePlugins(CraftServer.java:252)
at net.minecraft.server.MinecraftServer.i(MinecraftServer.java:298)
at net.minecraft.server.MinecraftServer.d(MinecraftServer.java:277)
at net.minecraft.server.MinecraftServer.a(MinecraftServer.java:227)
at net.minecraft.server.DedicatedServer.init(DedicatedServer.java:140)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:378)
at net.minecraft.server.ThreadServerApplication.run(SourceFile:539)
 

 

the variable config is null. As far as i can tell your alternating between the variable config which you should not have, and the method call.

 

Sagacious_Zed
config is a reference to my getters class. However, I guess for this I should be using static-access anyways, so I'll try using a static method.

 

config should never be static access.

 

Take a look at my source code, specifically PumpkiNibbleMain.java

config is not an instance of my configuration, but an instance of PumkiNibbleAPI.java (or in this case, PumpkiNibbleAPIDebug.java), which contains all my wrapper methods.

 

Solving this, all I had to do was make my API methods static, I also did

Code:

private static ConfigurationSection config = PumpkiNibbleMain.p.getConfig();

Just in case.