Skip to content
Code-Schnipsel Gruppen Projekte

fix: migration of all v2 fixes of the order guest module

Zusammengeführt Henning Leutz requested to merge next-1.x into 1.x

This commit addresses two main issues in EventHandler.php. Firstly, the fetching of the email from the GuestOrder session is moved at a different location in the flow where it makes more sense and doesn't get overridden prematurely.

Secondly, an exhaustive condition handling has been added to address cases where there might be a payment error or async payment error. The method fetches the current order via the URL hash which could be an OrderInProcess or an already existing Order.

The commit also ensures that the customer UUID is set correctly if the session exists. An update is made to the database to accommodate this change.

Minor changes have also been made to spacing and data-type declaration in GuestOrderUser.php to help with readability and consistency.

Related: #11 (closed)

Merge Request-Berichte

Merge Request Pipeline #16814 mit Warnungen bestanden

Merge Request Pipeline mit Warnungen bestanden für a7ba3020

Genehmigung ist optional

Merged von Henning LeutzHenning Leutz 1 week ago (Apr 1, 2025 7:59Vormittags UTC)

Details des Merges

  • Änderungen zusammengeführt in next mit 508ac440.
  • Hat den Quellbranch nicht gelöscht.

Pipeline #16815 mit Warnungen bestanden

Pipeline mit Warnungen bestanden für 508ac440 auf 1.x

Aktivität

Aktivitäten filtern
  • Zustimmungen
  • Beauftragte & Prüfer(innen)
  • Kommentare (von Bots)
  • Kommentare (von Benutzerinnen und Benutzern)
  • Commits & Branches
  • Bearbeitungen
  • Labels
  • Sperrstatus
  • Erwähnungen
  • Merge Request-Status
  • Nachverfolgung
Bitte registriere oder melde dich an um zu antworten
Lade