Material.getMaterial() problems

I want to change drop from one block to another.
Somewhere in my code:

Code:

Block block = event.getBlock();   //get block
 
System.out.println("DropEvent");
 
System.out.println(block.getTypeId());
System.out.println(block);
block.setType(Material.getMaterial("DIRT"));   //change type to DIRT
System.out.println(block.getTypeId());
System.out.println(block);
 
block.breakNaturally();    //should drop dirt

In testing envirioment the block at the beginning is STONE. After executing I got in console:

BUT when I change block.setType(Material.getMaterial("DIRT")); into e.g block.setType(Material.getMaterial("GLOWSTONE"));

and everything is working ok. Some kind of bug, or I'm doing something wrong? Tested on api 1.4.5.

 

Can you give me an brief explanation od what you are trying to do?

 

I have two blocks, let them be LAMP_ON and LAMP_OFF
In single case (that was only part of the code) if i get break event of LAMP_ON, i want it to drop item like it was LAMP_OFF.
But anyway, why using one ID works, but another doesn't? The same applies when using integer IDs instead of names...

 

In your code does it change dirt to glowstone?

 

My sample code works when I use Material.getMaterial("GLOWSTONE") (i get glowstone dust, whatever the block was), but when I use Material.getMaterial("DIRT") - the block remains unchanged.

I changed first post to make it more clear.

 

Its because when the block breaks you are replacing it with dirt so in actual fact nothing is happenening.

 

So then, why replacing it with dirt doesn't work, but with glowstone does?

 

so you wan't if you break dirt it replaces it with glowstone? instead of replacing the drops with glowstone?

 

well, not - when i got blockbreakevent, i check is it my block. if it is - i'm replacing it with this other block, after that call block.breaknaturally() and set event to cancelled
with glowstone hardcoded - it is working - so i'm breaking stone and getting glowstone dust. but with dirt hardcoded, after breaking stone i'm getting cobblestone, but following previous case i should get dirt as a drop, right?

I know, this may be not the best method to achieve that - but still, it should behave the same way in both cases, and it is actually not.

 

Code:JAVA

 public void onBlockBreak(BlockBreakEvent event){
Block block = event.getBlock(); //get block
if(block.getType() == Material.GLOWSTONE){
System.out.println("DropEvent");
 
System.out.println(block.getTypeId());
System.out.println(block);
block.setType(Material.DIRT); //change type to DIRT
System.out.println(block.getTypeId());
System.out.println(block);
 
block.breakNaturally(new ItemStack(Material.DIRT, 1)); //should drop dirt
}
}
}
 

kitor

 

Yes, this works, but in real code I have two variables: LAMP_ON and LAMP_OFF that holds block IDs:

Code:JAVA

 
public void doNotStole(BlockBreakEvent event) {
if (event.isCancelled()) {
return;
}
Block block = event.getBlock();
if (block.getTypeId() == LAMP_ON
&& block.getData() == LAMP_DATA) {
System.out.println("DropEvent");
block.setType(Material.DIRT);
block.breakNaturally();
event.setCancelled(true);
}
}

in this case it is working, but instead of Material.DIRT it should use block from LAMP_OFF variable. That's why I wanted to use getMaterial() method.

 

Does block.breakNaturally(new ItemStack(Material.DIRT, 1)); work?

 

This gives me no drop at all.

 

what about without the ',1' so its just new ItemStack(Material.DIRT));

 

No, it's not working, but I found quite different, but working sollution:

Code:JAVA

public void doNotStole(BlockBreakEvent event) {
        if (event.isCancelled()) {
            return;
        }
        Block block = event.getBlock();
        if (block.getTypeId() == LAMP_ON
                && block.getData() == LAMP_DATA) {
            System.out.println("DropEvent");
            block.setTypeId(0);
            ItemStack drop = new ItemStack(LAMP_OFF, 1);
            World world = block.getWorld();
            world.dropItemNaturally(block.getLocation(), drop);
            event.setCancelled(true);
        }
    }