h4kuna / fio

Read and send payment order for FIO bank, PSR-17 ready
54 stars 25 forks source link

Variabilní symbol by měl být interpretován jako string, nikolik jako integer #49

Closed lucasow closed 1 year ago

lucasow commented 1 year ago

$national->setVariableSymbol($vs); očekává string parametr.

Už se nám však stalo, že VS řetězec u dodavatele začíná 0. V případě, že bychom s proměnou nakládali jako s číslem, úvodní 0 by se z řetězce vypustila a došlo by tak k platbě se špatným VS.

I ISDOC podle dokumentace nakládá s VS jako se string: https://www.mvcr.cz/isdoc/soubor/isdoc-schema-6-0-1-20140526-zip.aspx

h4kuna commented 1 year ago

Ok, dokážeš udělat PR?

h4kuna commented 1 year ago

Oprava je v mastru bb8f6fed5077492f05ae268eef6fdc4d699f9459. Můžeš to testnout?

lucasow commented 1 year ago

Ahoj,

dekuju za zpravu.

Asi v tomhle jeste nejsem dostatence zkusenej, ale kdyz balicek znovu nainstaluju pomoci composeru, tak tve upravy nevidim. Kdyz si stahnu knihovnu primo z GitHubu, tak se tam problemu objevi jeste vic, jelikoz uz slozky maji jinou strukturu, nez balicek, ktery mi puvodne stahl composer. Napr FioFactory neni ve slozce Utils, ale primo v src.

Delam neco nestandarne nebo spatne?

Diky za pripadnou radu.

čt 30. 3. 2023 v 8:06 odesílatel Milan Matějček @.***> napsal:

Oprava je v mastru bb8f6fe https://github.com/h4kuna/fio/commit/bb8f6fed5077492f05ae268eef6fdc4d699f9459. Můžeš to testnout?

— Reply to this email directly, view it on GitHub https://github.com/h4kuna/fio/issues/49#issuecomment-1489747897, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI5O65LYO36J6X4ZTM46BV3W6UPGRANCNFSM6AAAAAAWLUJFEU . You are receiving this because you authored the thread.Message ID: @.***>

h4kuna commented 1 year ago

V composeru v require máš

"h4kuna/fio": "^3.0"

tak to uprav na dev-master

"h4kuna/fio": "dev-master"

a potom composer update h4kuna/fio

h4kuna commented 1 year ago

Released v3.0.2