Cmd Entering Command

I have a break block event and I don't know how to fix a problem. I want the cmd to send 10 bucks to a player for breaking a diamond block.

Code:

            @EventHandler
            public void onBlockBreak(BlockBreakEvent e) {
                Block b = e.getBlock();
                Player p = e.getPlayer();
                if (b.getType() == Material.DIAMOND_BLOCK) {
                    p.getInventory().addItem(new ItemStack(Material.AIR));
                    String cmd = "eco " + "give " + e.getPlayer() + " 10";
                    Bukkit.dispatchCommand(Bukkit.getConsoleSender(), cmd);

How do I fix it? It will not send any money to the player.

 

Ok, so where do I replace it? I get confused fast :/

 

Code:

            @EventHandler
            public void onBlockBreak(BlockBreakEvent e) {
                Block b = e.getBlock();
                Player p = e.getPlayer();
                if (b.getType() == Material.DIAMOND_BLOCK) {
                    p.getInventory().addItem(new ItemStack(Material.DIAMOND_BLOCK));
                    String cmd = "eco " + "give " + getPlayer().getName() + " 10";
                    Bukkit.dispatchCommand(Bukkit.getConsoleSender(), cmd);

So, I did what you said but it still wont work ,unless I put it in the wrong place. Can you show me how to do it? Maybe Fix it for me, you don't have to though.

 

@BladdonB Why is Eco and give seperate strings?

 

isnt the essentials command /eco give (player) ?

 

@BladdonB he means you can write "eco give"
getPlayer().getName() this is null dude

 

so what do I do then cause that's what the guy in the 2nd post said.

 

@BladdonB it's p.getname() or e.getplayer().getname();

 

@caderape I tried it and it still doesn't work

 

@BladdonB The event is called ? Do you have essentials on your server ?
Show your code

 

@BladdonB
Did you check if P is null? Did you check if the player Even exist? Did you register the Listener?Did you debug?

Please do the following before you post again.

 

I don't see a way #getPlayer() could return null unless some plugin is calling a new BlockBreakEvent with a null Player...

 

Did you register events ?

Code:

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

Also is better if you use Vault for this like this.

 

@caderape

Code:

package me.prison.main;

import org.bukkit.Bukkit;
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.inventory.ItemStack;
import org.bukkit.plugin.java.JavaPlugin;

public class Main extends JavaPlugin{
   
    //When the plugin is enabled!
            @Override
            public void onEnable(){
                getLogger().info("Prison autosell By: Bladdon is being enabled");
            }
           
            //when du plugin is disabledded
            public void OnDisable(){
                getLogger().info("Prison autosell By: Bladdon is being disabled");
            }
           
            @EventHandler
            public void onBlockBreak(BlockBreakEvent e) {
                Block b = e.getBlock();
                Player p = e.getPlayer();
                if (b.getType() == Material.DIAMOND_BLOCK) {
                    p.getInventory().addItem(new ItemStack(Material.AIR));
                    String cmd = "eco give " + e.getPlayer().getName() + " 10";
                    Bukkit.dispatchCommand(Bukkit.getConsoleSender(), cmd);
                    event.setCanceled(true);
                   
                }
               
               
            }
   

}

I do have essentials on my server.

 

@BladdonB
I suggest you learn Bukkit before you continue. You clearly do not have experience with making plugins as you fore got even add a Listener.

Please use this link and scroll down to "Learn Bukkit"
http://bukkit.org/threads/plugin-dev-sticky-learning-java-where-to-learn.350132/