Need help with shooting fire from sword

So basically, I need help figuring out how I would be able to make a Stone Sword "shoot" Fire (The Item) and when it hits a player... The player would be set on fire

I have a basic piece of code that lets the fire get shot out of the sword, but I don't know & need help on how I would be able to make it burn a player

 

AeroMC check if it hits a entity then if its true then set it on fire if not then cancel the event

 

Sorry for sounding noobish, but could you give me an example?

Code:java

package me.troll.abilities;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.Sound;
import org.bukkit.World;
import org.bukkit.entity.Entity;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Item;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.event.player.PlayerPickupItemEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.util.Vector;
 
    public class MAIN extends JavaPlugin implements Listener {
 
        public void onEnable() {
            Bukkit.getServer().getLogger().info("Plugin enabled!");
            Bukkit.getPluginManager().registerEvents(this, this);
        }
 
        @EventHandler
        public void fireFromSword(PlayerInteractEvent event) {
            Player player = event.getPlayer();
            World world = player.getWorld();
            if (player.getItemInHand().getType() == Material.STONE_SWORD) {
                if (event.getAction() == Action.RIGHT_CLICK_AIR) {
                    final Item grenade = world.dropItem(player.getEyeLocation(), new ItemStack(Material.FIRE));
                    grenade.setVelocity(player.getEyeLocation().getDirection());
                    for (Entity e : player.getNearbyEntities(5, 5, 5)) {
                        if (e.getType() == EntityType.PLAYER) {
                            player.setFireTicks(100);
                        }
                    }
                }
            }
        }
 
        //Method for "hit"
        @EventHandler
        public void onFireHit(PlayerPickupItemEvent e) {
        Player p = e.getPlayer();
        ItemStack i = e.getItem().getItemStack();
        //Checks if the item was a ninja star
        if (i.getType().equals(Material.FIRE)) {
        Item item = e.getItem();
        //Checks if it is moving through the air
        if (item.getVelocity().length() < 1){
        //If item is still, does nothing
        e.setCancelled(false);
        }else if (item.getVelocity().length() > 1 ) {
        //If item is moving, kills the player and removes the item drop from the world
        //You can just hurt the player instead by doing the method below
        //p.setHealth(p.getMaxHealth() - [amount]);
        //Otherwise, this kills the player.
        e.setCancelled(true);
        p.setHealth(0.0);
        item.remove();}}}
 
        //Method for throwing star
        @EventHandler
        public void onThrow(PlayerInteractEvent e) {
        Player p = e.getPlayer();
        //Checks if it is right clicked
        if (e.getAction() == Action.RIGHT_CLICK_AIR || e.getAction().equals(Action.RIGHT_CLICK_BLOCK)) {
        ItemStack hand = p.getItemInHand();
        //Checks if item is nether star
        if (hand.getType().equals(Material.NETHER_STAR)) {
        //Checks if item has name "Ninja Star"
        if (hand.getItemMeta().hasDisplayName()) {
        if (hand.getItemMeta().getDisplayName().equals("Ninja Star")) {
        //Throws the star
        Item item = p.getWorld().dropItem(p.getEyeLocation().add(new Vector(0, 1, 0)), hand);
        item.setVelocity(p.getEyeLocation().getDirection().multiply(3.0));
        p.getInventory().removeItem(new ItemStack(Material.NETHER_STAR, 1));
        item.setPickupDelay(2);
        //Tells the player that it was successful
        p.sendMessage("" + ChatColor.GRAY + ChatColor.ITALIC + "Works");
        }
        }
        }
        }
        }
 
 
        //Method for "hit"
        @EventHandler
        public void onHit(PlayerPickupItemEvent e) {
        Player p = e.getPlayer();
        ItemStack i = e.getItem().getItemStack();
        //Checks if the item was a ninja star
        if (i.getType().equals(Material.NETHER_STAR)) {
        if (i.getItemMeta().hasDisplayName()) {
        if (i.getItemMeta().getDisplayName().equals("Ninja Star")) {
        Item item = e.getItem();
        //Checks if it is moving through the air
        if (item.getVelocity().length() < 1){
        //If item is still, does nothing
        e.setCancelled(false);
        }else if (item.getVelocity().length() > 1 ) {
        //If item is moving, kills the player and removes the item drop from the world
        //You can just hurt the player instead by doing the method below
        //p.setHealth(p.getMaxHealth() - [amount]);
        //Otherwise, this kills the player.
        e.setCancelled(true);
        p.setHealth(0.0);
        item.remove();}}}
        }
        }
        }

 

AeroMC does ur code not work?

 

JordyPwner The only thing it does is shoots the items, but it won't set the player on fire when they get hit

EDIT: Also, how would I make it so the fire gets shot out repeatedly instead of one at a time?

 

AeroMC
My advise is that you do the following:
When spawning fire items with velocity, apply a custom name which is that of the player who 'shot' the fire and "active_fire" :

Code:

item.getItemStack().getItemMeta().setDisplayName(player+"active_fire");

Add this player to a list with "active_fire" :

Code:

 exemptPlayer.add(player+"active_fire");

Then detect when a player picks up an item with an event which contains if statements to determine correct conditions are met.

Final Example:

Code:

ArrayList<String> exemptPlayer = new ArrayList<>();
@EventHandler
public void playerpickupitem(PlayerPickupItemEvent event){
Player player = event.getPlayer();
Item item = event.getItem();
String itemName = item.getItemStack().getItemMeta().getDisplayName();
if(!(exemptPlayer.contains(itemName))){
/*if list doesnt contain item name, allow item to be picked up*/
event.setCancelled(false);
}
if(exemptPlayer.contains(itemName) && player.getName().contains(itemName)){
/*stop shooter from being on fire from own shot and item's name is listed*/
event.setCancelled(true);
}
if(exemptPlayer.contains(itemName) && !(player.getName().contains(itemName))){
/*if player is not shooter and item's name is listed*/
/*remove item (despawn it)*/
item.remove();
int timeonfire = 80;
/*20 ticks is aprox 1 second - when a server is running perfectly*/
player.setFireTicks(timeonfire);
/*got this to set player on fire for 4 seconds aprox, or 80 ticks*/
}
}

Note: This code is untested, and more than likely contains loop-holes and bugs.

Hope this helped,
- Minty

 

Thanks for the reply!
1 Question (Again, sorry for sounding like a noob, I've only recently got back into Bukkit from leaving it a long time): Where would I put

Code:java

item.getItemStack().getItemMeta().setDisplayName(player+"active_fire");

and

Code:Java

 
exemptPlayer.add(player+"active_fire");
 

 

Something like this:

Code:

ArrayList<String> exemptPlayer = new ArrayList<>();
@EventHandler
public void fireFromSword(PlayerInteractEvent event) {
Player player = event.getPlayer();
World world = player.getWorld();
if (player.getItemInHand().getType() == Material.STONE_SWORD) {
if (event.getAction() == Action.RIGHT_CLICK_AIR) {
Item grenade = world.dropItem(player.getEyeLocation(), new ItemStack(Material.FIRE));
/*apply displayname*/           
grenade.getItemStack().getItemMeta().setDisplayName(player.getName()+"active_fire");
grenade.setVelocity(player.getEyeLocation().getDirection());
/*add to exemptPlayer ArrayList*/
exemptPlayer.add(player.getName()+"active_fire");
}
}
}

Please notify me of any errors that may occur in the testing procedure of the plugin so I can be a good inspector.

Hope this also helped,
- Minty

Edit: If you like you could post your current code (with the above appended) and I'l tweak it for you - if your having further issues with it

 

When I copy 'n' paste the code you gave me, I get an error "item.getItemStack().getItemMeta().setDisplayName(player+"active_fire");"
"The method getItemStack() is undefined for the type ItemStack"

EDIT: Nevermind I fixed the error... But one question!
How would I be able to keep it coming without having to keep clicking, just by holding it in?

EDIT 2: It won't set the player on fire still

Code:java

package me.trollzor.kits;
 
import java.util.ArrayList;
 
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Item;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.block.Action;
import org.bukkit.event.entity.EntityDamageByEntityEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
 
public class Main extends JavaPlugin {
 
public void onEnable(){
getLogger().info("Enabled");
}
 
public void onDisable(){
getLogger().info("Disabled");
}
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
 
    // Scout Sword   
    ItemStack scoutSword = new ItemStack(Material.DIAMOND_SWORD, 1);
    scoutSword.addUnsafeEnchantment(Enchantment.DAMAGE_ALL, 1);
    ItemMeta im = scoutSword.getItemMeta();
    im.setDisplayName("§b§oKnife"); 
    scoutSword.setItemMeta(im);
 
    // Ninja Stars
    ItemStack ninjaStar = new ItemStack(Material.NETHER_STAR, 1);
    ItemMeta im1 = ninjaStar.getItemMeta();
    im1.setDisplayName("Ninja Star"); 
    ninjaStar.setItemMeta(im1);
 
    Player player = (Player) sender;
if(label.equalsIgnoreCase("knight")) {
    player.getInventory().clear();
    player.setHealth(15.0);
    player.setMaxHealth(15.0);
    ItemStack scoutHelmet = new ItemStack(Material.IRON_HELMET, 1);
    ItemStack scoutChestplate = new ItemStack(Material.IRON_CHESTPLATE, 1);
    ItemStack scoutLeggings = new ItemStack(Material.IRON_LEGGINGS, 1);
    ItemStack scoutBoots = new ItemStack(Material.IRON_BOOTS, 1);
    player.sendMessage("§9Class > §7You are now §aScout");
    player.addPotionEffect(new PotionEffect(PotionEffectType.SPEED, 10000, 1));
    player.getInventory().addItem(scoutSword);
    player.getInventory().setHelmet(scoutHelmet);
    player.getInventory().setChestplate(scoutChestplate);
    player.getInventory().setLeggings(scoutLeggings);
    player.getInventory().setBoots(scoutBoots);
    player.getInventory().addItem(new ItemStack(Material.GOLDEN_APPLE, 10));
 
}else{ if(label.equalsIgnoreCase("ninja")) {
    player.getInventory().clear();
    player.setHealth(15.0);
    player.setMaxHealth(15.0);
    player.removePotionEffect(PotionEffectType.SPEED);
    ItemStack heavyHelmet = new ItemStack(Material.LEATHER_HELMET, 1);
    ItemStack heavyChestplate = new ItemStack(Material.LEATHER_CHESTPLATE, 1);
    ItemStack heavyLeggings = new ItemStack(Material.LEATHER_LEGGINGS, 1);
    ItemStack heavyBoots = new ItemStack(Material.LEATHER_BOOTS, 1);
    player.sendMessage("§9Class > §7You are now §aNinja");
    player.addPotionEffect(new PotionEffect(PotionEffectType.SLOW, 1000000, 1));
    player.getInventory().addItem(new ItemStack(ninjaStar));
    player.getInventory().addItem(new ItemStack(ninjaStar));
    player.getInventory().addItem(new ItemStack(ninjaStar));
    player.getInventory().addItem(new ItemStack(ninjaStar));
    player.getInventory().addItem(new ItemStack(ninjaStar));
    player.getInventory().setHelmet(heavyHelmet);
    player.getInventory().setChestplate(heavyChestplate);
    player.getInventory().setLeggings(heavyLeggings);
    player.getInventory().setBoots(heavyBoots);
 
}else{ if(label.equalsIgnoreCase("kits")) {
    player.sendMessage("§7§m--------------------------------------------------");
    player.sendMessage("");
    player.sendMessage("§bKits:");
    player.sendMessage("");
    player.sendMessage("Scout");
    player.sendMessage("§7§o");
}
 
return false;
}
return false;
}
return false;
}
 
@EventHandler
public void onDamage(EntityDamageByEntityEvent event) {
    if (event.getDamager() instanceof Player
            && ((Player)event.getDamager()).getItemInHand().getType() == Material.STONE_SWORD) {
        ItemStack sword = ((Player)event.getDamager()).getItemInHand();
        if (sword.hasItemMeta()
                && sword.getItemMeta().hasDisplayName()
                && sword.getItemMeta().getDisplayName().equalsIgnoreCase("Knife")) {
            event.setDamage(7.5);
        }
    }
}
 
ArrayList<String> exemptPlayer = new ArrayList<>();
@EventHandler
public void fireFromSword(PlayerInteractEvent event) {
Player player = event.getPlayer();
World world = player.getWorld();
ItemStack item = event.getItem();
if (player.getItemInHand().getType() == Material.STONE_SWORD) {
if (event.getAction() == Action.RIGHT_CLICK_AIR) {
Item grenade = world.dropItem(player.getEyeLocation(), new ItemStack(Material.FIRE));
item.getItemMeta().setDisplayName(player+"active_fire");       
grenade.getItemStack().getItemMeta().setDisplayName(player.getName()+"active_fire");
grenade.setVelocity(player.getEyeLocation().getDirection());
exemptPlayer.add(player+"active_fire");
exemptPlayer.add(player.getName()+"active_fire");
}
}
}
}

 

AeroMC You need to use a runnable to keep it going after being clicked once.

Code:

new BukkitRunnable() {
 
  @override
  public void run() {
 
      //Code to make fire shoot
 
  }
 
}.runTaskTimer(timeBeforeRunnableStarts, intevalToRunAt);

 

GeorgeeeHD

I am soo(ooooo...) sorry about this, but could you show me an "example" I can't seem to get mine working :/ >.<

 

AeroMC i did give an example...

 

GeorgeeeHD I mean like could you show me what I have to put inside the runnable and how to set the timer thing, it won't work for me... I haven't used runnables before