[Bug] livery scheme selector list ignores intro month

[Ranran]

　　　　　(´・ω・｀)Hello.

┏━━━━━━━━━━━━━━━━━━━━┓
┃Ranran appears! らんらんがあらわれた！　.┃
┃　　　　　　　　　▼　　　　　　　　　　┃
┗━━━━━━━━━━━━━━━━━━━━┛

I found that livery scheme selector ignores the value of intro_month and creates a list by checking only intro_year.
Although there are options, you have to wait until the actual month of appearance, but the player can not understand anything because we can't check the actual month of livery scheme appearance in the game.

[freddyhayward]

The other issue is that there is no indication as to which livery schemes are applicable to a convoy, leading to endless scrolling through the entire list. The system is virtually inaccessible to new players, and frustrating for experienced players. If I am trying to paint a bus, why are hovertravel and ffestinog railway options?

[Spenk009]

May I suggest that while this receives attention, that use_timeline also acts on the selection disables the timeline of liveries and enables all? It would make pakset work easier and improve control for players wishing to build outside of timelines.

[Vladki]

1. the livery selector combo is obverlapping with list of stops and is thus not usable for direct choose, but only for scrolling
2. it should be limited to liveries that are used by vehicles using the line
3. when picking vehicles in depot, info about available liveries would be very nice.

When speaking about timeline, it would be nice to be able to turn off the timeline only for liveries. However in that case it would be necessary to show not only the livery scheme (like LMS/GWR/BR/Blackpool/Sheffield/...) but a specific livery (BR-Green, BR-Blue, ...). This would be extremely handy in multiplayer games, where players could use liveries to distinguish from other players.

[Ranran]

I have not yet found the cause of this bug. I suspect that livery scheme list has been updated in the processing of new_year or that the value of intro_month has not been gotten correctly.

The other issue is that there is no indication as to which livery schemes are applicable to a convoy, leading to endless scrolling through the entire list. The system is virtually inaccessible to new players, and frustrating for experienced players. If I am trying to paint a bus, why are hovertravel and ffestinog railway options?

This patch will reduce some annoyance.
I am currently planning to make a patch that like: eliminates the name of the airplane company from inside the bus depot.

[Ranran]

Subsequent research has shown that the intro month of the "livery" is completely ignored. As a result, the intro month of the "livery scheme" is also ignored. In other words, all intro_month is 1 which is the default value.

For example, according to simuconf.tab and dat settings, if you apply British-Rail for LNER-A1 in 1950, the BR-Early livery will be applied like this:

and it should be green in November 1951, but in January 1951 is already green.

Don't jump the gun! 

[Ranran]

I think I have identified and fixed the issue where the intro_month for liveries was not registered correctly. This was a simple mistake.
https://github.com/Ranran-the-JuicyPork/simutrans-extended/commit/31b1a57167044f887dee08e3542978228dc3cf97

Please confirm it.

[jamespetts]

My apologies: I have realised that I omitted to respond to this thread. I believe that this is now working. Thank you for your fix.