Player icon indicating copy to clipboard operation
Player copied to clipboard

Rewrite ControlGlobalSave

Open Ghabry opened this issue 6 months ago • 1 comments

Main difference is the logic when opening the save. Also moved it to ManiacPatch file to reduce complexity in the interpreter code.

Header cleanup (was necessary because of weird compile errors due to a Game_String include while moving code around)

Fix #3069


@Mimigris please test this. Should be at least closer than before xD

Ghabry avatar Jun 16 '25 11:06 Ghabry

Sorry for answering a bit late on this, just started my testing.

From what I can see, the Save.lgs file is not able to be created at the moment using the build of this pull request, only updated, meaning that storing data in a Save.lgs file is impossible unless said file already existed, so this would probably need to be fixed before doing bigger testings due to being at the heart of the command.

Mimigris avatar Jun 22 '25 16:06 Mimigris

Yeah my bad, the flag that tracked whether the global save is "opened" what only set when the file exists... Is fixed now

Ghabry avatar Jun 23 '25 14:06 Ghabry

Noticed a crash with the current setup. I have the Player set outside of a game folder, and I use it to open a folder containing the game (game in question being the EasyRPG TestGame made for Maniac). If I use the command Open Global Save in the command, then exit it, then make the program close by exiting the game browser, the Player will crash instead of closing as intended.

Mimigris avatar Jun 23 '25 16:06 Mimigris

ResetGameObjects was not called when doing it as you described.

I added this call now to Continue from the game browser. No idea why this wasn't used before, seems to work fine (likely some other bugfix before).

Ghabry avatar Jun 28 '25 15:06 Ghabry

Outside of the last point mentioned in #3069 , which has been explained in chat why it has not been adressed (tied to the path cache + not something that would be noticeable while playing normally), everything seems good from what I see 👍

Mimigris avatar Jun 29 '25 22:06 Mimigris