PayPal order_status automatisch setzen Problem

[noRiddle (revilonetz)]

Hallo Community.
Es gibt in diesem Forum einige Threads wo moniert wird, daß nach Abschluß einer Bestellung per
PayPal Kaufabwicklung (Modul paypal)
oder
PayPal Express (Modul paypalexpress)
und erfolgreicher Zahlung (payment_status in DB-Tabelle paypal 'complete')
der order_status in der DB-Tabelle orders nicht gesetzt wird welcher im Backend unter
Erweiterte Konfiguration => Partner => PayPal
bei "Bestellstatus OK" (Konfigurations-Konstante PAYPAL_ORDER_STATUS_SUCCESS_ID) gesetzt ist.

Eine Lösung des Problems wurde nirgends wirklich gegeben.

Nun habe ich mir aus aktuellem Anlass mal die zuständigen Dateien angesehen.
In /includes/classes/paypal_checkout.php gibt es die Funktion write_status_history() welche den orders_status in der DB-Tabelle orders setzt.
Aufgerufen wird die Funktion jeweils in den Modulen selbst in der Funktion after_process().

In der Funktion write_status_history() gibt es folgendes Konstrukt:

Code: PHP  [Auswählen]

if($ack=="SUCCESS"  || $ack=="SUCCESSWITHWARNING") {
    $o_status = PAYPAL_ORDER_STATUS_PENDING_ID;
} else {
    $o_status = PAYPAL_ORDER_STATUS_REJECTED_ID;
}

Dieses müsste, ohne alle Vorgänge und allen Code genauestens analysiert zu haben, nach meiner Meinung so lauten:

Code: PHP  [Auswählen]

if($ack=="SUCCESS"  || $ack=="SUCCESSWITHWARNING") {
    $o_status = PAYPAL_ORDER_STATUS_SUCCESS_ID; //set right status, noRiddle
} else {
    $o_status = PAYPAL_ORDER_STATUS_REJECTED_ID;
}

Zumindest aber müsste weiter unterteilt werden, möchte man z.B. bei $ack=="SUCCESSWITHWARNING" einen anderen Status setzen als bei $ack=="SUCCESS".

Fazit:
Der Status den man im Backend bei "Bestellstatus OK" (s.o.) bestimmt hat kann mit dem bisherigen Code gar nicht korrekt gesetzt werden.

Irre ich ?, übersehe ich etwas ?

Vielen Dank für Feedback, zumindest von einigen Wissenden und Fähigen hier im Forum.

Gruß,
noRiddle

Linkback: https://www.modified-shop.org/forum/index.php?topic=32740.0

[webald]

Wenn ich die Codestelle ansehe, wäre ich Deinre Meinung, aber ... kannst Du mal in einer Problem-DB nachsehen welche Werte in der Tabelle configuration für PAYPAL_ORDER_STATUS_SUCCESS_ID und PAYPAL_ORDER_STATUS_PENDING_ID eingetragen sind?

includes/modules/payment/paypal.php:

Code: PHP  [Auswählen]

                $new_config[] = array('','PAYPAL_ORDER_STATUS_SUCCESS_ID', (($PAYPAL_INST_ORDER_STATUS_SUCCESS_ID) ? $PAYPAL_INST_ORDER_STATUS_SUCCESS_ID : '2'), 111125, 10, 'xtc_get_order_status_name', 'xtc_cfg_pull_down_order_statuses(');
                $new_config[] = array('','PAYPAL_ORDER_STATUS_PENDING_ID', (($PAYPAL_INST_ORDER_STATUS_PENDING_ID) ? $PAYPAL_INST_ORDER_STATUS_PENDING_ID : '5'), 111125, 11, 'xtc_get_order_status_name', 'xtc_cfg_pull_down_order_statuses(');
 

In meiner DB stehen da nämlich andere Werte. PAYPAL_ORDER_STATUS_SUCCESS_ID = 5 und PAYPAL_ORDER_STATUS_PENDING_ID = 6.

Ohne da weiter im Code nachgesehen zu haben, würde ich sagen das sieht komisch aus.

[noRiddle (revilonetz)]

Der Code den du zitierst wird ausgeführt wenn das Modul neu installiert wird.
Ist es installiert stehen die gesetzten Werte in der DB.
Die PAYPAL_ORDER_STATUS_SUCCESS_ID ist korrekt gesetzt wie in Erweiterte Konfiguration => Partner => PayPal bei "Bestellstatus OK" bestimmt (was man ja auch im Backend an besagter Stelle sieht, ohne in die DB zu schauen, da der richtige Wert im Dropdown auf selected="selected" steht).

Du denkst also auch, daß in der Funktion write_status_history() ein Fehler ist ?

Gruß,
noRiddle

[Tomcraft]

Ich habe das mal in Ticket #462 aufgenommen.

Grüße

Torsten

[webald]

@noRiddle
wie gesagt ich bin da auch nicht in die Tiefe gegangen, aber so ganz erschließt sich mir diese Codestelle auch nicht. Ich habe jetzt mal den ein oder anderen xt-Clone angesehen, und da sieht das auch so aus. Die Müßten dann doch eigentlich die gleichen Probleme haben.

Anbei mal die ipn.php für callback/paypal/ipn.php. Mit dieser Version werden die von Paypal zurückgelieferten Daten in ein Logfile geschrieben. Bitte vorher ein Verzeichnis shoproot/paypaltest anlegen. Evtl. bringt das brauchbare Daten.

[noRiddle (revilonetz)]

@Tomcraft
Merci.

Ich habe mir mal die entsprechende Code-Stelle aus der /includes/classes/paypal_checkout.php in Gambio-Versionen angeschaut (und da funktioniert die orders_status-Änderung).
Dort ist die Codestelle anders formuliert aber im Endeffekt so wie ich vermute wie es sein soll:

Code: PHP  [Auswählen]

$ack = strtoupper($_SESSION['reshash']['ACK']);
if($ack != 'SUCCESS') {
    $o_status = PAYPAL_ORDER_STATUS_REJECTED_ID;
} else {
    $o_status = PAYPAL_ORDER_STATUS_SUCCESS_ID; // <= das ist das Entscheidende, noRiddle
}

@webald
Momentan habe ich leider nicht die Zeit und auch keine wirkliche Testumgebung um zu Logs zu testen, vielleicht jemand anders ?

Gruß,
noRiddle

[Bonner]

Hallo,

ich bau es mal ein..sobald Ergebnisse da sind, melde ich mich.

Bonner

[noRiddle (revilonetz)]

Hallo Stadt-Kollege .
Bitte beachten:

• Nimm nicht die Gambio-Version sondern die Version aus meinem ersten Post !
• Der Test macht natürlich nur Sinn wenn du bislang auch keine Order-Status-Änderungen hattest.

Danke für's Testen.

Gruß,
noRiddle

[webald]

Gerhard hat das wohl schon gelöst und das Ticket geschlossen.

[Tomcraft]

Ist in r8166 wie folgt korrigiert worden:

Code: PHP  [Auswählen]

      $ack = strtoupper($_SESSION['reshash']["ACK"]);
      if ($ack=="SUCCESS") {
        $o_status = PAYPAL_ORDER_STATUS_SUCCESS_ID;
      } elseif ($ack=="SUCCESSWITHWARNING") {
        $o_status = PAYPAL_ORDER_STATUS_PENDING_ID;
      } else {
        $o_status = PAYPAL_ORDER_STATUS_REJECTED_ID;
      }

Grüße

Torsten

[Bonner]

@webald
Das funktioniert nicht..hat bei mir jedoch nix reingeschrieben.

@noRiddle
Manchmal neige ich ja zu Leichtsinn und habe das dann einfach mal direkt im Livebetrieb eingebaut.
..und bei der kurz darauf erfolgten Bestellung hat das auch funktioniert. Ich schau mal, wie sich das weiterentwickelt, insbesondere bei Abbruch bzw. Ablehnung (was allerdings noch nie vorgekommen ist!).

Screenshots anbei.
Grüße auf die andere Rheinseite!

Bonner

[Bonner]

manchmal ist es wirklich so: gut Ding will Weile haben! 

Das Problem wird in diversen Threads angesprochen und keiner hat ne Lösung. Und dann klappt alles in 24 Stunden! Cool!
an alle!

Bonner

[webald]

weil sich einer wie noRiddle hingesetzt und den Code durchsucht hat und dabei eine Idee hatte....

[noRiddle (revilonetz)]

Naja, ein noRiddle hat sich nicht hingesetzt und den Code durchsucht, zumindest nicht einfach so .
Du weißt ja wie das ist. Man hat es schon oft gelesen/gehört, will immer mal danach schauen und kommt jedoch zeitlich nicht dazu.
Dann hat ein Kunde das Problem, man schaut mal genauer und schwupps haben alle etwas davon.

@Team
Danke für's schnelle Prüfen und Fixen.

@Bonner
Merci für deine Tests.
Grüße zurück auf die andere (richtige ? ) Rheinseite.

Gruß,
noRiddle

[Bonner]

Ja, so geht das. Wenn ich auf meine ToDo Liste schaue  ..aber ich bin ja nicht Dr. Kimble und auf der Flucht! 

@noRiddle
Grüße auf die andere Rheinseite (gibt es eigentlich eine richtige?  )..aber ich als Ruhrpottler sehe das sowieso entspannt 

Bonner