Solved PlayerRespawnEvent - Problem with setting PotionEffects

Hello!
I've a problem with PlayerRespawnEvent in my first plugin associated with kits and PvP. When I'm trying to give player an item or armor - it appears in player's inventory, but when I'm trying to give player some PotionEffects - it doesn't. With PlayerJoinEvent it works.
Here's the code. I'd be thankful to you for solving this trouble.

Code:java

public class HSKitsPvP extends JavaPlugin {
Logger pluginLogger;
public final PlayerJoinListener pj = new PlayerJoinListener();
public final PlayerRespawnListener pr = new PlayerRespawnListener();
@Override
public void onEnable() {
PluginManager pm = getServer().getPluginManager();
pm.registerEvents(this.pj, this);
pm.registerEvents(this.pr, this);
}
@Override
public void onDisable() {
}
 
}
 
public class PlayerRespawnListener implements Listener {
public static HSKitsPvP plugin;
@EventHandler
public void onPlayerRespawn(PlayerRespawnEvent event) {
Player player = event.getPlayer();
 
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 99999,
1));
player.addPotionEffect(new PotionEffect(PotionEffectType.HEALTH_BOOST,
99999, 20));
}
}
 
public class PlayerJoinListener implements Listener {
public static HSKitsPvP plugin;
 
@EventHandler
public void onPlayerJoin(PlayerJoinEvent event) {
Player player = event.getPlayer();
 
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 99999,
1));
player.addPotionEffect(new PotionEffect(PotionEffectType.HEALTH_BOOST,
99999, 20));
}
}

Also, there's one more question. How to change player's spawn when player joins the server? (I want to set up a couple of random spawn locations)
P.S. Sorry for bad English, trying to improve.

 

1) You have to use a delayed task (1 tick).
2) You can use Player.teleport().

 

Thank you!
Using of delayed task like this? I got error in console.

Code:java

 public void onPlayerRespawn(PlayerRespawnEvent event) {
final Player player = event.getPlayer();
plugin.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
public void run() {
player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 99999,
1));
player.addPotionEffect(new PotionEffect(PotionEffectType.HEALTH_BOOST,
99999, 20));
}}, 1L);
}

 

lexmint
Post your error

 

Yes, but:
1. Use runTaskLater, scheduleSyncDelayedTask is less efficient (done tests).
2. Check if player != null.

For your error, make sure plugin is not null.

 

player respawn event thrown before player is actually respawned, this same for nearly all event in bukkit.

 

Thank you for explanations, problem is solved! It was connected with null in the field "plugin" in PlayerRespawnListener - I was so inattentive