Bug in vehicle window

[wipi35]

As can be seen in the appended screenshot the train is 100% loaded, but it says: 0/140 boxes of piece goods loaded.
These 140 boxes appear in the category refrigerated goods.
Using R2500 with Pak 128-579

http://simutrans-germany.com/files/upload/simscr13.png

[gerw]

Same as http://forum.simutrans.com/index.php?topic=2560.0, but it seems that this bug isn't fixed. Can you provide a savegame?

[wipi35]

Can you provide a savegame?

I tried to upload at simutrans-germany.com but without success.
Don't understand why, savegame is 14MB, so smaller than max. filesize (20MB)

[wipi35]

R2500 is from before June 27, so according to Prissi it should be fixed.

[Zeno]

I did report it in february, and it was solved few days later, according to this post (seems to be same problem):
http://forum.simutrans.com/index.php?topic=1478.0

[gerw]

The patch of this old topic was applied in r2363, so the bug still exists.

[prissi]

Well, I need a savegame, because in my tests it did work as intended.

[wipi35]

As I said, I tried to upload but without success. Is max. size of 20MB correct, the biggest file I see is 2.83 MB.
In the meantime I'm now using R2568 with Pak128-648 and I can't reproduce it anymore. So may be solved after all? I'll keep an eye on it.

[gerw]

Imho, the relevant code for this bug didn't changed since r2500. Is this version in the first post correct?

[wipi35]

Is this version in the first post correct?

Sure, it is.

[gerw]

I've found the problem:

Have a look at freight_list_sorter_t::sort_freight.

- full_list is sorted by simconvoi::get_freight_info in a very fancy way: by the smallest index a freight in this category has.
- but compare_ware sorts only by the index of the freights
=> wrong display.

With the patch, both sort according to the catg_index (which is unique, even for special freights). Also a savegame for testing purposes is attached.

http://www-user.tu-chemnitz.de/~gerw/patches/pics/convoi_info.sve
http://www-user.tu-chemnitz.de/~gerw/patches/convoi_info.patch

[wipi35]

I've found the problem

My compliments!