Solved -removed-

-removed-

 

GrandmaJam
player1 and player2 are the same player?

try using this:

Code:

@EventHandler
public void onJoin(PlayerJoinEvent event)
{
Player player = event.getPlayer();
if (player.hasPermission("silent.join")) {
event.setJoinMessage(null);
player.sendMessage("§7Join> §c" + player.getName() + "§7 joined the game silenty.");
}
}

 

Thanks! That works perfectly now, however the join message for players who don't use silent join is broken. Code is attached.

Code:java

if (!player.isOp()) {
                  event.setJoinMessage("§c+ §7" + player.getName());
}

 

Do you have the closing bracket?

 

-removed-

 

There are numerous things wrong with the code.

First off: When you are sending the message saying that the player joined silently, you are just sending it to them. To send it to everyone who has the permission, make a for loop of all online players and check if they have the permission.

Second: You don't need the check to see if the player is op or not. Going off what I said above, you can check the permission then make an else statement. Example:

Code:java

if(onlinePlayers.hasPermission("perm.perm")) {
//Do stuff for people with the permission.
}else {
//Do stuff for people w/o the permission.
}

 

Don't call the player object 2 times
Edit: Call the player object BEFORE HAND as null, and change it on join.

 

-removed-

 

Words of advice. Learn Java before learning Bukkit. Along with that, learn Bukkit, too. You clearly don't have a good understanding of it.

Code:java

If player has a permission set the join message to something. 
else set the join message to something else. 
 

^ Pseudo code.

 

rete25iscool Imposter.

 

rete25iscool Thanks. But that still doesn't solve my problem.

 

-snip-

EDIT: Loop through the online players, check if they have a certain permission and if they do have it, set the join message for the players with the permission you're checking for.

Code:java

 for (Player player : Bukkit.getOnlinePlayers()) {
if (player.hasPermission("permission.permission") {
// Do stuff. 
}
}
 

Sorry, I misread your original issue and I'm typing on an iPad currently so sorry if there are any spelling errors.

 

GrandmaJam

Don't suppress warnings for no reason, especially when they're telling you EXACTLY why your code isn't working.

 

Player object has hasPermission, no need to call the object.

 

GrandmaJam
onlinePlayers needs to be the Player object when you make a for loop.

 

Regablith ChipDev mythbusterma rete25iscool Thank you all so much! I have done everything you said and the plugin now works

 

Great!!

 

GrandmaJam
Good