Issues with checking config in onEnable

So I'm trying to make a plugin where you can set the weather permanently. Don't ask why, I'm sure there are ways to do it with another plugin or even in the game.

Code:java

 
public void onEnable()
{
//reloadConfig();
 
weatherState = getConfig().getString("WeatherState");
 
System.out.println(weatherState);
 
if(getConfig().getString("WeatherState").equals("sun"))
{
getServer().broadcastMessage("sun");
getServer().getWorlds().get(0).setStorm(false);
}
else if(getConfig().getString("WeatherState").equals("storm"))
{
getServer().broadcastMessage("storm");
getServer().getWorlds().get(0).setStorm(true);
}
 
getServer().getPluginManager().registerEvents(this, this);
}

Whenever the plugin starts I want it to check the config to see if WeatherState is storm, sun, or off then set the weather to the corresponding setting. The if statement up there checking only works when I reload the server (/reload) but not when I stop and then start it. Any ideas as to why?

 

The first thing i was noticing looking through your code: Why are you calling

Code:java

getConfig().getString("WeatherState")

multiple times? You already have the String 'WeatherState', so use it. (I believe there's a third weather state called thuder btw)

For the weather: If it works on reloading but not on stopping/starting i guess it has something to do with the world not properly loaded yet. Try to shedule it some ticks after the initial start of the server.

Also i don't know if

Code:java

getServer().getWorlds().get(0)

is the best way to get the current world. (Didn't need to change a world without some reference inside of it yet)


I hope that helped a little bit.

 

I don't see why it possibly hAppends. Usually onenable fires each time plugin is enabled. That's weird. Try adding a delay before seting to whatever.

 

Any errors on startup?

 

jojo1541 I was experimenting with getting the state directly from the config or from the variable. I'll try adding ticks and see if it works. mine-care I know for a fact that it is running on start up and reload, but for some reason the if statement isn't working on start up, but on reload it works great. reider45 Not any that I've seen after testing it a bigillion times.

I also have a WeatherChangeEvent event that checks if the weather is being changed. If weatherState is set to sun it won't allow the weather to evaluate to storm and if it's set to storm it won't let the weather evaluate to sun. That isn't working either! When I stop and start the server it doesn't work but it works fine when I reload.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Code:java

package WT;
 
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.weather.WeatherChangeEvent;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
 
 
public class WeatherTest extends JavaPlugin implements Listener {
 
PluginManager PM;
 
public void onEnable() {
 
PM = getServer().getPluginManager();
PM.registerEvents(this, this);
 
getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
 
@Override
public void run() {
 
getServer().getWorlds().get(0).setStorm(true);
 
}
 
 
 
}, 30L, 30L);
}
 
public void onDisable() {
 
}
 
@EventHandler
public void onWeatherChangeEvent(WeatherChangeEvent Event) {
 
if (Event.toWeatherState()) {
Event.setCancelled(true);
getServer().broadcastMessage("Weather set to sun again");
}
 
}
 
 
}
 

works fine for me. The Event fires everytime the weather changes. (Per command, naturally and per plugin)

Maybe there's something wrong with your listener?