The International Simutrans Forum

Simutrans Extended => Simutrans-Extended bug reports => Simutrans-Extended development => Simutrans-Extended closed bug reports => Topic started by: Rollmaterial on September 16, 2017, 03:42:00 PM

Title: Inserting is broken
Post by: Rollmaterial on September 16, 2017, 03:42:00 PM
A vehicle other than one already present in the convoy cannot be inserted in front.
Title: Re: Inserting is broken
Post by: jamespetts on September 16, 2017, 05:52:46 PM
Thank you for the report. I believe that this may well arise from recent UI changes in the depot window - Ves: would you be able to look into this?
Title: Re: Inserting is broken
Post by: Ves on September 16, 2017, 10:41:02 PM
Hmm, are you sure?
I just tested on my gui-classes branch as well as the passenger-and-mail-classes on James Github, and there it does work. I do note, however, that some constraints are quite restrictive, so you would for instance need to remove the locomotive etc for any other vehicles to appear.

It do, however, crash for me in random occasions when I mess around with it! :o
It appears also on the latest passenger-and-mail-classes on your Github, James, and to relive the crash, do the following:

* Start this game: http://simutrans-germany.com/files/upload/classes_GUI_vehicle_tests_2.sve (http://simutrans-germany.com/files/upload/classes_GUI_vehicle_tests_2.sve) using passenger-and-mail-classes on your Github.
* On the station in the view, click on the train on the leftmost track and click "replace".
* Click on different passenger vehicles to append (or insert, if you want, doesnt matter) and you should find that it crashes.
Title: Re: Inserting is broken
Post by: Rollmaterial on September 16, 2017, 10:49:04 PM
Removing the engine doesn't work on the master branch. I only get vehicles present in the convoy.
Title: Re: Inserting is broken
Post by: jamespetts on September 16, 2017, 11:09:06 PM
This is a master branch issue, and I suspect that the issue is caused by the changes in gui-misc; Ves, can you test this on the master branch and see whether you can reproduce/fix that issue there?
Title: Re: Inserting is broken
Post by: Ves on September 17, 2017, 12:50:15 AM
Apologizes for this! I must have disturbed something when combining the comboboxes. I will have to look more tomorrow.

However, James, did you have a look on the issue I wrote?
Title: Re: Inserting is broken
Post by: jamespetts on September 17, 2017, 09:18:34 AM
I have not had a chance to look at the passenger-and-mail-classes branch yet as I am focussing on master branch fixes at present, but will have a look at that when I switch the branch back (switching branches takes a non-trivial amount of time, so I prefer to do a chunk of work on one branch before looking at the other).
Title: Re: Inserting is broken
Post by: jamespetts on September 18, 2017, 11:14:06 PM
I have now fixed the insertion failure: I should be grateful if anyone could confirm that the fix is working with the next nightly build (with a date of the 19th of September 2017 or later).
Title: Re: Inserting is broken
Post by: Ves on September 19, 2017, 04:54:00 AM
Oh, thanks! I did not have time yet to fix it. And again, apologies for creating the bug in the first place!