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

Fix #8093: Build+Refit changed game-state in command test run, and thus caused desyncs #8103

Merged
merged 3 commits into from
May 3, 2020
Merged

Fix #8093: Build+Refit changed game-state in command test run, and thus caused desyncs #8103

merged 3 commits into from
May 3, 2020

Commits on May 3, 2020

  1. Codechange: Unify the tests whether build+refit is in simulation-test… 

    … or real-run.
    @frosch123
    frosch123 committed May 3, 2020
    Configuration menu
    Copy the full SHA
    ad0362b View commit details
    Browse the repository at this point in the history

  2. Fix: When build+refit an engine, do not refit any free wagons that ma… 

    …y get attached.
    @frosch123
    frosch123 committed May 3, 2020
    Configuration menu
    Copy the full SHA
    8db81e8 View commit details
    Browse the repository at this point in the history

  3. Fix OpenTTD#8093: Build+Refit changed game-state in command test run,… 

    … and thus caused desyncs.

Use DC_AUTOREPLACE for actions that shall be reversibe, in this case:
- Do not rearrange free wagons in test-run.
- Do not discard OrderBackups.
The latter was not triggered by actual auto-replace, since it does not set a 'user'.
    @frosch123
    frosch123 committed May 3, 2020
    Configuration menu
    Copy the full SHA
    7727910 View commit details
    Browse the repository at this point in the history