Plugin issues...

Hi Everyone,

I am having some issues, with a simple plugin of mine. What i am expecting to happen is that when someone does the commands, they will get a message explaining some commands, and chat channels.

I think it has something to do with my plugin.yml, i have run it through the YML parser, and it looked like it would for any normal YML file.

Here is the plugin.yml: http://pastebin.com/s3mgNKWW

And my Code:

Code:java

 
package us.craftville.chat;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.java.JavaPlugin;
 
public final class Chat extends JavaPlugin {
@Override
public void onDisable(){
 
}
 
@Override
public void onEnable() {
 
}
 
public boolean onCommand(CommandSender cdsend, Command cmd, String label, String[] args){
if(cmd.getName().equalsIgnoreCase("Chat")){
if(args[0] == "channels"){
cdsend.sendMessage(ChatColor.AQUA + "~~CraftVille Chat Channels: ~~");
cdsend.sendMessage(ChatColor.WHITE + "G: Global chat (General Chat) - /g");
cdsend.sendMessage(ChatColor.YELLOW + "H: Help Chat (Server Questions) - /h");
cdsend.sendMessage(ChatColor.DARK_AQUA + "TC: Town Chat (Chat to town) - /tc");
cdsend.sendMessage(ChatColor.AQUA +"~~~ END ~~~");
}
else if(args[0] == "commands" ){
cdsend.sendMessage(ChatColor.AQUA + "~~ CraftVille Chat Commands ~~");
cdsend.sendMessage(ChatColor.WHITE + "/ignore - Ignore a player");
cdsend.sendMessage(ChatColor.WHITE + "/<channel> - Focus on <channel> Chat Channel");
}
else if(args.length == 0){
cdsend.sendMessage(ChatColor.RED + "~~ CraftVille Chat ~~");
cdsend.sendMessage(ChatColor.WHITE + "Please use one of the following Commands:");
cdsend.sendMessage(ChatColor.AQUA + "/chat Channels - For a list of channels");
cdsend.sendMessage(ChatColor.AQUA + "/chat commands - For a list of commands");
}
 
}
 
return true;
}
}
 

I appreciate the help, sorry if it is something really silly, i have worked on this for hours though.

 

I think the usage in the plugin.yml is wrong.

 

You need to check args.length ...

teunie75 that means nothing.

 

I just updated it, is this what it should have been?

Code:java

 
package us.craftville.chat;
 
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.java.JavaPlugin;
 
public final class Chat extends JavaPlugin {
    @Override
    public void onDisable(){
 
    }
 
    @Override
    public void onEnable() {
 
    }
 
    public boolean onCommand(CommandSender cdsend, Command cmd, String label, String[] args){
        if(cmd.getName().equalsIgnoreCase("Chat")){
            if (args.length == 1) {
                if(args[0] == "channels"){
                    cdsend.sendMessage(ChatColor.AQUA + "~~CraftVille Chat Channels: ~~");
                    cdsend.sendMessage(ChatColor.WHITE + "G: Global chat (General Chat) - /g");
                    cdsend.sendMessage(ChatColor.YELLOW + "H: Help Chat (Server Questions) - /h");
                    cdsend.sendMessage(ChatColor.DARK_AQUA + "TC: Town Chat (Chat to town) - /tc");
                    cdsend.sendMessage(ChatColor.AQUA +"~~~ END ~~~");
                }
                else if(args[0] == "commands" ){
                    cdsend.sendMessage(ChatColor.AQUA + "~~ CraftVille Chat Commands ~~");
                    cdsend.sendMessage(ChatColor.WHITE + "/ignore - Ignore a player");
                    cdsend.sendMessage(ChatColor.WHITE + "/<channel> - Focus on <channel> Chat Channel");
                }
 
            }
            else{
                cdsend.sendMessage(ChatColor.RED + "~~ CraftVille Chat ~~");
                cdsend.sendMessage(ChatColor.WHITE + "Please use one of the following Commands:");
                cdsend.sendMessage(ChatColor.AQUA + "/chat Channels - For a list of channels");
                cdsend.sendMessage(ChatColor.AQUA + "/chat commands - For a list of commands");
            }
 

What is wrong though, This does not help me get anywhere?

 

Looks good to me.

Edit: You are missing brackets { } and booleans need to return a, well, boolean.

Add 'return false; ' before the last bracket