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.

Sign up for GitHub

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

Jump to bottom

Loan inconsistencies between difficulty&economy #145

Closed
DorpsGek opened this issue Apr 29, 2006 · 2 comments
Closed

Loan inconsistencies between difficulty&economy #145

DorpsGek opened this issue Apr 29, 2006 · 2 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

Darkvater opened the ticket and wrote:

I was playing the Africa scenario where I tried to increase my loan (which was 100.000 pounds) but then found I couldn't because it was maxed out. However the difficulty window said the max loan was 300.000 pounds.

There is an inconsistency between the _economy struct (saved) that contains max_loan and is used for calculation and _opt/_opt_newgame that is used for setting this value and not saved. Presumably on using 'new game' the values are not copied over to the _economy struct? Play scenario works properly because there the Difficulty options are taken from the savegame (OPTS chunk).

This needs some clearing up for 0.4.8 imho.

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/145
@DorpsGek
Copy link
Member Author

glx wrote:

I didn't tried, but maybe adding StartupEconomy(); in StartScenario() (openttd.c:633) after _opt initialisation (line 661) can solve it.

Sorry I didn't made a diff.

This comment was imported from FlySpray: https://bugs.openttd.org/task/145#comment277

@DorpsGek
Copy link
Member Author

Darkvater closed the ticket.

Reason for closing: Fixed

r4947

This comment was imported from FlySpray: https://bugs.openttd.org/task/145

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek