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

Fix save wizard file identification check #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

klarkyson
Copy link

As of the DLC release the regulation contents can no longer be used to compare to a checksum for identification.

This PR opts instead to attempt to decrypt and decompress the regulation contents such that if it succeeds the program assumes the file is the save wizard format. This is a bit slower than the checksum, but at least it's consistent!

@llolnyj
Copy link

llolnyj commented Jul 6, 2024

The new hotfix is not working. It instantly crashes when I open SL2 file with it:(

@noamb14
Copy link

noamb14 commented Jul 6, 2024

it says that the save data is corrupted when I got the save file from the ps4 back to the pc

@klarkyson
Copy link
Author

Did either of you build from my branch?

I'm not having these issues. If you did build from my branch please post your saves and what game version you're using.

@gaseet
Copy link

gaseet commented Jul 7, 2024

Hello klarkyson, I just built from your branch, and while I can open my save, the scadutree blessing option isn't there, and all dlc items only have ids and no names, is this intended?

@noamb14
Copy link

noamb14 commented Jul 10, 2024

Did either of you build from my branch?

I'm not having these issues. If you did build from my branch please post your saves and what game version you're using.

Hi. It actually did work. I didnt press the import character buttin and thats why it didnt work.
If you start a ng+ it will still work tho right?

@babbatem
Copy link

babbatem commented Aug 2, 2024

thanks, @klarkyson. Does this include items from the DLC?

e.g. from this PR -- #49 (comment)

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

Successfully merging this pull request may close these issues.

5 participants