search

slevomat / csob-gateway

Client library for CSOB card payment gateway
MIT License
27 stars 10 forks source link

Chyba v příkladu v Usage #25

Closed arxeiss closed 7 years ago

arxeiss commented 7 years ago

Při vytváření ApiClient není v příkladu uvedeno, že třetí parametr má být ještě URL na bránu. A pak ani v jakém formátu.

Nevím proč je to navíc nepovinný parametr, když se stejně nikde v kódu nedoplňuje, a pak PHP ani nevyhodí chybu při jeho nevyplnění. Když verze kódu je navíc pro danou verzi API, nebylo by lepší do konstruktoru ApiClient předat parametr $isProduction a podle toho doplnit URL adresu ve třídě ApiClient automaticky? Takové řešení má třeba GoPay u své implementace.

janlanger commented 7 years ago

Díky za report, signaturu konstruktoru a dokumentaci jsem opravil.

Nastavování prostředí místo URL zvážíme pro příští verze.