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

Buy menu - refittable label can be confusing for articulated vehicle. #6113

Closed
DorpsGek opened this issue Sep 14, 2014 · 3 comments
Closed
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

andythenorth opened the ticket and wrote:

For articulated vehicles, buy menu shows capacity of each sub-vehicle as a list.
However '(refittable)' label is shown on first item of list, even if first sub-vehicle is not refittable (and subsequent sub-vehicles are).

Reported as an Iron Horse bug by Alberth. https://dev.openttdcoop.org/issues/7109
Might be specific to how Iron Horse is handling cargo on articulated vehicles, but I couldn't see anything obvious I could do differently.

To reproduce: get Iron Horse from Bananas, have a look at Combine Car in buy menu.

Reported version: Version?
Operating system: All


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

andythenorth wrote:

Screenshot...

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6113#comment13522

@DorpsGek
Copy link
Member Author

mb wrote:

Yes, that´s a bug. The (refittable) label is always shown for the first part vehicle even if that one´s not refittable, but only subsequent parts are.

Interestingly, the refittability lists seems to differ in different versions of OTTD, see screenshots. Mine is a 2-part vehicle, 1st part passengers only, 2nd part refittable to mail and valuables.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6113#comment13523

@DorpsGek
Copy link
Member Author

Alberth closed the ticket.

Reason for closing: Fixed

in r26849


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

@DorpsGek DorpsGek added flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) Vehicles 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/)
Projects
None yet
Development

No branches or pull requests

1 participant