[1.8.9] Custom Mob Drops Only With Specific Item?

[Monstrous_Apple]

My way still works fine for making it drop when using the sword only though? Why change it? And okay thanks for the chance to drop help I'll have a look at what I can do.

[Draco18s]

...take the holding line from your ore drop event?

[Draco18s]

...take the holding line from your ore drop event?

[Monstrous_Apple]

Yep

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			Object holding = new ItemStack(MAItems.SoulHarvester);
			if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

 

Didn't you say to literally only add this:

 

if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {

 

When I did that it said "holding cannot be resolved to a variable" so I tried making one which is this here:

 

Object holding = new ItemStack(MAItems.SoulHarvester);

 

So what did I actually do wrong? And by the way I can't get the

event.source.getEntity()

working...

[Monstrous_Apple]

Yep

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			Object holding = new ItemStack(MAItems.SoulHarvester);
			if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

 

Didn't you say to literally only add this:

 

if (holding  != null && ((ItemStack) holding).getItem() instanceof MASoulHarvester) {

 

When I did that it said "holding cannot be resolved to a variable" so I tried making one which is this here:

 

Object holding = new ItemStack(MAItems.SoulHarvester);

 

So what did I actually do wrong? And by the way I can't get the

event.source.getEntity()

working...

[Draco18s]

I said you need "This line equivalent"

 

I didn't say "add this and only this, nothing else."

 

I figured you were smart enough to go "oh hey, that holding variable, maybe I should copy that too."

[Draco18s]

I said you need "This line equivalent"

 

I didn't say "add this and only this, nothing else."

 

I figured you were smart enough to go "oh hey, that holding variable, maybe I should copy that too."

[Monstrous_Apple]

Okay well I have this and now it gives me an error on "ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);" on both of the "harvester" parts saying "harvester cannot be resolved or is not a field"?

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);
			if (holding != null && holding.getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

[Monstrous_Apple]

Okay well I have this and now it gives me an error on "ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);" on both of the "harvester" parts saying "harvester cannot be resolved or is not a field"?

 

//Cow Soul
	@SubscribeEvent
	public void addEntityDrop50 (LivingDropsEvent event) {
		if (event.entity instanceof EntityCow) {
			ItemStack holding = event.harvester.inventory.getStackInSlot(event.harvester.inventory.currentItem);
			if (holding != null && holding.getItem() instanceof MASoulHarvester) {
			ItemStack itemstack = new ItemStack(MASouls.CowSoul, 1);
			event.drops.add(new EntityItem(event.entity.worldObj, event.entity.posX, event.entity.posY, event.entity.posZ, itemstack));
		}
	}
}

[Draco18s]

Probably because the event has different fields.  Look at the LivingDropsEvent class and see if you can trace your way back to a player.

 

One exists.  It's not hard to find.

[Draco18s]

Probably because the event has different fields.  Look at the LivingDropsEvent class and see if you can trace your way back to a player.

 

One exists.  It's not hard to find.

[Monstrous_Apple]

I've been trying to do that for ages even before you said and before I asked for help but I can't figure out which ones to use...

[Monstrous_Apple]

I've been trying to do that for ages even before you said and before I asked for help but I can't figure out which ones to use...

[Ernio]

DamageSource#getEntity()

[Ernio]

DamageSource#getEntity()

[Monstrous_Apple]

Yep got that far, don't know what else to add?

[Monstrous_Apple]

Yep got that far, don't know what else to add?

[Ernio]

What is the problem? You get entity from source, get that entity's current item and check if it's yours.

[Ernio]

What is the problem? You get entity from source, get that entity's current item and check if it's yours.

[Monstrous_Apple]

I have this, hopefully it's correct so far...

ItemStack holding = event.source.getEntity().

 

I've tried adding this:

ItemStack holding = event.source.getEntity().getInventory()

 

And honestly I have no clue what else to do, I did try and get the item using .getInventory but that didn't work out?

[Monstrous_Apple]

I have this, hopefully it's correct so far...

ItemStack holding = event.source.getEntity().

 

I've tried adding this:

ItemStack holding = event.source.getEntity().getInventory()

 

And honestly I have no clue what else to do, I did try and get the item using .getInventory but that didn't work out?

[Ernio]

Jeuss christ. Learn some Java and how to use your IDE.

 

Should this work for all entities or for players?

 

You can make it work for entities by using EntityLivingBase#handHandler Capability.

If you only want players, just cast the damn entity and get player.inventory like you do in harvesting.

[Ernio]

Jeuss christ. Learn some Java and how to use your IDE.

 

Should this work for all entities or for players?

 

You can make it work for entities by using EntityLivingBase#handHandler Capability.

If you only want players, just cast the damn entity and get player.inventory like you do in harvesting.

[Monstrous_Apple]

Can you not show me an example of this using my code so I know how to do it? I tried doing what you said and it doesn't work...

[Monstrous_Apple]

Can you not show me an example of this using my code so I know how to do it? I tried doing what you said and it doesn't work...