Payone Giropay erwartet 4-16 stellige Referenznummer

[Shoekrates]

Isotope übergibt als Referenznummer die ID der Bestellung. Diese ist natürlich erst ab ID 1000 vierstellig, so dass es unterhalb dieser Grenze zur Ablehnung bei Giropay führt.

Die Fehlermeldung, die von Payone zurückgegeben wird, wenn es auf dieses Problem trifft, ist leider nicht sehr aussagekräftig: „Ungültiger Name des Kontoinhabers“.

Wir haben den Hinweis auf die Vorgabe von 4-16 Stellen bei der Zahlungsart Giropay vom Payone Support selbst bekommen.
Im PMI von Payone lässt sich diesbezüglich auch nichts konfigurieren, so dass die Anpassung letztlich in Isotope vorgenommen werden muss.

Da wir nicht einfach das Core-Zahlungsmodul von Isotope umbauen wollen, stellt sich uns die Frage, ob es diese Vorgaben bei anderen Zahlungsanbietern/-modulen eventuell auch gibt, was hier wiederverwendet werden könnte? Eigentlich ist das doch etwas, dass behoben werden müsste, oder?

[Shoekrates]

Ich erweitere mal die Infos, vielleicht hat ja doch jemand einen Hinweis für uns?

Installiert ist:
Contao 3.5.1
Isotope 2.3.4 Build 8
Isotope Zahlungsmodul SEPA-Lastschrift 2.0.5 Build 3
Isotope EU MwSt-ID (Umsatzsteuer-ID) Behandlung 0.3.4 Build 23

Kann vielleicht jemand, der etwas mehr Überblick über die Programmierung der Core-Zahlungsmodule hat, sagen, ob es ähnliche Probleme schon mal gegeben hat? Oder ob für andere Zahlungsanbieter ähnliche Vorgaben bestehen, an die wir unsere Lösung anlehnen können?

[Shoekrates]

Und nochmal etwas mehr:

Es geht um die Funktion checkoutForm() in der Datei /system/modules/isotope/library/Isotope/Model/PaymentPayone.php (ab Zeile 88).
Hier steht in Zeile 100:
'reference' => $objOrder->id,
Und genau das wird bei Giropay via Payone erst ab der Bestell-ID 1000 funktionieren, da Giropay eine mindestens 4-stellige Referenznummer benötigt.

[aschempp]

vierstellig wäre die Nummer aber auch wenn sie 0001 lautet, oder? Wäre Giropay eventuell damit zufrieden, könnt ihr das ausprobieren? Einfach ein str_pad($objOrder->id, 4, '0', STR_PAD_LEFT) für reference verwenden.

[Shoekrates]

Ist zwar schon was her, aber ich möchte wenigstens noch das Feedback geben, dass deine Änderung die Lösung war – es funktioniert.
Danke!

[aschempp]

Fixed in 5091b72