Solved Entity Tracking error on custom Entity

In one of my plugins I'm creating custom projectiles. They are bukkit projectiles though and not NMS projectiles. From NMS stuff it's an EntityItem. They work fine when I'm launching them. They fly how I want them to and work how I want them to. However, if I try to launch one after I had already launched one but before the first one is gone, it throws this error:

Code:

"Silently" catching entity tracking error.
net.minecraft.server.v1_6_R1.ReportedException: Adding entity to track
      at net.minecraft.server.v1_6_R1.EntityTracker.addEntity(EntityTracker.java:116)
      at net.minecraft.server.v1_6_R1.EntityTracker.track(EntityTracker.java:57)
      at net.minecraft.server.v1_6_R1.WorldManager.a(WorldManager.java:18)
      at net.minecraft.server.v1_6_R1.World.a(World.java:957)
      at net.minecraft.server.v1_6_R1.WorldServer.a(WorldServer.java:800)
      at net.minecraft.server.v1_6_R1.World.addEntity(World.java:950)
      at com.xern.jogy34.tardis.tardis.TARDIS.fireTimeWeapon(TARDIS.java:1695)
      at com.xern.jogy34.tardis.general.TARDISMain.onCommand(TARDISMain.java:1383)
      at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
      at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:189)
      at org.bukkit.craftbukkit.v1_6_R1.CraftServer.dispatchCommand(CraftServer.java:523)
      at net.minecraft.server.v1_6_R1.PlayerConnection.handleCommand(PlayerConnection.java:983)
      at net.minecraft.server.v1_6_R1.PlayerConnection.chat(PlayerConnection.java:901)
      at net.minecraft.server.v1_6_R1.PlayerConnection.a(PlayerConnection.java:858)
      at net.minecraft.server.v1_6_R1.Packet3Chat.handle(SourceFile:49)
      at net.minecraft.server.v1_6_R1.NetworkManager.b(NetworkManager.java:293)
      at net.minecraft.server.v1_6_R1.PlayerConnection.d(PlayerConnection.java:118)
      at net.minecraft.server.v1_6_R1.ServerConnection.b(SourceFile:37)
      at net.minecraft.server.v1_6_R1.DedicatedServerConnection.b(SourceFile:30)
      at net.minecraft.server.v1_6_R1.MinecraftServer.t(MinecraftServer.java:590)
      at net.minecraft.server.v1_6_R1.DedicatedServer.t(DedicatedServer.java:226)
      at net.minecraft.server.v1_6_R1.MinecraftServer.s(MinecraftServer.java:486)
      at net.minecraft.server.v1_6_R1.MinecraftServer.run(MinecraftServer.java:419)
      at net.minecraft.server.v1_6_R1.ThreadServerApplication.run(SourceFile:582)
Caused by: java.lang.IllegalStateException: Entity is already tracked!
      at net.minecraft.server.v1_6_R1.EntityTracker.addEntity(EntityTracker.java:96)
      ... 23 more

My basic code is:

Code:

MyProjectile proj = new MyProjectile(nmsWorld);
proj.setPosition(location); //I have switch cases that I use to determine this and the velocity
proj.setVelocity(vector); //My own method that sets the motX, motY, motZ which I use to determine how it flys.
nmsWorld.addEntity(proj);

This works fine for my mob entities and fireworks projectile but not my EntityItem projectiles. Anyone have any thoughts as to what would be causing this? If you want to see more code just ask.

 

What is on and around "com.xern.jogy34.tardis.tardis.TARDIS.fireTimeWeapon(TARDIS.java:1695)"

By "Caused by: java.lang.IllegalStateException: Entity is already tracked!" it seems the item is automatically tracked when you create it OR you've already added it somewhere else to the tracker.

 

Jogy34 Could i please see your MyProjectile class

 

That's where I actually add the entity to the NMS world. And it seems to think that every time I create a new one that it's actually the same one. The error only gets thrown when I try to fire one before the previous one has died.

The code I showed you wasn't my actual code it was just the basic part of what I was doing. I didn't really want to show you that because in order to show you everything I have to show you 4 different classes.
Anyways here you go:

Highest up class:

Code:java

 
package com.xern.jogy34.tardis.weapons;
 
import net.minecraft.server.v1_6_R1.World;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
 
public class MagneticBeacon extends TimeWeapon
{
    private static ItemStack me;
    public MagneticBeacon(World world)
    {
        super(world, Material.FLINT.getId(), 0);
    }
 
    @Override
    public TimeWeaponType getType()
    {
        return TimeWeaponType.MAGNETIC_BEACON;
    }
 
    @Override
    public ItemStack getWeaponStack()
    {
        return me;
    }
 
    static
    {
        me = new ItemStack(Material.FLINT);
 
        ItemMeta im = me.getItemMeta();
        im.setDisplayName(ChatColor.DARK_RED + "BASIC MAGNETIC BEACON");
 
        WeaponHelper.addWeapon(MagneticBeacon.class, me, 50);
    }
}
 

All of the overriden methods are for my plugin and don't have to do with EntityItem
TimeWeapon:

Code:java

 
package com.xern.jogy34.tardis.weapons;
 
import java.util.UUID;
 
import org.bukkit.Location;
import org.bukkit.util.Vector;
 
import net.minecraft.server.v1_6_R1.EntityItem;
import net.minecraft.server.v1_6_R1.ItemStack;
import net.minecraft.server.v1_6_R1.World;
 
public abstract class TimeWeapon extends EntityItem implements ITimeWeapon
{
    public TimeWeapon(World world, int id, int data)
    {
        super(world);
 
        this.uniqueID = UUID.randomUUID();
 
        this.id = id;
        this.setItemStack(new ItemStack(id, 1, data));
    }
 
    @Override
    public void shoot(double d0, double d1, double d2, float f, float f1)
    {
        WeaponHelper.shoot(d0, d1, d2, f, f1, this);
    }
 
    @Override
    public void l_()
    {
        WeaponHelper.l_(this);
    }
 
    @Override
    public void setVelocity(Vector v)
    {
        this.motX = v.getX();
        this.motY = v.getY();
        this.motZ = v.getZ();
    }
 
    @Override
    public void setPosition(Location loc)
    {
        super.setPosition(loc.getX(), loc.getY(), loc.getZ());
    }
}
 

ITimeWeapon (interface):

Code:java

 
package com.xern.jogy34.tardis.weapons;
 
import org.bukkit.Location;
import org.bukkit.inventory.ItemStack;
import org.bukkit.util.Vector;
 
import net.minecraft.server.v1_6_R1.IProjectile;
 
public interface ITimeWeapon extends IProjectile
{
    public TimeWeaponType getType();
    public ItemStack getWeaponStack();
    public void setVelocity(Vector v);
    public void setPosition(Location loc);
}
 

WeaponHelper class:

Code:java

 
package com.xern.jogy34.tardis.weapons;
 
import java.util.HashMap;
import java.util.Random;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.craftbukkit.v1_6_R1.CraftWorld;
import org.bukkit.inventory.ItemStack;
 
import com.xern.jogy34.tardis.events.TimeWeaponHitTARDISEvent;
import com.xern.jogy34.tardis.tardis.TARDIS;
 
import net.minecraft.server.v1_6_R1.Entity;
import net.minecraft.server.v1_6_R1.MathHelper;
 
public class WeaponHelper
{
    private static final Random rnd = new Random();
    public static void l_(Entity e)
    {
        if(e instanceof ITimeWeapon) 
        {
            ((ITimeWeapon)e).shoot(e.motX, e.motY, e.motZ, 1.5F, 1.0F);
            e.move(e.motX, e.motY, e.motZ);
            if(e.getBukkitEntity().getTicksLived() > 100) e.die();
 
            CraftWorld w = e.world.getWorld();
            int x = (int) (e.locX + e.motX);
            int y = (int) (e.locY + e.motY);
            int z = (int) (e.locZ + e.motZ);
            if(w.getBlockAt(x, y, z).getTypeId() != 0 || (e.motX == 0 && e.motY == 0 && e.motZ == 0))
            {
                TARDIS t = TARDIS.getTardisByLocation(new Location(w, x, y, z), false, false);
                if(t == null)
                {
                    e.die();
                }
                else
                {
                    TimeWeaponHitTARDISEvent event = new TimeWeaponHitTARDISEvent(t, ((ITimeWeapon)e));
                    Bukkit.getPluginManager().callEvent(event);
                    e.die();
                }
            }
        }
    }
 
    public static void shoot(double d0, double d1, double d2, float f, float f1, Entity e)
    {
        float f2 = MathHelper.sqrt(d0 * d0 + d1 * d1 + d2 * d2);
 
        d0 /= (double) f2;
        d1 /= (double) f2;
        d2 /= (double) f2;
        d0 += rnd.nextGaussian() * (double) (rnd.nextBoolean() ? -1 : 1) * 0.007499999832361937D * (double) f1;
        d1 += rnd.nextGaussian() * (double) (rnd.nextBoolean() ? -1 : 1) * 0.007499999832361937D * (double) f1;
        d2 += rnd.nextGaussian() * (double) (rnd.nextBoolean() ? -1 : 1) * 0.007499999832361937D * (double) f1;
        d0 *= (double) f;
        d1 *= (double) f;
        d2 *= (double) f;
        e.motX = d0;
        e.motY = d1;
        e.motZ = d2;
        float f3 = MathHelper.sqrt(d0 * d0 + d2 * d2);
 
        e.lastYaw = e.yaw = (float) (Math.atan2(d0, d2) * 180.0D / 3.1415927410125732D);
        e.lastPitch = e.pitch = (float) (Math.atan2(d1, (double) f3) * 180.0D / 3.1415927410125732D);
    }
 
    public static class WeaponStackAndPrice
    {
        public final ItemStack weaponStack;
        public final int price;
        public WeaponStackAndPrice(ItemStack weaponStack, int price)
        {
            this.weaponStack = weaponStack;
            this.price = price;
        }
    }
 
    private static HashMap<WeaponStackAndPrice, Class<? extends ITimeWeapon>> allWeapons = 
            new HashMap<WeaponStackAndPrice, Class<? extends ITimeWeapon>>();
 
    public static void addWeapon(Class<? extends ITimeWeapon> weapon, ItemStack weaponStack, int price)
    {
        allWeapons.put(new WeaponStackAndPrice(weaponStack, price), weapon);
    }
 
    public static WeaponStackAndPrice[] getAllWeaponStacksAndPrices()
    {
        return allWeapons.keySet().toArray(new WeaponStackAndPrice[0]);
    }
 
    public static ItemStack getStackFromWeapon(Class<? extends ITimeWeapon> weapon)
    {
        for(WeaponStackAndPrice wap : allWeapons.keySet())
        {
            if(allWeapons.get(wap).equals(weapon))
            {
                return wap.weaponStack;
            }
        }
        return null;
    }
 
    public static Class<? extends ITimeWeapon> isWeaponStack(ItemStack is)
    {
        for(WeaponStackAndPrice w : allWeapons.keySet())
        {
            ItemStack cur = w.weaponStack;
            if(is.getType().equals(cur.getType()))
            {
                if(is.getItemMeta().equals(cur.getItemMeta()))
                {
                    if(is.getEnchantments().equals(cur.getEnchantments()))
                    {
                        return allWeapons.get(w);
                    }
                }
            }
        }
        return null;
    }
}
 

I use this so that I don't have to keep rewriting the same code as not all of my weapon classes come from TimeWeapon as currently one of them is a firework and not an item

Anyone?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Does NMS automatically add the item to the tracker. It could be not that all the items are the same but that the item has been added already by another section.

Try printing put all the tracked items and see if they are the same or if it's what I'm thinking.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Entities are added to the tracker when you add them into the NMS world. And I already know that the tracker contains an EntityItem that it thinks is the same one as a new one that I try to fire which is one that was previously fired. If it didn't think it was already tracked then it wouldn't be throwing the error and it would actually be fireing and if it didn't think that the one that is already tracked is the one that was previously fired then it would be throwing the error at times when one wasn't out there that was previously fired or it would sometimes allow it to fire when one is already out there.

Anyone?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Anyone have any ideas?

EDIT: Sorry, I figured it out. I was just being kind of stupid. Turns out the entity tracker checks if an entity is already tracked by its id, not it's UUID its normal id. I was thinking that the id field in EntityItem was the id of the item it was going to render as so I kept setting it to the same thing.