Solved Send Message on BlockPlaceEvent

So I want to send a message to the players when Stone is placed. Just trying to learn the different classes and what no in Java.

my Code:

Code:java

package Dae.Spell;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class spell extends JavaPlugin implements Listener {
    public void onEnabled() {
        getLogger().info("Spell Cast has been enabled");
 
    }
    public void onDisable() {
        getLogger().info("Spell Cast has been disabled");
 
    }
    @EventHandler
    public void Bplace(BlockPlaceEvent bpe, Player p){
        if(bpe.getBlockPlaced().getType().equals(Material.STONE)){
            bpe.getPlayer().sendMessage(ChatColor.RED + "You placed Stone!");
        }
        else{
            bpe.getPlayer().sendMessage(ChatColor.BLUE + "Thats not stone!");
 
        }
 
    }
 
}
 

 

You do not have to have the Player variable in the BlockPlaceEvent void. Also, you have to remember to register your events, and then it should all work wonderfully.

 

Use this in your onEnable method in order to register the events. Then you need to remove the Player variable as ElliottOlson said.

Code:

            Bukkit.getServer().getPluginManager().registerEvents(listener, plugin);

 

AS ElliottOlson said,
Make sure you have the following:

Code:java

 Player player = event.getPlayer();

Code:java

Material block = event.getBlock().getType();

Then you can simply write

Code:java

player.sendMessage("You have placed" + block);

Also if you have it as a different calss to your main one, make sure you register the events as LightMC has given
Hope this helps,
Best Of Luck,
ProMCKingz

 

Your code is almost correct, however your function parameters are wrong. Here is my version:

Code:java

 @EventHandler
public void CheckForStone(BlockPlaceEvent e)
{
if (e.getBlockPlaced().getType().equals(Material.STONE))
{
// The block placed is stone.
}
else
{
// The block place is not stone.
}
}

Then just remember to register the listeners:

Bukkit.getServer().getPluginManager().registerEvents(this, this);

 

Code:java

package Dae.Spell;
 
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class spell extends JavaPlugin implements Listener {
    public void onEnabled() {
        getLogger().info("Spell Cast has been enabled");
        Bukkit.getServer().getPluginManager().registerEvents(this, this);
    }
    public void onDisable() {
        getLogger().info("Spell Cast has been disabled");
 
    }
    @EventHandler
    public void onBlockPlace(BlockPlaceEvent bpe){
        if(bpe.getBlockPlaced().getType().equals(Material.STONE)){
            bpe.getPlayer().sendMessage(ChatColor.RED + "You placed Stone!");
        }
        else{
            bpe.getPlayer().sendMessage(ChatColor.BLUE + "That's not stone!");
 
        }
 
    }
 
}
 

 

Thank you everyone! Im trying to learn how to code, I have many plugin ideas but learning is the main battle!

 

You should mark the thread as solved.

 

I used this and it didnt seem to work

 

malikdbuseck It has to be onEnable not onEnabled. Other than that, it seems like it will work.

 

Sorry, it was a typo I made