Solved Crafting permissions

I have been working on a plugin and it has so far worked. I am adding new crafting recipes and permissions for those recipes. The permissions work when a player doesn't have them but when they do have them they still can't craft the item

Code:java

 
public class CraftingEvent implements Listener
{ 
 
@EventHandler(priority = EventPriority.HIGHEST)
    public void PICE(PrepareItemCraftEvent event) 
{
        Player p = (Player) event.getView().getPlayer();
        System.out.println("Player: " + p.getDisplayName());
 
        if (!p.hasPermission("spawnercage.cage") && event.getInventory().getResult().getType()==Material.MOB_SPAWNER)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.creeper") && itemCheck(event, "Creeper Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.skeleton") && itemCheck(event, "Skeleton Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.spider") && itemCheck(event, "Spider Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.zombie") && itemCheck(event, "Zombie Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.slime") && itemCheck(event, "Slime Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.ghast") && itemCheck(event, "Ghast Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.pigman") && itemCheck(event, "Zombie Pigman Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.enderman") && itemCheck(event, "Enderman Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.cavespider") && itemCheck(event, "Cavespider Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.silverfish") && itemCheck(event, "Silverfish Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.blaze") && itemCheck(event, "Blaze Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.magma") && itemCheck(event, "Magma Cube Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.bat") && itemCheck(event, "Bat Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.witch") && itemCheck(event, "Witch Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.sheep") && itemCheck(event, "Sheep Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.cow") && itemCheck(event, "Cow Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.chicken") && itemCheck(event, "Chicken Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.squid") && itemCheck(event, "Squid Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.wolf") && itemCheck(event, "Wolf Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.mooshroom") && itemCheck(event, "Mooshroom Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.ocelot") && itemCheck(event, "Ocelot Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.horse") && itemCheck(event, "Horse Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }
 
        else if (!p.hasPermission("spawnercage.villager") && itemCheck(event, "Villager Cage")==true)
        {
            p.sendMessage(ChatColor.RED + "You don't have permission to craft this item!");
            event.getInventory().setResult(null);
        }     
    }
 
public boolean itemCheck(PrepareItemCraftEvent event, String name)
{
 
boolean returnValue = false; 
 
    ItemStack i = event.getInventory().getResult();
    if(i == null)return false; // Fail the test if there is no item in hand
    if(!i.hasItemMeta())return false; // Fail the test if item does not have metadata
    ItemMeta im = i.getItemMeta();
    if(!im.hasDisplayName())return false; // Fail the test if the item doesn't have a display name
    String dn = im.getDisplayName();
    List<String> lore = im.getLore();
    if(lore != null)
    {
   if(dn.indexOf(name) > -1 && i.getType() == ItemStack(Material.MOB_SPAWNER, 1))
   {
   returnValue = true;
   }
    }
 
    return returnValue;
}
 
private Material ItemStack(Material mobSpawner, int i) {
         // TODO Auto-generated method stub
         return null;
 }
}
 

 

Have you tried the API?

 

Ok? 17nhammond

bump.

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