PluginBugs / Issues-ItemsAdder

Repository used to keep track of issues of my plugin ItemsAdder
https://itemsadder.devs.beer
53 stars 21 forks source link

Breaking custom block will fire the PlayerItemBreakEvent wrongly #2687

Closed iamyou closed 1 year ago

iamyou commented 1 year ago

Terms

Discord tag (optional)

No response

What happened?

Breaking custom block will fire the PlayerItemBreakEvent wrongly

Steps to reproduce the issue

  1. making a test plugin to listen the PlayerItemBreakEvent
  2. placing a custom block and break it using tools

Server version

git-Purpur-1972 (MC: 1.19.4)*

ItemsAdder Version

3.4.1c

ProtocolLib Version

5.0.0

LoneLibs Version

1.0.24

FULL server log

No response

Error (optional)

java.lang.RuntimeException: Test exception
        at io.github.iamyou.testplugin.TestPlugin.onItemBreak(TestPlugin.java:25) ~[TestPlugin-1.0.jar:?]
        at com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor47.execute(Unknown Source) ~[?:?]
        at org.bukkit.plugin.EventExecutor$2.execute(EventExecutor.java:77) ~[purpur-api-1.19.4-R0.1-SNAPSHOT.jar:?]
        at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:81) ~[purpur-api-1.19.4-R0.1-SNAPSHOT.jar:git-Purpur-1972]
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[purpur-api-1.19.4-R0.1-SNAPSHOT.jar:?]
        at io.papermc.paper.plugin.manager.PaperEventManager.callEvent(PaperEventManager.java:54) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.callEvent(PaperPluginManagerImpl.java:126) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:617) ~[purpur-api-1.19.4-R0.1-SNAPSHOT.jar:?]
        at ia.m.kO.a(SourceFile:68) ~[ItemsAdder_3.4.1c.jar:?]
        at ia.m.er.a(SourceFile:472) ~[ItemsAdder_3.4.1c.jar:?]
        at ia.m.er.a(SourceFile:449) ~[ItemsAdder_3.4.1c.jar:?]
        at ia.m.er.a(SourceFile:376) ~[ItemsAdder_3.4.1c.jar:?]
        at ia.m.er.a(SourceFile:370) ~[ItemsAdder_3.4.1c.jar:?]
        at ia.m.cr.bc(SourceFile:207) ~[ItemsAdder_3.4.1c.jar:?]
        at org.bukkit.craftbukkit.v1_19_R3.scheduler.CraftTask.run(CraftTask.java:101) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at org.bukkit.craftbukkit.v1_19_R3.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:483) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1510) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:488) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1424) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1195) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:325) ~[purpur-1.19.4.jar:git-Purpur-1972]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]

Problematic items yml configuration file (optional)

No response

Other files, you can drag and drop them here to upload. (optional)

No response

Screenshots/Videos (you can drag and drop files or paste links)

No response

iamyou commented 1 year ago

Will causing #2219 because of https://github.com/Sefiraat/SlimeTinker/blob/2b6eeb317d61cc6fe387cc1ddd90c5380f75720d/src/main/java/io/github/sefiraat/slimetinker/listeners/DurabilityListener.java#L88

LoneDev6 commented 1 year ago

Thanks for the details. I somehow made a mistake and called PlayerItemBreakEvent instead of PlayerItemDamageEvent.