[1.10] SOLVED Remove apples from leaf drops?

[Cleverpanda]

Hello.

I need a way to remove apples from vanilla tree drops because I have added an apple tree and want it to be the only natural source of apples.

However, apples are not added to the list in getDrops() and use spawnEntityInorld() directly so it doesn't seem that I can remove it with a simple harvest event.

Does anyone know what I need to do?

[Cleverpanda]

So adding

event.getDrops().remove(new ItemStack(Items.APPLE));

to my event handler that looks for leaf blocks should work fine?

[Cleverpanda]

if(event.getDrops().contains(new ItemStack(Items.APPLE)))

{

int s = event.getDrops().size();

        for(int i=0;i<s;i++)

        {

        if(event.getDrops().get(i) == new ItemStack(Items.APPLE))

                {

        event.getDrops().remove(i);

                }

}

}

 

Why does this not work?

[Cleverpanda]

Because you told me to search the list manually and this is what I thought you meant?

I've had formal classes in Java. I thought that get() returned an itemStack, but I had actually checked getDrops(), which does. I fixed that statement.

[larsgerrits]

Object == new Object()

can never be true, as they are different Objects.

 

You have to check if the item of the

ItemStack

is equal to

Items.APPLE

.

 

Also, you can use a for-each loop instead of a normal loop.

[Draco18s]

1.7 code, but the principle is the same.

https://github.com/Draco18s/HarderStuff/blob/master/src/main/java/com/draco18s/wildlife/WildlifeEventHandler.java#L335-L349

 

You have to do it this way because you cannot compare item stacks with ==, you have to compare their contents.

[trollworkout]

Cleverpanda, how did you solve this issue? Can you post code?  I'm too tired to think.

Also what mod is this you're adding apples and how? Im making a mod i'm also adding apples. I think I will have a customize-able option to turn apples off tho and I can use oreLogApple or something similar to make sure I can use apple wood from other mods.