Bestellen ohne Auswahl einer Zahlweise

[Archetim]

Hallo Zusammen

seit ein paar Wochen kommen bei uns Bestellungen an, die als Zahlweise "no_payment" ausgewiesen haben.
Ich konnte nun das Problem auch im aktuellen Demo-Shop nachstellen.

Man bucht ein Guthaben auf und legt einen Artikel in den Warenkorb. Bei der Auswahl der Zahlweise genügt es, die Checkbox anzuklicken, damit das Guthaben für die Bestellung herangezogen wird. Obwohl das Guthaben die Kosten nicht abdeckt - d.h. ein Restbetrag wäre noch zu zahlen - kann man zum Checkout weitergehen - siehe Screenshot im Anhang.

Meiner Meinung nach liegt es an der Stelle in der "checkout_confirmation.php"

Code: PHP  [Auswählen]

// load the selected payment module
require_once (DIR_WS_CLASSES . 'payment.php');
if (isset ($_SESSION['credit_covers']) || !isset($_SESSION['payment'])) { //DokuMan - 2010-10-14 - check that payment is not yet set, SP1b
        $_SESSION['payment'] = 'no_payment'; // GV Code Start/End ICW added for CREDIT CLASS
}
$payment_modules = new payment($_SESSION['payment']);
 

Obwohl die Variable "credit_covers" nicht gesetzt ist, wird durch die oder-Anweisung "no_payment" gesetzt.
Es gibt einen Block der im SP1b geändert wurde, der meiner Meinung nach diesen Fall abfangen würde.

Code: PHP  [Auswählen]

// GV Code line changed , SP1b
//if ((is_array($payment_modules->modules) && (sizeof($payment_modules->modules) > 1) && (!is_object($$_SESSION['payment'])) && (!isset ($_SESSION['credit_covers']))) || (is_object($$_SESSION['payment']) && ($$_SESSION['payment']->enabled == false))) {
if ((is_array($payment_modules->modules) && (sizeof($payment_modules->modules) > 1) && (!isset($$_SESSION['payment']) || !is_object($$_SESSION['payment'])) && (!isset($_SESSION['credit_covers']))) || (isset($$_SESSION['payment']) && is_object($$_SESSION['payment']) && ($$_SESSION['payment']->enabled == false))) {
        xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, 'error_message=' . urlencode(ERROR_NO_PAYMENT_MODULE_SELECTED), 'SSL'));
}
 

Ich teste das gerade und berichte über meine Versuche. Sollte eventuell jemand eine Idee haben, so würde ich mich um jede Hilfe freuen.

Danke im Vorraus.

Nachtrag:

Wenn man im Block

Code: PHP  [Auswählen]

if(isset($_SESSION['payment']) && $_SESSION['payment'] != 'no_payment') { //web28 - 2012-04-27 - fix for coupon amount == order total
  if ((is_array($payment_modules->modules) && (sizeof($payment_modules->modules) > 1) && (!is_object($$_SESSION['payment'])) && (!isset ($_SESSION['credit_covers']))) || (is_object($$_SESSION['payment']) && ($$_SESSION['payment']->enabled == false))) {
    xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, 'error_message=' . urlencode(ERROR_NO_PAYMENT_MODULE_SELECTED), 'SSL'));
  }
}
 

das "!=" zu einem "==" ändert, funktioniert alles.

Code: PHP  [Auswählen]

if(isset($_SESSION['payment']) && $_SESSION['payment'] == 'no_payment') { //web28 - 2012-04-27 - fix for coupon amount == order total
  if ((is_array($payment_modules->modules) && (sizeof($payment_modules->modules) > 1) && (!is_object($$_SESSION['payment'])) && (!isset ($_SESSION['credit_covers']))) || (is_object($$_SESSION['payment']) && ($$_SESSION['payment']->enabled == false))) {
    xtc_redirect(xtc_href_link(FILENAME_CHECKOUT_PAYMENT, 'error_message=' . urlencode(ERROR_NO_PAYMENT_MODULE_SELECTED), 'SSL'));
  }
}
 

Aber ich habe noch nicht getestet, welche Seitenfeekte das haben könnte.



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

[karabey]

Das Problem habe ich auch festgestellt und ist nicht NUR an den Guthaben oder Coupons gebunden. Der Kunde kann bestellen wenn sogar keine Zahlungsart definiert ist.

[web28]

Für Coupons kann ich das im aktuellen 1.06 Demoshop nicht bestätigen, wenn der Couponwert nicht ausreicht wird ganz normal die Zahlungsauswahl angezeigt.

Mit Gutschein/Guthaben habe ich aber nicht getestet

[Archetim]

@karabey: Ich bin mir auch noch nicht sicher, ob es nur am Guthaben liegt. Aber eine andere Stelle habe ich noch nicht gefunden. Da es auch nicht reproduzierbar auftaucht, könnte es auch eine nicht initialisierte Variable sein. Aber ich programmiere erst seit 5 Jahren. Bin also absolut kein Profi.

@web28: Im Screenshot sieht man das Ergebnis aus dem aktuellen Demo-Shop, wenn man ein Guthaben aufbucht. Cupons habe ich nicht getestet.

Bis jetzt zumindestens sind keine Bestellungen mehr mit "no_payment" angekommen. Da die aber nicht regelmäßig oder nachvollziehbar erscheinen, ist es auch noch zu Früh zu sagen, das Problem wäre bei uns gelöst.

Ich wäre für jeden Denkanstoß dankbar.

Grüße an alle

[madeby]

Das Problem habe ich auch festgestellt und ist nicht NUR an den Guthaben oder Coupons gebunden. Der Kunde kann bestellen wenn sogar keine Zahlungsart definiert ist.

Ist bei mir auch vorgekommen!

Wie kann dass sein und vor allem was kann man dagegen tun?

gruss Roger

[Archetim]

Hallo Zusammen,

da aktuell noch jemand das Problem mit "no_payment" hat, ein Nachsatz.
Seit der Änderung im ersten Beitrag, sind bei uns keine Bestellungen mehr aufgetreten, die "no_payment"  als Zahlweise haben.

Grüße
Rene

[swisshurricane]

Hi,

Ich habe folgende Zahlungsweisen aufgesetzt:

- Vorauskasse
- Rechnung (minimum Orders 1)
-> Somit kann ein Neukunde die erste Bestellung nur per Vorauskasse tätigen.

Das funktioniert soweit seit Jahren gut.

Allerdings gibt es ein Problem sobald Jemand Firefox mit dem no script addon verwendet.
Dann kann man bestellen (checkout_payment.php) ohne eine Zahlungsweise auszuwählen.

Hat Jemand eine Idee wie man das korrigieren kann?

Gruss
Harry

[andreaswien]

Hatte soeben das erste Mal auch dieses "no_payment" Problem. Ich habe jetzt jedenfalls die Kundin gefragt wie sie zahlen wollte und was passiert ist. Da sie schon öfters bestellt hat bin ich jetzt leicht verwirrt was wirklich los ist.

[noRiddle (revilonetz)]

Habe gerade im Demoshop nachgestellt:
Bei deaktiviertem Javascript kann man ohne Auswahl einer Zahlungsweise auf die checkout_confirmation gehen.

Ich bin auch der Meinung, daß diese Zeile in der /checkout_confirmation.php

Code: PHP  [Auswählen]

if(isset($_SESSION['payment']) && $_SESSION['payment'] != 'no_payment') { //web28 - 2012-04-27 - fix for coupon amount == order total

so lauten muß

Code: PHP  [Auswählen]

if(isset($_SESSION['payment']) && $_SESSION['payment'] == 'no_payment') { //web28 - 2012-04-27 - fix for coupon amount == order total

denn weiter oben wird definiert

Code: PHP  [Auswählen]

if (isset ($_POST['payment']))
  $_SESSION['payment'] = xtc_db_prepare_input($_POST['payment']);

und

Code: PHP  [Auswählen]

if (isset ($_SESSION['credit_covers']) || !isset($_SESSION['payment'])) { //DokuMan - 2010-10-14 - check that payment is not yet set
  $_SESSION['payment'] = 'no_payment'; // GV Code Start/End ICW added for CREDIT CLASS
}
$payment_modules = new payment($_SESSION['payment']);

was bedeutet,
daß ohne eine ausgewählte Zahlart ($_POST['payment']) not "setted" ist
und folglich $_SESSION['payment'] == 'no_payment' ist.

Das ist ein Bug.

Gruß,
noRiddle

[Tomcraft]

Wir haben das bereits in Ticket #36 festgehalten.

Grüße

Torsten

[andreaswien]

Danke!
Wie ist dann die beste Vorgehensweise derzeit? Den Kunden zu bitten die Bestellung nochmals durchzuführen und die Zahlungsart nicht zu vergessen?
Im konkreten Fall hatte ich eben Glück, da die Kundin schon öfters bestellt hatte und ich sie auf Rechnung beliefere, also kein Herumhantieren mit paypal bzw. Kreditkarteneingabe.

[noRiddle (revilonetz)]

Was gefällt dir an der von mir geposteten Lösung nicht ?, zumindest als vorübergehenden Fix ?
Mit Coupons und Gutscheinen usw. habe ich noch nicht getestet und müsste mich da erst reindenken.

Jedenfalls ist mit dem von mir geposteten Mini-Fix auch bei deaktiviertem Javascript ein Überspringen der Zahlungsart nicht mehr möglich (ohne Coupons/Guthaben/Gutschein).

Gruß,
noRiddle

[andreaswien]

dass ich zu blöd war und das irgendwie nicht so richtig mitbekommen habe?
Probiere es gleich aus, danke schon mal1

[noRiddle (revilonetz)]

Nur zur Info:
Wenn du's ausprobierst, mit deaktiviertem Javascript.
Einmal ohne Fix und einmal mit Fix.
Dann könntest du auch meine Tests bestätigen .

Gruß,
noRiddle

[andreaswien]

noRiddle, super! Besten Dank!
Du weißt, dass in Wien schon einige Biere auf dich warten wenn Du mal herkommst