Bukkit.getOnlinePlayers Repeats

Hi,
I am trying to have a message displayed to everyone who has player visibility disabled every 30 seconds. in my onEnable, I have invisMsg() to start the loop. If there is only one player with player invisibility enabled, then the plugin displays the message once. If there are more than one, the message gets displayed several times. Here is my invisMsg code:

Code:java

public void invisMsg() {
        new BukkitRunnable(){
              @Override
              public void run(){
                for (Player p : Bukkit.getOnlinePlayers()) {
                    if (vanished.contains(p.getName())) {
                        p.sendMessage("§ePlayer visibility is disabled!  Click your Nether Star to see players again!");
                    }
                    invisMsg();
                }
              }
            }.runTaskLater(this, 30 * 20L);
    }

If anyone knows how to get around this, I would greatly appreciate it. Thanks!

 

Don't call invisMsg() recursively.
Make a repeating task instead. (.runTaskTimer()) and delete the "invisMsg()" call from the loop.
Because what you currently do is creating a new timer task for each player currently online,
causing multiple messaging tasks to run simultaneously.