Patch: change finances window grouping

Forum for technical discussions regarding development. If you have a general suggestion, problem or comment, please use one of the other forums.

Moderator: OpenTTD Developers

Post Reply
jawsper
Engineer
Engineer
Posts: 20
Joined: 08 Jun 2004 00:02
Location: Netherlands

Patch: change finances window grouping

Post by jawsper »

Hi all this is my first finished and working patch :D

To the point: I wasn't really happy with the way the new grouping works in the finances window, I'd want it like this:
Image

So I set off to work and made it work.

Here is what I did to the settings window: http://hostmysite.nl/Jawsper%20Inc.,%202581-02-24_1.png

Changed:
  • Added 3 entries to the language file
  • Added new list format to finances gui
  • Changed the setting "gui.expenses_layout" from a bool to a var. You probably need to change/remove that line from openttd.cfg to make this work!
I hope you like my first contribution. :mrgreen:

Also hope i made the patch files work properly :)


Update 5th of may 2009: Updated the file to one diff and made it r16183

Update 28th of october 2009 (wow it's been that long!): Updated to r17895
Attachments
finances_grouping_r17895.diff
Patch for r17895
(3.71 KiB) Downloaded 215 times
Last edited by jawsper on 28 Oct 2009 16:43, edited 3 times in total.
Patch: change finances window grouping
Patch: Tracking viewport

Warning: Unexpected end of file reached.
User avatar
CommanderZ
Tycoon
Tycoon
Posts: 1872
Joined: 07 Apr 2008 18:29
Location: Czech Republic
Contact:

Re: Patch: change finances window grouping

Post by CommanderZ »

This makes sense :)

Btw, you should put all the changes into one patch file. One diff can patch multiple source files.


And welcome to TT-Forums :]
jawsper
Engineer
Engineer
Posts: 20
Joined: 08 Jun 2004 00:02
Location: Netherlands

Re: Patch: change finances window grouping

Post by jawsper »

Well I tried to make one diff file, but Tortoise didn't want me to create a patch on a folder, so I made 3 patches and modified them to the right directory. I was going to merge the 3 diffs, but I was too afraid to break them :D


EDIT: Moved to first post
Patch: change finances window grouping
Patch: Tracking viewport

Warning: Unexpected end of file reached.
Noldo
Engineer
Engineer
Posts: 75
Joined: 16 Jun 2005 13:17
Location: Lappeenranta, Finland

Re: Patch: change finances window grouping

Post by Noldo »

I love this!

Another subtotal for operating profit and it's perfect.
User avatar
ever
Traffic Manager
Traffic Manager
Posts: 164
Joined: 26 Apr 2009 11:45

Re: Patch: change finances window grouping

Post by ever »

Well someone is obviously not an accountant.
jawsper
Engineer
Engineer
Posts: 20
Joined: 08 Jun 2004 00:02
Location: Netherlands

Re: Patch: change finances window grouping

Post by jawsper »

I'd add operating profit, but I have no clue what that is :)
Patch: change finances window grouping
Patch: Tracking viewport

Warning: Unexpected end of file reached.
User avatar
FooBar
Tycoon
Tycoon
Posts: 6553
Joined: 21 May 2007 11:47
Location: The Netherlands
Contact:

Re: Patch: change finances window grouping

Post by FooBar »

The "Operating Profit" is IMO already available: it's the last grand total...

I like this setup of the finance window!
Noldo
Engineer
Engineer
Posts: 75
Joined: 16 Jun 2005 13:17
Location: Lappeenranta, Finland

Re: Patch: change finances window grouping

Post by Noldo »

Always great to use the wrong terms. Anyway the subtotal that I would like to follow would include everything else but not Construction, New Vehicles and Loan interest (and maybe Other). So the incomes, running costs and property maintenance would be included.

This number would tell how much money is coming in without player intervention.
Also the effect of player action in from contructing and new vehicles is divided somewhat randomly between years so the big total doesn't really tell that much about how the company did on a specific year.
jawsper
Engineer
Engineer
Posts: 20
Joined: 08 Jun 2004 00:02
Location: Netherlands

Re: Patch: change finances window grouping

Post by jawsper »

I hope this is what you meant with operational profit :D

Also updated to r16410.

Will move this patch to first post after I decide what to actually call the selection "operational profit" :)
Attachments
finances_grouping_v3_r16410.diff
(4.01 KiB) Downloaded 253 times
Jawsper Inc., 2675-03-01.png
Jawsper Inc., 2675-03-01.png (64.29 KiB) Viewed 5050 times
Patch: change finances window grouping
Patch: Tracking viewport

Warning: Unexpected end of file reached.
Wasila
Tycoon
Tycoon
Posts: 1498
Joined: 15 Mar 2008 07:02

Re: Patch: change finances window grouping

Post by Wasila »

I don't konw if this is what he meant..but what about turnover? Turnover = overall money made before any expenses, even running costs.
jawsper
Engineer
Engineer
Posts: 20
Joined: 08 Jun 2004 00:02
Location: Netherlands

Re: Patch: change finances window grouping

Post by jawsper »

Updated to r17895 in first post :)

Edit: whoops! That last patch was messy, updated it :)
Patch: change finances window grouping
Patch: Tracking viewport

Warning: Unexpected end of file reached.
User avatar
Lord Aro
Tycoon
Tycoon
Posts: 2369
Joined: 25 Jun 2009 16:42
Location: Location, Location
Contact:

Re: Patch: change finances window grouping

Post by Lord Aro »

yes i know it's a bump :shock: but is it me or did this get added to trunk?
AroAI - A really feeble attempt at an AI

It is practically impossible to teach good programming to students that have had a prior exposure to BASIC: as potential programmers they are mentally mutilated beyond hope of regeneration. --Edsger Dijkstra
petert
Tycoon
Tycoon
Posts: 3008
Joined: 02 Apr 2009 22:43
Location: Massachusetts, USA

Re: Patch: change finances window grouping

Post by petert »

Lord Aro wrote:yes i know it's a bump :shock: but is it me or did this get added to trunk?
I can see why you are confused. There is already a group finances, but this patch changes the settings there. Look at this link:
User avatar
Lord Aro
Tycoon
Tycoon
Posts: 2369
Joined: 25 Jun 2009 16:42
Location: Location, Location
Contact:

Re: Patch: change finances window grouping

Post by Lord Aro »

i see. i didn't look at the pictures close enough

(please shoot me :oops: )
AroAI - A really feeble attempt at an AI

It is practically impossible to teach good programming to students that have had a prior exposure to BASIC: as potential programmers they are mentally mutilated beyond hope of regeneration. --Edsger Dijkstra
Post Reply

Return to “OpenTTD Development”

Who is online

Users browsing this forum: No registered users and 11 guests