Gambio 2007 verrechnet sich bei Versandkosten in Fremdwährung (USD)

[Elektrofuzzy]

Hallo Leute,

ich bin hier auf einen Bug gestoßen, der sich wie folgt äußert:
Kunde hat bei uns die Wahl, ob er in USD oder Euro bestellt. Stellt er auf USD um, kommt er in der Bestellabwicklung hier hin:

http://www.ineltron.de/Display/Grafik1.jpg

Auf den ersten Blick erkennbar: Die Summe stimmt nicht. Der Shop zeigt zwar alle Einzel-Beträge richtig an. Aber letztlich rechnet er in der Endsumme mit den Frachtkosten auf Eurobasis (diese betragen bei uns 10,12 Euro). Auf USD-Basis müsste eigentlich 31,53 Dollar rauskommen.
Scheint mir nur ein kleiner Bug, aber ich such mir hier einen Wolf: Wo genau rechnet Gambio 2007 (ist noch sehr nah am ursprünglichen xt:c) die Summe aus, die da angezeigt wird?

Heißer Kandidat in meinen Augen ist /shop/includes/modules/order_total/ot_shipping.php

Da wäre der Quelltext folgender:


<?php
/* -----------------------------------------------------------------------------------------
$Id: ot_shipping.php 1002 2005-07-10 16:11:37Z mz $

XT-Commerce - community made shopping
http://www.xt-commerce.com

Copyright (c) 2003 XT-Commerce
-----------------------------------------------------------------------------------------
based on:
(c) 2000-2001 The Exchange Project (earlier name of osCommerce)
(c) 2002-2003 osCommerce(ot_shipping.php,v 1.15 2003/02/07); www.oscommerce.com
(c) 2003 nextcommerce (ot_shipping.php,v 1.13 2003/08/24); www.nextcommerce.org

Released under the GNU General Public License
---------------------------------------------------------------------------------------*/

class ot_shipping {
var $title, $output;

function ot_shipping() {
global $xtPrice;
$this->code = 'ot_shipping';
$this->title = MODULE_ORDER_TOTAL_SHIPPING_TITLE;
$this->description = MODULE_ORDER_TOTAL_SHIPPING_DESCRIPTION;
$this->enabled = ((MODULE_ORDER_TOTAL_SHIPPING_STATUS == 'true') ? true : false);
$this->sort_order = MODULE_ORDER_TOTAL_SHIPPING_SORT_ORDER;


$this->output = array();
}

function process() {
global $order, $xtPrice;

if (MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING == 'true') {
switch (MODULE_ORDER_TOTAL_SHIPPING_DESTINATION) {
case 'national':
if ($order->delivery['country_id'] == STORE_COUNTRY) $pass = true; break;
case 'international':
if ($order->delivery['country_id'] != STORE_COUNTRY) $pass = true; break;
case 'both':
$pass = true; break;
default:
$pass = false; break;
}

if ( ($pass == true) && ( ($order->info['total'] - $order->info['shipping_cost']) >= $xtPrice->xtcFormat(MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPIN G_OVER,false,0,true)) ) {
$order->info['shipping_method'] = $this->title;
$order->info['total'] -= $order->info['shipping_cost'];
$order->info['shipping_cost'] = 0;
}
}

$module = substr($_SESSION['shipping']['id'], 0, strpos($_SESSION['shipping']['id'], '_'));

if (xtc_not_null($order->info['shipping_method'])) {
if ($_SESSION['customers_status']['customers_status_show_price_tax'] == 1) {
// price with tax

$shipping_tax = xtc_get_tax_rate($GLOBALS[$module]->tax_class, $order->delivery['country']['id'], $order->delivery['zone_id']);
$shipping_tax_description = xtc_get_tax_description($GLOBALS[$module]->tax_class, $order->delivery['country']['id'], $order->delivery['zone_id']);

//nc_bugfix BOF

//removed $tax = $xtPrice->xtcFormat(xtc_add_tax($order->info['shipping_cost'], $shipping_tax),false,0,false)-$order->info['shipping_cost'];
$tax = xtc_add_tax($order->info['shipping_cost'], $shipping_tax) - $order->info['shipping_cost'];
//removed $tax = $xtPrice->xtcFormat($tax,false,0,true);
//echo "<br>333333333:".$tax;

//nc_bugfix EOF

$order->info['shipping_cost'] = xtc_add_tax($order->info['shipping_cost'], $shipping_tax);
$order->info['tax'] += $tax;
$order->info['tax_groups'][TAX_ADD_TAX . "$shipping_tax_description"] += $tax;
$order->info['total'] += $tax;
} else {

if ($_SESSION['customers_status']['customers_status_show_price_tax'] == 0 && $_SESSION['customers_status']['customers_status_add_tax_ot'] == 1) {

$shipping_tax = xtc_get_tax_rate($GLOBALS[$module]->tax_class, $order->delivery['country']['id'], $order->delivery['zone_id']);
$shipping_tax_description = xtc_get_tax_description($GLOBALS[$module]->tax_class, $order->delivery['country']['id'], $order->delivery['zone_id']);
$tax = $xtPrice->xtcFormat(xtc_add_tax($order->info['shipping_cost'], $shipping_tax),false,0,false)-$order->info['shipping_cost'];
$tax = $xtPrice->xtcFormat($tax,false,0,true);

$order->info['tax'] = $order->info['tax'] += $tax;
$order->info['tax_groups'][TAX_NO_TAX . "$shipping_tax_description"] = $order->info['tax_groups'][TAX_NO_TAX . "$shipping_tax_description"] += $tax;
}
}
$this->output[] = array('title' => $order->info['shipping_method'] . ':',
'text' => $xtPrice->xtcFormat($order->info['shipping_cost'], true,0,true),
'value' => $xtPrice->xtcFormat($order->info['shipping_cost'], false,0,true));
}
}

function check() {
if (!isset($this->_check)) {
$check_query = xtc_db_query("select configuration_value from " . TABLE_CONFIGURATION . " where configuration_key = 'MODULE_ORDER_TOTAL_SHIPPING_STATUS'");
$this->_check = xtc_db_num_rows($check_query);
}

return $this->_check;
}

function keys() {
return array('MODULE_ORDER_TOTAL_SHIPPING_STATUS', 'MODULE_ORDER_TOTAL_SHIPPING_SORT_ORDER', 'MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING', 'MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING_OVER', 'MODULE_ORDER_TOTAL_SHIPPING_DESTINATION', 'MODULE_ORDER_TOTAL_SHIPPING_TAX_CLASS');
}

function install() {
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, set_function, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_STATUS', 'true','6', '1','xtc_cfg_select_option(array(\'true\', \'false\'), ', now())");
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_SORT_ORDER', '3','6', '2', now())");
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, set_function, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING', 'false','6', '3', 'xtc_cfg_select_option(array(\'true\', \'false\'), ', now())");
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, use_function, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING_OVER', '50', '6', '4', 'currencies->format', now())");
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, set_function, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_DESTINATION', 'national','6', '5', 'xtc_cfg_select_option(array(\'national\', \'international\', \'both\'), ', now())");
xtc_db_query("insert into " . TABLE_CONFIGURATION . " (configuration_key, configuration_value, configuration_group_id, sort_order, use_function, set_function, date_added) values ('MODULE_ORDER_TOTAL_SHIPPING_TAX_CLASS', '0','6', '7', 'xtc_get_tax_class_title', 'xtc_cfg_pull_down_tax_classes(', now())");
}

function remove() {
xtc_db_query("delete from " . TABLE_CONFIGURATION . " where configuration_key in ('" . implode("', '", $this->keys()) . "')");
}
}
?>


Kann jemand helfen? Ich bin für jeden Tipp dankbar.

Gruß,
Fuzzy

[Elektrofuzzy]

Hallo nochmal,

habe den Fehler eingegrenzt, auf diese Zeile hier in der order.php:


$this->info['total'] = $this->info['subtotal'] + $this->info['shipping_cost'];


Der ganze Abschnitt in der ORDER.PHP sieht so aus:


if ($_SESSION['customers_status']['customers_status_show_price_tax'] == '0') {
$this->info['total'] = $this->info['subtotal'] + $xtPrice->xtcFormat($this->info['shipping_cost'], false,0,true);
if ($_SESSION['customers_status']['customers_status_ot_discount_flag'] == '1') {
$this->info['total'] -= ($this->info['subtotal'] /100 * $_SESSION['customers_status']['customers_status_ot_discount']);
}
} else {

//nc_patch BOF

// $this->info['total'] = $this->info['subtotal'] + $xtPrice->xtcFormat($this->info['shipping_cost'],false,0,true);
//echo '<br>debug: Subtotal:'.$this->info['subtotal'] .' und shipping-cost:'. $this->info['shipping_cost'].' und $xtPrice(shipping-cost):'. $xtPrice->xtcFormat($this->info['shipping_cost'],false,0,true);

$this->info['total'] = $this->info['subtotal'] + $this->info['shipping_cost'];

//nc_patch EOF


Welche Parameter muss ich der Funktion xtPrice und der xtcFormat übergeben, damit es passt?

Mittels

$xtPrice->xtcFormat($this->info['shipping_cost'], false,0,true);

rechnet er schonmal den Dollar ein, aber vergisst noch die Mehrwehrtsteuer.

Gruß und Danke,
Fuzzy