Grundlegendes Problem in checkout_process.php?

Bei den Zahlungsarten wie PayPal, Kreditkarten usw. wird ja nach der Bestellbestätigung auf die Bezahlseite verzweigt.

In der checkout_process.php wird dabei eine Bestellung angelegt, und die Lagerbestände werden aktualisiert.

Wenn die Zahlung dann aber abgebrochen wird, sind m.E. die Lagerbestände falsch, da die ja schon vor der Verzweigung zur Zahlungsseite aktualisiert, und bei Fehlschlagen der Zahlung nicht korrigiert werden.

Habe ich etwas übersehen?

EDIT:

Diese Daten dürfen also erst nach wirklich positivem Bestellabschluss aktualisiert werden.

Ebenso die Zuordnung zu Kampagnen.

 

Keine Meinung dazu von der Gambio GmbH?

Wenn meine Erkenntnis stimmt (wovon ich mal ausgehe), muss das sicher schnell gelöst werden!

 

Das wundert mich eigentlich.....

Weil im Fehlerfall die "checkout_process.php" nicht gestartet wird (in der die eMails verschickt werden), sondern die Zahlungsseite..

 

Ja, genau das ist das Problem,,,,

m.E. kann es aber keine unterschiedlichen Ansichten geben, ob das richtig oder falsch ist: es ist definitiv falsch!

Es darf natürlich nicht sein, dass der Lagerbestand verändert wird, bevor sicher ist, dass der Kunde auch bezahlt hat.

Und das Problem ist auch einfach lösbar:

man darf den Bestand erst dann anpassen, wenn die Bezahlung erfolgt ist.

Und das ist einfach realisierbar, auch innerhalb der "checkout_process.php".

In der "checkout_succes.php" ist es insofern schwieriger, weil der Warenkorb nicht mehr existiert.

 

Aber im Fehlerfall wird ja die "checkout_process.php" nicht aktiviert, sondern die Zahlungsseite "checkout_payment.php"

Und nur in der "checkout_process.php" werden die Mails versendet, und nur nach erfolgreicher Zahlung!

Aus meiner Sicht gibt es nur das Problem mit der Lagerbestandsführung, die zu früh erfolgt.

Verwendest Du Deinen "one-page-checkout"?

Gibt es evtl. da ein Problem?

 

Ich habe mir das mal angesehen, und folgende Problemlösung erarbeitet:

die Idee ist, dass man im Falle einer externen Zahlungsart die Lagerbestand-SQLs, die erzeugt werden, nicht direkt anwendet, sondern in der Session zwischenspeichert.

Und die erst dann anwendet, wenn der Zahlungs-Prozess erfolgreich abgeschlossen ist.

Dann kann man die vorhandene Logik weitestgehend beibehalten, und muss nur moderat in den Code eingreifen.

In "checkout_process.php" folgendes ändern:

Die beiden folgenden Änderungen sind notwendig, weil bei mir das verwendete "$$_SESSION['payment']" "NULL" ist, und so durch "$GLOBALS[$_SESSION['payment']]" ersetzt wird...

(Mit "$$_SESSION['payment']" verlässt man sich m.E. darauf, dass "register_globals" auf "on" gesetzt ist.)

Vor

PHP:

if (isset ($_SESSION['tmp_oID']) && is_int($_SESSION['tmp_oID'])) {

einfügen

PHP:

//Avenger
$payment=$GLOBALS[$_SESSION['payment']];
//Avenger

Alle

PHP:

$$_SESSION['payment']

ersetzen mit

PHP:

$payment

PHP:

            xtc_db_query("update ".TABLE_PRODUCTS." set products_quantity = '".$stock_left."' where products_id = '".xtc_get_prid($order->products[$i]['id'])."'");
            // BOF GM_MOD:
            if (($stock_left < 1) && (STOCK_ALLOW_CHECKOUT == 'false') && GM_SET_OUT_OF_STOCK_PRODUCTS == 'true') {
                xtc_db_query("update ".TABLE_PRODUCTS." set products_status = '0' where products_id = '".xtc_get_prid($order->products[$i]['id'])."'");
            }

ersetzen mit

PHP:

      //Avenger      
      //Delayed stock updates
      //For external payment-methods adjust stock only after real payment! 
      //In that case store updates sql in session and do not apply directly!
      $update="update ".TABLE_PRODUCTS." set ";
      $where=" where products_id = '".xtc_get_prid($order->products[$i]['id'])."'";
      $stock_update_sql=$update."products_quantity = '".$stock_left."'".$where;      
      $status_update_sql=$update."products_status = '0' ".$where;      
      $no_stock_left=$stock_left < 1 && (STOCK_ALLOW_CHECKOUT == 'false') && GM_SET_OUT_OF_STOCK_PRODUCTS == 'true';
      if ($tmp)                  
      {
        $_SESSION{'stock_update_sql'}[]=$stock_update_sql;
        // BOF GM_MOD:
        if ($no_stock_left) 
        {
          $_SESSION{'status_update_sql'}[]=$status_update_sql;
        }
      }
      else
      {
        xtc_db_query($stock_update_sql);
        // BOF GM_MOD:
        if ($no_stock_left) 
        {
          xtc_db_query("update ".TABLE_PRODUCTS." set products_status = '0' where products_id = '".xtc_get_prid($order->products[$i]['id'])."'");
        }
      }
      //Avenger

Nach

PHP:

if (!$tmp) {

einfügen

PHP:

  //Avenger     
  //Delayed stock updates
  //For external payment-methods adjust stock only after real payment! 
  //In that case updates sql have been stored in session and not have been applied directly!
  $updates_sql_names=array('stock_update_sql','status_update_sql');
  foreach ($updates_sql_names as $updates_sql_name)
  {
    $update_sqls=$_SESSION{$updates_sql_name};
    if ($update_sqls)
    {
      foreach ($update_sqls as $update_sql)
      {
        xtc_db_query($update_sqls);
      }
      unset($_SESSION{$updates_sql_name});
    }
  }
  //Avenger

Wie immer gilt:

Anwendung ausschließlich auf eigenes Risiko des Verwenders.

Eine Gewährleistung jeglicher Art ist ausgeschlossen.

Vor Einbau eine Sicherung der Shop-Programme vornehmen.

Erst im Testshop testen.

 

Das war noch nicht ganz vollständig.....

In "checkout_process.php" folgendes ändern:

Nach

PHP:

   Copyright (c) 2012 Gambio GmbH

einfügen

PHP:

   Copyright (c) 2013 Avenger, entwicklung@powertemplate.de
   Delayed stock updates
   For external payment-methods adjust stock only after real payment!
   In that case store updates sql in session and do not apply directly!

Die beiden folgenden Änderungen sind notwendig, weil bei mir das verwendete "$$_SESSION['payment']" "NULL" ist, und so durch "$GLOBALS[$_SESSION['payment']]" ersetzt wird...

(Mit "$$_SESSION['payment']" verlässt man sich m.E. darauf, dass "register_globals" auf "on" gesetzt ist.)

Vor

PHP:

if (isset ($_SESSION['tmp_oID']) && is_int($_SESSION['tmp_oID'])) {

einfügen

PHP:

//Avenger
$payment=$GLOBALS[$_SESSION['payment']];
$updates_names=array('stock_update_sql','status_update_sql','properties_update_data','attributes_update_sql');
//Avenger

Alle

PHP:

$$_SESSION['payment']

ersetzen mit

PHP:

$payment

Nach

PHP:

        $tmp_status = $payment->tmpStatus;

einfügen

PHP:

    //Avenger     
    //Initialize stock update parameters arrays
    foreach ($updates_names as $updates_name)
    {
      $_SESSION{$updates_name}=array();
    }
    //Avenger

PHP:

            xtc_db_query("update  ".TABLE_PRODUCTS." set products_quantity = '".$stock_left."' where  products_id = '".xtc_get_prid($order->products[$i]['id'])."'");
            // BOF GM_MOD:
            if (($stock_left < 1) && (STOCK_ALLOW_CHECKOUT ==  'false') && GM_SET_OUT_OF_STOCK_PRODUCTS == 'true') {
                xtc_db_query("update ".TABLE_PRODUCTS." set  products_status = '0' where products_id =  '".xtc_get_prid($order->products[$i]['id'])."'");
            }

ersetzen mit

PHP:

      //Avenger      
      //Delayed stock updates
      //For external payment-methods adjust stock only after real payment!
      //In that case store updates sql in session and do not apply directly!
      $update="UPDATE ".TABLE_PRODUCTS." SET ";
      $products_id=xtc_get_prid($order->products[$i]['id']);
      $where=" WHERE products_id = '".$products_id."'";
      $stock_update_sql=$update."products_quantity = '".$stock_left."'".$where;      
      $status_update_sql=$update."products_status = '0' ".$where;      
      $no_stock_left=$stock_left < 1 && (STOCK_ALLOW_CHECKOUT == 'false') && GM_SET_OUT_OF_STOCK_PRODUCTS == 'true';
      if ($tmp)                  
      {
        $_SESSION{'stock_update_sql'}[]=$stock_update_sql;
        // BOF GM_MOD:
        if ($no_stock_left)
        {
          $_SESSION{'status_update_sql'}[]=$status_update_sql;
        }
      }
      else
      {
        xtc_db_query($stock_update_sql);
        // BOF GM_MOD:
        if ($no_stock_left)
        {
          xtc_db_query(xtc_db_query($status_update_sql));
        }
      }
      //Avenger

PHP:

        if(($t_use_combis_quantity == 0 && STOCK_CHECK ==  'true' && ATTRIBUTES_STOCK_CHECK == 'true') ||  $t_use_combis_quantity == 2){
            $t_quantity_change = $order->products[$i]['qty'] * -1;
            $val = $coo_properties->change_combis_quantity($t_combis_id, $t_quantity_change);
        }

ersetzen mit

PHP:

       if(($t_use_combis_quantity == 0 && STOCK_CHECK == 'true'  && ATTRIBUTES_STOCK_CHECK == 'true') || $t_use_combis_quantity  == 2){
        //Avenger
    //Delayed stock updates
    //For external payment-methods adjust stock only after real payment! 
        $t_quantity_change = -$order->products[$i]['qty'];
        if ($tmp)
        {
          $_SESSION['properties_update_data'][$t_combis_id]=$t_quantity_change;
        }
        else
        {
          $val = $coo_properties->change_combis_quantity($t_combis_id, $t_quantity_change);
        }
        //Avenger
      }

PHP:

            xtc_db_query("UPDATE ".TABLE_PRODUCTS_ATTRIBUTES." set
                                                       attributes_stock=attributes_stock - '".$order->products[$i]['qty']."'
                                                       where
                                                       products_id='".$order->products[$i]['id']."'
                                                        and  options_values_id='".$order->products[$i]['attributes'][$j]['value_id']."'
                                                        and  options_id='".$order->products[$i]['attributes'][$j]['option_id']."'
                                                       ");

ersetzen mit

PHP:

      //Avenger
    //Delayed stock updates
    //For external payment-methods adjust stock only after real payment! 
      $sql="
        UPDATE 
          ".TABLE_PRODUCTS_ATTRIBUTES." 
        SET
           attributes_stock=attributes_stock - '".$order->products[$i]['qty']."'
        WHERE
           products_id='".$products_id."'
           AND options_values_id='".$order->products[$i]['attributes'][$j]['value_id']."'
           AND options_id='".$order->products[$i]['attributes'][$j]['option_id']."'";
      if ($tmp)                  
      {
        $_SESSION['attributes_update_sql'][]=$sql;
      }
      else
      {
        xtc_db_query($sql);
      }
      //Avenger

Nach

PHP:

if (!$tmp) {

einfügen

PHP:

  //Avenger     
  //Delayed stock updates
  //For external payment-methods adjust stock only after real payment! 
  //In that case updates sql have been stored in session and not have been applied directly!
  foreach ($updates_names as $updates_name)
  {
    $update_sqls=$_SESSION{$updates_name};
    if ($update_sqls)
    {
      if ($updates_name=='properties_update_data')
      {
        $coo_properties = MainFactory::create_object('PropertiesControl');
        foreach ($update_sqls as $t_combis_id=>$t_quantity_change)
        {
          $val = $coo_properties->change_combis_quantity($t_combis_id, $t_quantity_change);
        }
      }
      else
      {
        foreach ($update_sqls as $update_sql)
        {
          xtc_db_query($update_sqls);
        }
      }
    }
    unset($_SESSION{$updates_name});
  }
  //Avenger

Wie immer gilt:

Anwendung ausschließlich auf eigenes Risiko des Verwenders.

Eine Gewährleistung jeglicher Art ist ausgeschlossen.

Vor Einbau eine Sicherung der Shop-Programme vornehmen.

Erst im Testshop testen.

 

Ich kann ja schon ganz schön viel, aber zaubern (noch?) nicht.

Allerdings mit meinem USERMOD-Konzept kann man auch das updatesicher machen, da man im USERMOD-Bereich ein modifizierte Kopie davon anlegen kann.

Die Lösung liegt ja vor....

 

Das hatten wir ja auf dem User-Meeting schon diskutiert....

Dein Ansatz ist eigentlich logisch, weil die Bestellung im Prinzip nicht mit der Zahlung verbunden ist.

Daher wäre es logisch direkt nach Bestellung die Bestätigung zu versenden, statt auf die erfolgreiche Zahlung zu warten.

Gambio arbeitet aber nicht so, sondern bestätigt bei externen Zahlungen erst, wenn die Zahlung Erfolg hatte.

 

Im Moment führt es aber dazu, dass der Lagerbestand auch sinkt, wenn die Bezahlung abgebrochen wird, und der Besteller auf Nimmerwiedersehen.verschwindet....