interaction should be reliable now.

Author: EternalBlueFlame

src/main/java/train/common/core/handlers/PacketHandler.java

@@ -22,6 +22,12 @@
 
 public class PacketHandler {
 	private static final IMessageHandler[] HANDLERS = new IMessageHandler[]{
+			new IMessageHandler<IMessage, IMessage>() {
+				@Override
+				public IMessage onMessage(IMessage message, MessageContext ctx) {
+					return null;
+				}
+			},
 			new IMessageHandler<IMessage, IMessage>() {
 				@Override
 				public IMessage onMessage(IMessage message, MessageContext ctx) {
@@ -56,6 +62,8 @@ public static void init(){
 				Side.SERVER);
 		Traincraft.keyChannel.registerMessage(HANDLERS[0], PacketRemove.class,
 				6, Side.SERVER);
+		Traincraft.keyChannel.registerMessage(HANDLERS[1], PacketInteract.class,
+				7, Side.SERVER);
 		Traincraft.rotationChannel.registerMessage(PacketRollingStockRotation.Handler.class,
 				PacketRollingStockRotation.class, 2, Side.CLIENT);
 		 Traincraft.modChannel.registerMessage(PacketSetJukeboxStreamingUrl.Handler.class,

src/main/java/train/common/core/network/PacketInteract.java

@@ -0,0 +1,48 @@
+package train.common.core.network;
+
+import cpw.mods.fml.common.network.simpleimpl.IMessage;
+import io.netty.buffer.ByteBuf;
+import net.minecraft.client.Minecraft;
+import net.minecraft.entity.Entity;
+import net.minecraft.entity.player.EntityPlayer;
+import net.minecraftforge.common.DimensionManager;
+import train.common.api.AbstractTrains;
+
+/**
+ * <h1>Mount packet</h1>
+ * This is intended to be a replacement for
+ * @see net.minecraft.network.play.client.C02PacketUseEntity
+ * because for whatever reason, the stupid thing refuses to send for our entities.
+ * @author Eternal Blue Flame
+ */
+public class PacketInteract implements IMessage {
+    /**the ID of the entity to dismount from*/
+    private int entityId, dimensionId, playerId;
+
+    public PacketInteract() {}
+    public PacketInteract(int entityId) {
+        this.entityId = entityId;
+        this.dimensionId= Minecraft.getMinecraft().thePlayer.worldObj.provider.dimensionId;
+        this.playerId=Minecraft.getMinecraft().thePlayer.getEntityId();
+
+    }
+    /**reads the packet on server to get the variables from the Byte Buffer*/
+    @Override
+    public void fromBytes(ByteBuf bbuf) {
+        entityId = bbuf.readInt();
+        dimensionId=bbuf.readInt();
+        playerId=bbuf.readInt();
+        Entity e = DimensionManager.getWorld(dimensionId).getEntityByID(entityId);
+        Entity p = DimensionManager.getWorld(dimensionId).getEntityByID(playerId);
+        if (e instanceof AbstractTrains && p instanceof EntityPlayer) {
+            e.interactFirst((EntityPlayer)p);
+        }
+    }
+    /**puts the variables into a Byte Buffer so they can be sent to server*/
+    @Override
+    public void toBytes(ByteBuf bbuf) {
+        bbuf.writeInt(entityId);
+        bbuf.writeInt(dimensionId);
+        bbuf.writeInt(playerId);
+    }
+}

src/main/java/train/common/entity/CollisionBox.java

@@ -24,6 +24,7 @@
 import net.minecraftforge.fluids.IFluidHandler;
 import train.common.Traincraft;
 import train.common.api.EntityRollingStock;
+import train.common.core.network.PacketInteract;
 import train.common.core.network.PacketRemove;
 
 
@@ -61,6 +62,10 @@ public String getCommandSenderName() {
 
     @Override
     public boolean interactFirst(EntityPlayer p_130002_1_) {
+        if(worldObj.isRemote){
+            Traincraft.keyChannel.sendToServer(new PacketInteract(host.getEntityId()));
+            return true;
+        }
         return host != null && host.interactFirst(p_130002_1_);
     }
 
@@ -82,6 +87,7 @@ public void onUpdate() {
     public boolean attackEntityFrom(DamageSource damageSource, float p_70097_2_) {
         if(worldObj.isRemote){
             Traincraft.keyChannel.sendToServer(new PacketRemove(host.getEntityId(), damageSource==null?-1:damageSource.getEntity().getEntityId()));
+            return true;
         }
         return host != null && host.attackEntityFromPart(this, damageSource, p_70097_2_);
     }