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

For installer, detect if the specified folder contains instances folder. #2230

Closed
1 task done
aMelonRind opened this issue Mar 21, 2024 · 4 comments
Closed
1 task done
Labels
enhancement New feature or request

Comments

@aMelonRind
Copy link

Role

I'm prism launcher user that was on older version and didn't have auto update enabled.

Suggestion

I want the installer to warn the user if the specified folder contains instances folder.

Benefit

so the user won't accidently delete their instances. (assuming if the installer actually deletes old files)

This suggestion is unique

  • I have searched the issue tracker and did not find an issue describing my suggestion, especially not one that has been rejected.

You may use the editor below to elaborate further.

tldr: I'm dumb and almost deleted all my instances, although I do have some backups.

The accident:
I need to test a mod with NeoForge, and found out NeoForge support is on prism 8.0, so I need to update the launcher.
I might be blind, but I didn't see the update button in 7.2.
Look into the help menu, didn't find anything related to updates. then noticed the news at the bottom says:

Prism Launcher Release 8.2, now available

Click into it, read it, and the last line says:

You can grab the latest download here for your respective platform.

So, I thought downloading the installer is the only way to update.
Downloaded and open the installer, it says select a folder. I open the mods folder from a shortcut, then noticed the path is slightly different from the default one in the installer, and the folder name is the same, then I thought I need to change the path, and I did it.
Then the installation ran for a minute, the progress says there's 1.2gb of files, and it's continually increasing. I think something went terribly wrong, the cancel button is not active, so I terminated the installer in task manager.

Good news is that I didn't lose any file because free space didn't increase, and it still launches correctly.

@aMelonRind aMelonRind added the enhancement New feature or request label Mar 21, 2024
@Trial97
Copy link
Member

Trial97 commented Mar 21, 2024

Prism installer doesn't delete the instance folder, nor does the uninstall of prism.
Usually, prism has two folders that it cares about:

  • where the executable is located
  • and the data directory = where your configs and instances are usually located

So when reinstalling/uninstalling prism the only affected folder is where the executable is located and even there the files that are modified are limited to the ones created by the prism installer.
So when you install my suggestion is if you do not have a reason to change the install path do not do it. But if you do prism will not care about where is installed as the data directory will always point toward the same location on the system.
I will close this as not planned.
If you have trouble installing/updating let me know in this issue or better you can join Prism discord and ask there.
Here is the invite link if needed: https://prismlauncher.org/discord

@Trial97 Trial97 closed this as not planned Won't fix, can't repro, duplicate, stale Mar 21, 2024
@aMelonRind
Copy link
Author

I see. Appreciate your answer.

So when you install my suggestion is if you do not have a reason to change the install path do not do it.

Normally I won't do that, but I thought I should because at the past I'm using portable version of MultiMc which is in a custom location, so I thought my prism install might be on custom location too, so I looked up the path and changed it.

Good to know that it doesn't actually remove other files! But why is it scanning all the files, although they will remain untouched?

@aMelonRind
Copy link
Author

If you have trouble installing/updating let me know in this issue or better you can join Prism discord and ask there.

Thanks, I've already installed it correctly before opening this issue.

@Trial97
Copy link
Member

Trial97 commented Mar 21, 2024

Good to know that it doesn't actually remove other files! But why is it scanning all the files, although they will remain untouched?

I will presume that is to collect the files that need to be deleted.
The portable update is another story as you usually just need to extract over the old portable(there may be caveats to that like before extracting you would need to delete some files)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants