Verbesserungsvorschlag: Checkout Success

[Murmeltier]

@GTB

Deine Lösung funktioniert prima. Aber wie lager ich den Text in die Sprachdatei aus?

In der lang_german.conf habe ich bei [chekout_success] folgendes hinzugefügt:

Code: PHP  [Auswählen]

text_payment_description ='Bitte überweisen Sie den Gesamtbetrag von <b>'. $order->info['total'] .'</b> mit Angabe des Verwendungszwecks <b>"Bestellnummer: '. $last_order .'"</b> an:<br/><br/>''

Ich weiß aber nicht, was ich in der checkout_success.php machen muss. Ich habe es so versucht (durch Raten):

Code: PHP  [Auswählen]

$payment_info = #text_payment_description#;

Funktioniert leider nicht...

[web28]

Code: PHP  [Auswählen]

{#text_payment_description#}
 

funktioniert nur über Smarty in den html Vorlagen

Da wir hier in einer PHP Datei sind musst Du in der "german.php" eine Sprachkonstante definieren, einfach am Ende einbauen

Code: PHP  [Auswählen]

define ('PAYMENT_INFO_MONEYORDER', 'Dein Text');
 

In der PHP Datei dann wie gewohnt:

Code: PHP  [Auswählen]

$payment_info = PAYMENT_INFO_MONEYORDER;
 

Gruss Web28

[Murmeltier]

Danke, dass funktioniert. Einen kleinen Fehler gibt es allerdings noch: Leider kann die german.php die Variablen nicht ausgeben (oder ich habe einen Fehler bei der Deklaration gemacht).

german.php

Code: PHP  [Auswählen]

define ('PAYMENT_INFO_MONEYORDER', 'Bitte überweisen Sie den Gesamtbetrag von <b>'. $order->info['total'] .'</b> mit Angabe des Verwendungszwecks <b>"Bestellnummer: '. $last_order .'"</b> an:<br/><br/>');

[Murmeltier]

Ich habe das ganze mittlerweile mit einem Workaround lösen können:

german.php

Code: PHP  [Auswählen]

// Sprachcode für geänderte checkout_success.php
define ('PAYMENT_INFO_EUSTANDARD_Text1', 'Bitte überweisen Sie den Gesamtbetrag von <b>');
define ('PAYMENT_INFO_EUSTANDARD_Text2', '</b> mit Angabe des Verwendungszwecks <b>"Bestellnummer: ');
define ('PAYMENT_INFO_EUSTANDARD_Text3', '"</b> an:<br/><br/>');

checkout_sucess.php

Code: PHP  [Auswählen]

$payment_info = PAYMENT_INFO_EUSTANDARD_Text1 . $order->info['total'] . PAYMENT_INFO_EUSTANDARD_Text2 . $last_order . PAYMENT_INFO_EUSTANDARD_Text3;

[Sardaukar]

Vielen Dank an Murmeltier.

Da es lediglich ein Workaround ist, würde ich gerne wissen wie sich dies auch schöner lösen lässt?

Kann jemand hierbei behilflich sein?

[web28]

Schaut euch PHP: sprintf an.

@Murmeltier
Die Variablen sind in der "german.php" nicht bekannt, deshalb ging das nicht.

[franck]

Hallo memocorex,

das ist eigentlich keine schlechte Idee. Es ist auch in der Tat einfacher umzusetzen:

/checkout_sucess.php

nach dem hier:

Code: PHP  [Auswählen]

$last_order = $orders['orders_id'];
$order_status = $orders['orders_status'];
 

das hier einfügen:

Code: PHP  [Auswählen]

//BOF - GTB - anzeige von Zahlungsinformationen
require_once (DIR_WS_CLASSES . 'order.php');
$order = new order($last_order);
if ($order->info['payment_method'] == 'moneyorder') {
  require_once (DIR_WS_CLASSES . 'payment.php');
  $payment_modules = new payment($order->info['payment_method']);
  if (is_array($payment_modules->modules)) {
      $confirmation = $payment_modules->confirmation();
      $payment_info = 'Bitte überweisen sie den Gesamtbetrag von <b>'. $order->info['total'] .'</b> mit Angabe der Bestellnummer <b>'. $last_order .'</b> an:<br/><br/>';
      $payment_info .= $confirmation['title'];
      if (isset($confirmation['fields'])) {
        for ($i = 0, $n = sizeof($confirmation['fields']); $i <$n; $i++) {
          $payment_info .= '<table><tr>
                                <td>' . xtc_draw_separator('pixel_trans.gif', '10', '1') . '</td>
                                <td class="main">' . $confirmation['fields'][$i]['title'] . '</td>
                                <td>' . xtc_draw_separator('pixel_trans.gif', '10', '1') . '</td>
                                <td class="main">' . stripslashes($confirmation['fields'][$i]['field']) . '</td>
                              </tr></table>';
        }
      }
  $smarty->assign('PAYMENT_INFORMATION', $payment_info);
  }
}
//EOF - GTB - anzeige von Zahlungsinformationen
 

dann brauch man in der checkout_success.html nur mehr an gewünschter Stelle das hier einfügen:

Code: PHP  [Auswählen]

{$PAYMENT_INFORMATION}
 

Gruss Gerhard

Hallo,

ich habe alles einbaut nach diesen Anleitung, funktioniert auch super aber ich würde auch gerne das so haben mit den 4 Zeilen für Kontonummer , Bankleitzahl, IBAN und BIC.

Wie und wo genau muss an den Code was geändert werden damit es funktioniert ?
Wäre es nicht einfacher direkt die Datei "/includes/modules/payment/moneyorder.php" anzupassen das es Felder gibt zum Eintragen der Daten im Admin Panel?

Für jede Hilfe bin ich dankbar.

Gruß Franck

[iche89]

Hallo.

Ich komm gerade nicht weiter.
Ich habe das wie beschrieben eingefügt :

Code: PHP  [Auswählen]

//BOF - GTB - anzeige von Zahlungsinformationen
require_once (DIR_WS_CLASSES . 'order.php');
$order = new order($last_order);
if ($order->info['payment_method'] == 'moneyorder') {
  require_once (DIR_WS_CLASSES . 'payment.php');
  $payment_modules = new payment($order->info['payment_method']);
  if (is_array($payment_modules->modules)) {
      $confirmation = $payment_modules->confirmation();
      $payment_info = 'Bitte überweisen sie den Gesamtbetrag von <b>'. $order->info['total'] .'</b> mit Angabe der Bestellnummer <b>'. $last_order .'</b> an:<br/><br/>';
      $payment_info .= $confirmation['title'];
      if (isset($confirmation['fields'])) {
        for ($i = 0, $n = sizeof($confirmation['fields']); $i <$n; $i++) {
          $payment_info .= '<table><tr>
                                <td>' . xtc_draw_separator('pixel_trans.gif', '10', '1') . '</td>
                                <td class="main">' . $confirmation['fields'][$i]['title'] . '</td>
                                <td>' . xtc_draw_separator('pixel_trans.gif', '10', '1') . '</td>
                                <td class="main">' . stripslashes($confirmation['fields'][$i]['field']) . '</td>
                              </tr></table>';
        }
      }
  $smarty->assign('PAYMENT_INFORMATION', $payment_info);
  }
}
//EOF - GTB - anzeige von Zahlungsinformationen

Aber jetzt kommt folgende Fehlermeldung :

Fatal error: Call to a member function get_quantity() on a non-object in /kunden/325561_03051/SportAgility/includes/cart_actions.php on line 175

in der "cart_actions.php" Zeile 175 steht :

Code: PHP  [Auswählen]

$act_qty = $_SESSION['cart']->get_quantity(xtc_get_uprid($quickie['products_id'], 1));

Allerdings habe ich das nicht gemacht :

[...]
dann brauch man in der "checkout_success.html" nur mehr an gewünschter Stelle das hier einfügen:

Code: PHP  [Auswählen]

{$PAYMENT_INFORMATION}

[...]

Weil ich nicht weiß wo ich das einfügen soll. ich kann das ja nicht einfach irgendwo einfügen oder ?

Habe ich etwas falsch gemacht ?

[cillex]

Ich habe diese Lösung bisher auch immer für die eustandardtransfer verwendet. Seit dem Update auf die 1.06 geht es so aber nicht mehr, da es offenbar eine Änderung in der includes/modules/payment/eustandardtransfer.php gegeben hat (Zeile 49-51). Ob auch die moneyorder.php betroffen ist, habe ich nicht getestet, gehe aber mal davon aus. In naher Zukunft wird die eustanardtransfer ja eh für SEPA-Überweisungen nötig sein.

Lange Rede, kurzer Sinn:
Anstatt der MODULE_PAYMENT_EUTRANSFER_TEXT_DESCRIPTION wird auf der checkout_success.php jetzt die MODULE_PAYMENT_EUTRANSFER_TEXT_INFO angezeigt. Somit werden also nach erfolgter Bestellung keine Bankdaten mehr angezeigt.

Die Anpassung hat vermutlich damit zusammen, dass auf der checkout_confirmation jetzt die Zahlungsinformationen zu sehen sein sollen.

Vielleicht hat ja jemand eine Idee, wie man das Problem lösen kann.

VG Cillex

[online-beobachter]

Hallo,

gibt es dazu auch eine Lösung für den Sop 2.0.4.2?
Alle meine Bemühungen diesbezüglich waren in der neueren Shopversion leider erfolglos.
Das reine anzeigen der Bankverbindung schön und gut aber ohne genauen Endbetrag und Bestellnummer nicht ganz durchdacht.

Mir ist es sowieso ein Rätsel warum das kein Standard ist bei Vorkasse, es kommt ziemlich häufig vor das Kunden Ihre eMail-Adresse selbst nicht kennen oder einfach falsch schreiben.
Jede dieser Bestellungen ist eine verlorene /erfolglose, da man fast nie mehr einen Kontakt zum Kunden bekommt wenn er sich nicht zufällig nochmals meldet.
Das wäre sehr aber sehr einfach zu vermeiden...

Danke
Michael

[Timm]

Habe Ticket #1169 dahingehend ergänzt, weil ich das auch wichtig finde.

Gruß Timm

[online-beobachter]

Hallo Timm,

ich habe noch die Version 2.0.4.2, und ein update kommt momentan nicht in Frage.
Weißt Du wo ich die geänderten Dateien bzw. die Änderungen zum Fix finde? Ich würde das gerne nachrüsten.
Früher wurden die ja immer direkt angezeigt wenn man auf das Ticket klickt, das scheint aber nicht mehr so zu funktionieren.

Gruß Michael

[Timm]

Hallo Michael

Du könntest im Ticket mal nett fragen, ob sie ein changeset anhängen können. Normal werden changesets aber nur bei wichtigen Sachen angehangen, die vor der nächsten Veröffentlichung eines Fix bedürfen.

Ansonsten bleibt nur der Weg über einen Dateivergleich von Version 2.0.5.1 gegen 2.0.4.2.

Gruss Timm

[online-beobachter]

Ich werde mal versuchen herauszufinden welche Dateien konkret betroffen sind.
Ich hab den Eindruck das das Shopsystem seit V2 immer komplexer wird.😕