Plugin is null

I love the amount of documentation on how to use other classes >_<

When i place a block i receive a lovely NPE which points to plugin, being null.

Main

Code:

package net.stuntguy3000;
 
import java.util.logging.Logger;
 
import net.stuntguy3000.event.*;
 
import org.bukkit.Bukkit;
import org.bukkit.plugin.java.JavaPlugin;
 
public class TSPlugin extends JavaPlugin {
   
    public final TSUtil TSUtil = new TSUtil(this);
    public final evt_build evt_build = new evt_build(this);
   
    public void onEnable() {
        Logger log = Bukkit.getLogger();
       
        getConfig().options().copyDefaults(true);
        saveConfig();
       
        this.getServer().getPluginManager().registerEvents(this.evt_build, this);
    } 
   
    public void onDisable() {
       
    }
}

TSUtil

Code:

package net.stuntguy3000;
 
import org.bukkit.entity.Player;
 
public class TSUtil {
    public TSPlugin plugin;
   
    public TSUtil(TSPlugin instance) {
        this.plugin = instance;
    }
   
    public String test(Player p)
    {
        String value = plugin.getServer().getName();
       
        return value;
    }
}

evt_build

Code:

package net.stuntguy3000.event;
 
import net.stuntguy3000.TSPlugin;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.event.hanging.HangingBreakByEntityEvent;
import org.bukkit.event.hanging.HangingPlaceEvent;
 
public class evt_build implements Listener {
    public TSPlugin plugin;
   
    public evt_build(TSPlugin instance) {
        this.plugin = instance;
    }
   
    public net.stuntguy3000.TSUtil TSUtil = new net.stuntguy3000.TSUtil(plugin);
   
    @EventHandler
    public void BlockPlaceEvent(BlockPlaceEvent event)
    {
        Player p = event.getPlayer();
       
        if (!p.hasPermission("theship.editblocks"))
        {
            event.setCancelled(true);
        }
       
        TSUtil.test(p);
    }
}

 

Make a variable for the classes, and initialize them in the onEnable

 

Could you provide a code example? Thanks

 

Code:

public final TSUtil tsutil;
public final evt_build evt_build;

and in onEnable do

Code:

tsutil = new TSUtil(this);
evt_build = new evt_build(this);

 

Strange errors,

"tsutil = new TSUtil(this);
evt_build = new evt_build(this);"

Eclipse wants to remove the "final" modifiers, and the classname has a error under it about tsutil not being defined. I removed the final modifiers and the error still exists.

 

Try rename the variable, since its the same name as the class it maybe be causing the error

 

That's probably the reason ^^

also don't use underscores it doesn't look good im vert petty about things like that

 

public TSUtil tsutilTest;
public evt_build evt_buildTest;

public void onEnable() {
Logger log = Bukkit.getLogger();

getConfig().options().copyDefaults(true);
saveConfig();

tsutilTest = new TSUtil(this);
evt_buildTest = new evt_build(this);

this.getServer().getPluginManager().registerEvents(this.evt_buildTest, this);
}

No fix :/

TnT md_5 Looking for help

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

 

.......... for one u bumped after 2 minutes -_-. secondly... you don't ask them for plugin help questions. read bukkit staff profile pages and rules before you go tagging staff

 

Jeez i was just asking help from the 'professional's -.-

...Since this thread is going nowhere...

 

We don't have be be professionals to help. In evt_build, you create a new TsUtil class with the plugin value, which is null at the time. When you do TSUtil, you'd want to use plugin.tsutil, instead of the version you created.

 

I don't get what you mean ...?

 

1) TnT isn't even a plugin developer
2) No I won't help you, since tagging me does not entitle you to priority support.

 

I had this issue before - I fixed it by creating a new class and COPYING + PASTING into it.

 

stuntguy3000 to give you a head start here is a modified version of your code

Since this seems to be a test-plugin transporting the plugin instance everytime seems ok to me but be aware that you should only pass what its needed to a class to avoid overhead.

E.g. when you want to use the Server instance then only pass getServer() to the other class instead of the whole plugin instance. Or if you only need the name just transport the name as a String like new MyClass(this.getServer().getName()) and the constructor would be public MyClass(String serverName);

Code hidden in spoiler tags

Show Spoiler Hide Spoiler

Main

Code:

package net.stuntguy3000;
 
import java.util.logging.Logger;
 
import net.stuntguy3000.event.*;
 
import org.bukkit.Bukkit;
import org.bukkit.plugin.java.JavaPlugin;
 
public class TSPlugin extends JavaPlugin {   
    public Logger log;
 
    public void onEnable() {
        this.log = Bukkit.getLogger();
     
        getConfig().options().copyDefaults(true);
        saveConfig();
 
        this.getServer().getPluginManager().registerEvents(new EVT_Build(this), this);
    }
 
    public void onDisable() {
     
    }
}

EVT_Build

Code:

package net.stuntguy3000.event;
 
import net.stuntguy3000.TSPlugin;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.event.hanging.HangingBreakByEntityEvent;
import org.bukkit.event.hanging.HangingPlaceEvent;
 
public class EVT_Build implements Listener {
    protected TSPlugin plugin;
    protected TSUtil tsUtil;
 
    public EVT_Build(TSPlugin instance) {
        this.plugin = instance;
        this.tsUtil = new TSUtil(this.plugin);
    }
   
    @EventHandler()
    public void BlockPlaceEvent(BlockPlaceEvent event)
    {
        Player p = event.getPlayer();
     
        if (!p.hasPermission("theship.editblocks")) {
            event.setCancelled(true);
        }
     
        player.sendMessage("Test returned: " + tsUtil.test(p));
    }
}

TSUtil

Code:

package net.stuntguy3000;
 
import org.bukkit.entity.Player;
 
public class TSUtil {
    protected TSPlugin plugin;
 
    public TSUtil(TSPlugin instance) {
        this.plugin = instance;
    }
 
    public String test(Player p) {
        String value = plugin.getServer().getName();
       
        return value;
    }
}

 

Its not actually going to be a test plugin though...what is another method.

I want to know, how this fixes it...

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

 

stuntguy3000

I don't know! But it worked !

 

Code:java

Code:java

 
[FONT=arial]package net.stuntguy3000.event;[/FONT]
 
[FONT=arial]import net.stuntguy3000.TSPlugin;[/FONT]
[FONT=arial]import net.stuntguy3000.TSUtil;[/FONT]
[FONT=arial]import org.bukkit.entity.Player;[/FONT]
[FONT=arial]import org.bukkit.event.EventHandler;[/FONT]
[FONT=arial]import org.bukkit.event.Listener;[/FONT]
[FONT=arial]import org.bukkit.event.block.BlockBreakEvent;[/FONT]
[FONT=arial]import org.bukkit.event.block.BlockPlaceEvent;[/FONT]
[FONT=arial]import org.bukkit.event.hanging.HangingBreakByEntityEvent;[/FONT]
[FONT=arial]import org.bukkit.event.hanging.HangingPlaceEvent;[/FONT]
 
[FONT=arial]public class evt_build implements Listener {[/FONT]
[FONT=arial] public TSPlugin plugin;[/FONT]
 
[FONT=arial] public evt_build(TSPlugin instance) {[/FONT]
[FONT=arial] this.plugin = instance;[/FONT]
[FONT=arial] }[/FONT]
 
 
[FONT=arial] @EventHandler[/FONT]
[FONT=arial] public void BlockPlaceEvent(BlockPlaceEvent event)[/FONT]
[FONT=arial] {[/FONT]
[FONT=arial] Player p = event.getPlayer();[/FONT]
 
[FONT=arial] if (!p.hasPermission("theship.editblocks"))[/FONT]
[FONT=arial] {[/FONT]
[FONT=arial] event.setCancelled(true);[/FONT]
[FONT=arial] }[/FONT]
 
[FONT=arial] TSUtil.test(p);[/FONT]
[FONT=arial] }[/FONT]
[FONT=arial]}[/FONT]

There that SHOULD work WTF with the [font.] thing?

 

Bukkit documentation covers its own stuff, it doesn't have to teach you Java, the Java documentation does that.

And are you dragging and dropping the text ? Or if you're not, then use [code] instead, [syntax] is always screwing with stuff.