search

h4kuna / fio

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

Chyba při vytváření instance FioFactory #50

Closed dominikmiskovec closed 1 year ago

dominikmiskovec commented 1 year ago

Při vytváření instance FioFactory kód hlásí chybu.

h4kuna\Fio\FioFactory::createClientInterface(): Return value must be of type Psr\Http\Client\ClientInterface, GuzzleHttp\Client returned

h4kuna commented 1 year ago

Můžeš mi poslat výpis balíků z composeru? Protože testy prochází a tento kód se testuje. Takže Muselo dojít k nainstalování nějakého balíku se kterým to úplně neladí.

Zajímá mě guzzlehttp/guzzle ale když pošleš celý composer show nebudu se zlobit.

Edit: chybu se mi zatím nepodařilo nasimulovat. A testuju pro guzzlehttp/guzzle 7.5.0

h4kuna commented 1 year ago

Dokážeš aktualizovat guzzlehttp/guzzle na ^7.0?

dominikmiskovec commented 1 year ago

Jasný, vyzkouším. Taky mě to teď napadlo. Dám hned vědět.

h4kuna commented 1 year ago

Opravu zkusím udělat jen aktualizace pokud je možná bude rychlejší.

h4kuna commented 1 year ago

Oprava nepůjde, až guzzle 7.0+ implementoval Psr\Http\Client\ClientInterface. To znamená, že mohu dát jen hlášení o této skutečnosti.

h4kuna commented 1 year ago

Nebo je také možné použít symfony/http pak je potřeba FioFactory podědit a upravit metody, co instancují guzzle, aby se použilo symfony. Ale asi nechceš mít v projektu dvě knihovny co dělají to samé, možná je dočasně než vyřešíš update guzzle.

dominikmiskovec commented 1 year ago

Jo už vidím, jedna knihovna má dependency na starší verzi guzzle, musím knihovnu aktualizovat nebo vyměnit za jinou.

Tímpádem problém vyřešený :) Díky moc!