Solved Custom Jail Plugin Problems

So Im making a jail plugin for my server and im not a big fan of the essentials jail. So I thought why not make my own! Im new to java and for some reason ive tried everything to fix the errors with teleporting and i just cant do it.


Here is the code:

Code:java

                    if (cmd.getName().equalsIgnoreCase("jail")) {
                        if (getConfig().getConfigurationSection("jails") == null) {
                                p.sendMessage(ChatColor.GRAY + "[" + ChatColor.AQUA + "Jails" + ChatColor.GRAY + "]" + ChatColor.RED + "No Jail Has Been Set!");
                                return true;
 
                    if(args.length == 0){
                        p.sendMessage(ChatColor.GRAY + "[" + ChatColor.AQUA + "Jails" + ChatColor.GRAY + "]" + ChatColor.RED + "Please Specify a Player.");
                        return true;
 
                    if (args.length == 1) {
                  double world = getConfig().getDouble("jail.world");
                  double x = getConfig().getDouble("jail.x");
                  double y = getConfig().getDouble("jail.y");
                  double z = getConfig().getDouble("jail.z");
                  p.teleport(new Location(world ,x ,y ,z));
                  p.sendMessage(ChatColor.RED + "You Have Been Jailed For Being Naughty!");

 

look at line 11 .. the world is no double .. it's an World Object ..

 

Like I said im pretty new at Java but what would i put instead of a double because if i dont use double it wont work

after i make everything an object it works but then i still get an error because it says the the whole code is unreachable?

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

 

jaybirdpontiac69

Well is a world a number with a decimal point?

 

no.

 

jaybirdpontiac69

Exactly:

int = basically a number that is not too big or too small that doesn't have a decimal points
double = a number with a decimal point
long = A huge number
byte = Not that big, around -16 to 16
short = A bit more than byte

They are called Primitive Data Types. Well, is a world a number or anything? No, a world is a world. So replace double world to World world

 

jaybirdpontiac69
Do World world = Bukkit.getWorld(stringFromConfig);

(You may want to check if it is null).

 

Thanks! But i have a problem where after i put return true i want the command to end. But when i do, everything under it is now unreachable.

 

jaybirdpontiac69

return true stops the code, you need to do it before you close the boolean

 

Thanks!

Yes thats to teleport yourself though. I want to be able to do "/jail (player)" and have that player get moved to a spot where they cant /spawn out. Not just "/jail (player)" and teleport yourself.

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

 

jaybirdpontiac69

Is your problem solved?

 

not exactly. With the /jail command i cant jail other players just myself even though i put their name in and ive googled it already. None of the solutions work.

 

jaybirdpontiac69 that's because you need to fetch the player via the argument

 

jaybirdpontiac69 You can the getPlayer method of Bukkit:
(Assuming p is the sender)

Code:

String targetPlayerName = args[0];
Player targetPlayer = Bukkit.getPlayer(targetPlayerName);
if (targetPlayer != null) {
    targetPlayer.teleport(...)
}
// p.teleport teleports you

You should check if args.length is equal to 1. e.g: if (args.length == 1) { ... rest of your code .. }
That should solve your problem.

 

Thank you very much for that! That bassically sums up my problems with this! Now bukkit has a new fun jail plugin!

 

jaybirdpontiac69 Congrats'!