ScreenShot_2022-06-08 19-39-53.png
[SPIGOT-7040] [1.19] Block physics event doesn't fire for signs anymore Created: 08/Jun/22 Updated: 25/Dec/24 |
|
| Status: | Open |
| Project: | Spigot |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Minor |
| Reporter: | Irmo van den Berge | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | 1.19, BlockPhysicsEvent, Sign, redstone | ||
| Environment: |
Windows 7 64 bit JDK 17 |
||
| Attachments: |
ScreenShot_2022-06-08 19-39-53.png
|
| Version: | This server is running CraftBukkit version 3507-Spigot-fa893f0-c3f219e (MC: 1.19) (Implementing API version 1.19-R0.1-SNAPSHOT) |
| Plugin: | BKCommonLib / Traincarts |
| Guidelines Read: | Yes |
| Description |
|
I'll start with: I'm not sure if this can be considered a bug, or if the original behavior was a bug. My plugin depends heavily on sign block physics events fired when redstone near them changes state. This is so that a redstone signal can trigger a sign, kind of like CraftBook. Since Minecraft 1.19, at least on the latest Spigot 1.19 snapshot, this event is no longer fired. It does fire for other blocks (like the redstone wire/torch, etc.) |
| Comments |
| Comment by Irmo van den Berge [ 10/Jun/22 ] |
|
I've since patched this issue by tracking the signs in chunk load / placement / etc. in my own hashmap. More performant than the old block search method to boot. So for me, this issue is no longer important. If others care they can chime in. |
| Comment by Irmo van den Berge [ 10/Jun/22 ] |
|
Yes, Im sure this isn't really a change in spigot / craftbukkit itself, more of an unintended parity change in the server itself. I can work around this change by tracking where signs are so I don't have to do expensive block type lookups in block physics events, so it's not a blocker for me... So I'm more looking for some feedback of whether this is something the team deems worthy of changing in the server, or I'm better off modifying my own plugin's logic. |
| Comment by md_5 [ 10/Jun/22 ] |
|
I think this is more to do with changes to the 1.19 physics engine than anything to do with Spigot. |
