Hi! NPE on villager right click (3.5)
SimSonic opened this issue · 19 comments
Should be fixed.
4b4bf7c
At the moment, as we do not have unit tests (something I am currently implementing), we do not have a CI server. However, it is a possibility for the future
I've sent a version with some fixed bugs to someone else, but here it is if you wanna test it too: https://dl.dropboxusercontent.com/u/2485556/EpicQuest.jar
Still the same...
org.bukkit.event.EventException
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:305) ~[Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62) ~[Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:502) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:487) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.PlayerConnection.a(PlayerConnection.java:1268) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.PacketPlayInUseEntity.a(SourceFile:52) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.PacketPlayInUseEntity.a(SourceFile:11) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.PacketHandleTask.run(SourceFile:13) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_11]
at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_11]
at net.minecraft.server.v1_8_R1.MinecraftServer.z(MinecraftServer.java:683) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.DedicatedServer.z(DedicatedServer.java:316) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.MinecraftServer.y(MinecraftServer.java:623) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at net.minecraft.server.v1_8_R1.MinecraftServer.run(MinecraftServer.java:526) [Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
at java.lang.Thread.run(Thread.java:745) [?:1.8.0_11]
Caused by: java.lang.NullPointerException
at randy.questentities.QuestEntity.NextInteraction(QuestEntity.java:78) ~[?:?]
at randy.listeners.PlayerInteractListener.onPlayerInteractEntity(PlayerInteractListener.java:92) ~[?:?]
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_11]
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_11]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_11]
at java.lang.reflect.Method.invoke(Method.java:483) ~[?:1.8.0_11]
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:301) ~[Spigot_v1.8_2015-01-06.jar:git-Spigot-e497721-96d09e3]
... 14 more
Yes I did it 3-5 times.
/q questentity create Notch Welcome!
Welcome! — initial quest from plugin's config.
right click on villager = the same console error.
Ah, I see. You should reference quests by their file name, not their display name :)
On this topic, I've created a new task for more config checks, as I see more people have trouble with setting up some things. In any case, happy it works now!
The reason why I made it reference the file name is because then the plugin will always keep track of the quest, so you can change whatever you want in the quest while always making it work. It's also the reason for the big quest changing thing!
I'm trying to use your plugin first time and see some hardness:
- For some reason Reset_Time = -1 is not working.
- /q reload during quests allows to get many instances of single quest.
- somehow killing mobs is shared between all players given quest to kill mobs :)
- Villager is not teleporting to his point.
Maybe I do something wrong and I need just more time to find correct usage. I will try tomorrow :)
Another notes:
- Maybe quests and quest givers should be in separate files like it is now, but for per-player progress mysql database can be used. I can help with db design and code.
- My server is not english speaking so I need possibility to make translation.
Thanks for the reporting the bugs, can you create issues for them?
Also, I have added the suggestions here already: https://trello.com/b/lrlUoJYq/epicquest-suggest-new-ideas-to-be-added-via-pm-on-bukkit
You can vote for what you want first :)