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 some debug info showing despite being disabled in the UI #12205

Merged
merged 1 commit into from
Apr 21, 2022

Conversation

appgurueu
Copy link
Contributor

@appgurueu appgurueu commented Apr 21, 2022

This fixes a bug reported by @GoodClover and introduced in #12020, namely here: F5 is now being ignored, only the priv and the flag are used to check whether some debug info (such as GenericCAO hp & armor groups) is shown.

src/client/game.cpp Outdated Show resolved Hide resolved
@sfan5 sfan5 added the Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines label Apr 21, 2022
@GoodClover
Copy link
Contributor

Tested, seems to work: doesn't show when debug is closed.
Not sure if I misunderstand though, is it not meant to be available without debug privs? as it still shows without them.

@appgurueu
Copy link
Contributor Author

Tested, seems to work: doesn't show when debug is closed. Not sure if I misunderstand though, is it not meant to be available without debug privs? as it still shows without them.

It is meant to be available (=allowed) without debug privs for compatibility reasons if and only if the basic debug HUD flag is not explicitly set to false by the modder. If you have the debug priv it will override the HUD flag.

@sfan5 sfan5 merged commit 4558793 into minetest:master Apr 21, 2022
@appgurueu appgurueu deleted the fix-de-bug branch March 31, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix 🐛 PRs that fix a bug @ Client / Controls / Input Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants