Keine Versandbest?tigung mehr (?)

[Nachtrag]

Nach ein bisschen “probieren” :: die Mail wird nicht versendet, wenn das Syncen der Trackingdaten zu PayPal fehl schl?gt (Woocommerce-Problem, wie es scheint) … zumindest funktionieren im Staging alle Bestellungen, die entweder Vorkasse sind oder bei denen aus anderen Gründen keine Tracking-ID zu PayPal gesyncet werden soll.

Die Woocommerce-eigene “Bestellung abgeschlossen” – Mail ist von dem Problem nicht betroffen – die wird immer versendet.

[Nachtrag 2]

Im Log steht dann sowas:

2022-11-24T17:10:10+00:00 ERROR Error saving shipment #61

kommt, wie mir scheint, aus src/Shipment.php :: 2760

• This reply was modified 1 year, 11 months ago by wolkenkrieger.

[Nachtrag 3]

Jupp, bei

do_action( "woocommerce_gzd_{$hook_postfix}shipment_after_save", $this, $is_new );

fliegt die Exception ??

Ich habe den Machern von PayPalPayments hier schon Bescheid gegeben.

Das ist gerade ein echtes Problem, weil meine Kunden keine Versandbenachrichtigung bekommen :/

[Nachtrag 4]

Quickfix:

add_filter( 'woocommerce_paypal_payments_sync_gzd_tracking', '__return_false' );

unterbindet den Versuch, den Shipment-Status an PayPal zu übermitteln, nachdem das Shgipment als solches gespeichert wurde.

Der Ball liegt also nun im Spielfeld der PayPalPayment-Macher ??

Hi,

oh, mir war bislang nicht bewusst, dass man dort eine Kompatibilit?t speziell für Germanized integriert hat. Ich habe ein ?hnliches Problem in unserem Help Desk reported bekommen und konnte das bislang nicht zuordnen. Danke für die Analyse – da muss man wohl im PP Plugin nacharbeiten.

Grü?e

da muss man wohl im PP Plugin nacharbeiten

Naja. Ja, eigentlich, schon. Aber aus meiner Sicht müsstet ihr dort auch Hand anlegen. Alles, was das Shipment ansich betrifft, kann man ja ruhig in einem try-catch drin haben.

Sonderbehandlungen oder Hooks sollten imho selbst eingekapselt sein.

Du siehst ja, was passiert: die eigentliche Funktionalit?t l?uft. Eine externe Quelle funkt aber jetzt derma?en dazwischen, dass ganz wichtige Core-Funktionalit?t (n?mlich der Status-Switch und die Trigger, die dadurch ausgel?st werden sollen/müssen) nicht mehr da ist.

Ich hab in der Shipment.php einfach mal das letzte do_action() auskommentiert und alles l?uft, wie es soll –> Staus wird ge?ndert und Mail wird versendet.

Da solltet ihr wirklich drüber nachdenken ??

Hi,

Sonderbehandlungen oder Hooks sollten imho selbst eingekapselt sein.

Was meinst du mit eingekapselt? Auch im Woo Core wird das sehr ?hnlich gehandhabt. Das System mit Hooks & Filtern ist eben Vor- und Nachteil zugleich. Das Werfen einer Exception sollte ja nur erfolgen, wenn wirklich ein harter Fehler auftritt – wenn ein Entwickler das falsch macht, dann betrifft das eben die jeweilige Erweiterung. ?rgerlich natürlich, dass das eine relativ weit verbreitete Erweiterung ist, die offenbar nicht ordentlich getestet wurde.

Grü?e

Was meinst du mit eingekapselt?

Wenn du das do_action() selber in einen try-catch-Block packst, gehst du sicher, dass dein Code sauber ausgeführt wird und du dich um Fehler kümmern kannst, die in deiner Verantwortung liegen.

Das ist sicher nicht die sonst gelebte Praxis in Woocommerce – weis ich ??

Das Problem jetzt ist aber, dass eine ganz ganz wichtige Funktionalit?t nicht mehr da ist. Es gibt sonst keine M?glichkeit, die Versandmail zu versenden, richtig?

Selbstredend ist das eigentliche Problem eine besch*ssene Fehlerbehandlung im Payment-Plugin. Klar.

Hi,

Wenn du das do_action() selber in einen try-catch-Block packst, gehst du sicher, dass dein Code sauber ausgeführt wird

Der Sinn dahinter ist natürlich, dass es durchaus m?glich sein soll, die Ausführung des weiteren Codes bei harten Fehlern zu unterbrechen, auch für Erweiterungen.

Ich würde dir vorerst empfehlen eine ?ltere Version von PayPal Payments zu installieren, bis das Problem gekl?rt bzw. in einem PayPal Payments Update behoben ist.

Grü?e