処理の流れを追ってみました。

＞変更した箇所は 514行目あたりからの処理で
＞public function getNewShipmentItem(Order $Order, Product $Product, ProductClass $ProductClass, $quantity)
上記の変更を加えられていますが、ここに処理が来た時点で、
$Shipping->getDeliveryFee()->getFee()に配送業者別の配達料金が1度セットされているようです。

それが原因で1回分送料が多いのでは無いでしょうか？

function registerPreOrder() {} の中で、

// お届け先情報を作成
$Order = $this->getNewShipping($Order, $Customer, $deliveries); ← この中でfunction setShippingDeliveryFee() {} を呼んで、その中で$Shipping->setDeliveryFee($deliveryFee);を処理しています。

// 受注明細情報、配送商品情報を作成
$Order = $this->getNewDetails($Order); ← この中でfunction getNewShipmentItem() {}を呼んでいます。

この辺を修正すれば、送料計算が上手くいくように思います。

----------------
株式会社シロハチ
■ECCUBE2系、3系構築カスタマイズご相談ください。
■EC-CUBE3マニュアル
■blog