Send a message to a user on login?

I have this:

Code:

@EventHandler
    public void onPlayerLogin(PlayerLoginEvent event) {
        Player player = event.getPlayer();
        player.sendMessage("Unimportant");
           
       
    }

however it does not want to work. I just started with Bukkit developing today, and wouldn't be surprised if it was a simple, easy-to-fix mistake.

Thank you!

 

You are looking for PlayerJoinEvent as opposed to PlayerLoginEvent. Reason for that is during Login you are staring at a nice "Logging in...".

 

use this:

Code:java

@EventHandler
public void onPlayerLogin(PlayerLoginEvent event){
final Player player = event.getPlayer();
Bukkit.getServer().getScheduler().scheduleSyncTask(plugin, new Runnable(){
public void run(){
player.sendMessage("Unimportant");
}
}
}

EDIT: And half ninja'd by Giant

 

travja that scheduler is not really required though. It should work just fine without. Edit, yours would also still listen to the Login event instead of the Join event...

 

Giant True, but I find even with PlayerJoinEvent that they aren't quite in game yet and therefore won't do anything with them...

 

travja During PlayerJoinEvent the player is in the game even though the player himself might still see a black screen. Anything you do to the player at that point, will be happening in the game. It might however happen that a spam happy plugin like Essentials is beating you to being the last plugin to send the message, causing yours to fall out of view...

The reason why teleporting during the event is not too clever is because you would break off a chunkload mid loading. And I suppose it may cause the client to crash...

 

Hmm. Two things.

1. An error occurs that says that plugin cant be resolved to a variable.
2. Another one on one of the brackets that says a ) and a ; are missing.

Thank you for your help!

 

replace plugin with this and replace scheduleSyncDelayedTask with scheduleSyncTask

 

TobyG123 simply changing Login to Join in your own code will work just as well...

 

Giant TobyG123 if it doesn't, add the scheduler.

 

Giant Still nothing.

I dont care, here is the ENTIRE plugin:

Code:

package net.arcadecraft;
 
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerLoginEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class AlphaAccess extends JavaPlugin {
 
    public void onEnable(){
        getLogger().info("AlphaAccess has been enabled!");
        getLogger().info("Custom ArcadeCraft plugin by ");
       
    }
 
    public void onDisable() {
        getLogger().info("AlphaAccess has been disabled!");
    }
    @EventHandler
    public void onPlayerJoin(PlayerJoinEvent event) {
        Player player = event.getPlayer();
        player.sendMessage(ChatColor.GREEN + "Welcome to ArcadeCraft Alpha," + ChatColor.BLUE + player + ChatColor.GREEN + "!" + ChatColor.GOLD + "If you are a developer or have Alpha Access, please use /access to authenticate.");
           
   
    }
   
   
   
 
 
   
   
   
}

The scheduler gave me a "Plugin could not be assigned to a variable or something like that.

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

 

You forgot to register the event in your onEnable...
Add getServer().getPluginManager().registerEvents(this, this) to there.
And also make sure you extend JavaPlugin AND implement Listener!

 

Add the scheduler as it is in my post.

EDIT: Ninja'd by Giant with correct answer!

 

Thank you Giant and travja!

 

Your very welcome!
oooooh! I've reached the msg spam limit!

 

Giant
When I log in, it shows me as something like CraftPlayer{name=TobyG123}

 

TobyG123 Oh, that is because you output the full Player object. Which then goes into it's toString() method...

What you want to do to fix that is change the player in the message to player.getName()! Like so:

Code:

player.sendMessage(ChatColor.GREEN + "Welcome to ArcadeCraft Alpha," + ChatColor.BLUE + player.getName() + ChatColor.GREEN + "!" + ChatColor.GOLD + "If you are a developer or have Alpha Access, please use /access to authenticate.");

 

Oh! I went to sleep without thanking you!

Thank you!

 

No worries, I K.O.ed at 7 or 8 am anyhow...