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

Jump to bottom

Building drive through road stop on town-owned one-way road crashes game. #7626

Closed
James103 opened this issue Jun 18, 2019 · 5 comments
Closed

Building drive through road stop on town-owned one-way road crashes game. #7626

James103 opened this issue Jun 18, 2019 · 5 comments
Milestone
1.10.0

Comments

@James103
Copy link
Contributor

Version of OpenTTD

20190603-master-g846fc8fe09
(and probably any version after merge of PR #7547)

Expected result

The drive through road stop is built and the game does not crash.

Actual result

When you try to build a drive through road stop on any town-owned one-way road, it fails and crashes the game with Assertion failed at line 311 of d:\a\1\s\src\company_cmd.cpp: owner != OWNER_TOWN || tile != 0.
crash.zip

Steps to reproduce

  1. Load up a saved game/scenario with town-owned one-way roads, e.g. Cindini.
  2. Make sure that the game is unpaused or that "command_pause level" == 3 (Advanced Settings -> Limitations -> When paused allow: All actions)
  3. Make sure that "road_stop_on_town_road" == true (Advanced Settings -> Limitations -> Allow drive through stops on town owned roads: On)
  4. Try to build a road stop on a town-owned one-way road. If it would be successful (before the money check), then it instead crashes the game.
@EgyLynx
Copy link

EgyLynx commented Jun 24, 2019

Are you mean these where cam at vechiles? Are youre think at game "wait" vechiles at right/Left and when one side is not possible it isnt know how continue? So there should be message tells at it not work that way...

@James103
Copy link
Contributor Author

image

  1. Select one of the drive-through road stops, which looks like this (for buses). Note that this crash affects only the drive through road stops, not the road vehicle bays.
    image

  2. If the highlighted tile is owned by towns AND one-way, then the game will crash when building a drive through road stop there. In this case, both conditions are true.

  3. When building a drive through road stop under these conditions, the game crashes because one or more special cases weren't handled properly (in this case the special case of drive-through stop on town-owned road)

JGRennison added a commit to JGRennison/OpenTTD-patches that referenced this issue Jul 11, 2019
@JGRennison
Fix building of road stops on town-owned one-way roads
2164eb4 
See: OpenTTD/OpenTTD#7626
@Ben79487
Copy link

It's probably due to the fact that bus stops can't be one-way, therefore the game tries to build a one-way bus stop but can't.

@stormcone stormcone mentioned this issue Jul 12, 2019
Merged
@James103
Copy link
Contributor Author

Still crashes in 20190723-master-g2e686ad5d5. Crash files have been attached.
Why haven't the PR fixing this crash, #7627, been merged already? It's been more than a month since the PR was opened, and 3 and a half weeks since it was approved to merge.

@LordAro
Copy link
Member

LordAro commented Jul 30, 2019 •
edited

Because there are some questions about the fix. I should've removed the review, tbh

@nielsmh nielsmh added this to the 1.10.0 milestone Sep 1, 2019
@nielsmh nielsmh closed this as completed in 2d9eb1c Sep 6, 2019
nielsmh pushed a commit to nielsmh/OpenTTD that referenced this issue Sep 7, 2019
@ddm999 @nielsmh
Fix OpenTTD#7626: Allow building drive-through stops over one-way/blo…
c6df08a 
…cked roads owned by towns (instead of crashing).
LordAro pushed a commit that referenced this issue Sep 7, 2019
@ddm999 @LordAro
Fix #7626: Allow building drive-through stops over one-way/blocked ro…
da086a3 
…ads owned by towns (instead of crashing).
douiwby pushed a commit to douiwby/OpenTTD that referenced this issue Apr 16, 2020
@ddm999 @douiwby
Fix OpenTTD#7626: Allow building drive-through stops over one-way/blo…
d6d4444 
…cked roads owned by towns (instead of crashing).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.10.0
Development

No branches or pull requests
5 participants
@nielsmh @LordAro @EgyLynx @James103 @Ben79487