search

axotion / laravel-dotpay

Paczka do Laravela umożliwająca wysyłanie danych bezpośrednio po API
MIT License
12 stars 11 forks source link

Update Client.php #6

Closed mgralikowski closed 6 years ago

mgralikowski commented 6 years ago

Fix 403 error caused by special chars like polish "ó" for example in recipient company name. Json_encode with 320 flag is recommended by DotPay Docs (API Panelu Administracyjnego Sprzedawcy wersja 1.35.4.2, Page 7).

Description

Zmiana metody wysyłania zmiennych,

Motivation and context

403 error. Bez tego wysłanie odbiorcy przelewu np. ze znakiem ó nie przejdzie. Niestety DotPay nie pomagał ogólnym errorem i znalezienie tego błędu zajęło mi kilka godzin. Błąd ujawnia się w nietypowych sytuacjach więc mógł nie zostać zauważony.

How has this been tested?

Testy użytkowe.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

axotion commented 6 years ago

Sprawdziłem i rzeczywiście taki wyjątek występuje. Dzięki za PR'a!

mgralikowski commented 6 years ago

Hej, można zaktualizować repo/packagist bo composer pobiera bez tej poprawki?

axotion commented 6 years ago

Wypuściłem release z tym fixem. Powinien być już dostępny na packagist.