CommandBook Could not pass event PlayerLoginEvent to CommandBook

CMDBOOK-2190 - Reported by unusualbob

On player login a stacktrace appears in the console.

13:11:48 [SEVERE] Could not pass event PlayerLoginEvent to CommandBook v539-57d1a9e
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_4_6.PlayerList.attemptLogin(PlayerList.java:275)
        at net.minecraft.server.v1_4_6.PendingConnection.d(PendingConnection.java:121)
        at net.minecraft.server.v1_4_6.PendingConnection.c(PendingConnection.java:45)
        at net.minecraft.server.v1_4_6.DedicatedServerConnectionThread.a(DedicatedServerConnectionThread.java:44)
        at net.minecraft.server.v1_4_6.DedicatedServerConnection.b(SourceFile:29)
        at net.minecraft.server.v1_4_6.MinecraftServer.r(MinecraftServer.java:598)
        at net.minecraft.server.v1_4_6.DedicatedServer.r(DedicatedServer.java:224)
        at net.minecraft.server.v1_4_6.MinecraftServer.q(MinecraftServer.java:494)
        at net.minecraft.server.v1_4_6.MinecraftServer.run(MinecraftServer.java:427)
        at net.minecraft.server.v1_4_6.ThreadServerApplication.run(SourceFile:849)
Caused by: java.lang.NullPointerException
        at com.sk89q.commandbook.session.SessionComponent.onLogin(SessionComponent.java:325)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:601)
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
        ... 13 more

CraftBukkit version git-Bukkit-1.4.6-R0.3-7-g48dcb3e-b2591jnks (MC: 1.4.6) (Implementing API version 1.4.6-R0.4-SNAPSHOT)

Jan 07 '13 18:01 LadyCailinBot

Comment by unusualbob

Aha figured out its directly related to if the player is opped. Opped players get a stacktrace, regulars do not.

Jan 07 '13 19:01 LadyCailinBot

Comment by Dark_Arc

I think it may be trying to load a session that doesn't exist... Though that's just from glancing over it...

Jan 08 '13 02:01 LadyCailinBot

Comment by unusualbob

Nope, session file definitely exists. Just to make sure I deleted it and it recreated it and still is broken. It only gives me the error while I am opped, as soon as I deop myself and login its fine and dandy. Reop and relog, stacktrace.

Jan 08 '13 02:01 LadyCailinBot

Comment by zml2008

I cannot reproduce your error locally. However, I've added some code to the latest CommandBook dev build that will give me more information about what's wrong. Get the dev build at: http://build.sk89q.com/job/CommandBook/540/artifact/target/commandbook-2.2.1-SNAPSHOT.zip

Jan 08 '13 03:01 LadyCailinBot

Comment by unusualbob

Sorry about the delay getting back to you, here's the new stack.

09:18:32 [SEVERE] Could not pass event PlayerLoginEvent to CommandBook v540-e0b472a
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_4_6.PlayerList.attemptLogin(PlayerList.java:275)
        at net.minecraft.server.v1_4_6.PendingConnection.d(PendingConnection.java:121)
        at net.minecraft.server.v1_4_6.PendingConnection.c(PendingConnection.java:45)
        at net.minecraft.server.v1_4_6.DedicatedServerConnectionThread.a(DedicatedServerConnectionThread.java:44)
        at net.minecraft.server.v1_4_6.DedicatedServerConnection.b(SourceFile:29)
        at net.minecraft.server.v1_4_6.MinecraftServer.r(MinecraftServer.java:598)
        at net.minecraft.server.v1_4_6.DedicatedServer.r(DedicatedServer.java:224)
        at net.minecraft.server.v1_4_6.MinecraftServer.q(MinecraftServer.java:494)
        at net.minecraft.server.v1_4_6.MinecraftServer.run(MinecraftServer.java:427)
        at net.minecraft.server.v1_4_6.ThreadServerApplication.run(SourceFile:849)
Caused by: java.lang.NullPointerException: Session class com.sk89q.commandbook.locations.TeleportSession does not have a valid factory!
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:204)
        at com.sk89q.commandbook.session.SessionComponent.onLogin(SessionComponent.java:326)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:601)
        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
        ... 13 more

Jan 09 '13 14:01 LadyCailinBot

Comment by Darkhand81

Same issue here, problem with PlayerLoginEvent when an op logs in:

http://pastebin.com/9haebUvw

Jan 20 '13 17:01 LadyCailinBot

Comment by zml2008

This should be fixed in the most recent dev build (#542+).

Jan 28 '13 04:01 LadyCailinBot