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

Canal on River #6277

Closed
DorpsGek opened this issue Apr 7, 2015 · 2 comments
Closed

Canal on River #6277

DorpsGek opened this issue Apr 7, 2015 · 2 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Apr 7, 2015

Samu opened the ticket and wrote:

This patch restores rivers when demolishing canals that have been built on river tiles. Demolishing them again will actually clear the river tiles.

- A behaviour was changed: Leveling land with rivers or lakes will not immediately demolish them, they must be cleared first.
- A better error message was added to complement the behaviour change.
- Store canal on river bit flag on m1 bit 7 for convenience.
- Moved completed industry flag from m1 bit 7 to m1 bit 4, and added a conversion for old savegames.
- Updated the documentation regarding these changes.

Attachments

Reported version: trunk
Operating system: Windows


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

DorpsGek commented Apr 8, 2015

Samu wrote:

There was a problem with v5, I was setting canal on river too early for industries, and that was causing assertion. This fixes it.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6277#comment13860

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 7, 2018
@andythenorth
Copy link
Contributor

I think we'll pass on this one, thanks.

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

No branches or pull requests

2 participants