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

All TMPE data silently corrupted/removed on any level load error #89

Closed
penma opened this issue Dec 25, 2017 · 2 comments
Closed

All TMPE data silently corrupted/removed on any level load error #89

penma opened this issue Dec 25, 2017 · 2 comments
Labels
enhancement A feature needs to be modified and improved

Comments

@penma
Copy link

penma commented Dec 25, 2017

When for any reason a saved game contains data but it cannot be loaded, it is silently ignored and destroyed after saving. This should obviously not happen and the user should be given some kind of notice. (The average user will probably not know what to do anyway when presented with any kind of error dialog, but just silently killing all data and letting user notice only days later is not a good thing either - this way they might at least not destroy their only copy of a savegame)

Seems code is already there, just the dialog box display commented out:

//UIView.library.ShowModal<ExceptionPanel>("ExceptionPanel").SetMessage("An error occurred while loading", "Traffic Manager: President Edition detected an error while loading. Please do NOT save this game under the old filename, otherwise your timed traffic lights, custom lane arrows, etc. are in danger. Instead, please navigate to http://steamcommunity.com/sharedfiles/filedetails/?id=583429740 and follow the steps under 'In case problems arise'.", true);

(noticed because for some reason a local build of current git master is unable to load data saved by current steam workshop version, don't know why, that's a different issue though)

@VictorPhilipp VictorPhilipp added the enhancement A feature needs to be modified and improved label Dec 30, 2017
@VictorPhilipp
Copy link
Owner

I disabled the message because it does not provide any value to the user. At the time it shows (during loading) damage has already been done (during saving). If it shows during saving then I would also have to provide countermeasures (e.g. skip saving). That in turn could lead to new problems where TMPE's internal state may become misaligned with the game state.

@VictorPhilipp
Copy link
Owner

moved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A feature needs to be modified and improved
Projects
None yet
Development

No branches or pull requests

2 participants