Page 1 of 1

[PATCH] Life time profit on vehicle windows

Posted: 29 Mar 2015 11:38
by burty
I've been thinking about this one for a while.

There is now a lifetime profit value against vehicles which can be viewed alongside the profit this year and profit last year.

V1
[*] Stores lifetime profit (gets updated at end of year)
[*] Displays on the vehicle info window where the other profits are shown.

This currently isn't saved into the save games.

Re: [PATCH] Life time profit on vehicle windows

Posted: 05 Apr 2015 18:00
by burty
V2 attached.

This now saves the lifetime profit in to the save game and loads it.

Re: [PATCH] Life time profit on vehicle windows

Posted: 08 Apr 2015 20:58
by Wahazar
Good idea, I always wondered why there is lack of such information.
I'm using first patch (for game/multiplayer compatibility), hope that it will be in trunk, combined with group info :)

BTW, how profit_lifetime variable is initialized?

[PATCH] Life time profit on vehicle windows

Posted: 08 Apr 2015 21:15
by burty
Ahh... Yeah I just realised that on new game it doesn't get initialised :S
I don't know How I missed that!

I'm glad you like it, I hope it gets in trunk as well. Ive written another small patch that meddles with the share system and dividends :)


Sent from my iPhone using Tapatalk

Re: [PATCH] Life time profit on vehicle windows

Posted: 10 Apr 2015 20:05
by xarick
Hi. Can it also be added to the Sort by when listing vehicles? That would be awesome.

Re: [PATCH] Life time profit on vehicle windows

Posted: 11 Apr 2015 01:07
by burty
I don't see why not. I shall look in to it in the week


Sent from my iPhone using Tapatalk

Re: [PATCH] Life time profit on vehicle windows

Posted: 26 Dec 2015 13:17
by burty
V4
[*] NEW: Option to use Lifetime instead of last years profit for news alert.

V3
[*] New: Sort by lifetime profit in vehicle lists.

Re: [PATCH] Life time profit on vehicle windows

Posted: 01 Jan 2016 14:07
by JGR
I've made a few slight tweaks to version 3 of this patch.
  • Display the current lifetime profit, instead of the lifetime profit at the start of the current year
  • Change the lifetime profit string format to avoid breaking translations
The update in version 4 is not multiplayer safe, as the AI event for unprofitable vehicles depends on the current client display setting for which profit to use.
I'm not sure that it makes sense in general for profit warnings to use the lifetime profit, as that effectively means that older vehicles which later become unprofitable will not show a warning.

Edit: updated.