Bukkit openInventory() help?

Hello, if you could help me, I would immensely appreciate it. For some reason, the following code will not work as it is supposed to. When a player left clicks the air, it should open an inventory, which it fails to do so. Here is the code:

@EventHandler
public void onVChestOpen(PlayerInteractEvent event) {
Player player = event.getPlayer();
if ((event.getAction() == Action.LEFT_CLICK_AIR)) {
Inventory quickwarps = Bukkit.createInventory(player, 54, "Quick Warps");
player.openInventory(quickwarps);

There are no errors in the console, no incorrect lines in Eclipse, no in-game errors, just nothing happens. I am using CraftBukkit 1.5.2 R1. Thanks a bunch!

 

I don't think you are registering your events?

 

In English lol? I'm not a super nerd at Java.

 

Connor2weirdness In your onEnable() put

Code:java

getServer().getPluginManager().registerEvents(new <Class>(), this);

 

Yeah I do have all that it's just that what's the point pasting the whole thing in when I know where the error is? But oh well, here's the whole thing: http://pastebin.com/RrQfRL2S

 

You don't have that in your onEnable... you have to put it into it :|

 

Ok I just tried it, I got a red line at <Class> Thanks everyone for the help so far!

 

Replace <Class> with the class your listener is in

 

I then get a line under registerEvents

bump please help

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

 

oh sorry I'll be more specific...
getServer().getPluginManager().registerEvents(new <CLASS NAME (No .java)>(this), this);

 

Ok now I've got rid of the red line error, but it still does not work. Here is the new code: http://pastebin.com/DCHDEsPT

 

getServer().getPluginManager().registerEvents(new INVTELE(this), this);

 

Any ideas why?

 

You didn't put "this" inside "new INVTELE(), this);"
should be: new INVTELE(this), this);

 

When I do that, I get an error, it asks me to create a constructor which when I do, makes no difference apart from removing the line.

 

Wait... if your listener is in your main class you do it like this:
getServer().getPluginManager().registerEvents(this, this);

 

Yep the class name is INVTELE which I had already done: getServer().getPluginManager().registerEvents(new INVTELE(), this);

 

Please show your INVTELE.class

 

Still nothing, and what is a listener o.0

 

If INVTELE.class has your Listener & onEnable() ...
you have to register it like this:
getServer().getPluginManager().registerEvents(this, this);


... A listener is the thing with
@EventHandler
that listens for an event...

 

http://pastebin.com/2Xvg4KfL

As said above, no difference.

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

 

And is it an external Listener you have to register it like this:
getServer().getPluginManager().registerEvents(new <Listenerclass>(if needed something as parameter),this);

 

That should work

 

Does not work, used the following: getServer().getPluginManager().registerEvents(new INVTELE,this);
Thanks guys o.0

 

This won't work because java wants to have the brackets.
getServer().getPluginManager().registerEvents(new INVTELE(),this);
Will work.

 

(A note to lobnews: These are brackets: {}. These are parentheses: (). And, you also forgot to add the 'this' in between your two parentheses.)

Okay, so, let's clear this up:

If the event that you've created a Listener for (supposedly with no knowledge of any Bukkit slang for such methods and classes) is in the same class that extends JavaPlugin and has your onEnable(), it would be this in your onEnable():

Code:

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

If it's NOT in that main class, and it's an external class named, say, "INVTELEListener", then it'd look like this in your onEnable().

Code:

getServer().getPluginManager().registerEvents(new INVTELEListener(this),this);

 

Ok so I entered getServer().getPluginManager().registerEvents(this,this); in onEnable(); and the plugin still doesn't work. No errors or anything. Thanks.

 

Sigh, post the code and any stack traces you might have.

 

http://pastebin.com/6rZPfLAD. Thank you.

 

Connor2weirdness Registering your events isn't "super nerd talk"; it's basic Java.