Fehler in "checkout_process"?

Thema wurde von Avenger, 14. Juli 2014 erstellt.

  1. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Was ist eigentlich der Sinn dahinter, bei bestimmten Zahlungsarten keine Lagerbestandsprüfung vorzunehmen???

    Irgendwie finde ich das nicht richtig, auch da sollte doch eine Bestandsprüfung erfolgen.

    PHP:
    if(STOCK_ALLOW_CHECKOUT == 'false' 
        
    && $_SESSION['payment'] != 'paypalng' // PayPalNG
        
    && $_SESSION['payment'] != 'paypal' 
        
    && $_SESSION['payment'] != 'paypal_gambio' 
        
    && $_SESSION['payment'] != 'paypalgambio_alt' 
        
    && $_SESSION['payment'] != 'sofortueberweisung_direct' 
        
    && $_SESSION['payment'] != 'pn_sofortueberweisung' 
        
    && $_SESSION['payment'] != 'clickandbuy_v2' 
        
    && $_SESSION['payment'] != 'postfinance_epayment' 
        
    && strpos($_SESSION['payment'], 'sofort_') === false
        
    && strpos($_SESSION['payment'], 'moneybookers') === false
        
    && strpos($_SESSION['payment'], 'vrepay') === false
        
    && strpos($_SESSION['payment'], 'ipayment') === false)
    {
     
  2. Moritz (Gambio)

    Moritz (Gambio) Administrator

    Registriert seit:
    26. April 2011
    Beiträge:
    5.786
    Danke erhalten:
    2.693
    Danke vergeben:
    903
    Hallo,

    das sind alles Module, die die checkout_process.php mehrmals aufrufen. Die Bestandsüberprüfung darf aber nur ein einziges Mal ausgeführt werden, so dass diese Module ausgeschlossen werden. Elegant ist das nicht, stimmt.
     
  3. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Um das mehrfache Auswerten auszuschließen könnte man das doch mit

    PHP:
    if (!isset ($_SESSION['tmp_oID'])) {
    prüfen, oder?
     
  4. Avenger

    Avenger G-WARD 2012/13/14/15

    Registriert seit:
    26. April 2011
    Beiträge:
    4.771
    Danke erhalten:
    1.478
    Danke vergeben:
    89
    Das ist nicht nur eine Frage der Eleganz, sonder eine Fehlerursache.
     
  5. Moritz (Gambio)

    Moritz (Gambio) Administrator

    Registriert seit:
    26. April 2011
    Beiträge:
    5.786
    Danke erhalten:
    2.693
    Danke vergeben:
    903
    Ist nun für die nächste Version so umgesetzt.