h4kuna / fio

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

Key cold'nt be null. #45

Closed R3sidento closed 2 years ago

R3sidento commented 2 years ago

Podařilo se mi to rozjet podle návodu.

Nicméně transakce se vypíše ale pod jí tato chybová hláška.

Fatal error: Uncaught h4kuna\Fio\Exceptions\InvalidState: Key cold'nt be null. in /data/web/virtuals/154265/virtual/www/t-mobile/vendor/h4kuna/fio/src/Response/Read/TransactionAbstract.php:77 Stack trace: #0 /data/web/virtuals/154265/virtual/www/t-mobile/vendor/h4kuna/fio/src/Response/Read/TransactionAbstract.php(100): h4kuna\Fio\Response\Read\TransactionAbstract->key() #1 /data/web/virtuals/154265/virtual/www/t-mobile/_cron/banka_platby.php(22): h4kuna\Fio\Response\Read\TransactionAbstract->valid() #2 {main} thrown in /data/web/virtuals/154265/virtual/www/t-mobile/vendor/h4kuna/fio/src/Response/Read/TransactionAbstract.php on line 77

Za boha nevím co s tím.

h4kuna commented 2 years ago

Myslíš, že by jsi mi mohl poskytnut ten json co se stáhne? Evidentně se tam dostala null hodnota, která by tam neměla být a nebo tam má být a já to mám blbě navržené.

h4kuna commented 2 years ago

Už to vidím, není potřeba.

h4kuna commented 2 years ago

zkus dev-master když to bude ok, vydám další verzi

R3sidento commented 2 years ago

Testovací verze je dustupná zde.

https://ospld-brno.cz/t-mobile/_cron/banka_platby.php

pá 21. 10. 2022 v 11:44 odesílatel Milan Matějček @.***> napsal:

Už to vidím, není potřeba.

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

-- Bušek Martin E-mail - @.*** Telefon - 607 / 826 131

R3sidento commented 2 years ago

dev-master je Ok chyba zmiznula, díky za FIX

[image: image.png]

Ještě takový možná hloupý dotaz. procházet ten object je docela masáž z ohledem na to kolik to vrací dat. napadlo mě, jestli jak máte již hotovou metodu getInfo(), nebylo dobrý přidat metody pro každý parametr toho co to vrací.

Např.: getVS(), getSS(), getNameAccountTo(), atd... člověk by si pak jen metodou zavolal přesně to co potřebuje podle dokumentace a nemusel by se probíjet tím objektem.

pá 21. 10. 2022 v 11:49 odesílatel Martin Bušek @.***> napsal:

Testovací verze je dustupná zde.

https://ospld-brno.cz/t-mobile/_cron/banka_platby.php

pá 21. 10. 2022 v 11:44 odesílatel Milan Matějček < @.***> napsal:

Už to vidím, není potřeba.

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

-- Bušek Martin E-mail - @.*** Telefon - 607 / 826 131

-- Bušek Martin E-mail - @.*** Telefon - 607 / 826 131

h4kuna commented 2 years ago

Já knihovnu teď osobně nikde nepoužívám, takže nad tím nepřemýšlím, ale klidně udělej PR, kouknu na to.

h4kuna commented 2 years ago

vyřešeno, vydána verze 2.0.7