How to fix this error ?

Hey, I am having an issue..
Im checking if the material is the material which is defined in the config..

Code:java

 
String material = this.getConfig().getString("TeleportItem");
 
if(e.getItem().getType().equals(Material.getMaterial(material))) {
}

But im getting errors in the console:

http://pastebin.com/A4VRNAAA

What have I done wrong??
Thanks

 

1. Caused by: java.lang.NullPointerException
2. at main.HubBukkit.onPlayerInteractWithItemTpItem(HubBukkit.java:460) ~[?:?]

Line 460, what's wrong with that one (What is line 460)
And what's in your config
And did you try a string without the config, so you can easily debug?

 

Code:Java

 
if (e.getItem().getType(Material.getMaterial(material))) {
 
}
 

 

Techno
Don'works, getting red lines!

 

Minesuchtiiii
Should work like this:

Code:java

 
String teleportItem = getConfig().getString("TeleportItem");
if (e.getItem().getType() == Material.getMaterial(teleportItem)) {
// Do stuff here...
}
 

What does the whole @EventHandler part look like?

 

CraftBang
I know how to read the error ^^
So I posted line 460!!!
and it's if(e.getItem().getType() == (Material.getMaterial(material))) {}

GameplayJDK
Whats the difference? You just gave the string another name... doesn't change anything!

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

 

Minesuchtiiii
So, in case you know how to read the error, as said, you should check if your config is correct, because it seems that the Material.getMaterial(material) returns null, but a material can't be null -> It throws an error.

 

GameplayJDK It works cause i get the material from the string.. but get an error in console!

 

Minesuchtiiii And you get the string from your config. Have you checked if the action is a eight/left click air/block?

 

GameplayJDK

Hole event handler:

Code:java

  @EventHandler
  public void onPlayerInteractWithTpItem(PlayerInteractEvent e) {
      final Player p = e.getPlayer();
 
      String material = this.getConfig().getString("TeleportItem");
 
      if(e.getAction() == Action.RIGHT_CLICK_AIR | e.getAction() == Action.RIGHT_CLICK_BLOCK) {
          if(e.getItem().getType() == Material.getMaterial(material)) {
              if(p.hasPermission("serverhub.itemtp")) {
 
                  if(bo == false && ba == false) {
                  final FileConfiguration cfg = this.getConfig();
 
                  if(cfg.get("hubs." + "primary" + ".x") != null) {
 
                double x = cfg.getDouble("hubs." + "primary" + ".x");
                double y = cfg.getDouble("hubs." + "primary" + ".y");
                double z = cfg.getDouble("hubs." + "primary" + ".z");
                double yaw = cfg.getDouble("hubs." + "primary" + ".yaw");
                double pitch = cfg.getDouble("hubs." + "primary" + ".pitch");
                String n = cfg.getString("hubs." + "primary" + ".world");
 
                World world = Bukkit.getWorld(n);
 
              final Location hubb = new Location(world, x, y, z);
              hubb.setYaw((float)yaw);
              hubb.setPitch((float)pitch);
 
              bo = true;
 
            count = Bukkit.getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
 
                @Override
                public void run() {
 
                    if(d != 0) {
                    p.sendMessage("§7[§cHubTeleport§7] §6Teleporting in §b" + d + " §6seconds!");
                    d--;
 
                    } else if(d == 0) {
                        p.teleport(hubb);
                        p.sendMessage("§7[§cHubTeleport§7] §6Teleporting...");
                        d = cfg.getInt("ItemTeleportDelay");
                        Bukkit.getScheduler().cancelTask(count);
                        bo = false;
                    }
 
                }
 
              }, 0,20L );
 
 
              } else {
                  p.sendMessage(this.px + "§cYou don't have set a primary hub yet!");
                  return;
              }
              } else {
                  return;
              }
              } else {
                  p.sendMessage(this.px + "§cYou don't have permissions!");
          }
          } else {
              return;
 
          }
      } else {
          return;
      }
 
  }

 

I'd recommend using type id's when working with this sort of stuff - It's deprecated, but it works fine.

 

Voltex
Ya, but for this I'll take the type :/!

 

bump

fixed:
if(p.getItemInHand().equals(material) {

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