From b6962222ee5ae22b801131ce00a90a83b9c70d7d Mon Sep 17 00:00:00 2001 From: smatz Date: Wed, 3 Sep 2008 10:30:07 +0000 Subject: [PATCH] (svn r14234) -Fix: feeder share was computed wrong when splitting cargo packet --- src/cargopacket.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/cargopacket.cpp b/src/cargopacket.cpp index e50532376f..44c3dfbbe7 100644 --- a/src/cargopacket.cpp +++ b/src/cargopacket.cpp @@ -218,19 +218,21 @@ bool CargoList::MoveTo(CargoList *dest, uint count, CargoList::MoveToAction mta, /* Can move only part of the packet, so split it into two pieces */ if (mta != MTA_FINAL_DELIVERY) { CargoPacket *cp_new = new CargoPacket(); + + Money fs = cp->feeder_share * count / cp->count; + cp->feeder_share -= fs; + cp_new->source = cp->source; cp_new->source_xy = cp->source_xy; cp_new->loaded_at_xy = (mta == MTA_CARGO_LOAD) ? data : cp->loaded_at_xy; cp_new->days_in_transit = cp->days_in_transit; - cp_new->feeder_share = cp->feeder_share / count; + cp_new->feeder_share = fs; /* When cargo is moved into another vehicle you have *always* paid for it */ cp_new->paid_for = (mta == MTA_CARGO_LOAD) ? false : cp->paid_for; cp_new->count = count; dest->packets.push_back(cp_new); - - cp->feeder_share /= cp->count - count; } cp->count -= count;