File cannot be null - It isn't...

Hello! So my plugin keeps returning null everytime I right click a sign.
Basically I:
1) Create a sign
2) Syntax:
# [HP]
# [ Teleport name ]
3) Set the coords of the teleport name by doing: /HP set [name] [world] [x] [y] [z]
4) When they right click the sing they get teleported to the coordinates in the config.

But, it always returns null :/ By the way. If anyone could tell me how to access a config from a class that does not extend JavaPlugin that would be awesome!

Code (Main):

Code:java

package me.CraftCreeper6.main;
 
import me.CraftCreeper6.teleport.Teleportation;
 
import org.bukkit.ChatColor;
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 Teleportation tp = new Teleportation(this);
 
    public void onEnable(){
 
        System.out.print("[HubblePort] Enabled!");
 
        getServer().getPluginManager().registerEvents(tp, this);
 
    }
 
    public void onDisable(){
 
        System.out.print("[HubblePort] Disabled!");
 
    }
 
    public boolean onCommand(CommandSender sender, Command cmd,
            String commandLabel, String[] args) {
        Player p = (Player) sender;
 
        if(commandLabel.equalsIgnoreCase("HP") && args[0].equalsIgnoreCase("Set") && args[1] !=null && args[2] !=null && args[3] !=null && args[4] !=null && args[5] !=null){
 
            getConfig().set(args[1] + ".World", args[2]);
            getConfig().set(args[1] + ".X", args[3]);
            getConfig().set(args[1] + ".Y", args[4]);
            getConfig().set(args[1] + ".Z", args[5]);
 
            saveConfig();
 
            p.sendMessage(ChatColor.BLUE + "[" + ChatColor.RED + "HP" + ChatColor.BLUE + "]" + ChatColor.AQUA + " Succesfully set location for: " + args[1] + " to - World: " + args[2] + " X: " + args[3] + " Y: " + args[4] + " Z: " + args[5]);
 
        }
 
        return true;
    }
}

Code (Teleportation):

Code:java

package me.CraftCreeper6.teleport;
 
import me.CraftCreeper6.main.Main;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.block.Block;
import org.bukkit.block.Sign;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
import org.bukkit.util.Vector;
 
public class Teleportation extends JavaPlugin implements Listener {
 
    public static Main plugin;
 
    public Teleportation(Main instance) {
        plugin = instance;
    }
 
    public void tP(Player p, World world, int x, int y, int z){
 
        Location loc = new Location(world, x, y, z);
 
        p.teleport(loc);
 
    }
 
    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        final Player p = event.getPlayer();
        Block block = event.getClickedBlock();
 
        if (block != null && block.getType() == Material.SIGN_POST
                || block != null && block.getType() == Material.WALL_SIGN) {
            Sign sign = (Sign) block.getState();
            Action action = event.getAction();
            if (action == Action.RIGHT_CLICK_BLOCK) {
                if (sign.getLine(0).equalsIgnoreCase("[HP]")
                        || sign.getLine(0).equalsIgnoreCase(ChatColor.BLUE + "[" + ChatColor.RED + "HP" + ChatColor.BLUE + "]")) {
 
                        sign.setLine(0, ChatColor.BLUE + "[" + ChatColor.RED + "HP" + ChatColor.BLUE + "]");
                        sign.update();
 
                        if(p.hasPermission("HP.Create")){
 
                            p.sendMessage(ChatColor.BLUE + "[" + ChatColor.RED + "HP" + ChatColor.BLUE + "]" + ChatColor.AQUA + " Remember to set the destination using: /HP set [Name] [World, X , Y, Z]");
 
                        }
 
                        final int x = (int) getConfig().get(sign.getLine(1) + ".X");
                        final int y = (int) getConfig().get(sign.getLine(1) + ".Y");
                        final int z = (int) getConfig().get(sign.getLine(1) + ".Z");
                        final World world = (World) getConfig().get(sign.getLine(1) + ".World");
 
                        p.setVelocity(p.getLocation().getDirection().multiply(-3));
                        p.setVelocity(new Vector(p.getVelocity().getX(), 9.0D, p.getVelocity().getZ()));
 
                        Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
 
                            @Override
                            public void run() {
 
                                tP(p, world, x, y, z);
 
                                p.sendMessage(ChatColor.BLUE + "[" + ChatColor.RED + "HP" + ChatColor.BLUE + "]" + ChatColor.AQUA + " You have been teleported!");       
 
                            }
 
                        }, 40);
 
                } else {
 
                }
            }
        }
    }
}

Any help appreciated!

 

CraftCreeper6
Is there a Stack Trace you can provide?

EDIT:
Regarding the config file from another class:
https://forums.bukkit.org/threads/config-from-another-class.61899/

 

RAFA_GATO_FOFO
Yes, sorry forgot to post the stack trace here:

Code:

[01:09:11 ERROR]: Could not pass event PlayerInteractEvent to HubblePort v1.0
org.bukkit.event.EventException
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:427) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.jav
a:62) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.j
ava:481) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.j
ava:466) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.craftbukkit.v1_7_R1.event.CraftEventFactory.callPlayerInte
ractEvent(CraftEventFactory.java:191) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b29
74jnks]
        at net.minecraft.server.v1_7_R1.PlayerInteractManager.interact(PlayerInt
eractManager.java:374) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.PlayerConnection.a(PlayerConnection.java
:628) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.PacketPlayInBlockPlace.a(SourceFile:60)
[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.PacketPlayInBlockPlace.handle(SourceFile
:9) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.NetworkManager.a(NetworkManager.java:146
) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.ServerConnection.c(SourceFile:134) [craf
tbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:6
55) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:2
50) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:5
45) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java
:457) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:6
17) [craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
Caused by: java.lang.IllegalArgumentException: File cannot be null
        at org.apache.commons.lang.Validate.notNull(Validate.java:203) ~[craftbu
kkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.configuration.file.YamlConfiguration.loadConfiguration(Yam
lConfiguration.java:171) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.plugin.java.JavaPlugin.reloadConfig(JavaPlugin.java:118) ~
[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at org.bukkit.plugin.java.JavaPlugin.getConfig(JavaPlugin.java:112) ~[cr
aftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        at me.CraftCreeper6.teleport.Teleportation.onPlayerInteract(Teleportatio
n.java:58) ~[?:?]
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.7.0
_45]
        at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:1.7.0
_45]
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1
.7.0_45]
        at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.7.0_45]
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.ja
va:425) ~[craftbukkit.jar:git-Bukkit-1.7.2-R0.2-b2974jnks]
        ... 15 more

 

CraftCreeper6
I don't know if that'll fix it but I see that you've extended both classes as JavaPlugin.
Again I don't know if this is the issue (or even an issue at all) but I've learned that only the main class should have it.

 

That is most likely the issue, as having the second JavaPlugin, it will likely be looking for a Teleportation folder and its respective configuration. As for accessing the config outside of that class, the easiest way is to maintain a reference to the JavaPlugin class, i.e. a private field that is set in the constructor:

Code:java

 
public class Teleportation extends Listener {
 
  private Main parent_;
 
  public Teleportation (Main parent) {
    parent_ = parent;
  }
  // etc etc
}

Then you can use:

Code:

parent_.getConfig()

Anywhere in your code and it will work correctly.

P.S. I would strongly advise you to rename the "Main" class to the name of your plugin, it would make more sense.

 

mythbusterma Convention actually discourages starting variables with an underscore.

 

Only extend JavaPlugin in your main class. Other classes are not other plugins

 

1Rogue
mythbusterma
Desided to move it all into my Main class ; I figured out why; Config:

Code:

TnT:
  World: Main
  X: '100'
  Y: '70'
  Z: '100'

As you see the int's are in ' ' for some reason, can anyone help?

 

Sorry I meant to suffix it, fixed.

When setting values via code those quotes will surround the value, and in reading the values they are inconsequential. That isn't your problem.

 

CraftCreeper6
I've answered this question on your other post.

 

mythbusterma Pretty sure it's against convention to end it with an underscore, too. Nothing wrong with just calling it "parent"

 

The problem is with getters and constuctors using the name parent as an argument, and I prefer that notation to the "this.parent = parent" which I feel is less comprehensible. Coding guidelines I use: http://geosoft.no/development/javastyle.html (see rule number 8)

 

You're setting them as strings, not ints.