Solved Stopping Block From Dropping Item

So here's what I'm trying to do. When there's an explosion, I want to check if a block blown up is snow or a snow block, and then if not, set it back to what it was. This code below works for just that, but I noticed that if the block isn't snow/snowblock, it still drops the item as if it were an explosion. Also, it doesn't reset signs. How do I fix those two problems?

Code:java

@EventHandler
public void onEntityExplode(EntityExplodeEvent event) {
    if (event.getEntityType() == EntityType.PRIMED_TNT) {
        List<Block> blist = event.blockList();
        for (final Block block : blist) {
            final Material type = block.getType();
            if (type != Material.SNOW_BLOCK && type != Material.SNOW) {
                Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                    @Override
                    public void run() {
                        block.setType(type);
                    }
                }, 1);
            } else {
                Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                    @Override
                    public void run() {
                        block.setType(type);
                    }
                }, 2400L);
            }
        }
    }
}

 

Rather than setting the block type back to normal, try:

Code:

  if (..some check to see if we want to deny the block exploding...) {
    event.blockList().remove(block);
  }

Hmm, just noticed you're scheduling the replacements - not sure how to work this into it. Perhaps in the main thread, remove the block as per above, then set it to air (to emulate the explosion) and let the scheduled task set it back to the original type?

 

That didn't work, I already tried that. It still blew the block up.

 

Hmm, looking at your code, when you use the blockList.remove() does it not give you a ConcurrentModificationException?

I just doubled checked my code (here) and I take a copy of the list first to iterate over (as you can't remove items from the list you are currently iterating over).

Eg:

Code:

  List<Block> blockListCopy = new ArrayList<Block>();
  blockListCopy.addAll(event.blockList());
 
  for(Block block : blockListCopy) {
    if (drop.isDenied()) event.blockList().remove(block);
  }

 

It does =3
But I fixed the huge problem thing awhile ago

Code:java

@EventHandler
public void onEntityExplode(EntityExplodeEvent event) {
    Entity entity = event.getEntity();
    if (event.getEntityType() == EntityType.PRIMED_TNT) {
        List<Block> blist = event.blockList();
        Location loc = entity.getLocation();
        event.setCancelled(true);
        loc.getWorld().createExplosion(loc, 0);
        for (final Block block : blist) {
            final Material type = block.getType();
            if (block.getType() == Material.SNOW_BLOCK || block.getType() == Material.SNOW) {
                block.setType(Material.AIR);
                Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(plugin, new Runnable() {
                    @Override
                    public void run() {
                        block.setType(type);
                    }
                }, 2400L);
            }
        }
    }
}