Make vanilla mobs drop items

[dude22072]

Im trying to make a vanilla cow drop a new item.

 

ModLivingDropsEvent:

package dudesmods.lozmod;

import net.minecraft.entity.item.EntityItem;
import net.minecraft.entity.passive.EntityCow;
import net.minecraft.item.Item;
import net.minecraftforge.event.ForgeSubscribe;
import net.minecraftforge.event.entity.living.LivingDropsEvent;
import net.minecraft.entity.EntityLiving;

public class ModLivingDropsEvent {

          public static double rand;

          @ForgeSubscribe
          public void onEntityDrop(ModLivingDropsEvent event) {
        	  if(event.entityLiving instanceof EntityCow) {
        		  event.entityLiving.dropItem(LOZmod.greenTannedLeather.itemID);
        	  }
          }
                  
}

 

it gives me a red underline on both of the "entityLiving" parts saying it cannot be resolved or is not a field.

[WeeziMonkey]

It should be like this:

 

   
@ForgeSubscribe
     public void onEntityDrop(LivingDropsEvent event) {

              if (event.entityLiving instanceof EntityCow) {
                 //The integer at the end relates to how many potato will be dropped.
                  event.entityLiving.dropItem(LOZmod.greenTannedLeather.itemID, 1);
              }
       }
}

 

From: http://darkhaxspace.weebly.com/living-drops.html

 

Took me 5 seconds googling.

[Draco18s]

I'd just like to point out that there is no way in hell you should have thought that your code would work:

 

public void onEntityDrop(ModLivingDropsEvent event) {

 

So this function takes a parameter of...your class.  No red flags yet.  Weird, but not--technically--incorrect.

 

event.entityLiving

 

Uh.  Your class doesn't have that property.  It only has

 

public static double rand;

 

Because ModLivingDropsEvent doesn't extend or implement any classes.

 

So either:

a) You gave 'event' the wrong type (as the above posters have pointed out, it should be LivingDropsEvent)

b) You didn't properly extend a base class (e.g. ModLivingDropsEvent extends LivingDropsEvent, which in this case is unnecessary).

[SanAndreaP]

I'd just like to point out that there is no way in hell you should have thought that your code would work:

 

public void onEntityDrop(ModLivingDropsEvent event) {

 

So this function takes a parameter of...your class.  No red flags yet.  Weird, but not--technically--incorrect.

 

event.entityLiving

 

Uh.  Your class doesn't have that property.  It only has

 

public static double rand;

 

Because ModLivingDropsEvent doesn't extend or implement any classes.

 

So either:

a) You gave 'event' the wrong type (as the above posters have pointed out, it should be LivingDropsEvent)

b) You didn't properly extend a base class (e.g. ModLivingDropsEvent extends LivingDropsEvent, which in this case is unnecessary).

 

b doesn't work since Forge makes an instance of the Event class everytime the event is fired. An unregistered subclass would crash / simply not work if you try to register it and 1) don't put it into the net.minecraftforge(.client).event package 2) don't fire it manually with an instance of your subclass

[dude22072]

OK. Eclipse messed with my stuff for some reason. fixed now.