Änderung einer Bestellung durch Kunden nach Abschluss möglich!

Der Kunde kann theoretisch Bestelldaten nach einer durchgeführten Bestellung ändern!

Reproduzieren

1. Tab öffnen und ein Produkt in den Warenkorb tun
2. Zum Bestellprozess wechseln und den Warenkorb offen lassen
3. Neuen Tab öffnen und hier auch zum Bestellprozess wechseln
4. In Tab 2 den Bestellprozess zu Ende führen -> Bestellung ist abgeschlossen und Zahlung durchgeführt (z.B. PayPal)
5. Wieder zu Tab 1 wechseln und hier im noch geöffneten Warenkorb die Artikel-Menge ändern und bestätigen

Bug

-> Die abgeschlossene Bestellung wird jetzt editiert und die Mengen und Preisberechnung ändert sich dadurch in der Bestellung!

added priority: High type: Bug labels

changed due date to June 08, 2020

mentioned in commit 0ca55245

Ich hab nen Fix dafür geschrieben ... aber ob ich damit so glücklich bin weis ich noch nicht. Die zu speichernden Daten einer Bestellung werden nun überschrieben.

if (QUI::isFrontend()
    && $this->isSuccessful()
    && !QUI::getUsers()->isSystemUser($PermissionUser)) {
    // if order is successful
    // only some stuff are allowed to changed

    $_data = [
        'payment_id'      => $data['payment_id'],
        'payment_method'  => $data['payment_method'],
        'payment_data'    => $data['payment_data'],
        'payment_address' => $data['payment_address'],
        'comments'        => $data['comments'],
        'history'         => $data['history']
    ];

    $data = $_data;
}

D.h. wenn im Frontend folgendes eintrift: Bestellung ist successful UND es der Nutzer kein System Nutzer gerade ist, dann werden nur payment, comments, history daten gespeichert.

Das Backend sollte dies nicht betreffen und ich denke nicht das im Frontend noch Addressen etc geändert werden sollten.

Irgendwelche Einwände für den Patch? @peat @mor ?

Sollte mMn passen.

closed via commit 0ca55245

sollte passen