Solved Equip Armour error

I am making it so that when a player right clicks a sign saying "[Equip]", it will equip Iron Armour on them.

Code:java

 
package noah.plugins.armourequip;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.block.Sign;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener {
 
public void onEnable() {
// Register
Bukkit.getServer().getPluginManager().registerEvents(this, this);
}
 
public void onDisable() {
 
}
 
// Main
@EventHandler
public void equipS(PlayerInteractEvent equip) {
Player p = equip.getPlayer();
 
if (equip.getClickedBlock() != null) {
if (equip.getClickedBlock().getType() == Material.WALL_SIGN || equip.getClickedBlock().getType() == Material.SIGN_POST || equip.getClickedBlock().getType() == Material.SIGN) {
Sign block = (Sign) equip.getClickedBlock().getState();
 
if (block.getLine(0).equalsIgnoreCase("[Equip]")) {
 
p.getInventory().setHelmet(new ItemStack(Material.IRON_HELMET));
p.getInventory().setChestplate(new ItemStack(Material.IRON_CHESTPLATE));
p.getInventory().setLeggings(new ItemStack(Material.IRON_LEGGINGS));
 
p.getInventory().setBoots(new ItemStack(Material.IRON_BOOTS));
 
p.sendMessage(ChatColor.RED + "[" + ChatColor.GREEN + "EquipArmour" + ChatColor.RED + "] " + ChatColor.YELLOW + "Armour has been equipped!");
 
}
}
}
 
if (!(p.hasPermission("ArmourEquip.equipsign"))) {
p.sendMessage("" + ChatColor.RED + ChatColor.BOLD + "You dont have permission!");
}
 
}
 
}
 

 

Looks fine. What's the error?

 

adam753
So, I right click the sign and it sends the message, then I hit F5, no armour on me...

 

Try p.updateInventory()?

 

Barinade
It's deprecated.

 

Doesn't mean it won't work, deprecation means there is either a better method or it is suggested to not use it, and I'm pretty sure in this case it's deprecated because it's not always needed, only for aesthetics, so it's waste of performance

I may be wrong, that's just my guess

Deprecated. This method should not be relied upon as it is a temporary work-around for a larger, more complicated issue.