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

Makes the upgrade/convert tool upgrade trains stationed in depots between incompatible rail types #6617

Closed
DorpsGek opened this issue Aug 31, 2017 · 1 comment
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

zegentzy opened the ticket and wrote:

Ever upgraded from rail to monorail (or manglevs)? Annoyed at having to recreate all your trains? I got the solution for you!

This patch allows the upgrading of depots containing parked trains. It also autoreplaces all trains in said depots.

Now you can:
1- Set autoreplace to upgrade rail engines/wagons to monorail (or manglave) engines/wagons.
2- Send all trains to a depot
3- Use the upgrade/convert tool on the depot!
4- Watch as all your trains are now monorail/manglev, no order cloning of 400 trains required!

Changes:
1- Made autoreplace gui show incompatible rail types - as far as I can tell trains don't upgrade to non-compatible rail types of the depot can't buy the rail type so this should have no effect on anything
2- Reworked Upgrade rails command to still upgrade depot tiles if they have trains on em' only if all trains are stationed in the depot. It then calls autoreplace on all those trains.

Attachments

Reported version: 1.7.1
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6617
@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) enhancement labels Apr 7, 2018
@andythenorth
Copy link
Contributor

Although cross-railtype upgrading is micro-management, eliminating it is not a current goal. Thanks for the patch, but we'll have to pass on this one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

2 participants