NullPointer on RespawnEvent

I'm stuck on having my player respawn at a certain location. I have put together the following code but am getting a NullPointer on the int x line.

Code:

    @EventHandler
    public void onPlayerRespawn(PlayerRespawnEvent p)
    {
        Player player = p.getPlayer();
        int x = plugin.getConfig().getInt("Plugin." + "Spawn." + "X");
        int y = plugin.getConfig().getInt("Plugin." + "Spawn." + "Y");
        int z = plugin.getConfig().getInt("Plugin." + "Spawn." + "Z");
        p.setRespawnLocation(new Location(player.getWorld(), x, y, z));
    }

 

Sounds like the config doesn't exist.

 

Nope it exists:

Code:

Plugin:
  Spawn:
    X: -114.5
    Y: 66
    Z: 239.5

 

I mean the Config object, not the config.yml

 

Based on your code we can only continue guessing. My turn:

plugin is null.

 

You need to set a delayedTask in the respawn method. It seems to not work immediately and needs to be delayed. I had to do this on a plugin I made where it gave the player a potion effect on respawn.

Code:

@EventHandler
    public void onPlayerRespawn(PlayerRespawnEvent p)
    {
        final Player player = p.getPlayer();
        plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                    public void run() {
        int x = plugin.getConfig().getInt("Plugin." + "Spawn." + "X");
        int y = plugin.getConfig().getInt("Plugin." + "Spawn." + "Y");
        int z = plugin.getConfig().getInt("Plugin." + "Spawn." + "Z");
        p.setRespawnLocation(new Location(player.getWorld(), x, y, z));
    }
 
    }, 20); // ******** The 20. is the delay for the task in TICKS
                        // ********
}

 

This wont fix the NPE if plugin is null but move it up to your plugin.getServer()...

 

This didn't fix it AmShaegar How do I fix the plugin being null?

 

make a constructor

Code:

//pretend 'main' was your main class and //'thisclass' is the class you're having the NPE in
 
main plugin;
thisclass(main c){
plugin=c;
}

that way when you run plugin.whatever, youre accessing the main config

 

Add a plugin instance

Code:

Mainpluginnamehere plugin;
 
    public Listenernamehere(Mainpluginnamehere instance) {
        plugin = instance;
    }

 

Ok so I have multiple classes, a main and the event class. Where would this go? And is Mainpluginnamehere my Main class?

 

Yes main class is Mainpluginnamehere and you can put this in any or all listener classes

 

Couldn't get that to work either.

 

Always follow the same scheme for Listeners:

Code:

public class PlayerListener implements Listener {
   
  private Plugin plugin;
   
  public PlayerListener(Plugin plugin) {
    this.plugin = plugin;
  }
}

 

Ok I tried this but didn't work. I decided to take a break over night so I could think about it and now the nullpointer is on:

Code:

final Player player = p.getPlayer();

EDIT: So doesn't that mean I should check to see if the entity respawning is a player?

 

Are you sure? Is p still the PlayerRespawnEvent? If so it cannotbe null. Bukkit does not pass null as event.

 

I tested that and it didn't work then I changed the timer thing to Bukkit.getServer() and I got an error saying plugin was null.

Wait no. This is line 41, the error line: Oh and p is still the event

Code:

        plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){

Ok so now the lines with the errors are my x y z integers. I found this by replacing my x y z with actual numbers and when I died I was sent to those coordinates. So now I am 100% sure its something with that config.

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