I made a left click event so that left clicking will fire out a fireball (I will make it specific to some items after I get it to work)

I have got this to work on a right click event in the tool class and it works fine.

package com.ChaoticSoul.MoreArmor.items.tools;

import com.ChaoticSoul.MoreArmor.Main;
import com.ChaoticSoul.MoreArmor.init.ModItems;
import com.ChaoticSoul.MoreArmor.util.IHasModel;

import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.projectile.EntityLargeFireball;
import net.minecraft.item.ItemStack;
import net.minecraft.item.ItemSword;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.math.Vec3d;
import net.minecraft.world.World;

public class ToolSword extends ItemSword implements IHasModel
{
	
	public ToolSword(String name, ToolMaterial material)
	{
		super(material);
		setUnlocalizedName(name);
		setRegistryName(name);
		setCreativeTab(CreativeTabs.COMBAT);
		
		
		ModItems.ITEMS.add(this);
	}
	
	@Override
	public void registerModels() 
	{
		Main.proxy.registerItemRenderer(this, 0, "inventory");
		
	}
	
	public ActionResult<ItemStack> onEntitySwing(EntityLivingBase entityLiving, ItemStack stack, EnumHand handIn, World worldIn) {
		ItemStack item = entityLiving.getHeldItem(handIn);
		Vec3d aim = entityLiving.getLookVec();
		EntityLargeFireball fireball = new EntityLargeFireball(worldIn, entityLiving, 1, 1, 1);
		
		fireball.setPosition(entityLiving.posX, entityLiving.posY, entityLiving.posZ);
		fireball.accelerationX = aim.x * 0.1; fireball.accelerationY = aim.y * 0.1; fireball.accelerationZ = aim.z * 0.1;
		worldIn.spawnEntity(fireball);
		
		item.damageItem(1, entityLiving);
		return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, item);
	}
	@Override
	public ActionResult<ItemStack> onItemRightClick(World worldIn, EntityPlayer playerIn, EnumHand handIn) {
		ItemStack item = playerIn.getHeldItem(handIn);
		Vec3d aim = playerIn.getLookVec();
		EntityLargeFireball fireball = new EntityLargeFireball(worldIn, playerIn, 1, 1, 1);
		
		fireball.setPosition(playerIn.posX, playerIn.posY, playerIn.posZ);
		fireball.accelerationX = aim.x * 0.1; fireball.accelerationY = aim.y * 0.1; fireball.accelerationZ = aim.z * 0.1;
		worldIn.spawnEntity(fireball);
		
		item.damageItem(1, playerIn);
		return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, item);
	}

}
	

I then made a left click event which spawns and fires the fireball correctly but the fireball just travels through all entities and blocks without interacting in any way.

package com.ChaoticSoul.MoreArmor.events;

import net.minecraft.entity.player.EntityPlayer;
import net.minecraft.entity.projectile.EntityLargeFireball;
import net.minecraft.item.ItemStack;
import net.minecraft.util.ActionResult;
import net.minecraft.util.EnumActionResult;
import net.minecraft.util.EnumHand;
import net.minecraft.util.math.Vec3d;
import net.minecraft.world.World;
import net.minecraftforge.event.entity.player.PlayerInteractEvent;
import net.minecraftforge.event.entity.player.PlayerInteractEvent.LeftClickEmpty;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

public class MAEventHandler {
	@SubscribeEvent
	public ActionResult<ItemStack> leftClick(LeftClickEmpty event) {
		ItemStack item = event.getEntityPlayer().getHeldItemMainhand();
		Vec3d aim = event.getEntityPlayer().getLookVec();
		World worldIn = event.getEntityPlayer().getEntityWorld();
		EntityPlayer playerIn = event.getEntityPlayer(); 
		EntityLargeFireball fireball = new EntityLargeFireball(worldIn, playerIn, 1, 1, 1);
		
		fireball.setPosition(playerIn.posX, playerIn.posY, playerIn.posZ);
		fireball.accelerationX = aim.x * 0.1; fireball.accelerationY = aim.y * 0.1; fireball.accelerationZ = aim.z * 0.1;
		worldIn.spawnEntity(fireball);
		
		item.damageItem(1, playerIn);
		return new ActionResult<ItemStack>(EnumActionResult.SUCCESS, item);
	}

}

 

Edited July 19, 20196 yr by clearcut

You should always spawn your entities on the server.  Use: !world.isRemote()

Also, model registration is client only, so it should go in a proxy and not in you Item Class

The LeftClickEmpty event is client-side only.

Use packets to notify the server of the left click.

7 hours ago, DavidM said:

The LeftClickEmpty event is client-side only.

Use packets to notify the server of the left click.

Thanks for the reply, but I do not know how to do this as I am still new to this.