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.

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.

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).

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

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