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

Various fixes from my development branch #60

Merged
merged 8 commits into from
Sep 10, 2024

Conversation

mertenpopp
Copy link

No description provided.

Linux version: Need to read 'sabbel.dat' first for language settings

Also fixing DoesDirectoryExist() and using std::filesystem.
They were fine when I first implemented my new financial reports.
Re-added them now.
* Classic mission 04 now uses correct route utilization
    * Previously, even though boss said that routes must be 20% utilized, game would check for 20% plane utilization
* Addon mission 09: Fixed counting of Uhrig flights
    * Note that computer players always have and still are cheating in this mission
* First class mission 07: Only need to have two repaired planes, not all of them in case more than two were bought
* Evolution mission 02: Only need to have five planes with full safety upgrades, not all of them in case more than five were bought

Now in statistics menu:
* Number of freight jobs completed
* Income from freight jobs
* Total tons transported
@WizzardMaker
Copy link
Owner

Uhh nice!

I've got these points for the next release change list, correct my if I missed something:

Fixes:

  • Updated translation of various statistic texts
  • Classic mission 04 now uses correct route utilization
    • Previously, even though boss said that routes must be 20% utilized, game would check for 20% plane utilization
  • Addon mission 09: Fixed counting of Uhrig flights
    • Note that computer players always have and still are cheating in this mission
  • First class mission 07: Only need to have two repaired planes, not all of them in case more than two were bought
  • Evolution mission 02: Only need to have five planes with full safety upgrades, not all of them in case more than five were bought

Additions:

  • Added new statistics:
    • Number of freight jobs completed
    • Income from freight jobs
    • Total tons transported
  • Added various quick test modes for developers
    • /quick start argument
    • AUTORUN cheat code in game

Other parts like cmake changes are more developer oriented, so I omitted them from the list

@mertenpopp
Copy link
Author

Yes, looks good.

I will continue to cherry-pick my other changes.

My biggest feature is a new computer player, one that does not cheat and still usually outperforms the old computer player.
However, it does not work in network games yet and will break savegame compatibility.

Do you have working network play and compatibility with old savegames on your branch?

@WizzardMaker
Copy link
Owner

Network play should work, at least Direct IP.

Savegame compatibility depends on the version - everything past version 1.2 Hotfix 3 should work, but to be honest is not really tested

@WizzardMaker WizzardMaker merged commit 696e454 into WizzardMaker:develop Sep 10, 2024
5 checks passed
@mertenpopp mertenpopp deleted the fix_init branch September 29, 2024 19:51
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.

2 participants