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

[Bug 🐞]: Incorrect mod settings descriptions/labels #74

Closed
johnnycocas opened this issue Oct 3, 2022 · 3 comments · Fixed by #83
Closed

[Bug 🐞]: Incorrect mod settings descriptions/labels #74

johnnycocas opened this issue Oct 3, 2022 · 3 comments · Fixed by #83
Assignees
Labels
bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code

Comments

@johnnycocas
Copy link

johnnycocas commented Oct 3, 2022

Brief description of your issue (aka the problem)

In-game mod settings are not being correctly mapped to the text keys in the Localization file, as shown in one of the attached print screens containing the contents of the DLL included in the mod.

Operating system

Win10

KSP version

1.12.3.3173

Version of addon

2.2.0.0

Steps to reproduce the behavior or issue

  1. Open "Settings"
  2. Open "Difficulty Options"
  3. Select "OhScrap! (OHS) Options"

Expected behavior

In-game settings text keys being mapped with the prefix "#OHS".

Actual behavior

Settings keys are being mapped with the prefix "#FND".

Environment

Mod: v2.2.0.0
KSP: v1.12.3.3173
Installation: CKAN

Log files

KSP.log

Screenshots

In-game screenshot:
image

DLL contents:
image

Localization file (expected values):
image

Additional context

Edited to add a label to each image.

How did you download and install this?

CKAN (auto install with dependencies)

@zer0Kerbal zer0Kerbal added the bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code label Oct 3, 2022
@zer0Kerbal zer0Kerbal added this to the 2.2.99.0-prerelease milestone Oct 3, 2022
@zer0Kerbal
Copy link
Owner

zer0Kerbal commented Oct 3, 2022

@johnnycocas thank you for a wonderful bug report (the report is wonderful - not the bug)

if not - will be for the next - will comment later -

again, thank you!

@zer0Kerbal
Copy link
Owner

I believe this has been fixed internally - just not released (sooner than soon)

@johnnycocas
Copy link
Author

johnnycocas commented Oct 4, 2022

Yeah the code in the repository seemed to be correct, at least on the 2.2.99.0 branch (master seemed to contain legacy code) so I assumed maybe the DLL was compiled with old code and said code was changed afterwards. Weird thing nonetheless, iirc "FND" should be for "Foundations", another of your mods, correct? Maybe some code found a way across to the other side 😁

Don't thank me too much, I may find more bugs in the future xD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants