Page 1 of 3

Patch: Filter by cargo [in trunk]

Posted: 19 Apr 2008 16:56
by gule
- in trunk, thanks to frosch and srb

- updated to r14564

- updated to r13085

Hello!

I split my patch Build and refit, into Filter by cargo and Build and refit. Here is only filter part of the patch.
You can filter vehicles by cargo that they can transport.

Re: Patch: Filter by cargo [r12791]

Posted: 19 Apr 2008 17:11
by Digitalfox
Hey this is cool :mrgreen:

So what happens when theres wagons that support multiple cargos ( can't compile right now ) ?

Nice work :D

Re: Patch: Filter by cargo [r12791]

Posted: 19 Apr 2008 17:20
by gule
Digitalfox wrote:So what happens when theres wagons that support multiple cargos ( can't compile right now ) ?
When you select cargo you see all wagons which support this cargo, as default cargo or they can be refited to. So if wagon support coal and goods, you will see this wagon in coal filter and goods filter.

Re: Patch: Filter by cargo [r12791]

Posted: 19 Apr 2008 23:14
by cmoiromain
this patch looks pretty cool and really useful for me! Since I don't know how to compile, I'll wait until it gets in an IN build by someone (this shouldn't take long, when we see how many there are ATM...)

Re: Patch: Filter by cargo [r12791]

Posted: 19 Apr 2008 23:26
by rbenevid
This patch is so useful...... hope it gets in the trunk soon.

Re: Patch: Filter by cargo [r12791]

Posted: 20 Apr 2008 03:43
by leno4
this is ownage thanks.

Re: Patch: Filter by cargo [r12791]

Posted: 20 Apr 2008 08:20
by Roujin
trunk plx :D

Re: Patch: Filter by cargo [r12791]

Posted: 20 Apr 2008 08:43
by Trond
Yes, trunk plx! :wink:

Re: Patch: Filter by cargo [r12791]

Posted: 12 May 2008 11:44
by Ar4i
This one very useful feature with all those train and industry packs. Especially given the fact that some trainsets would actually continuously add/remove different wagons through the whole game and would provide 2-3-4 wagons for one type of cargo.

Re: Patch: Filter by cargo [r12791]

Posted: 12 May 2008 12:20
by Timitry
And it is especially useful as now you can have different trainsets loaded at once, and as there are wagons for each trainset it sometimes gets quite confusing ;-)

Re: Patch: Filter by cargo [r12791]

Posted: 12 May 2008 12:55
by zombie
Hi.

Thanks for this great patch. I attached an updated patch file.

Kind regards

Zombie

Re: Patch: Filter by cargo [r12791]

Posted: 12 May 2008 15:26
by zypa
I would really love to see this patch in the trunk.
Is there any chance for that?

Re: Patch: Filter by cargo [r12791]

Posted: 13 May 2008 07:24
by Ammler
Heya

Is this patch extendable for i.e also filtering for the different GRF sets, since trunk allowes dynamic vehicle pool? (Dunno, if there is more, maybe you could split it to a Filterframework and then add different types of filter?)

Greets
Ammler

Re: Patch: Filter by cargo [r12791]

Posted: 13 May 2008 22:00
by zombie
Hi.

Can anyone please provide a recent patch file? The shopping dialog has been transformed into a class and I'm failing doing it myself. My knowledge of C/C++ is kind of limited.

Kind regards

Zombie

Re: Patch: Filter by cargo [r13085]

Posted: 14 May 2008 13:02
by gule
zombie wrote:Can anyone please provide a recent patch file?
Updated version in first post.

Re: Patch: Filter by cargo [r13085]

Posted: 03 Jun 2008 13:18
by Tiberius
Update to r13374 for anyone who's interested in it :D

EDIT: change to svn-diff style, the code itself isn't altered.

Re: Patch: Filter by cargo [r13085]

Posted: 08 Jun 2008 10:11
by planetmaker
I like this patch as it avoids browsing through dozens of not needed wagons. Bonus: it's client only :P

A nice addition might be to sort engines, too: universal service, freight only & pax only.

Re: Patch: Filter by cargo [r13085]

Posted: 08 Jun 2008 11:18
by zypa
This is a _really_ useful patch.
Is there a chance that it will be included in the ottd trunk?

Re: Patch: Filter by cargo [r13085]

Posted: 11 Jul 2008 15:01
by Tiberius
Fixed a nasty negative array subscription, that crashes VS2008 optimized binary.

Re: Patch: Filter by cargo [r13085]

Posted: 14 Jul 2008 22:10
by planetmaker
small bump to make it apply again to trunk