Solved trouble getting a material from config.yml

Im having some trouble getting a material from the config.
Basically when the player right clicks the item defined under "Item" in the config the message should be displayed to the user defined under "Message" in the config. But its not working for me. What am I doing wrong?

Main Class:

Code:java

public void onPlayerInteract(PlayerInteractEvent e) {
 
if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.LEFT_CLICK_BLOCK) {
Player player = e.getPlayer();{
 
Material m = Material.getMaterial(this.getConfig().getString("Item").toUpperCase());
if (e.getPlayer().getItemInHand().getType().equals(m)){
 
//player.sendMessage(getConfig().getString("Message").replaceAll("&", "§"));
player.sendMessage(ChatColor.translateAlternateColorCodes('&', getConfig().getString("Message")));
 
 
 
}
}
}
}

config.yml:

Code:

Message: 'This is a test message!'
Item: EMERALD

Help would be much appreciated!

 

coolmonkeyguy

Try This

Code:java

public void onPlayerInteract(PlayerInteractEvent e) {
if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.LEFT_CLICK_BLOCK) {
Player player = e.getPlayer();{
 
Material m = Material.getMaterial(this.getConfig().getString("Item").toUpperCase());
if (e.getPlayer().getItemInHand().getType().equals(m)){
 
//player.sendMessage(getConfig().getString("Message").replaceAll("&", "§"));
player.sendMessage(ChatColor.translateAlternateColorCodes('&', getConfig().getString("Message")));
 
 
 
}
}
}

 

Try this

Code:java

public void onPlayerInteract(PlayerInteractEvent e) {
 
if(e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction() == Action.LEFT_CLICK_AIR || e.getAction() == Action.RIGHT_CLICK_BLOCK || e.getAction() == Action.LEFT_CLICK_BLOCK) {
Player player = e.getPlayer();
 
Material m = Material.getMaterial(this.getConfig().getString("Item").toUpperCase());
if (e.getPlayer().getItemInHand().getType().equals(m))}
//player.sendMessage(getConfig().getString("Message").replaceAll("&", "§"));
player.sendMessage(ChatColor.translateAlternateColorCodes('&', getConfig().getString("Message")));
}
}
}

You had a stray open and close brace.

 

It still does not seem to work. I think it has to do with recalling "Item" and getString. but im still not sure. and is my config correct for using an emerald?

 

This is what i used in my plugin

Code:java

ItemStack tempItem = new ItemStack(Material.getMaterial(tempConfig.getString(keys[j])), 64, 0);

 

i dont know whats wrong but it just wont work.

 

Use item ID's instead? Not the most practical...but it works.

 

coolmonkeyguy Where in the event does it stop/get stuck?

 

have you tried using the valueOf method, or the matchMaterial method? Test them out.

 

i figured out what was wrong. i forgot to put @EventHandler
GG

Thanks for the help agent6262