Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vehicle owner gets paid for whole cargo feeder share #2427

Closed
DorpsGek opened this issue Nov 23, 2008 · 6 comments
Closed

Vehicle owner gets paid for whole cargo feeder share #2427

DorpsGek opened this issue Nov 23, 2008 · 6 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

yorick opened the ticket and wrote:

If companies transfer cargo to a station that can be accessed by other companies(oil rigs, also with the infrastructure sharing patch), and someone else delivers the cargo, the one delivering the cargo will get the whole feeder share, and not the other companies. This is currently not quite common, but with IS or cargodest, it will be...

The attached patch fixes the problem, but uses a lot of memory space.

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/2427
@DorpsGek
Copy link
Member Author

yorick wrote:

Wrong savegame version...

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427#comment5059

@DorpsGek
Copy link
Member Author

Rubidium wrote:

This patch only replaces this problem with: last vehicle owner has to pay for delivery if his/her leg is slower than the legs before the final leg.
So the patch only makes it worse as in the current case the first legs might not get paid but in the new version the last leg might be paying a hell of a lot due to the feeder share, e.g. A moves cargo from south-east to north-west and B (accidentally) moves it back to south-east. In that case B has to pay the 'revenue' of A and it's running cost, whereas in the current system A only pays it's running costs which is (significantly) less than feeder share + running cost.

So the only real solution I see is disallowing transferring onto oil rigs (or other owner none stations).


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427#comment5065

@DorpsGek
Copy link
Member Author

yorick wrote:

"So the only real solution I see is disallowing transferring onto oil rigs (or other owner none stations)." <-- what if cargotest/IS would hit the trunk?


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427#comment5066

@DorpsGek
Copy link
Member Author

Swallow wrote:

As a possible solution, the distance each company moves the cargo could be stored. Final payment would then be divided between all companies, with each of them receiving a share proportional to the transported distance.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427#comment5067

@DorpsGek
Copy link
Member Author

Rubidium wrote:

An option is that upon unloading at a shared station the feedershare paid for the cargo is added to the 'balance' of the company owning the vehicle. Then once it's loaded again that feedershare is substracted from the 'balance'. This would account the different legs to the companies that made them and the final delivering company must 'just' assure that it only picks up cargo that's going to be profitable for him. This way you do not need the extra 100+ bytes in the cargopacket.


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427#comment7037

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Won't fix

for now; especially because it's primarily an infrastructure sharing issue


This comment was imported from FlySpray: https://bugs.openttd.org/task/2427

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

1 participant