Fabric Waystones

Fabric Waystones

16M Downloads

[Bug]: Global waystones become discovered and dont hide

Linguardium opened this issue ยท 1 comments

commented

Is there an existing issue for this?

  • I have searched the existing issues

Description

If you use a global waystone, it is added to the discovered waystones. The hiding global waystones is ignored if the you have ever used the waystone in question.

This creates a situation where clicking a global waystone once prevents you from ever hiding that waystone from the list without hiding ALL waystones from the list

To Reproduce

set waystone global
use waystone to make it "discovered"
uncheck show global waystones

Expected behavior

Hiding globals should take priority over discovered status. If someone intentionally hides global waystones, they should be hidden.

Mod Version

v3.0.8

Minecraft Version

1.19.2

Java Version

Java 17

Fabric Version

N/A, logic issue not FAPI issue

Relevant log output

No response

Anything else?

issue still present in master branch

((PlayerEntityMixinAccess) player).discoverWaystone(blockEntity);

if this was inside the bracket above, it would skip this if it were a global waystone

commented

The line of code you pointed out, I need that there in case the player ""discovers"" a global Waystone, which in the future gets toggled as non-global.
Maybe I can make a config option in the future...