Coordinate Teleportation & Inventory Updating

Hey Guys!
This plugin works fine so far, but only things wrong is that the inventory glitches. If I right-click the sign, one item would be given, if I did it again, only one item would be given, but when I do it again, it gives me the full kit. I'm guessing I need to update the inventory, but "p.updateInventory" doesn't work.
Also, I can't find a way how to make it so the player would be teleported to a certain world/coordinate, if you could help... It'd be much appreciated!

Here's the code:

Code:java

package net.theaura.AuraKits;
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.block.Sign;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.block.SignChangeEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class AuraKits extends JavaPlugin implements Listener {
 
public final Logger logger = Logger.getLogger("Minecraft");
public static AuraKits plugin;
 
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
PluginDescriptionFile pdfFile = this.getDescription();
this.getServer().getConsoleSender().sendMessage(ChatColor.GREEN + "[" + ChatColor.RED + pdfFile.getName() + ChatColor.GREEN + "]" + ChatColor.GOLD + " Has been enabled!");
}
 
public void onDisable() {
PluginDescriptionFile pdfFile = this.getDescription();
this.getServer().getConsoleSender().sendMessage(ChatColor.GREEN + "[" + ChatColor.RED + pdfFile.getName() + ChatColor.GREEN + "]" + ChatColor.DARK_RED + " Has been disabled!");
}
@EventHandler
public void GAME1(SignChangeEvent sign){
Player p = sign.getPlayer();
if(sign.getLine(0).equalsIgnoreCase("[" + ChatColor.RED + "aurakits" + ChatColor.BLACK + "]")){
if(sign.getLine(1).equalsIgnoreCase(ChatColor.BOLD + "tank")){
p.sendMessage(ChatColor.YELLOW + "Tank class has successfully been added");
}
 
}
}
@EventHandler
public void clicksign(PlayerInteractEvent e) {
Player p = e.getPlayer();
if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
Block b = e.getClickedBlock();
if (b.getType() == Material.SIGN_POST|| b.getType() == Material.WALL_SIGN) {
Sign sign = (Sign) b.getState();
if (sign.getLine(0).equalsIgnoreCase("[" + ChatColor.RED + "aurakits" + ChatColor.BLACK + "]")) {
if (sign.getLine(1).equalsIgnoreCase(ChatColor.BOLD + "tank")) {
if(p.hasPermission("AuraKits.kit.tank")){
//HEALING
p.setHealth(20.0);
p.setFireTicks(0);
p.setFoodLevel(20);
//ITEMSTACKS
ItemStack ihelmet = new ItemStack(Material.IRON_HELMET, 1);
ItemStack ichestplate = new ItemStack(Material.IRON_CHESTPLATE, 1);
ItemStack ileggings = new ItemStack(Material.IRON_LEGGINGS, 1);
ItemStack iboots = new ItemStack(Material.IRON_BOOTS, 1);
ItemStack isword = new ItemStack(Material.IRON_SWORD, 1);
ItemStack bow = new ItemStack(Material.BOW, 1);
ItemStack arrow = new ItemStack(Material.ARROW, 1);
//ENCHANTS
bow.addEnchantment(Enchantment.ARROW_INFINITE, 1);
//ITEMS
p.getInventory().addItem(ihelmet);
p.getInventory().addItem(ichestplate);
p.getInventory().addItem(ileggings);
p.getInventory().addItem(iboots);
p.getInventory().addItem(isword);
p.getInventory().addItem(bow);
p.getInventory().addItem(arrow);
//MESSAGE
p.sendMessage(ChatColor.GREEN + "You have been given the tank kit!");
}else{
p.sendMessage(ChatColor.DARK_RED + "You don't have permission to use tank kit!");
}
}
}
}
}
}
}

EDIT: Fixed some bad grammar XD

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

 

Some things:
Check if the player inventory contains certain items, if it does don't give him that item.
I would also set all the items in an Array, to make it easier.

Code:

p.teleoprt(new Location(world, x, y, z);

 

calebbfmv

// Completely off topic
Omg, your signature is so true right now... i'm going to have to remember all this stuff after all :|

 

for the items, I did it separately because it would be easier to add enchantments. I'll try it out, but if I remember correctly... it didn't work before. Thanks for your help though .

And how would I do that? since I'm not too good at coding since I forgot a ton of things as I took my break from it. calebbfmv

and to top it off, I get this error; http://prntscr.com/20mw1c

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

 

Make a World variable called arenas

 

Told me to do this, http://prntscr.com/20nbox
is that right?

 

Make sure your world is an actual world.

Code:

World arena = Bukkit.getServer().getWorld("worldNameHere");