magento-1
magento-1 copied to clipboard
prevent success action for non-paid orders
Dieser PR verhindert, dass bei noch nicht bezahlten Bestellungen die Success Action des OnepageControllers aufgerufen werden kann.
Es gibt zwar noch die zusätzlichen Informationen zur Bezahlung, die in den Kommentaren der Bestellung stehen, aber insbesondere bei Shops die auf Events (wie z.B. checkout_onepage_controller_success_action) lauschen ist das sehr kritisch. Meiner Meinung nach sollten nicht bezahlte Bestellungen nicht in der Lage sein erfolgreich abzuschließen.
Reproduzierung:
- PayPal Bestellung triggern und weiterleiten lassen
- /checkout/onepage/success aufrufen
Thanks for your contribution. Seems valid to me, we'll review and test.
Hi @marvasDE ich habe den PR mal ausgecheckt und dann sowohl mit PayPal als auch Paydirekt getestet. Bei beiden war das Verhalten unverändert, wenn ich deiner Reproanleitung folge. Dort wurde dann nämlich die Erfolgsseite angezeigt. Ich hätte jetzt anhand des Codes ein anderes Verhalten erwartet. Kannst du nochmal nachschauen?
Hi @hreinberger Ich habe dies eben nochmals verifiziert. Ich werde beim Versuch die Erfolgsseite aufzurufen auf den Warenkorb weitergeleitet. Getestet wurde mit Magento 1.9.4.2 (bzw 1.9.4.1 beim Erstellen des PRs) mit PayPal als Zahlungsmethode.
Ha, jetzt klappts bei mir auch! Der Aufruf von checkout/onepage/success leitet zum Warenkorb weiter. Mir ist noch aufgefallen, dass ein Aufruf unserer custom redirecturl ( https://github.com/PAYONE-GmbH/magento-1/blob/11d8583c3e5c961c0af0edb2f6c33bd9190e3814/app/code/community/Payone/Core/Helper/Url.php#L59 ) weiterhin möglich ist. Wir schauen mal intern, ob wir das noch robuster gestalten können.