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

Lenient road stop removal #6262

Closed
DorpsGek opened this issue Mar 22, 2015 · 2 comments
Closed

Lenient road stop removal #6262

DorpsGek opened this issue Mar 22, 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

adf88 opened the ticket and wrote:

This patch makes the road stop removal tool behave lenient.

Currently the tool throws an error if any road stop within the dragged area fails to be removed. It can be a road stop of other company, a road stop with a vehicle inside etc. The patch modifies this behaviour. If there are any road stops that can be removed, they will be removed despite of any other road stops that couldn't be removed.

Note - ai/script backward compatibility is preserved. Script front-ends use the removal command only with single-tile areas. For single-tile areas the behaviour of the command is not altered.

I'm attaching two variants of the patch. The first one can be applied over clean trunk. The second one can be applied over #6252

Attachments

Reported version: trunk
Operating system: All


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

adf88 wrote:

Note - rail station removal tool is already "lenient".


This comment was imported from FlySpray: https://bugs.openttd.org/task/6262#comment13834

@DorpsGek
Copy link
Member Author

planetmaker closed the ticket.

Reason for closing: Implemented

In r27225. Thanks for the patch :)


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

@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
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

1 participant