My Own /setspawn & /spawn Wont work! HELP

Hey Guys! I am trying to make a plugin, that makes players tp to spawn.
i got some commands:
/towny = Tp you to the set spawn
/settowny = Sets the spawn

i really need some help beacuse it wont read my /towny command!
it reads my /settowny command and it works but when i am trying to /towny it will do nothing
PLEASE HELP!

My Code:

Code:java

package me.BechGameplay.townytp;
 
import java.util.logging.Logger;
 
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class main extends JavaPlugin{
    public final Logger logger = Logger.getLogger("Minecraft");
    public static main plugin;
 
    @Override
    public void onEnable(){
        this.getConfig().options().copyDefaults(true);
        this.saveConfig();
    }
 
    @Override
    public void onDisable(){
 
 
        this.saveConfig();
    }
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String CommandLabel, String[] args){
    Player player = (Player) sender;
 
        if(cmd.getName().equalsIgnoreCase("settowny")){
 
            if(sender instanceof Player){
 
                if(player.hasPermission("townytp.settowny")){
 
                    this.getConfig().set("Spawn" + ".X", player.getLocation().getBlockX());
                    this.getConfig().set("Spawn" + ".Y", player.getLocation().getBlockY());
                    this.getConfig().set("Spawn" + ".Z", player.getLocation().getBlockZ());
                    this.getConfig().set("Spawn" + ".Yaw", player.getLocation().getYaw());
                    this.getConfig().set("Spawn" + ".Pitch", player.getLocation().getPitch());
                    this.getConfig().set("Spawn" + ".World", player.getLocation().getWorld());
 
                    player.sendMessage(ChatColor.GREEN + "Spawn set!");
 
                }else if((cmd.getName().equalsIgnoreCase("towny"))){
 
                    if(sender instanceof Player){
 
                        if(player.hasPermission("townytp.towny")){
 
                            int spawnX = this.getConfig().getInt("Spawn" + ".X");
                            int spawnY = this.getConfig().getInt("Spawn" + ".Y");
                            int spawnZ = this.getConfig().getInt("Spawn" + ".Z");
                            int spawnYaw = this.getConfig().getInt("Spawn" + ".Yaw");
                            int spawnPitch = this.getConfig().getInt("Spawn" + ".Pitch");
                            Object world = this.getConfig().get("Spawn" + ".World");
 
                            Location spawn = new Location((World) world, spawnX, spawnY, spawnZ, spawnYaw, spawnPitch);
 
                            player.teleport(spawn);
 
                            player.sendMessage(ChatColor.GREEN + "Velkommen Til Towny Spawn!");
 
                }
            }
                    }
                }
            }
        return false;
    }
}

MY PLUGIN.YML

Code:

name: TownyTp
version: 1.0
main: me.BechGameplay.townytp.main
author: BechGameplay
website: www.google.com
commands:
    towny:
        description: tp dig til townyspawn
        permission: townytp.towny
    settowny:
        description: set Spawn
        permission: townytp.settowny
       
permissions:
  townytp.settowny:
    description: Set Towny Spawn
    default: op
  townytp.towny:
    description: Tp Dig Til Towny
    default: op

PLEASE HELP ME!

 

Remove the else and put the towny command on the next line

Code:java

                }else if((cmd.getName().equalsIgnoreCase("towny"))){
 

 

Arcticmike

Didnt Work......

 

Okay there i see it here's a fix

Code:java

    public boolean onCommand(CommandSender sender, Command cmd,
            String CommandLabel, String[] args){
    Player player = (Player) sender;
 
        if(cmd.getName().equalsIgnoreCase("settowny")){
 
            if(sender instanceof Player){
 
                if(player.hasPermission("townytp.settowny")){
 
                    this.getConfig().set("Spawn" + ".X", player.getLocation().getBlockX());
                    this.getConfig().set("Spawn" + ".Y", player.getLocation().getBlockY());
                    this.getConfig().set("Spawn" + ".Z", player.getLocation().getBlockZ());
                    this.getConfig().set("Spawn" + ".Yaw", player.getLocation().getYaw());
                    this.getConfig().set("Spawn" + ".Pitch", player.getLocation().getPitch());
                    this.getConfig().set("Spawn" + ".World", player.getLocation().getWorld());
 
                    player.sendMessage(ChatColor.GREEN + "Spawn set!");
 
                }
                }
            } else if((cmd.getName().equalsIgnoreCase("towny"))){
 
                if(sender instanceof Player){
 
                    if(player.hasPermission("townytp.towny")){
 
                        int spawnX = this.getConfig().getInt("Spawn" + ".X");
                        int spawnY = this.getConfig().getInt("Spawn" + ".Y");
                        int spawnZ = this.getConfig().getInt("Spawn" + ".Z");
                        int spawnYaw = this.getConfig().getInt("Spawn" + ".Yaw");
                        int spawnPitch = this.getConfig().getInt("Spawn" + ".Pitch");
                        Object world = this.getConfig().get("Spawn" + ".World");
 
                        Location spawn = new Location((World) world, spawnX, spawnY, spawnZ, spawnYaw, spawnPitch);
 
                        player.teleport(spawn);
 
                        player.sendMessage(ChatColor.GREEN + "Velkommen Til Towny Spawn!");
 
            }
        }
                }
        return false;
    }

 

Arcticmike

I will try it out. Thx! I will Tell you if it Works xD

 

Arcticmike
Can you help me how to make Cooldown and Delay on the Teleport!??

 

BechGameplay Please, just please, use google or any other search engine before asking. It's not that we won'te help, problem is that you will not learn how to do it. Learn it by yourself and if you really get stuck somewhere I'm sure there will be someone to help you with errors you can't solve.
Here's what you are asking for:
http://wiki.bukkit.org/Scheduler_Programming

 

Thx! I know but i couldnt find anything on the net!
Thx For Helping!

 

Well did it work?

 

Yes it worked so Fine! Thx You!

 

BechGameplay To make a delay, just make a bukkit runnable. This reads ticks. I believe 20 ticks is a second so if you want a 5 second delay, you would have to make a new runnable and set it to 100 ticks

 

You may want to be a little more wary of YouTube tutorials. Right away I see this:

Code:java

public final Logger logger = Logger.getLogger("Minecraft");
public static main plugin;

You are also casting sender to a Player before your instanceof check. You shouldn't declare a Player object until after the instanceof check.

You're saving the world object to the config and trying to read it back. When saving locations, you don't actually want to save the whole world; you just want to save its name, and Bukkit will get the world for you. To properly save the world's name:

Code:java

getConfig().set("Spawn" + ".World", player.getLocation().getWorld().getName());

To retrieve the world from the config:

Code:java

String worldName = getConfig().getString("Spawn" + ".World");
World world = getServer().getWorld(worldName);

Now that we've cleared up these issues, the actual problem with your command setup is that you are checking for your second command within the conditional block for the first one. After the "player.sendMessage(ChatColor.GREEN + "Spawn set!");" line, you are 3 conditionals deeper into the code. To get back to the main onCommand method body, you need to close all three brackets. If you're using Eclipse or another IDE, you should notice how it likes to tab back to the original indentation.

After making these changes, the code should follow this form:

Code:java

if (cmd.getName().equalsIgnoreCase("settowny")) {
  if (sender instanceof Player) {
    Player player = (Player) sender;
    if (player.hasPermission("townytp.settowny")) {
      // ...
    }
  }
} else if ((cmd.getName().equalsIgnoreCase("towny"))) {
  if (sender instanceof Player) {
    Player player = (Player) sender;
    if (player.hasPermission("townytp.towny")) {
      // ...
    }
  }
}
// ...

 

Because you are likely using 'Towny' as a plugin on your server, the command /towny is most likely already in use.