Solved BlockBreakEvent oak/spruce/jungle logs

It cant seem to tell when an oak/spruce/jungle log is broken, any ideas why not?
There are no console errors

Code:java

package com.kassestral.plugins.ilevel.skills;
 
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
 
import com.kassestral.plugins.ilevel.Main;
import com.kassestral.plugins.ilevel.Processor;
 
public class Woodcutting implements Listener{
 
    private static Main plugin;
    public Woodcutting(Main plugin){
        Woodcutting.plugin = plugin;
    }
 
    @SuppressWarnings("deprecation")
    @EventHandler
    public void onChopTree(BlockBreakEvent e){
 
 
        Block log = e.getBlock();
        Player p = e.getPlayer();
        String level = "Woodcutting";
 
        //Oak Log
        if(log.equals(Material.LOG) && log.getData() == (byte)0){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)4){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)8){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
 
        //Spruce Log
        if(log.equals(Material.LOG) && log.getData() == (byte)1){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)5){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)9){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
 
        //Jungle Log
        if(log.equals(Material.LOG) && log.getData() == (byte)3){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)6){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)10){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
    }
}

 

Kassestral
Did you register your events?

 

Kassestral eew static instance . Have you debugged the block?

 

System.out.println("Put in some debug code");

 

Gater12 fireblast709 AdamQpzm
I debugged it by sending players the String "woodcutting' just so I could tell if it was picking up the block being broke, it registers the blockbreakevent but not if the block is a wood log block (oak/spruce/jungle)

 

Kassestral second look at the code, you are comparing a Block with Material xD. Compare log.getType() == Material.LOG

 

fireblast709

I tried to change it to a material originally

Code:java

Block log = e.getBlock();

to

Code:java

Material log = e.getBlock().getType();

but it comes up with this error on this line

Code:java

log.getData() == (byte)0);

the error is

Code:java

Incompatible operand types Class<capture#1-of ? extends MaterialData> and byte

 

Kassestral
Can you post the whole code if you change it to Material?

 

Code:java

package com.kassestral.plugins.ilevel.skills;
 
import org.bukkit.Material;
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.BlockBreakEvent;
 
import com.kassestral.plugins.ilevel.Main;
import com.kassestral.plugins.ilevel.Processor;
 
public class Woodcutting implements Listener{
 
    private static Main plugin;
    public Woodcutting(Main plugin){
        Woodcutting.plugin = plugin;
    }
 
    @SuppressWarnings("deprecation")
    @EventHandler
    public void onChopTree(BlockBreakEvent e){
 
 
        Material log = e.getBlock().getType();
        Player p = e.getPlayer();
        String level = "Woodcutting";
 
        //Oak Log
        if(log.equals(Material.LOG) && log.getData() == (byte)0){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)4){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)8){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.oak"));
            p.sendMessage(level);
        }
 
        //Spruce Log
        if(log.equals(Material.LOG) && log.getData() == (byte)1){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)5){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)9){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.spruce"));
        }
 
        //Jungle Log
        if(log.equals(Material.LOG) && log.getData() == (byte)3){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)6){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
        if(log.equals(Material.LOG) && log.getData() == (byte)10){
            Processor.addXP(p, level, plugin.getConfig().getInt("Woodcutting.jungle"));
        }
    }
}

zombiekiller753

 

Kassestral
So, what you need is

Block log = e.getBlock()

then, do

if (log.getType().equals(Material.LOG))