Lobby Plugin Error NEED HELP

So I had this person make a plugin for me where if I do /lobby I go to specified cords. The problem is that when I do /lobby after a server restart, It says internal error on the game. I got the error code on console and here is the link: http://pastebin.com/FHqW5p2Z The only way to fix this for now it to either do /reload and it will work or that i have to do /setlobby or reload the plugin. Please help me as fast as you can. If you need to talk to me more and want to see the plugin come to my server at mc.universaldiamondzgaming.com Thank you in advance.

 

does it teleport you into an other world?
you get some errors with WE teleport and Multiverse teleport

 

nope i stay in the same place and it says internal error

 

"Caused by: java.lang.NullPointerException"

can you post the code of the plugin?

 

I know what the error is. The person who made you the plugin didn't make the locations save over restart/reload. So everytime you restart/reload it loses the location data that is why you get the error because the location is null.

 

I made the same error in a plugin a few weeks ago, i saved the config befor i loaded it

 

uhhh how can i exactly?

uhh ok i'll tell him that

So guys what is the fix to it?

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

 

That is an impressive stack trace. I needed to say it.

 

Solution anyone

 

we need to see the code

 

what program do i need to get the code?

 

you can ask him for the code

 

awww man ok wait can i give u the link of the download?

Link is given by The_Doctor_123: https://forums.bukkit.org/threads/lobby-plugin.160421/#post-1763299

I will give this link to the plugin producer once he comes on which will be a few hours.

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

 

Ah, yes, I am quite stumped as to why it is not working. Here is the full src(1 Class):

Code:

package com.yahoo.brettbutcher98.LobbyPlugin;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
   
    public Location loc = new Location(null, 0, 0, 0);
   
    @Override
    public void onEnable()
    {
        getConfig().options().copyDefaults(true);
        saveConfig();
        loc.setX(getConfig().getInt("Lobby.x"));
        loc.setY(getConfig().getInt("Lobby.y"));
        loc.setZ(getConfig().getInt("Lobby.z"));
        loc.setPitch((float) getConfig().getDouble("Lobby.Pitch"));
        loc.setYaw((float) getConfig().getDouble("Lobby.Yaw"));
        loc.setWorld(Bukkit.getWorld(getConfig().getString("Lobby.World")));
    }
   
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if (cmd.getName().equalsIgnoreCase("lobby"))
        {
            if (args.length != 0)
            {
                if (args[0].equalsIgnoreCase("reload") && sender.hasPermission("lobby.set"))
                {
                    loc.setX(getConfig().getInt("Lobby.x"));
                    loc.setY(getConfig().getInt("Lobby.y"));
                    loc.setZ(getConfig().getInt("Lobby.z"));
                    loc.setPitch((float) getConfig().getDouble("Lobby.Pitch"));
                    loc.setYaw((float) getConfig().getDouble("Lobby.Yaw"));
                    loc.setWorld(Bukkit.getWorld(getConfig().getString("Lobby.World")));
                    sender.sendMessage(ChatColor.GREEN + "Config Reloaded!");
                }
            }
            else
            {
                sender.sendMessage(colorize(getConfig().getString("LobbyTeleportMessage")));
                Bukkit.getPlayer(sender.getName()).teleport(loc); // Line of error.
            }
            return true;
        }
        else if (cmd.getName().equalsIgnoreCase("setlobby"))
        {
            sender.sendMessage(colorize(getConfig().getString("LobbySetMessage")));
            loc = Bukkit.getPlayer(sender.getName()).getLocation();
            getConfig().set("Lobby.x", loc.getBlockX());
            getConfig().set("Lobby.y", loc.getBlockY());
            getConfig().set("Lobby.z", loc.getBlockZ());
            getConfig().set("Lobby.Yaw", loc.getYaw());
            getConfig().set("Lobby.Pitch", loc.getPitch());
            getConfig().set("Lobby.World", loc.getWorld().getName());
            saveConfig();
            return true;
        }
        return false;
    }
   
    public String colorize(String string)
    {
        string = string.replace("&", "§");
        return string;
        }
}

The issue seems to be that it works for the first like hour or so after a reload/startup and then it just throws NullPointerExceptions pointing to the teleport line.

 

just a complete side point here..
Bukkit.getPlayer(sender.getName()).teleport(loc)
i think i'm a fan of the second way...
sender.teleport(loc)

 

I'm not a fan of errors, though. (The method does not exist.)

 

you are correct i'm way too tired apparently to be helping people on here
if (sender instance of Player){
Player player = (Player)sender;
player.teleport(loc);
} else {
System.out.print("Silly console, programs are not allowed to teleport");
}

 

For the sake of only needing the player object on one line of code, I had done what I did.

 

Code:java

package com.yahoo.brettbutcher98.LobbyPlugin;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
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 Location loc = new Location(null, 0, 0, 0);
 
    @Override
    public void onEnable()
    {
        getConfig().options().copyDefaults(true);
        saveConfig();
        loc.setX(getConfig().getInt("Lobby.x"));
        loc.setY(getConfig().getInt("Lobby.y"));
        loc.setZ(getConfig().getInt("Lobby.z"));
        loc.setPitch((float) getConfig().getDouble("Lobby.Pitch"));
        loc.setYaw((float) getConfig().getDouble("Lobby.Yaw"));
        loc.setWorld(Bukkit.getWorld(getConfig().getString("Lobby.World")));
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args)
    {
        if (cmd.getName().equalsIgnoreCase("lobby"))
        {
            if (!(sender instanceof Player))
            {
                sender.sendMessage("You can't use /lobby on console");
                return true;
            }
            if (args.length != 0)
            {
                if (args[0].equalsIgnoreCase("reload") && sender.hasPermission("lobby.set"))
                {
                    loc.setX(getConfig().getInt("Lobby.x"));
                    loc.setY(getConfig().getInt("Lobby.y"));
                    loc.setZ(getConfig().getInt("Lobby.z"));
                    loc.setPitch((float) getConfig().getDouble("Lobby.Pitch"));
                    loc.setYaw((float) getConfig().getDouble("Lobby.Yaw"));
                    loc.setWorld(Bukkit.getWorld(getConfig().getString("Lobby.World")));
                    sender.sendMessage(ChatColor.GREEN + "Config Reloaded!");
                }
            }
            else
            {
                sender.sendMessage(colorize(getConfig().getString("LobbyTeleportMessage")));
                Player player = (Player)sender;
                player.teleport(loc);
            }
            return true;
        }
        else if (cmd.getName().equalsIgnoreCase("setlobby"))
        {
            if (!(sender instanceof Player))
            {
                sender.sendMessage("You can't use /setlobby on console");
                return true;
            }
            sender.sendMessage(colorize(getConfig().getString("LobbySetMessage")));
            Player player = (Player)sender;
            loc = player.getLocation();
            getConfig().set("Lobby.x", loc.getBlockX());
            getConfig().set("Lobby.y", loc.getBlockY());
            getConfig().set("Lobby.z", loc.getBlockZ());
            getConfig().set("Lobby.Yaw", loc.getYaw());
            getConfig().set("Lobby.Pitch", loc.getPitch());
            getConfig().set("Lobby.World", loc.getWorld().getName());
            saveConfig();
            return true;
        }
        return false;
    }
 
    public String colorize(String string)
    {
        string = string.replace("&", "§");
        return string;
    }
}

fixed it

 

How is that fixed? Looks like you just caught the console from executing the commands and slapped on this:

Code:

Player player = (Player)sender
player.teleport(loc);

Instead of this:

Code:

Bukkit.getPlayer(sender.getName()).teleport(loc);

They both provide the same functionality.

 

no i tested it
the error has been fixed if used

Code:

Player player = (player)sender;
player.teleport(loc);

 

That would be impossible, they should do EXACTLY the same thing. Also, you see, the error takes time to kick in. It takes about an hour from reload/startup to receive it.

 

Look, if you trying to get a player from sender's name, it will return null.
You can cast the sender object to player and it will work.
So, use that line instead your line:
((Player) sender).teleport(loc)

 

Is it me or does it look like your plugin caused an error with every plugin on the server listening to a teleport event?

 

lol yes it did

 

How? It works every time I use "Bukkit.getPlayer(sender.getName())." I even tested the plugin on my server before uploading, and it worked fine. For the third time, I'm going to say this: It takes time for the errors to start happening after a reload/startup.

I saw at least 2 plugins it conflicted with in the error, and I don't think debs copied the whole thing.

 

Ok great news guys! Lambo re did the code and got it fixed! The problem was the message not saving wen doing /lobby. So he just put in the message I wanted. Everythings good now. Thanks for all your help!

 

Nvm still not solved. Turns out lambo has a hack code in it which he tried to hack my server. The issue still continues.

 

...what? o,o

 

yup