Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sign formspec updates while used #9

Merged
merged 2 commits into from Dec 9, 2023
Merged

Fix sign formspec updates while used #9

merged 2 commits into from Dec 9, 2023

Conversation

Niklp09
Copy link
Member

@Niklp09 Niklp09 commented Dec 5, 2023

Please read https://gitea.your-land.de/your-land/bugtracker/issues/5553

EDIT(wsor - lazy and tired of opening the link above, this from flux):
i figured out why i could't replicate this locally initially - it only happens when using the mt-mods fork of display_modpack, not the pyrollo fork. i've traced it to this commit: 1f8e99c9c5

the issue is that when a player is looking at a node formspec, and the node formspec is modified, what the player sees is also modified. but looking at that commit above, right-clicking the sign should only modify the formspec if it needs to have that fix applied. however, i'm having this issue multiple times on certain signs, and also for new signs that were placed well after the fix was applied. something else might be modifying the sign formspec, perhaps...

@Niklp09 Niklp09 added the bugfix fixes a bug label Dec 5, 2023
@wsor4035
Copy link

wsor4035 commented Dec 5, 2023

why not just remove the on rightclick and let it update on save rather than having an lbm run for all eternity? unless im missing something here

@fluxionary
Copy link
Member

why not just remove the on rightclick and let it update on save rather than having an lbm run for all eternity? unless im missing something here

run_at_every_load = false. the impact of the LBM will be minimal.

Copy link

@wsor4035 wsor4035 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, didnt test

@wsor4035 wsor4035 merged commit afe8f51 into master Dec 9, 2023
2 checks passed
@OgelGames
Copy link

OgelGames commented Dec 9, 2023

I see no reason to add a LBM, there isn't anything that needs updating before the formspec is opened. Just removing the on_rightclick should be enough to fix this.

@wsor4035
Copy link

wsor4035 commented Dec 9, 2023

I see no reason to add a LBM, there isn't anything that needs updating before the formspec is opened. Just removing the on_rightclick should be enough to fix this.

while i agree, reading this, the lbm in theory updates it before the user opens it(applying the formspec escape), rather than closing it, so the issue is solved before hand

@Niklp09 Niklp09 deleted the fix_fs branch December 13, 2023 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants