lclpyt / lclpgravestones Goto Github PK

This is a great plugin and I love the simplicity of it however the code preventing the chest from exploding doesn’t seem to work. Creepers tnt and beds in other dimensions can still blow the chest up on death causing item loss

It would be really cool if there was a command for admin, which shows player gravestone log (all gravestones that was created for the whole time) and command that can restore players most recent gravestone.
And command far all players that shows coordinates of their last grave.
And if there was a config file, where you can toggle showing gravestone coordinates in console

It keeps appearing in my server console every time i open chest

[Server thread/ERROR]: Could not pass event PlayerInteractEvent to LCLPGraveStones v1.0
org.bukkit.event.EventException: null
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:319) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:589) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:576) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at org.bukkit.craftbukkit.v1_16_R1.event.CraftEventFactory.callPlayerInteractEvent(CraftEventFactory.java:465) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.PlayerInteractManager.a(PlayerInteractManager.java:444) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.PlayerConnection.a(PlayerConnection.java:1284) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.PacketPlayInUseItem.a(PacketPlayInUseItem.java:27) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.PacketPlayInUseItem.a(PacketPlayInUseItem.java:1) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.PlayerConnectionUtils.lambda$0(PlayerConnectionUtils.java:19) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.TickTask.run(SourceFile:18) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.IAsyncTaskHandler.executeTask(SourceFile:144) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.IAsyncTaskHandler.executeNext(SourceFile:118) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.MinecraftServer.aZ(MinecraftServer.java:943) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.MinecraftServer.executeNext(MinecraftServer.java:936) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.IAsyncTaskHandler.awaitTasks(SourceFile:127) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.MinecraftServer.sleepForTick(MinecraftServer.java:920) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.MinecraftServer.v(MinecraftServer.java:852) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at net.minecraft.server.v1_16_R1.MinecraftServer.lambda$0(MinecraftServer.java:164) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_271]
Caused by: java.lang.NoSuchMethodError: org.bukkit.inventory.Inventory.isEmpty()Z
at work.lclpnet.gravestones.event.EventListener.onInteractGravestone(EventListener.java:182) ~[?:?]
at sun.reflect.GeneratedMethodAccessor24.invoke(Unknown Source) ~[?:?]
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:1.8.0_271]
at java.lang.reflect.Method.invoke(Unknown Source) ~[?:1.8.0_271]
at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:315) ~[spigot-1.16.1.jar:git-Spigot-0287a20-7560f5f]
... 20 more

Describe the bug
Breaking the chest does not drop items but instead makes them disappear with the chest.

To Reproduce
Steps to reproduce the behavior:

1. Die.
2. Retrieve items via breaking the chest.
3. Chest breaks, but nothing is dropped.

Expected behavior
Items should be dropped when the chest is broken. Otherwise the chest needs to be unbreakable.

Desktop (please complete the following information as far as possible):

• Plugin Version: 1.2
• Minecraft Version: 1.16.5
• Server: Paper
• Java Version: 1.8

Additional context
Sometimes there are too many items in the chest so players will unknowingly break their chest instead of retrieving all their items individually. This makes it very frustrating as you have high hopes that you can retrieve your items.

I love this plugin as it is very detailed. With the msg on your death marker, and the sound effects of the chest disappearing after getting all the items. Sadly the bug is too game breaking as the potential of players unknowingly breaking their chest exist. I will be using this plugin again when it is fixed ^^

There is one problem about gravestones. They keep items with "Curse of vanishing" instead of deleting them. It's pretty bad for vanilla servers, where curse of vanishing becomes Useless.

And could you please make LCLPGraveStones compatible with player head drops? Because now these 2 plugins doesn't work together, and when you kill a player with items, the head just doesn't drop (if player's inventory clear, then everything works just fine).