Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[2.3.5] Bug - PostFinance PSP #1662

Closed
HumanDigital opened this issue Jun 9, 2016 · 8 comments
Closed

[2.3.5] Bug - PostFinance PSP #1662

HumanDigital opened this issue Jun 9, 2016 · 8 comments
Labels
bug
Milestone
2.4.0

Comments

@HumanDigital
Copy link

Beim Bezahlungsprozess wird unter anderem auch die Telefonnummer der Billing-Address an PostFinance weiter gegeben. Wenn in der Telefonnummer eine runde Klammer enthalten ist, läuft die PostFinance auf einen Fehler.
Das Problem aus unserer Sicht ist, dass wir im Registationsprozess oder beim Anlegen einer neuen Versandadresse beim Telefonnummer-Feld, contaoseitig nicht unterbinden können, dass Klammern eingefügt werden, da die Eingabeprüfung "Telefonnummer" runde Klammern erlaubt.

Wir haben als Abhilfe in der Datei PSP.php, in der Funktion preparePSPParams() OWNERTELNO, einen Regex eingefügt, der uns nur die Zahlen durchlässt, ist aber halt eben nicht updatesicher...

Wir würde uns freuen, wenn in der nächsten Version ein solcher Filter drin wäre.
Vielen, vielen Dank für Eure super Arbeit. Wir haben viel Freude am Shop.
@HumanDigital HumanDigital changed the title Bug PostFinance PSP [2.3.5] Bug - PostFinance PSP Jun 9, 2016
@aschempp
Copy link
Member

Könnt ihr die nötigen Änderungen bitte als Pull Request zur Verfügung stellen?

@HumanDigital
Copy link
Author

HumanDigital commented Jun 10, 2016 •
edited by aschempp
Loading

Können wir, ist aber noch eine quick und dirty Lösung…
Dann würde ich noch genau herausfinden wollen, was die PSP zulässt.
Da wir zur Zeit grad in der Arbeit ersaufen, brauchen wir dafür etwas Zeit … kommt aber ;-)

@aschempp aschempp added the bug label Jul 4, 2016
@aschempp
Copy link
Member

aschempp commented Aug 4, 2016

ping

HumanDigital added a commit to HumanDigital/core that referenced this issue Aug 11, 2016
@HumanDigital
Update PSP.php
c207030 
Hallo Andreas,

anbei unsere Lösung zu dem Issue isotope#1662. Bezüglich Problemen, die bei Postfinance auftauchen, wenn im Telefonfeld runde Klammern verwendet werden.

Liebe Grüsse
Heidi
@HumanDigital HumanDigital mentioned this issue Aug 11, 2016
Open
@HumanDigital
Copy link
Author

HumanDigital commented Aug 21, 2016 •
edited by aschempp
Loading

Hallo Andreas,

ich habe am 11. August in Github einen Branch erstellt, die Anpassung gemacht, und einen Pull Request gemacht mit einer Nachricht an Dich.
Als ich das abgeschickt habe konnte ich meinen Anpassungen sehen … wenn ich jetzt den Pull Request aufrufe, sehe ich nur noch die Nachricht an Dich, aber nicht mehr die Änderungen im Quellcode …

Ist der Pull Request wie gewünscht bei Dir angekommen oder fehlt da noch was …

Vielen Dank für kurze Info

Liebe Grüsse
Heidi

@aschempp
Copy link
Member

Ich habe leider keinen Pull Request bekommen, kannst du den nochmals machen?

@Toflar
Copy link
Member

Toflar commented Aug 29, 2016

Ist kein PR, der Commit wurde aber referenziert: HumanDigital@c207030

@HumanDigital
Copy link
Author

HumanDigital commented Aug 29, 2016 •
edited by aschempp
Loading

Hallo Ihr Lieben,

anbei ein Screenshot von dem, was ich auf GitHub sehe …
Könnt Ihr anhand dessen sehen, was ich tun muss, damit es für Euch ein Pull Request ist?

Danke vielmal
und liebe Grüsse

Heidi

@aschempp aschempp added this to the 2.4.0 milestone Aug 30, 2016
aschempp added a commit that referenced this issue Aug 30, 2016
@aschempp
Remove unsupported characters from phone number in Postfinance paymen…
e4965b9 
…t method (#1662)
@aschempp
Copy link
Member

Übernommen in e4965b9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
2.4.0
Development

No branches or pull requests
3 participants
@Toflar @aschempp @HumanDigital