The International Simutrans Forum

Simutrans Extended => Simutrans-Extended bug reports => Simutrans-Extended development => Simutrans-Extended closed bug reports => Topic started by: Bittereinder on February 17, 2024, 09:17:59 AM

Title: [BUG] Crash when using "Upgrade"
Post by: Bittereinder on February 17, 2024, 09:17:59 AM
Hello,

Version used:

- Latest nightly build 17/02/2024
- Pakset: Pak128.Britain-Ex

Reproduction:

- Go to "Line Management"
-> Select your Line
-> "Replace vehicles"
-> Select "Upgrade" in the Vehicles drop down lis
-> Crash to desktop.

Video: https://www.youtube.com/watch?v=BdcaGJbszA4

I had the same issues with trains as with buses.

This does NOT occur when sending the individual train to the depot manually and using "Upgrade" in the depot.


Title: Re: [BUG] Crash when using "Upgrade"
Post by: RESTRICTED ACCOUNT on February 18, 2024, 12:18:09 PM
Thank you for reporting the bug and apologize for the inconvenience.
However, it will definitely improve the quality of the game.

On first impression I wonder if this dialog should have sell options. ???
I think it is necessary to review related processing...
Title: Re: [BUG] Crash when using "Upgrade"
Post by: RESTRICTED ACCOUNT on February 18, 2024, 02:40:58 PM
I fixed the crashing bug. Please check pull request #637.

Also, unrelated to this issue, I have confirmed that the upgrade operation in the replace dialog is broken, and the upgrade will not be performed correctly.

I need to look into this.

EDIT:
Also a bug where upgrade was not performed correctly in the replace dialog has been fixed.
However, strangely it requires two clicks. For some reason the first click doesn't respond....
Title: Re: [BUG] Crash when using "Upgrade"
Post by: Bittereinder on February 18, 2024, 03:29:47 PM
Quote from: Ranran(retired) on February 18, 2024, 02:40:58 PMI fixed the crashing bug. Please check pull request #637.

Also, unrelated to this issue, I have confirmed that the upgrade operation in the replace dialog is broken, and the upgrade will not be performed correctly.

I need to look into this.

EDIT:
Also a bug where upgrade was not performed correctly in the replace dialog has been fixed.
However, strangely it requires two clicks. For some reason the first click doesn't respond....

This is my first time bug reporting.

1) How do I test this? Do I have to wait for tonight's nightly build?

2) I have also noticed that the upgrade icon was not disappearing after upgrading a VEHICLE (bus) through the Line Management window. Perhaps this is the unrelated bug you are talking about. As far as I understand the game, this is the most up to date version of the bus in my 1925 game:

(https://i.imgur.com/BhXKjbN.png)
Title: Re: [BUG] Crash when using "Upgrade"
Post by: jamespetts on February 25, 2024, 03:15:46 PM
Excellent, thank you for (1) the report; and (2) the fix. Now incorporated.