change_order E-Mail ohne Inhalt bei Download-Artikeln

[Igotcha]

Hallo zusammen,

EDIT:

Konnte das Problem jetzt lokalisieren:

Die Statusänderungsmail ist nur leer bei Statusänderungen von Aufträgen mit reinen Downloadartikeln und dem Preis von 0,00 EUR. Bei einer gemischten Bestellung von Downloadartikeln und anderen, ist die Statusmail korrekt.

In der Datenbank sehe ich, dass im Falle der Bestellung eines reinen Downloadartikels mit Preis 0,00 die Felder "payment_class" und "language" in der Tabelle "orders" nicht gefüllt sind. Deshalb kommt es zu folgendem Fehler:

Warning: Smarty error: unable to read resource: "xtc5/admin/mail//change_order_mail.html" in D:\xampp\htdocs\bbqtop\includes\classes\Smarty_2.6.26\Smarty.class.php on line 1097

Das hilft mir jetzt aber nicht weiter, da an dieser Stelle nur die Fehlermeldung erzeugt wird:

Code: PHP  [Auswählen]

    function trigger_error($error_msg, $error_type = E_USER_WARNING)
    {
        trigger_error("Smarty error: $error_msg", $error_type);
    }
 

Wenn ich das so richtig sehe, könnte der Fehler daran liegen, dass die Variable order->info['language'] leer ist, deshalb auch der doppelte // in der Fehlermeldung

In der admin/orders.php Zeile 127:

Code: PHP  [Auswählen]

$html_mail = $smarty->fetch(CURRENT_TEMPLATE.'/admin/mail/'.$order->info['language'].'/change_order_mail.html');
$txt_mail = $smarty->fetch(CURRENT_TEMPLATE.'/admin/mail/'.$order->info['language'].'/change_order_mail.txt');
 

Wie gesagt, das kommt nur vor, wenn ein einziger Artikel als Download bestellt wurde, der 0,00 EUR kostet (setze ich den Preis auf 1,00 ist die Bestätigungsmail in Ordnung).



Linkback: https://www.modified-shop.org/forum/index.php?topic=9988.0

[franky_n]

Hallo Igotcha,

jo das liegt daran weil Du ja bereits gesagt hast das bei Downloadartikeln auch die beiden Felder leer sind.

Änder mal in der checkout_process.php in Z.130 das:

Code: PHP  [Auswählen]

        $sql_data_array = array ('customers_id' => $_SESSION['customer_id'], 'customers_name' => $order->customer['firstname'].' '.$order->customer['lastname'], 'customers_firstname' => $order->customer['firstname'], 'customers_lastname' => $order->customer['lastname'], 'customers_cid' => $order->customer['csID'], 'customers_vat_id' => $_SESSION['customer_vat_id'], 'customers_company' => $order->customer['company'], 'customers_status' => $_SESSION['customers_status']['customers_status_id'], 'customers_status_name' => $_SESSION['customers_status']['customers_status_name'], 'customers_status_image' => $_SESSION['customers_status']['customers_status_image'], 'customers_status_discount' => $discount, 'customers_street_address' => $order->customer['street_address'], 'customers_suburb' => $order->customer['suburb'], 'customers_city' => $order->customer['city'], 'customers_postcode' => $order->customer['postcode'], 'customers_state' => $order->customer['state'], 'customers_country' => $order->customer['country']['title'], 'customers_telephone' => $order->customer['telephone'], 'customers_email_address' => $order->customer['email_address'], 'customers_address_format_id' => $order->customer['format_id'], 'delivery_name' => $order->delivery['firstname'].' '.$order->delivery['lastname'], 'delivery_firstname' => $order->delivery['firstname'], 'delivery_lastname' => $order->delivery['lastname'], 'delivery_company' => $order->delivery['company'], 'delivery_street_address' => $order->delivery['street_address'], 'delivery_suburb' => $order->delivery['suburb'], 'delivery_city' => $order->delivery['city'], 'delivery_postcode' => $order->delivery['postcode'], 'delivery_state' => $order->delivery['state'], 'delivery_country' => $order->delivery['country']['title'], 'delivery_country_iso_code_2' => $order->delivery['country']['iso_code_2'], 'delivery_address_format_id' => $order->delivery['format_id'], 'payment_method' => $order->info['payment_method'], 'payment_class' => $order->info['payment_class'], 'shipping_method' => $order->info['shipping_method'], 'shipping_class' => $order->info['shipping_class'], 'cc_type' => $order->info['cc_type'], 'cc_owner' => $order->info['cc_owner'], 'cc_number' => $order->info['cc_number'], 'cc_expires' => $order->info['cc_expires'], 'date_purchased' => 'now()', 'orders_status' => $tmp_status, 'currency' => $order->info['currency'], 'currency_value' => $order->info['currency_value'], 'customers_ip' => $customers_ip, 'comments' => $order->info['comments']);
 

in das:

Code: PHP  [Auswählen]

        $sql_data_array = array ('customers_id' => $_SESSION['customer_id'], 'customers_name' => $order->customer['firstname'].' '.$order->customer['lastname'], 'customers_firstname' => $order->customer['firstname'], 'customers_lastname' => $order->customer['lastname'], 'customers_cid' => $order->customer['csID'], 'customers_vat_id' => $_SESSION['customer_vat_id'], 'customers_company' => $order->customer['company'], 'customers_status' => $_SESSION['customers_status']['customers_status_id'], 'customers_status_name' => $_SESSION['customers_status']['customers_status_name'], 'customers_status_image' => $_SESSION['customers_status']['customers_status_image'], 'customers_status_discount' => $discount, 'customers_street_address' => $order->customer['street_address'], 'customers_suburb' => $order->customer['suburb'], 'customers_city' => $order->customer['city'], 'customers_postcode' => $order->customer['postcode'], 'customers_state' => $order->customer['state'], 'customers_country' => $order->customer['country']['title'], 'customers_telephone' => $order->customer['telephone'], 'customers_email_address' => $order->customer['email_address'], 'customers_address_format_id' => $order->customer['format_id'], 'delivery_name' => $order->delivery['firstname'].' '.$order->delivery['lastname'], 'delivery_firstname' => $order->delivery['firstname'], 'delivery_lastname' => $order->delivery['lastname'], 'delivery_company' => $order->delivery['company'], 'delivery_street_address' => $order->delivery['street_address'], 'delivery_suburb' => $order->delivery['suburb'], 'delivery_city' => $order->delivery['city'], 'delivery_postcode' => $order->delivery['postcode'], 'delivery_state' => $order->delivery['state'], 'delivery_country' => $order->delivery['country']['title'], 'delivery_country_iso_code_2' => $order->delivery['country']['iso_code_2'], 'delivery_address_format_id' => $order->delivery['format_id'], 'payment_method' => $order->info['payment_method'], 'payment_class' => $order->info['payment_class'], 'shipping_method' => $order->info['shipping_method'], 'shipping_class' => $order->info['shipping_class'], 'cc_type' => $order->info['cc_type'], 'cc_owner' => $order->info['cc_owner'], 'cc_number' => $order->info['cc_number'], 'cc_expires' => $order->info['cc_expires'], 'date_purchased' => 'now()', 'orders_status' => $tmp_status, 'currency' => $order->info['currency'], 'currency_value' => $order->info['currency_value'], 'customers_ip' => $customers_ip, 'language' => $_SESSION['language'], 'comments' => $order->info['comments']);

Dann sollte die Language bei der nächsten Download Bestellung vorhanden sein.

Viele Grüße

Franky

[Igotcha]

Schau ich gleich mal, aber wie gesagt, das Problem tritt nur bei Downloadartikeln auf und, wenn diese 0,00 EUR kosten.

Also kann an einem generellen SQL-Statement eigentlich nicht liegen, da muss noch irgendwo eine Bedingung diesbzgl. vorhanden sein.

EDIT: Boah, ihr seid ja fies.

Genau diesen Fall habt ihr in einem SQL-Statement abgefangen und die Felder einfach nicht befüllt...