Solved What the heck is wrong with this?

Hey Guys,
Quick question.

I have some code I have been testing, and it is not up to snuff.
Here are the symptoms:
- No errors are made.
- When I fire the events, it only shows me, not all players. (e.g. Block Break)

The idea is that if someone breaks a block, or something else, it will notify everyone who has toggle debug on.
What am I doing wrong?
Please help.
-Blue

Code:java

package core.bluemustache.modanalyzer;
 
import java.util.HashMap;
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.block.Block;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Entity;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.block.BlockBreakEvent;
import org.bukkit.event.block.BlockEvent;
import org.bukkit.event.block.BlockFadeEvent;
import org.bukkit.event.block.BlockFormEvent;
import org.bukkit.event.block.BlockPlaceEvent;
import org.bukkit.event.entity.EntityDeathEvent;
import org.bukkit.event.player.PlayerInteractEntityEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.PluginManager;
import org.bukkit.plugin.java.JavaPlugin;
 
public class PluginCore extends JavaPlugin
  implements Listener
{
  public static HashMap<Player, String> active = new HashMap<Player, String>();
  public static int tid = 0;
  public static int running = 0;
  int interval = 30;
 
  @SuppressWarnings("deprecation")
public void onEnable()
  {
    getLogger().info("is now enabled");
    PluginManager pm = getServer().getPluginManager();
    pm.registerEvents(this, this);
    tid = Bukkit.getServer().getScheduler().scheduleAsyncRepeatingTask(this, new Runnable()
    {
      public void run()
      {
        System.runFinalization();
        System.gc();
      }
    }
    , 0L, this.interval * 20);
  }
 
  public void onDisable()
  {
    getLogger().info("is now disabled");
  }
 
  @EventHandler
  public void onBreak(BlockBreakEvent e) {
    Block blk = e.getBlock();
    Player pl = e.getPlayer();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("BROKEN: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("PLAYER: " + pl.getName() + " + " + pl.getDisplayName());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onDerp(BlockEvent e) {
    Block blk = e.getBlock();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("BLOCK: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onPlace(BlockPlaceEvent e) {
    Block blk = e.getBlock();
    Player pl = e.getPlayer();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("PLACED: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("PLAYER: " + pl.getName() + " + " + pl.getDisplayName());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onForm(BlockFormEvent e)
  {
    Block blk = e.getBlock();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("FORM: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onFade(BlockFadeEvent e) {
    Block blk = e.getBlock();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("FADE: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onDeath(EntityDeathEvent e) {
    Entity ent = e.getEntity();
    Location loc = ent.getLocation();
    EntityType typ = ent.getType();
    for (Player p : Bukkit.getOnlinePlayers())
      if (active.containsKey(p)) {
        p.sendMessage("DEATH: " + ent.getType() + " + " + typ.getTypeId() + " + " + typ.getName());
        p.sendMessage("LOCATION: " + loc.getX() + " " + loc.getY() + " " + loc.getZ());
        p.sendMessage("");
      } else {
        return;
      }
  }
 
  @EventHandler
  public void onClick(PlayerInteractEvent e) {
    Player p = e.getPlayer();
    Block blk = e.getClickedBlock();
    if (active.containsKey(p)) {
      if (e.getAction() == Action.RIGHT_CLICK_BLOCK) {
        p.sendMessage("CLICK: " + blk.getType() + " + " + blk.getTypeId() + " + " + blk.getData());
        p.sendMessage("LOCATION: " + blk.getX() + " " + blk.getY() + " " + blk.getZ());
        p.sendMessage("");
      }
    }
    else;
  }
 
  @EventHandler
  public void onEntityInteract(PlayerInteractEntityEvent e)
  {
    Player p = e.getPlayer();
    Entity ent = e.getRightClicked();
    Location loc = ent.getLocation();
    EntityType typ = ent.getType();
    if (active.containsKey(p)) {
      p.sendMessage("ENTITY: " + ent.getType() + " + " + typ.getTypeId() + " + " + typ.getName());
      p.sendMessage("LOCATION: " + loc.getX() + " " + loc.getY() + " " + loc.getZ());
      p.sendMessage("");
    }
    else;
  }
 
  public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args)
  {
    if (!(sender instanceof Player)) {
      sender.sendMessage("Nope, Chuck Testa! ;)");
    }
    else if (commandLabel.equalsIgnoreCase("toggledebug")) {
      Player player = (Player)sender;
      if (active.containsKey(player)) {
        active.remove(player);
        player.sendMessage("OFF: Debugging disabled!");
        player.sendMessage("");
      } else {
        active.put(player, null);
        player.sendMessage("ON: Debugging activated.");
        player.sendMessage("");
      }
    }
 
    return false;
  }
}

 

Would a bukkit brodcast work?

 

BlueMustache Why are you using a HashMap? use an ArrayList or a HashSet, do not store player objects, store their UUIDS or Names.

 

Gamesareme
I want it to be only announced to those that have it enabled.
DevRosemberg
Because I am using the hashmaps, and they work.
Also, this is for a pre-uuid server. (I hate UUID's, and it is no reason, it wouldn't work, which it does.)

 

BlueMustache
You should use a List or a Set. HashMap should be use if you want to associate a value with an object. A List or a Set well... creates a list or a set of some object.

You should use player names, if not UUIDs.

 

Gater12
Examples?
If I had known to do this, I would have done it.
I just want to print those events to everyone who has toggledebug enabled, no matter who or what triggered the event.

 

BlueMustache

Code:java

/* Way to create an ArrayList It will store String objects */
List<String> foo = new ArrayList<String>();
 
/* For Java 7, you can use diamond operators to infer */
List<String> foo = new ArrayList<>();

The documentation is here: http://docs.oracle.com/javase/7/docs/api/java/util/ArrayList.html

To create a Set is similiar.
Documentation: http://docs.oracle.com/javase/7/docs/api/java/util/HashSet.html

 

Solved!
Thanks.