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

Version mismatch "not loading" error appears for disabled mods #556

Closed
xen-42 opened this issue Sep 25, 2023 · 3 comments · Fixed by #567
Closed

Version mismatch "not loading" error appears for disabled mods #556

xen-42 opened this issue Sep 25, 2023 · 3 comments · Fixed by #567
Labels
bug Something isn't working

Comments

@xen-42
Copy link
Member

xen-42 commented Sep 25, 2023

Get a log error saying

Not loading Raicuparta.QuantumSpaceBuddies, as the current game version 1.1.14.768 is different to the latest version 1.1.13.456

Even though QSB is disabled.

@xen-42 xen-42 added the bug Something isn't working label Sep 25, 2023
@misternebula
Copy link
Member

this shouldnt be possible?

var shouldLoad = _modVersionChecker.CheckModVersion(modData) && _modVersionChecker.CheckModGameVersion(modData, latestGameVersion);

method that prints those errors is CheckModGameVersion, and CheckModVersion will return false if a mod is disabled

@xen-42
Copy link
Member Author

xen-42 commented Mar 11, 2024

No, that method returns true if a mod is disabled. That was changed to fix #369 where it would log errors about versions for disabled mods.

@xen-42
Copy link
Member Author

xen-42 commented Mar 11, 2024

Could've fixed both issues by doing this instead
var shouldLoad = modData.Config.Enabled && _modVersionChecker.CheckModVersion(modData) && _modVersionChecker.CheckModGameVersion(modData, latestGameVersion);
Otherwise its only in LoadMod (line 141) that it decides to ignore disabled mods

misternebula added a commit that referenced this issue Mar 19, 2024
@misternebula misternebula mentioned this issue Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants