What is wrong with this code?

So im trying to make a plugin that when a player is wearing a zombie head, they are disguised and cant be targeted by zombies for a 30 seconds. I dont get any errors or anything it just doesn't work.

Can anyone tell me why?

Code:java

  @EventHandler
  public void sneakbyzombies(EntityTargetLivingEntityEvent event) {
      Entity target = event.getTarget();
      Player player  = (Player) target;
      if (event.getEntity().getType() == EntityType.ZOMBIE) {
          if (event.getTarget().getType() == EntityType.PLAYER) {
              if (player.getInventory().getHelmet().getType() == Material.SKULL) {
                  event.setTarget(null);
              }
          }
 
      }
      }
  @EventHandler
  public void timedhead(InventoryInteractEvent event) {
      HumanEntity player = event.getWhoClicked();
      ItemStack helmet = player.getInventory().getHelmet();
      if (Settings.getConfig().getBoolean("zombies.disguise.enabled") && (player.getInventory().getHelmet().getType() == Material.SKULL)); {
                try {
                    Thread.sleep(Settings.getConfig().getInt("zombies.disguise.time")*100);
                    player.getInventory().remove(helmet);
                } catch (InterruptedException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
          }
      }
  }

 

Oh yeah, because you're putting the entire server to slumber for a second or two.

Use a scheduler.

 

@xTrollxDudex what do you mean?

 

Don't sleep the main thread, use a task or separate class that extends thread, etc.

 

@1Rogue i still dont understand, can you show me or somthing?

 

nrs23 Thread.sleep stops the server for the specified amount of time. Try to use the Bukkit scheduledSyncDelayedTask()