Projekt

Allgemein

Profil

Aktionen

GX-Abgewiesen #42700

geschlossen

Shop scheint gesetzten Haken bei AGB/WRB im Checkout zu vergessen

Von Marco Bruchmann vor mehr als 8 Jahren hinzugefügt. Vor mehr als 2 Jahren aktualisiert.

Status:
Abgewiesen
Priorität:
Normal
Zugewiesen an:
Daniel Schnadt
Kategorie:
Checkout
Zielversion:
-
Beginn:
Abgabedatum:
% erledigt:

0%

Geschätzter Aufwand:
Steps to reproduce:
Release Notes Langtext:

Beschreibung

Ausgangssituation: Unter Konfiguration -> Allgemeines -> Rechtliches ist „Widerrufsrecht im Bestellvorgang bestätigen“ (GM_CHECK_WITHDRAWAL) nicht aktiviert.

Ablauf: Man durchläuft den Checkout, setzt auf checkout_payment den AGB-Haken und geht weiter auf die checkout_confirmation. Klickt man jetzt in die Adressleiste und drückt Enter, landet man wieder auf der checkout_payment, die einem vorwirft, man hätte den AGB-Haken nicht gesetzt.

Ursache: Die checkout_confirmation erzeugt eine Ausgabe bei Aufruf per POST und verlässt sich auch darauf, so aufgerufen zu werden. Diese Grundannahme ist jedoch ungültig, da auch die pre_confirmation_check()-Methoden der Zahlungs- und Zusammenfassungsmodule aufgerufen werden, die Weiterleitungen auslösen können, an deren Ende die checkout_confirmation per GET aufrufbar sein muss.

Lösungsansatz kurzfristig: Bei $_SERVER['REQUEST_METHOD'] === 'POST' darf keine Ausgabe erzeugt werden. Die $_POST-Parameter müssen in geeigneter Weise ausgewertet werden und anschließend eine Weiterleitung auf checkout_confirmation ausgelöst werden, so dass der Aufruf der pre_confirmation_check()-Methoden und die Ausgabe aus einem GET-Aufruf heraus erfolgen.

Lösungsansatz langfristig: Seiten sollten grundsätzlich $_SERVER['REQUEST_URI'] als Action für POST-Formulare verwenden, ihre POST-Daten also selbst verarbeiten, und erst danach auf die Folgeseite weiterleiten, so dass letztere immer zuerst per GET geladen wird.


Aktionen #1

Von Moritz Bunjes vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde auf 59 gesetzt
  • Betroffene Versionen 2.2.2.1 wurde hinzugefügt
  • Betroffene Versionen Unbestimmt wurde gelöscht
Aktionen #3

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 59 zu 2.4.0.0 beta1 geändert
Aktionen #4

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.4.0.0 beta1 zu 2.4.1.0 beta1 geändert
Aktionen #5

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.4.1.0 beta1 zu 2.4.0.0 beta1 geändert
Aktionen #6

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.4.0.0 beta1 zu 2.4.1.0 beta1 geändert
Aktionen #7

Von Michael Kroenke vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.4.1.0 beta1 zu 2.5.1.0 beta1 geändert
Aktionen #8

Von Moritz Bunjes vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.5.1.0 beta1 zu 73 geändert
Aktionen #9

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 73 zu 2.5.2.0 beta1 geändert
Aktionen #10

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.5.2.0 beta1 zu 2.5.2.0 geändert
Aktionen #11

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.5.2.0 zu 73 geändert
Aktionen #12

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 73 zu 2.5.3.0 beta1 geändert
Aktionen #13

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.5.3.0 beta1 zu 132 geändert
Aktionen #14

Von Till Tepelmann vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 132 zu 2.6.0.0 beta1 geändert
Aktionen #15

Von Daniel Wu vor mehr als 8 Jahren aktualisiert

  • Zielversion wurde von 2.6.0.0 beta1 zu 132 geändert
Aktionen #16

Von Till Tepelmann vor etwa 8 Jahren aktualisiert

  • Zielversion wurde von 132 zu 2.6.1.0 beta1 geändert
Aktionen #17

Von Daniel Wu vor etwa 8 Jahren aktualisiert

  • Zielversion wurde von 2.6.1.0 beta1 zu 132 geändert
Aktionen #18

Von Till Tepelmann vor etwa 8 Jahren aktualisiert

  • Zielversion wurde von 132 zu 133 geändert
Aktionen #19

Von Sergej Kronhardt vor mehr als 7 Jahren aktualisiert

  • Status wurde von Gemeldet zu Feedback geändert
  • Zugewiesen an wurde auf Daniel Schnadt gesetzt
Aktionen #23

Von Mirko Janssen vor fast 7 Jahren aktualisiert

  • Status wurde von Feedback zu Abgewiesen geändert
Aktionen #25

Von Till Tepelmann vor etwa 6 Jahren aktualisiert

  • Tracker wurde von GX-Bug zu GX-Abgewiesen geändert
Aktionen #26

Von Moritz Bunjes vor mehr als 2 Jahren aktualisiert

  • Zielversion 133 wurde gelöscht
Aktionen

Auch abrufbar als: Atom PDF