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

Change: Allow overbuilding station and waypoint tiles #10618

Merged
merged 1 commit into from Apr 16, 2023

Conversation

2TallTyler
Copy link
Member

@2TallTyler 2TallTyler commented Apr 9, 2023

Motivation / Problem

#9852 made overbuilding free, but blocked overbuilding without changing any tiles. This is very common when building stations using NewGRF station sets, where platforms can have different canopies, footbridges, etc., so having it blocked is frustrating.

Description

It doesn't hurt anybody to allow overbuilding, even if graphics are the same, so I just removed the error entirely. Costs are still calculated correctly, which was the point of #9852.

Rotating road stations is still only possible for bay stations, since drive-through stations will throw an error about the underlying road.

Limitations

Checklist for review

Some things are not automated, and forgotten often. This list is a reminder for the reviewers.

  • The bug fix is important enough to be backported? (label: 'backport requested')
  • This PR touches english.txt or translations? Check the guidelines
  • This PR affects the save game format? (label 'savegame upgrade')
  • This PR affects the GS/AI API? (label 'needs review: Script API')
    • ai_changelog.hpp, gs_changelog.hpp need updating.
    • The compatibility wrappers (compat_*.nut) need updating.
  • This PR affects the NewGRF API? (label 'needs review: NewGRF')

@2TallTyler 2TallTyler added backport requested This PR should be backport to current release (RC / stable) size: trivial This Pull Request is trivial labels Apr 9, 2023
@FLHerne
Copy link
Contributor

FLHerne commented Apr 9, 2023

It's explicitly intended by the author of #9852, so the commit message should perhaps be 'Change' rather than 'Fix'. I suspect it (or its consequences for custom stations) may have been overlooked by reviewers though.

If no new rail stations are built or rotated, the command returns a STR_ERROR_ALREADY_BUILT error.
If there already is a rail waypoint, the command returns a STR_ERROR_ALREADY_BUILT error.

I agree that allowing overbuilding is preferable, the current behaviour would be very inconvenient for building NewGRF stations.

@2TallTyler 2TallTyler changed the title Fix eabb9db: Allow overbuilding station and waypoint tiles Change: Allow overbuilding station and waypoint tiles Apr 9, 2023
@2TallTyler
Copy link
Member Author

Ah, good point. Commit title changed.

This will need backporting to 13.1 or we'll get tons of bug reports if we release without it. 😉

@glx22
Copy link
Contributor

glx22 commented Apr 10, 2023

No need to backport, the original change is only in master.

@2TallTyler 2TallTyler removed the backport requested This PR should be backport to current release (RC / stable) label Apr 10, 2023
@2TallTyler 2TallTyler enabled auto-merge (squash) April 14, 2023 13:56
@2TallTyler 2TallTyler merged commit bc44158 into OpenTTD:master Apr 16, 2023
19 checks passed
@2TallTyler 2TallTyler deleted the station-overbuild branch April 16, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: trivial This Pull Request is trivial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants