CRAFTBOOK-2770 - Reported by Brian.Wood

Getting the following errors in Craftbook following 1.6.4 update.

2013-10-26 22:34:40 [SEVERE] Could not pass event BlockBreakEvent to CraftBook v3.7-SNAPSHOT:3279-6cc4c81,master
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427)
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
at net.minecraft.server.v1_6_R3.PlayerInteractManager.breakBlock(PlayerInteractManager.java:279)
at net.minecraft.server.v1_6_R3.PlayerInteractManager.dig(PlayerInteractManager.java:175)
at net.minecraft.server.v1_6_R3.PlayerConnection.a(PlayerConnection.java:539)
at net.minecraft.server.v1_6_R3.Packet14BlockDig.handle(SourceFile:46)
at net.minecraft.server.v1_6_R3.NetworkManager.b(NetworkManager.java:296)
at net.minecraft.server.v1_6_R3.PlayerConnection.e(PlayerConnection.java:116)
at net.minecraft.server.v1_6_R3.ServerConnection.b(SourceFile:37)
at net.minecraft.server.v1_6_R3.DedicatedServerConnection.b(SourceFile:30)
at net.minecraft.server.v1_6_R3.MinecraftServer.t(MinecraftServer.java:592)
at net.minecraft.server.v1_6_R3.DedicatedServer.t(DedicatedServer.java:227)
at net.minecraft.server.v1_6_R3.MinecraftServer.s(MinecraftServer.java:488)
at net.minecraft.server.v1_6_R3.MinecraftServer.run(MinecraftServer.java:421)
at net.minecraft.server.v1_6_R3.ThreadServerApplication.run(SourceFile:583)
Caused by: java.lang.ClassCastException: org.bukkit.material.MaterialData cannot be cast to org.bukkit.material.Tree
at com.sk89q.craftbook.mech.TreeLopper.searchBlock(TreeLopper.java:91)
at com.sk89q.craftbook.mech.TreeLopper.searchBlock(TreeLopper.java:102)
at com.sk89q.craftbook.mech.TreeLopper.searchBlock(TreeLopper.java:102)
at com.sk89q.craftbook.mech.TreeLopper.searchBlock(TreeLopper.java:102)
at com.sk89q.craftbook.mech.TreeLopper.onBlockBreak(TreeLopper.java:63)
at com.sk89q.craftbook.MechanicManager.dispatchBlockBreak(MechanicManager.java:205)
at com.sk89q.craftbook.bukkit.MechanicListenerAdapter.onBlockBreak(MechanicListenerAdapter.java:201)
at sun.reflect.GeneratedMethodAccessor139.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
... 16 more

Comment by me4502

Do you use tree looper with a non-log block?

Comment by Brian.Wood

This was while cutting trees down, so if I understand your question, the answer is no.

Comment by me4502

I mean in the config, do you have tree Lopper enabled on extra blocks?

Comment by Brian.Wood

I have it set to all the defaults, except I have gold axes exempted so that players can use gold axes if they need to work on log blocks. Here's the relevant section of the config file:

tree-lopper:
    enable: true
    block-list:
    - 17
    tool-list:
    - 271
    - 275
    - 258
    - 279
    max-size: 30
    allow-diagonals: false
    enforce-data: false
    place-saplings: true

Comment by Brian.Wood

Not sure if related, but I also get this on server startup:

2013-10-27 11:02:20 [SEVERE] java.lang.NullPointerException
2013-10-27 11:02:20 [SEVERE] at com.sk89q.craftbook.bukkit.Updater.versionCheck(Updater.java:422)
2013-10-27 11:02:20 [SEVERE] at com.sk89q.craftbook.bukkit.Updater.access$400(Updater.java:45)
2013-10-27 11:02:20 [SEVERE] at com.sk89q.craftbook.bukkit.Updater$UpdateRunnable.run(Updater.java:503)
2013-10-27 11:02:20 [SEVERE] at java.lang.Thread.run(Thread.java:724)

Comment by me4502

That error is just a connection error

Comment by me4502

Is that error still applicable on the latest dev build? If so, please post the error created by that build. Currently as the code is different that error can't be used to trace the issue

Comment by Brian.Wood

What's the best link for the dev build?

Comment by me4502

builds.enginehub.org

Comment by Brian.Wood

Latest dev doesn't throw an error, so that's good! Only thing I notice now is it's not planting sapling when "place-sapling" is "true".

(related: what is "enforce-data"? not in the wiki docs)

Comment by Brian.Wood

Wait. I lied. It places the sapling if I break a block other than the first. As long as I hit the second block up, that works.

Comment by me4502

Fixed in latest dev build.

Also, enforce data makes it only break other blocks that has the same data values.