Paper icon indicating copy to clipboard operation
Paper copied to clipboard

Lodestone Compass not working properly with unloaded chunks

Open Pitrex111 opened this issue 1 year ago • 1 comments

Expected behavior

When destroying Lodestone linked compasses stops pointing to lodestone location

Observed/Actual behavior

Compass in some situation still points out towards lodestone location even if Lodestone is destroyed

Steps/models to reproduce

  1. Place Lodestone and link a compass to it
  2. Put linked compass to your enderchest
  3. Destroy the Lodestone
  4. Unload the chunk the Lodestone was placed (i teleported away)
  5. Remove linked compass from your enderchest (right now it will still point towards former direction)
  6. Load the chunk lodestone was placed (i teleported back)
  7. Upon loading compass, acknowledges lodestone removal and start pointing in random direction

Plugin and Datapack List

[21:31:13 INFO]: Server Plugins (14): [21:31:13 INFO]: Bukkit Plugins: [21:31:13 INFO]: - ICraft2.0-Special, *JoinFullServer, Multiverse-Core, Multiverse-Inventories, Multiverse-Portals, NoChatReports, *PermissionsEx, PexTabCompleter, ProtocolLib, *SimplePrefix-Reloaded [21:31:13 INFO]: TreasureChestX, *VoidWorld, WorldEdit, WorldGuard

[21:31:24 INFO]: There are 3 data pack(s) enabled: [vanilla (built-in)], [file/bukkit (world)], [paper (built-in)] [21:31:24 INFO]: There are no more data packs available

Paper version

[21:35:54 INFO]: Checking version, please wait... [21:35:54 INFO]: This server is running Paper version 1.21-119-master@100d75a (2024-07-29T16:00:13Z) (Implementing API version 1.21-R0.1-SNAPSHOT) You are running the latest version Previous version: 1.21-109-5a5035b (MC: 1.21)

Other

There is also breakage in my custom plugin behaviour regarding storing serialazed linked compass in PersistentDataContainer, but I believe resolving shown issue will also fix that issue, so I'm not providing data at this moment for the plugin

Pitrex111 avatar Jul 29 '24 19:07 Pitrex111

Explicitly patched by paper. I guess we could add a configuration option to re-enable this, however keeping such behaviour on by default is pretty deadly as it induces sync chunk loads.

lynxplay avatar Jul 29 '24 19:07 lynxplay