ClickListener

Before I start this, I wanna say that up in the class, I have all of the other valid things, and they all work. It is just this small part that doesn't work in this entire class.

Ok, when I click this, nothing happens, though stuff should happen. In the fc (player config), the thing is saving, its just this part isn't working.

Code:java

Wool red = new Wool(DyeColor.RED);
            ItemStack enable = red.toItemStack(1);
            ItemMeta sunim = enable.getItemMeta();
            sunim.setDisplayName(ChatColor.RED + "" + ChatColor.BOLD + "Click to Enable");
            enable.setItemMeta(sunim);
 
            Wool green = new Wool(DyeColor.GREEN);
            ItemStack disable = green.toItemStack(1);
            ItemMeta greenim = disable.getItemMeta();
            greenim.setDisplayName(ChatColor.GREEN + "" + ChatColor.BOLD + "Click to Disable");
            disable.setItemMeta(greenim);
 
if(fc.getBoolean("gadgets.trampoline.enabled") == true){
                levelinven.setItem(13, disable);
            }
            if(fc.getBoolean("gadgets.trampoline.enabled") == false){
                levelinven.setItem(13, enable);
            }
 
//There is some other stuff here, not important, its all working.
 
if(item.getItemMeta().getDisplayName().equals(ChatColor.GREEN + "" + ChatColor.BOLD + "Click to Disable")) {
            Wool red = new Wool(DyeColor.RED);
            ItemStack enable = red.toItemStack(1);
            ItemMeta enableim = enable.getItemMeta();
            enableim.setDisplayName(ChatColor.RED + "" + ChatColor.BOLD + "Click to Enable");
            enable.setItemMeta(enableim);
 
            fc.set("gadgets.trampoline.enabled", false);
            try {
                fc.save(playerFile);
            } catch (IOException e1) {
                e1.printStackTrace();
            }
 
            levelinven.setItem(13, enable);
 
        }if(item.getItemMeta().getDisplayName().equals(ChatColor.RED + "" + ChatColor.BOLD + "Click to Enable")) {
            Wool green = new Wool(DyeColor.GREEN);
            ItemStack disable = green.toItemStack(1);
            ItemMeta greenim = disable.getItemMeta();
            greenim.setDisplayName(ChatColor.GREEN + "" + ChatColor.BOLD + "Click to Disable");
            disable.setItemMeta(greenim);
 
            fc.set("gadgets.trampoline.enabled", true);
            try {
                fc.save(playerFile);
            } catch (IOException e1) {
                e1.printStackTrace();
            }
 
            levelinven.setItem(13, disable);
 
        }

 

Have you tried debug messages yet?

 

Yep, every other line I put a player.sendMessage("hi"), didn't even get to the first line when I clicked the item.

 

Do you register events? Did you forget the annotation? Is it the correct event?

 

Barinade Like I said, no errors, no internal errors, nothing. I did register the events, and other stuff IN THE CLASS is working. It is litterly only this part of the class.

 

It doesn't work that way. Something you did or didn't do when you should have is the reason this is happening.

A missing annotation wouldn't give an error, wrong event wouldn't give an error, and whether or not the events are registered wouldn't give an error. These are things you just need to check.

If you have an event that says something every time, no conditions, like so:
@EventHandler
public void interact(PlayerInteractEvent e) {
System.out.println("Somebody did something!");
}

and it's not writing to the console then you probably didn't register your events (assuming the event and annotation are correct)

How about you give more code. Your entire listener, event, and onEnable should be enough.

 

As Barinade about debugging you could also print variables to check for false where true should be ect.
Also a Tip if you have a boolean and you want to use it in a if, you don't have to do boolean==true you can simply put it as:
If(boolean){

}
This if will work only of boolean is true, to do the same for false use the ! Operator from of the boolean.

 

You don't even need to check them to print
System.out.println(boolean)

 

And let me just repeat this, OTHER STUFF IN THE CLASS WORKS.
Error is at line 202: http://pastebin.com/m1BfJCLK
http://pastebin.com/3KrBMxrZ

 

You should be using else if if you're checking the same variable

Add an else statement to that if block and print to console what the display name is.
Perhaps you have the bold & green backwards?

 

Barinade Its almost like the green wool doesn't exist. Even when I click it, with the else tag, nothing happens.

 

Add a debug message just before and just after the if statement, see if it's even reaching it

 

Barinade Its printing the debug messages...this is so weird.

 

You really need to start being more specific. Which debug message?

 

Barinade The one before and after. Its reading those, its just not getting to the part.

Hello...

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

 

My guess is you're thinking the code is supposed to do one thing where in reality it does something completely different.

 

Barinade yes the tip was general not about the debug

 

Chrisdamonster

Also this:

Code:java

fc.getBoolean("gadgets.trampoline.bought.level5") == false || !fc.isSet("gadgets.trampoline.bought.level5")

This format is pointless. If anything isSet should be run first, however since getBoolean returns false if nothing is set you don't really need isSet in there..

 

if(fc.getBoolean("gadgets.trampoline.enabled"))

is sufficient enough as well with using an else statement.