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

when the KSP_VERSION is lower than the version of the game, it doesn't check the VERSION at all. #19

Closed
linuxgurugamer opened this issue Oct 10, 2017 · 1 comment · May be fixed by #20

Comments

@linuxgurugamer
Copy link

Most likely the cause of issue #9 and #10

This has been bugging me, and I think I've found a real bug:

It seems that when the KSP_VERSION is lower than the version of the game, it doesn't check the VERSION at all.

I'm basing this on the following:

AVC file in folder:

{
"NAME":"K2CommandPodContinued",
"URL":"http://ksp-avc.cybutek.net/version.php?id=428",
"VERSION":
{
"MAJOR":1,
"MINOR":1,
"PATCH":8,
"BUILD":1
},
"KSP_VERSION":
{
"MAJOR":1,
"MINOR":3,
"PATCH":1
},
"KSP_VERSION_MIN":
{
"MAJOR":1,
"MINOR":2,
"PATCH":0
},
"KSP_VERSION_MAX":
{
"MAJOR":1,
"MINOR":3,
"PATCH":99
}
}
AVC downloaded from URL (run through a JSON formatter):

{
"NAME": "K2CommandPodContinued",
"URL": "http://ksp-avc.cybutek.net/version.php?id=428",
"VERSION": {
"MAJOR": 1,
"MINOR": 1,
"PATCH": 8,
"BUILD": 2
},
"KSP_VERSION": {
"MAJOR": 1,
"MINOR": 3,
"PATCH": 0
},
"KSP_VERSION_MIN": {
"MAJOR": 1,
"MINOR": 2,
"PATCH": 0
},
"KSP_VERSION_MAX": {
"MAJOR": 1,
"MINOR": 3,
"PATCH": 99
}
}
In the output_log.txt:

KSP-AVC -> R:\KSP_1.3.1_dev\GameData\JFJohnny5\K2.version
NAME: K2CommandPodContinued
URL: http://ksp-avc.cybutek.net/version.php?id=428
DOWNLOAD: NULL
GITHUB: NULL
VERSION: 1.1.8.1
KSP_VERSION: 1.3
KSP_VERSION_MIN: 1.2
KSP_VERSION_MAX: 1.3.99
CompatibleKspVersion: False
CompatibleKspVersionMin: True
CompatibleKspVersionMax: True
CompatibleGitHubVersion: True

(Filename: C:/buildslave/unity/build/artifacts/generated/common/runtime/UnityEngineDebugBindings.gen.cpp Line: 42)

KSP-AVC -> http://ksp-avc.cybutek.net/version.php?id=428
NAME: K2CommandPodContinued
URL: http://ksp-avc.cybutek.net/version.php?id=428
DOWNLOAD: NULL
GITHUB: NULL
VERSION: 1.1.8.2
KSP_VERSION: 1.3
KSP_VERSION_MIN: 1.2
KSP_VERSION_MAX: 1.3.99
CompatibleKspVersion: False
CompatibleKspVersionMin: True
CompatibleKspVersionMax: True
CompatibleGitHubVersion: True
UpdateAvailable: False
You can clearly see that in the log, it specifically says that no update is available, AND in-game, it shows the mod in green

when I changed the website to reflect the correct version of the game. It's obviously using the data from the website, and, IMHO, is also obviously now totally useless to verify that mods are ok after the game has updated.

The specific circumstance I ran into was when checking parts mods, which have a very relaxed MIN and MAX values; but if the base KSP_VERSION is not matching the current game, it doesn't raise a flag at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant