Solved Commands not registering

The video explains it all



Thanks for any help!

 

SoTotallyRoary
It appears as all of your commands (such as /bank check) are registered as one work, like /bankcheck Try entering that and see if it helps

 

That worked but the other commands do not

 

SoTotallyRoary so even if you entered your commands exactly as they appeared in the plugin.yml, they dont work?

lol oops, i meant one word ^^^^^^

Can you by any chance see if you can help me with my problem? HashMaps

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

bobthefish
Here's the second video to help


Code:

Code:java

package sototallyroary.superbank.assassinhero;
 
import java.util.logging.Logger;
 
import net.milkbowl.vault.chat.Chat;
import net.milkbowl.vault.economy.Economy;
import net.milkbowl.vault.permission.Permission;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.plugin.RegisteredServiceProvider;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener{
 
    public static final Logger logger = Logger.getLogger("Minecraft");
    public static Economy Econ = null;
    public static Permission perms  = null;
    public static Chat chat = null;
 
 
    public void onEnable(){
        logger.info("SuperBank Has Been Enabled Successfully!");
        getConfig().options().copyDefaults(true);
        saveConfig();
        if (!setupEconomy()){
            logger.severe(String.format("[%s] - Disabled due to no Vault dependency found!", getDescription().getName()));
            getServer().getPluginManager().disablePlugin(this);
            return;
        }
        setupPermissions();
        setupChat();
 
 
    }
 
        private boolean setupEconomy(){
            if(getServer().getPluginManager().getPlugin("Vault") == null){
                return false;
            }
            RegisteredServiceProvider<Economy> rsp = getServer().getServicesManager().getRegistration(Economy.class);
            if (rsp == null){
                return false;
            }
            Econ = rsp.getProvider();
            return Econ != null;
 
        }
 
        private boolean setupChat(){
            RegisteredServiceProvider<Chat> rsp = getServer().getServicesManager().getRegistration(Chat.class);
            chat = rsp.getProvider();
                return chat !=null;
            }
 
        private boolean setupPermissions(){
            RegisteredServiceProvider<Permission> rsp = getServer().getServicesManager().getRegistration(Permission.class);
            perms = rsp.getProvider();
            return perms !=null;
        }
 
        public void onDisable(){
        logger.info("SuperBank Has Been Disabled Successfully!");
        logger.info(String.format("[%s] Disabled Version %s", getDescription(),getName(), getDescription().getVersion()));
        saveConfig();
        }
 
        public static double amount;
 
    @EventHandler
    public void PlayerJoinEvent(PlayerJoinEvent event){
        if(!event.getPlayer().hasPlayedBefore()){
            Econ.createBank("WorldBank", getName());
        }
 
        }
 
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args){
        Player player = (Player) sender;
        if(cmd.getName().equalsIgnoreCase("BankCheck")){
            player.hasPermission("Bank.Check");
            Econ.isBankOwner("WorldBank", getName());
          Econ.getBalance(getName());
        }
 
        if (cmd.getName().equalsIgnoreCase("BankDeposit")){
            player.hasPermission("Bank.Deposit");
            Econ.bankDeposit(getName(), amount);
            player.sendMessage("You have deposited" + amount + "from your bank!");
            Econ.getBalance(getName());
        }
 
        if(cmd.getName().equalsIgnoreCase("BankList")){
            player.hasPermission("Bank.List");
            Econ.getBanks();
        }else{
            player.sendMessage("You Do NOT Have Permissions");
        }
        if(cmd.getName().equalsIgnoreCase("BankWithdraw")){
            player.hasPermission("Bank.Withdraw");
            Econ.bankWithdraw(getName(), amount);
            player.sendMessage("You have withdrawn" + amount + "from your bank!");
            Econ.getBalance(getName());
        }
 
        if(cmd.getName().equalsIgnoreCase("BankDelete")){
            player.hasPermission("Bank.Delete");
            Econ.deleteBank(getName());
        }
        return true;
 
}
}

plugin.yml

Code:

name: SuperBank
main: sototallyroary.superbank.assassinhero.Main
version: 1.0.0
author: SoTotallyRoary
commands:
  Bank Check:
    description: Allows you to check the balance of your bank
    usage: /Bank Check
  Bank Deposit:
    description: Allows you to deposit into your bank
    usage: /Bank Deposit
  Bank Withdraw:
    description: Allows you to withdraw money from your bank
    usage: /Bank Withdraw
  Bank List:
    description: Allows you to check the list of banks
    usage: /Bank List
  Bank Delete:
    description: Allows you to delete a bank USE ADMIN ONLY
    usage: /Bank Delete
permissions:
    Bank.Check:
      description: Allows the use of command Bank Check
      default: true
    Bank.Deposit:
      description: Allows the use of command Bank Deposit
      default: true
    Bank.Withdraw:
      description: Allows the use of command Bank Withdraw
      default: true
    Bank.List:
      description: Allows the use of command Bank List
      default: op
    Bank.Delete:
      description: Allows the use of command Bank Delete
      default: op

 

SoTotallyRoary First, your welcome, 2nd, you see where it says:

Code:java

        if (cmd.getName().equalsIgnoreCase("BankDeposit")){

Between "Bank" and "Deposit" you need a space, so it should look like:

Code:java

        if (cmd.getName().equalsIgnoreCase("Bank Deposit")){

And also, typically commands are only one word long, with argument, if you want me to show you how to do that then I can, but for now, i believe that adding that space should work

 

It's working now. I've just gotta bug fix Thank you so much!!

 

SoTotallyRoary
Did I fix it? Or was it something else, sorry just would like to know

 

You fixed it I changed it to a one word command as you said and now it works!

 

SoTotallyRoary
ok, great! now, do you want any help with arguments for the commands? or are you good?

 

I'm good now pal. Thanks!