Force zombies to be able to break doors

Hello!

As far as I know, since 1.7 not all zombies are able to break doors, only those tagged with "CanBreakDoor".

I extended EntityZombie to have custom pathfinding along with few other things and now I'm trying to make each ExtendedZombie break doors.

Here is my code:

Code:java

public class ExtendedZombie extends EntityZombie {
 
    private void setTargets() //Set pathfinding targets
    {
        try {
            Field bField = PathfinderGoalSelector.class.getDeclaredField("b");
            bField.setAccessible(true);
            Field cField = PathfinderGoalSelector.class.getDeclaredField("c");
            cField.setAccessible(true);
            bField.set(goalSelector, new UnsafeList<PathfinderGoalSelector>());
            bField.set(targetSelector, new UnsafeList<PathfinderGoalSelector>());
            cField.set(goalSelector, new UnsafeList<PathfinderGoalSelector>());
            cField.set(targetSelector, new UnsafeList<PathfinderGoalSelector>());
 
 
           //These two lines actually should do the job. "bu" is the CanBreakDoor boolean.
            setField(EntityZombie.class,"bu",true);
            this.goalSelector.a(1, (PathfinderGoalBreakDoor)(getField(EntityZombie.class,"bs")));
 
            this.goalSelector.a(2, new PathfinderGoalMeleeAttack(this, EntityHuman.class, 1.0D, false));
            this.goalSelector.a(5, new PathfinderGoalMoveTowardsRestriction(this, 1.0D));
 
            this.targetSelector.a(1, new PathfinderGoalHurtByTarget(this, true));
            this.targetSelector.a(2, new PathfinderGoalNearestAttackableTarget(this, EntityHuman.class, 0, true));
 
        } catch (Exception exc) {
            exc.printStackTrace();
        }
    }
 
    private void setField(Class<?> clazz, String field, Object obj) //Helper method
    {
        try
        {
            Field f = clazz.getDeclaredField(field);
            f.setAccessible(true);
            f.set(this,obj);
        } catch (Exception ex)
        {
            ex.printStackTrace();
        }
    }
 
    private Object getField(Class<?> clazz, String field) //Helper method
    {
        try
        {
            Field f = clazz.getDeclaredField(field);
            f.setAccessible(true);
            return f.get(this);
        } catch (Exception ex)
        {
            ex.printStackTrace();
        }
        return null;
    }
 
    @Override //Called by LoadFromNbt
    public void a(boolean flag)
      {
        System.out.println("FLAG: "+flag); //Debug
        if ((boolean)(getField(EntityZombie.class,"bu")) != flag)
        {
          setField(EntityZombie.class,"bu",flag);
          if (flag) {
            this.goalSelector.a(1, (PathfinderGoalBreakDoor)(getField(EntityZombie.class,"bs")));
          } else {
            this.goalSelector.a((PathfinderGoalBreakDoor)(getField(EntityZombie.class,"bs")));
          }
        }
      }
 
    public ExtendedZombie(World world)
    {
        super(world);
 
        setField(EntityInsentient.class,"navigation",new NavigationEx(this,world)); //Custom pathfinding, no overrides yet
 
        setTargets();
        a(true); //Try to make him break blocks
    }
 
    @Override
    public void aC() {
        super.aC();
        this.getAttributeInstance(GenericAttributes.e).setValue(3D); //Set damage
        this.getAttributeInstance(GenericAttributes.a).setValue(40D); //Set HP
        this.getAttributeInstance(GenericAttributes.b).setValue(32); //Set view rage 
    }
 
    @SuppressWarnings({ "unchecked", "rawtypes" })
    public static void registerMob()
    {
        //Standard registration of mob
    }
}

Actually, the funny thing is that this code works well when I restart the server. I mean, if I spawn a zombie then he doesn't break doors (although he is using my AI methods and performing good) but when I don't kill him and restart the server (let him get loaded by ChunkLoader, spawned and NbtLoaded, calling a(boolean flag)), he can break the door perfectly as I wish.

I know this may seem hard, but I'm hope there is anyone who can help me in solving this problem...

 

The BreakDoor goal is probably checking if the zombie has the tag CanBreakDoors. I'm not very familiar with NMS, and I didn't look any of this up, but it seems like the BreakDoor goal caches the CanBreakDoors, and since booleans are not objects, it does not change with the zombie. Try calling a(true) before the setTargets() call. I'll see if I can find anything to back this theory up.

Actually, if you check the BreakDoor goal source, it checks if the difficulty is on hard. It doesn't make much sense, but it may be your problem.
Source

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

 

epicfacecreeper
Checked it a lot of times, and if it wasn't hard then they wouldn't break doors after restart
Gonna check with a before targets...

epicfacecreeper
Nothing has changed. :/

Furthermore, String ".getBoolean("CanBreakDoors")" is only referenced in EntityZombie, so it cannot check the NBT tags in PathfinderGoal...

epicfacecreeper
Okay, after 4 hours I found what was causing the wrong behaveiour.

I was setting the new Navigation after the super constructor. In EntityZombie.java we have:

Code:java

public EntityZombie(World world)
  {
    super(world);
    getNavigation().b(true); //NOTICE!!!
    this.goalSelector.a(0, new PathfinderGoalFloat(this));
    this.goalSelector.a(2, new PathfinderGoalMeleeAttack(this, EntityHuman.class, 1.0D, false));
    //More goals, blah blah
  }

getNavigation().b(true) allows pathfinding to cross the doors >.<

Calling it before setTargets() did the job.
Finally, my code looks like before, except for the constructor:

Code:java

    public ExtendedZombie(World world)
    {
        super(world);
 
        setField(EntityInsentient.class,"navigation",new NavigationEx(this,world));
        this.getNavigation().b(true);
 
        setTargets();
 
        a(true);
    }

Thanks for your interest!!!

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