Solved Command not working (no error) (need help)

Discussion in 'Plugin Development' started by dajako, Nov 23, 2013.

Thread Status:
Not open for further replies.
  1. Command shtuphz:
    Code:java
    1. public boolean onCommand(CommandSender sender, Command cmd, String lbl, String[] args){
    2. String name = sender.getName();
    3. Player player = (Player) sender;
    4. if(cmd.getName().equalsIgnoreCase("team")){
    5. if (args.length < 1) {
    6. player.sendMessage("Correct usage: /team create/join/leave/info");
    7. }else if (args[0] == "create") {
    8. player.sendMessage("Test1");
    9. if (args[0] != "" || args[1] != ""){
    10. player.sendMessage("Test2");
    11. getConfig().createSection("team." + args[1]);
    12. getConfig().set("team." + args[1], Arrays.asList(inteam));
    13. getConfig().getStringList("team." + args[1]).add(name);
    14. getConfig().set("team." + args[1], Arrays.asList(inteam));
    15. getConfig().createSection("team." + args[1] + ".password");
    16. getConfig().set("team." + args[1] + ".password", args[2]);
    17. getConfig().createSection("playersteam." + name);
    18. getConfig().set("playersteam." + name, args[1]);
    19. saveConfig();
    20. player.sendMessage("Test3");
    21. return true;
    22. }else{player.sendMessage("Correct usage: /team create <name> <password>"); return true;}
    23. }else if (args[0] == "info") {
    24. player.sendMessage("Your current team: " + getConfig().getString("playersteam." + name));
    25. return true;
    26. }
    27. return true;
    28. }
    29. return false;
    30. }


    plugin.yml:
    Code:
    name: Custom
    main: me.dajakos.server.plugin
    version: 1.0
     
    commands:
      team:
    There are no errors. If I type /team I get send "Correct usage: /team create/join/leave/info" as I should but I don't get the "Test1" message or any of the other debug messages from typing "/team create" .

    Thanks in advance!
     
  2. Offline

    Gater12

    dajako Shouldn't use else if to check for args. Try his instead:
    Code:java
    1.  
    2. if(args.length == 0){
    3. player.sendMessage("Correct usage: blah blah blah");
    4. return true;
    5. }
    6. if(args.length > 1){
    7. if(args[0].equalsIgnoreCase("WHATEVER")){
    8. //CODE GOES HERE
    9. return true;
    10. }
    11. if(args[0].equalsIgnoreCase("WHATEVERHERE")){
    12. //CODE GOES HERE
    13. return true;
    14. }
    15. }
    16.  
     
    dajako likes this.
  3. Thanks, I'll try that now!

    Gater12 Worked great, thanks.

    EDIT by Moderator: merged posts, please use the edit button instead of double posting.
     
    Last edited by a moderator: Jun 5, 2016
Thread Status:
Not open for further replies.

Share This Page