[1.7.10] New mob AI task doesn't apply to all hostile mobs?

[IceMetalPunk]

I have a new AI task I'm trying to add to all hostile mobs (except bosses). It simply makes them head to a Moonstone block (a custom block in my mod) if they're within 10 blocks of one. However, it seems it's only applying to certain mobs and not others. I've tested with all the hostile mob spawn eggs, and this is the result:

 

It works with Witches, Skeletons, Silverfish, Creepers, and Zombies, who all properly move towards the Moonstone.

 

It does not work with Spiders, Slimes, Zombie Pigmen, Endermen, Blazes, Magma Cubes, or Ghasts, who completely ignore the Moonstone.

 

I'm not sure why it's being so selective. Here's my event listener code to add the task to mobs:

 

	@SubscribeEvent
public void onJoinWorld(EntityJoinWorldEvent event) {

                /* If this is a mob, handle specific cases that don't extend EntityMob, then the general case. Don't be too general, as dragons implement IMob as well, and we don't want them using this AI */
	if (event.entity instanceof IMob) {
		if (event.entity instanceof EntityGhast) {
			((EntityGhast)event.entity).tasks.addTask(4, new EntityAIMoonstoneAttract((EntityLiving) event.entity, 1.0d));
		}
		else if (event.entity instanceof EntityMagmaCube) {
			((EntityMagmaCube)event.entity).tasks.addTask(4, new EntityAIMoonstoneAttract((EntityLiving) event.entity, 1.0d));
		}
		else if (event.entity instanceof EntitySlime) {
			((EntitySlime)event.entity).tasks.addTask(4, new EntityAIMoonstoneAttract((EntityLiving) event.entity, 1.0d));
		}
                        else if (event.entity instanceof EntityWither) { return; }
		else if (event.entity instanceof EntityMob) {
			((EntityMob)event.entity).tasks.addTask(4, new EntityAIMoonstoneAttract((EntityLiving) event.entity, 1.0d));
		}
	}
}

 

And here's the task code itself (the EntityAIMoonstoneAttract class):

 

package com.IceMetalPunk.weatherworks;

import net.minecraft.block.Block;
import net.minecraft.entity.EntityLiving;
import net.minecraft.entity.ai.EntityAIBase;

public class EntityAIMoonstoneAttract extends EntityAIBase {
private EntityLiving entity=null; 
    private double xPosition;
    private double yPosition;
    private double zPosition;
    private double speed;
    private static final String __OBFID = "CL_00001608";

    public EntityAIMoonstoneAttract(EntityLiving p_i1648_1_, double p_i1648_2_)
    {
        this.entity = p_i1648_1_;
        this.speed = p_i1648_2_;
        this.setMutexBits(1);
    }

    /**
     * Returns whether the EntityAIBase should begin execution.
     */
    public boolean shouldExecute()
    {
        // Get entity block position
    	int bX=(int)Math.floor(this.entity.posX);
    	int bY=(int)Math.floor(this.entity.posY);
    	int bZ=(int)Math.floor(this.entity.posZ);
    	boolean foundMoonstone=false;
    	int xx=0, yy=0, zz=0;
    	
        // Check a 10x10x10 cube around the entity, stopping if we find a Moonstone
    	outerLoop:
    	for (xx=-10; xx<=10; ++xx) {
    		for (yy=-10; yy<=10; ++yy) {
    			for (zz=-10; zz<=10; ++zz) {
    				Block block=this.entity.worldObj.getBlock(bX+xx, bY+yy, bZ+zz);
    				if (block instanceof BlockMoonstone) {
    					foundMoonstone=true;
    					break outerLoop;
    				}
    			}
    		}
    	}

        // Update with the position of the found Moonstone (if there's one) and allow AI execution
    	if (!foundMoonstone) {
    		return false;
    	}
        else
        {
            this.xPosition = bX+xx;
            this.yPosition = bY+yy;
            this.zPosition = bZ+zz;
            return true;
        }
    }

    /**
     * Returns whether an in-progress EntityAIBase should continue executing
     */
    public boolean continueExecuting()
    {
        return !this.entity.getNavigator().noPath();
    }

    /**
     * Execute a one shot task or start executing a continuous task
     */
    public void startExecuting()
    {
        this.entity.getNavigator().tryMoveToXYZ(this.xPosition, this.yPosition, this.zPosition, this.speed);
    }
}

 

It's especially strange to me since Endermen, at least, are definitely children of the EntityMob class, and I'm specifically handling EntitySlimes in the hook, yet neither mob seems to care about the Moonstone.

[Choonster]

Not all mobs use the new AI task system in 1.7.10, some still use the old AI system. If you want to handle these mobs, you may need to manually execute their AI tasks from

LivingUpdateEvent

.

[IceMetalPunk]

Ohhhh, okay. Seeing as this is my first look into custom AI, I'll give it a shot and see how it goes...I'll post here if I run into any other problems with it. Thanks.

 

*EDIT* Awesome, I got it all to work! I just had to basically copy over the pathfinding code to the LivingUpdateEvent handler for those certain mobs that don't use the new tasks system, like you mentioned, and everything works perfectly now Thank you!