What am I doing wrong? I can't bear the fact that I don't know!

Hey y'all.

So here's the thing:
My bukkit seems to be spamming with lots of errors when I try to perform a command.


The script I made has not been finished yet, what I am trying to make is a warpingsystem. I am just a beginner and I thought I'd be nice to learn scripting by making this plugin. The plugin now only can save warps, but for some reason it won't work.

The error occurs somewhere around line 48.
So here's the SCRIPT:

Code:java

package io.github.liquidian.FastTravel;
 
import java.util.logging.Logger;
 
import org.bukkit.command.Command;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class FastTravel extends JavaPlugin{
 
        public final Logger logger = Logger.getLogger("Minecraft");
        public static FastTravel plugin;
        public final Location[] warpLocations = new Location[100];
        public final String[] warpName = new String[100];
 
        public void loadConfig(){
            getConfig().options().copyDefaults(true);
            saveConfig();
        }
 
        @Override
        public void onEnable() {
            loadConfig();
            PluginDescriptionFile pdfFile = this.getDescription();
            this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " Has Been Enabled!");
        }
 
        @Override
        public void onDisable () {
            PluginDescriptionFile pdfFile = this.getDescription();
            saveConfig();
            this.logger.info(pdfFile.getName() + " Has Been Disabled!");
        }
 
        public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String [] args){
            Player player = (Player) sender;
            if(commandLabel.equalsIgnoreCase("fasttravellocationset") || commandLabel.equalsIgnoreCase("ftlocationset") || commandLabel.equalsIgnoreCase ("ftls"));
                if(args.length == 0){
                    player.sendMessage(ChatColor.RED + "/ftls <warpname>");}
 
                else if(args.length == 1){
                    this.logger.info(ChatColor.GRAY + "..Loading Config");   
                    this.logger.info(ChatColor.GRAY + "..Saving Data");
                    Location loc = player.getLocation();
                    getConfig().set("warps." + args[2] + ".x", Double.valueOf(loc.getX()));
                    getConfig().set("warps." + args[2] + ".y", Double.valueOf(loc.getY()));
                    getConfig().set("warps." + args[2] + ".z", Double.valueOf(loc.getZ()));
                    getConfig().set("warps." + args[2] + ".world", loc.getWorld().getName());
                    getConfig().set("warps." + args[2] + ".yaw", Float.valueOf(loc.getYaw()));
                    getConfig().set("warps." + args[2] + ".pitch", Float.valueOf(loc.getPitch()));
                    player.sendMessage("Public Warp " + args[2] + " created.");
 
                    saveConfig();
                    this.logger.info(ChatColor.GRAY + "..Data Succesfully Saved!");                   
                    return true;}
 
                else if(args.length > 1){
                player.sendMessage(ChatColor.RED + "Too many arguments! Correct use: /ftls <warpname>.");}
        return false;
        }   
}

And here's the error:


Thanks in advance!
- Liquidian

 

I would presonally change few things around.
Youre checking the amount of args, you check if its exactly 1... which is wrong because then in config you get 3rd arg from the command...

Do just args lenght > 1

Also args go the following way

arg[0] is the first one
arg[1] is the second one

And so on, you also need to register your commands if you havent done ao

 

Adriani6 Thanks for the support. Btw, how can I register my commands?

 

I'm guessing in your plugin.yml

 

I have solved it by doing this.

Code:java

                    getConfig().set("warps." + args[0] + ".x", Double.valueOf(loc.getX()));
                    getConfig().set("warps." + args[0] + ".y", Double.valueOf(loc.getY()));
                    getConfig().set("warps." + args[0] + ".z", Double.valueOf(loc.getZ()));
                    getConfig().set("warps." + args[0] + ".world", loc.getWorld().getName());
                    getConfig().set("warps." + args[0] + ".yaw", Float.valueOf(loc.getYaw()));
                    getConfig().set("warps." + args[0] + ".pitch", Float.valueOf(loc.getPitch()));

PS: Sorry for posting twice.

NDUGAR Thankyou!

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

 

liquid1an
commands:
basic:
description: This is a demo command.
usage: /<command> [player]
permission: <plugin name>.basic
permission-message: You don't have <permission>

Sorry for not tags around the code as im on my phone. Also change basic with one of your commands.

Also I seen you check commandlabel I would use cmd.getName().equalsIgnoreString("ftls")

And change basic in plugin.yml to ftls

Or register all 3 commands you check I guess

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

 

@Adriani Thanks, it all worked. I actually already had that plugin.yml thing done, you really really helped me by eplaining the args[0] thing.