EntityTargetLivingEntityEvent with condition

Hey I want zombies only have a target when the condition is true
my code works fine but i have one problem
if a zombie set his target and the condition true but now the condition changes and its false how can i update the target
at the moment i have folowing code

Code:java

@EventHandler(priority = EventPriority.HIGHEST)
    public void onZombieTargetPlayer(EntityTargetLivingEntityEvent event)
    {
        if ((event.getEntity() instanceof Zombie)
                && event.getTarget() instanceof Player)
        {
            final Player p = (Player) event.getTarget();
            final Zombie z = (Zombie) event.getEntity();
 
            List<Entity> nearBys = z.getNearbyEntities(z.getLocation().getX(),
                    z.getLocation().getY(), z.getLocation().getZ());
 
            int maxWanted = 0;
            for (Entity e : nearBys)
            {
                if (e instanceof Player)
                {
                    Player NBP = (Player) e;
                    if ((int)getMetadata(NBP, "WantedLevel") > maxWanted)
                    {
                        System.out.println("4 targed changed");
                        maxWanted = (int)getMetadata(NBP, "WantedLevel");
 
                        event.setTarget(NBP);
 
                    }
                }
            }
 
            if ((int)getMetadata((Player)event.getTarget(),"WantedLevel") <= 0)
            {
                event.setCancelled(true);
            }
        }
    }

 

} else {
event.setCancelled(true);
return;
}

That SHOULD do it, at least that's what worked when I was testing it with creepers a few weeks back.

 

valon750
are you sure i mean the target is already set isnt it?

i tryed it and it doesent work after the zombie has set me as target and than i change the condition to false he still wants to atack me

 

ToastHelmi

When tested with a simple..

if(event.getTarget() instanceof Player){
event.setCancelled(true);
return;
}

It cancelled the event just fine.

 

valon750
i think you think i have a other proplem
the event is also cancelled fine without return;

my problem is a zombie is searching his target when the condition is true
zombietarget = player

but now the condition turns into false
but still

zombietarget = player

but what i want is
zombietarget = null or zombietarget = new player

if the zombie have a target it will not call a
EntityTargetLivingEntityEvent
again

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