search

nemiah / phpFinTS

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

Migrate away from utf8_encode() and utf8_decode() #401

Closed Philipp91 closed 1 year ago

Philipp91 commented 1 year ago

Fixes #400

Philipp91 commented 1 year ago

@nemiah Danke. Es wäre übrigens schöner, die Commits nicht zu squashen. Die Funktion macht Sinn, wenn Leute zig Commits für eine kleine Sache hochladen, gerade wenn dann im Laufe eines Reviews noch mehr Commits à la "Kommentare bearbeitet" dazu kommen. Aber zumindest in meinem Fall fasse ich die Änderungen immer zu Commits zusammen, für sich alleine Sinn machen, ggf. dann mit rebase -i. Hier z.B. ist der Breaking Change (ext-mbstring) in einem Commit, dann der große Haufen Test-Code in einem weiteren, und der kleinere Rest in einem dritten. Dann kann man gezielter drüber schauen ohne ewig zu scrollen.

nemiah commented 1 year ago

Kann ich machen, ich hab nur auf den Knopf geklickt und das war vorausgewählt :grin:

Magellanth commented 1 year ago

@nemiah Könnte man zeitnah eine Release Version mit diesem Stand machen? Auf PHP 8.2 wirft utf_encode Deprecated Meldungen...

nemiah commented 1 year ago

Done :heavy_check_mark: