PHP 7.1 + Typehints

[mhujer]

TODO:

• [x] declare(strict_types = 1);
• [x] DateTimeImmutable
• [x] PHP 7.1
• [x] doplnit typehinty pro primitivní typy
• [x] id transakce je int nebo float?
• [x] otestovat na reálných datech

Closes #8

[mhujer]

@soukicz Až se k tomu dostaneš, tak bych tě poprosil o:

1. review po commitech
2. otestování u tebe a na tvých datech (když jsem zkusil naparsovat celou historii svého účtu u Fia, tak to pár špatných typehintů odhalilo)

[mhujer]

@soukicz ping?

[fmasa]

@mhujer Co void return typy u metod?

[fmasa]

Fio API PHP works with PHP 7.1 or higher.

[mhujer]

@fmasa jo, to je dobrý point, ty by tam mohly být taky 👍

[fmasa]

Jinak je něco jinýho, co brání mergnutí? :slightly_smiling_face: Můžu ještě zkusit projet naše transakce a zjistit, jestli všechno funguje.

[mhujer]

@fmasa jo, to by bylo super, pokud bys to otestoval, já to nemám nikde v produkci

[fmasa]

@mhujer Tak potvrzuju, i na našich produkčních datech funguje ok. :slightly_smiling_face:

Zvážil bych jen, jestli by getId() nemělo vracet místo float spíš string, protože API vždycky vrací 10 numerických znaků.

[mhujer]

@fmasa @soukicz Díky, konečně mergnuto!

[mhujer]

Releasnuto 4.0.0

[soukicz]

Tak přeci jen problém. Pokusím se to izolovat a poslat PR

[fmasa]

To by asi taky nemělo být float anyway.

[soukicz]

Dělá to u EUR účtu, tak pošlu aspoň failující test, pokud nevymyslím rovnou opravu.

[soukicz]

Tak nakonec plaším jen s prkotinou - https://github.com/mhujer/fio-api-php/pull/12