Solved Need Help With Signs!!!

So im trying to make a plugin where when you create a sign with the first line "Ws" it tells you that you created it just to make sure its working. whenever i try it nothing happens, but the plugin still loads. Heres the code.

Code:java

package me.toaster;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.SignChangeEvent;
import org.bukkit.plugin.PluginManager;
 
public class main implements Listener{
 
 
PluginManager pm = Bukkit.getServer().getPluginManager();
 
 
public static Bukkit plugin;
 
public void BukkitListener(Bukkit instance) {
plugin = instance;
}
@EventHandler
public void onSignCreate(SignChangeEvent sign){
Player player = sign.getPlayer();
if(sign.getLine(0).equalsIgnoreCase("Ws")){
player.sendMessage("Sign Worked");
sign.setLine(0, "WaterShow");
}
}
}
 

 

Mr_toaster111
You need to register the events. Create a "onEnable" method and add the following:

Code:java

pm.registerEvents(this, this);

 

I added it still nothing
:/

 

Hello Mr. Toaster,
It seems to me that you are not doing anything when the plugin is enabled, or disabled. In order to register events, the events need to be registered onEnable. Here is how I believe you should be writing your code!

Code:java

/*
** I would recommend placing this plugin in a directory lower than what you have here.
** For example: package me.toaster.pluginname;
*/
package me.toaster;
 
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.SignChangeEvent;
 
public class main implements Listener{
 
    @Override
    public void onEnable() {
        getServer().getPluginManager().registerEvents(this,this);
    }
 
    @EventHandler
    public void onSignChange(SignChangeEvent event){
        Player player = event.getPlayer();
        if(event.getLine(0).equalsIgnoreCase("Ws")){
            player.sendMessage("Sign Worked");
            event.setLine(0, "WaterShow");
        }
    }
}

 

Thanks for the Reply!, unfortunately i tried creating an onEnable() with the registering in it like you did but again when i tested it, put Ws on the first line it did nothing i dont understand why its not working!

Well i followed a simple Events Video and it worked thx to all contributers!

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

 

He forgot to add the "extends JavaPlugin" to the "public class main" - line.

@Mr_toaster111 Please close this thread as solved. Ty