Disabling Enchanting Problem.

So, I simply disabled enchanting by canceling the Enchant event. But, I want to be able to enchant Bows and Diamond Pickaxes aswell, so, I set the enchanting Dpicks priority higher, but now you can enchant everything else as-well, so, please help me out here. (DPick enchanting is in "PickEnchanting.java" and the disabling of enchanting is in the "EnchantingEvent.java")

Main.java:

Code:

package me.kyrp.Elite;
 
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
    private EnchantingEvent PlayerListener = new EnchantingEvent(this);
    private PickEnchanting PlayerListener1 = new PickEnchanting(this);
{
}
 
   
   
   
    public void onEnable(){
        getConfig().options().copyDefaults(true);
        saveConfig();
       
        PluginManager pm = getServer().getPluginManager();
        pm.registerEvents(this.PlayerListener, this);
       
        PluginManager pcm = getServer().getPluginManager();
        pcm.registerEvents(this.PlayerListener1, this);
    }
    }
    

EnchantingEvent.java (This disables enchanting.)

Code:

package me.kyrp.Elite;
 
import org.bukkit.ChatColor;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.enchantment.EnchantItemEvent;
 
public class EnchantingEvent implements Listener{
 
        public Main plugin;
     
        public EnchantingEvent(Main instance){
                plugin = instance;
        }
     
        @EventHandler(priority = EventPriority.LOWEST)
        public void EnchantItem(EnchantItemEvent e){
            e.getItem();
            e.setCancelled(true);
            e.getEnchanter().sendMessage(ChatColor.RED + "You cannot enchant " + e.getItem() + "on elite sonicpvp");
           
 
       
 
 
 
        {
        }
        }
}

PickEnchanting.java (Supposed to enable Enchanting of pickaxes.)

Code:

package me.kyrp.Elite;
 
import org.bukkit.Material;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.enchantment.EnchantItemEvent;
 
public class PickEnchanting implements Listener{
 
        public Main plugin;
     
        public PickEnchanting(Main instance){
                plugin = instance;
        }
     
        @EventHandler(priority = EventPriority.HIGHEST)
        public void EnchantItem(EnchantItemEvent e){
            e.getItem().getType();
            Material.DIAMOND_PICKAXE.getData();
            e.setCancelled(false);
     
        }
}

 

Just use one event and check if the item that is being enchanted is a Diamond Pickaxe. If it is not, have it cancel. You only need one event here, not two different classes.

Code:java

 
@EventHandler
public void onEnchant(EnchantItemEvent e) {
 
if (e.getItem().getType() != Material.DIAMOND_PICKAXE)) { // This checks if the item is NOT a diamond pickaxe.
e.setCancelled(true); //Cancels the event because it isn't a Diamond Pick.
// Send any messages here.
}
 
 

This should do the trick. Or something like it. This is off the top of my head.

 

That worked, but what about bow enchants?

 

Maybe add

1. if (e.getItem().getType() != Material.BOW)) { // This checks if the item is NOT a diamond pickaxe.
2. e.setCancelled(true);

 

Nope.

Anyone got any ideas?

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

 

Separate arguments in your if statement with && (this means AND).

Code:java

@EventHandler
public void onEnchant(EnchantItemEvent e) {
 
if (e.getItem().getType() != Material.DIAMOND_PICKAXE) && e.getItem().getType() != Material.BOW) { // This checks if the item is NOT a diamond pickaxe OR a bow.
e.setCancelled(true); //Cancels the event because it isn't a Diamond Pick.
// Send any messages here.
}

That checks if it is a DIAMOND_PICKAXE or a BOW. If it is neither, it runs the e.setCancelled(true) code.

 

Thanks, I'll test if it works!