nemiah / phpFinTS

PHP library to communicate with FinTS/HBCI servers
MIT License
130 stars 42 forks source link

Neuer Release möglich? #11

Closed ampaze closed 5 years ago

ampaze commented 5 years ago

Wäre es möglich einen neuen Release / neue Version zu machen, damit man via Composer nicht die Minimum Stability überschreiben muss?

nemiah commented 5 years ago

Ok, v1.1 ist draußen ;)

ampaze commented 5 years ago

Vielen Dank,

aber https://packagist.org/packages/nemiah/php-fints kennt es leider nicht und damit findet composer es auch nicht.

nemiah commented 5 years ago

Ok, jetzt sollte es klappen…

ampaze commented 5 years ago

Sorry für die Verspätung. Es klappt jetzt! Lieben Dank 👍

Könnte ich dich bitten noch eine neue Version machen, mit den MT940 Änderungen? Besteht Interesse die MT940 Dialekte die ich gebaut habe zu integrieren? Wäre die Frage wie genau das aussehen kann.

nemiah commented 5 years ago

Alles klar, ich mache das Release gleich.

Die Dialekte können wir gerne einbauen. Es geht dabei ums Abholen der Umsätze, oder?

ampaze commented 5 years ago

Danke :)

Ja es geht um das Abholen der Umsätze, die Dialekte von denen ich spreche sind eigentlich eher Workarounds für Banken die sich nicht an die MT940 Specs halten.

Ich habe jetzt z.B. die Klasse SpardaMT940 und kann diese bei mir händisch für die Abrufe von Sparda nutzen. Allerdings würde phpFinTS bei Abrufen von Sparda natürlich diese Klasse nicht automatisch nutzen, dafür fehlt dann ein Mechanismus zu erkennen das ein abweichender MT940 Parser genutzt werden müsste. Wie der aussehen könnte, wäre die Frage.

nemiah commented 5 years ago

Ich könnte mir vorstellen, dass die "Dialekte" automatisch in der getStatementOfAccount() über die BLZ verwendet werden. Weil sonst müssten wir die Auswahl letztlich dem Benutzer überlassen und er wird auch nichts anderes tun, als den "Dialekt" für seine Bank zu verwenden.

ampaze commented 5 years ago

Leider hat die Sparda nicht nur eine BLZ und auch nicht nur ein BIC. Aber vlt könnte man es über die FinTS Url mappen. Aber ich weiß natürlich nicht ob nur Sparda die URL nutzt (fints.bankingonline.de) oder ob jede andere Bank die auch diese URL nutzt dieselben Workarounds benötigt. 😞

nemiah commented 5 years ago

Also ich habe gerade durch die fints_insitute.csv durchgeschaut. Nur bei Sparda-Banken ist fints.bankingonline.de eingetragen. Ich würde es daher einfach mal an der URL festmachen. Ich persönlich schätze die Wahrscheinlichkeit hoch ein, dass es bei allen Sparda-Banken mit dieser URL gleich ist.

ampaze commented 5 years ago

Okay 👍, dann werde ich dazu einen Pull Request machen. Kann aber noch ein paar Tage dauern.

ampaze commented 5 years ago

Hab einen Pull-Request dazu gemacht, was meinst du?

nemiah commented 5 years ago

Sorry, war viel los die letzten Tage. Habs mir gerade angesehen und sieht gut aus, finde ich :smiley:

ampaze commented 5 years ago

Ja dachte mir schon sowas 😀 Danke!

Wenn du noch Zeit hast die nächsten Tage, würde ich mich auch noch über eine neue Version / Release freuen.

nemiah commented 5 years ago

Ist erledigt :wink:

ampaze commented 5 years ago

Ganz vergessen den Fall zu schließen :) Danke!