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]: current dev version leaves TweakScale folder behind when uninstalling #3936

Closed
1 task done
JonnyOThan opened this issue Dec 5, 2023 · 4 comments · Fixed by #3942
Closed
1 task done

[Bug]: current dev version leaves TweakScale folder behind when uninstalling #3936

JonnyOThan opened this issue Dec 5, 2023 · 4 comments · Fixed by #3942
Assignees
Labels
Bug Something is not working as intended Core (ckan.dll) Issues affecting the core part of CKAN

Comments

@JonnyOThan
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Operating System

windows 10

CKAN Version

5684cd6

Games

KSP 1

Game Version

1.12.5

Did you make any manual changes to your game folder (i.e., not via CKAN)?

No response

Describe the bug

uninstalling tweakscale can leave folders behind in the dev build. This does NOT occur on the current official release, 1.33.2

Steps to reproduce

  1. install tweakscale
  2. run the game (I'm not sure if this step is required)
  3. close the game
  4. uninstall tweakscale with ckan

Relevant log output

No response

@JonnyOThan JonnyOThan added the Bug Something is not working as intended label Dec 5, 2023
@HebaruSan HebaruSan self-assigned this Dec 5, 2023
@HebaruSan HebaruSan added the Core (ckan.dll) Issues affecting the core part of CKAN label Dec 5, 2023
@HebaruSan
Copy link
Member

I'm not able to reproduce this; my GameData is clean after the uninstall.
Do I have to load a save game? I loaded the main menu and then exited.

@JonnyOThan
Copy link
Contributor Author

JonnyOThan commented Dec 10, 2023

I'm not able to reproduce this; my GameData is clean after the uninstall. Do I have to load a save game? I loaded the main menu and then exited.

Quite possibly, TweakScale creates a config.xml file inside its PluginData but that might not happen until the save is loaded.

@HebaruSan
Copy link
Member

HebaruSan commented Dec 10, 2023

Thanks. After starting a save, launching to high suborbital, and exiting, I see that file created:

image

This shows up at uninstall as expected (though the label sizing needs some adjustment):

image

... and clicking Delete removes the xml file, the "Scale" folder, and the PluginData folder, but leaves behind Plugins and TweakScale:

image

Cool, now I can start debugging...

@HebaruSan
Copy link
Member

Probably related to #3890.

The unusual thing about this use case is that the mod is not only creating a file in a CKAN-created folder, it's creating a folder in a CKAN-created folder and creating a file inside that. So the updated logic is probably not considering that possibility.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working as intended Core (ckan.dll) Issues affecting the core part of CKAN
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants