search

pay-now / paynow-php-sdk

PHP Library for Paynow API
https://docs.paynow.pl
MIT License
16 stars 10 forks source link

[2.0] odpowiedź serwera w Paynow\Service\Payment::authorize() może być pusta #31

Open jacekkarczmarczyk opened 4 years ago

jacekkarczmarczyk commented 4 years ago

STR:

  1. utwórz zamówienie na kwotę < 1zł
  2. uruchom Paynow\Service\Payment::authorize()
  3. uruchom to samo ponownie z tym samym idempotencyKey

W odpowiedzi przychodzi pusty obiekt, czyli redirectUrl/paymentId/status są null - a to powoduje błąd:

TypeError: Return value of Paynow\Response\Payment\Authorize::getPaymentId() must be of the type string, null returned Paynow\Response\Payment\Authorize::getPaymentId(), line 34

Teoretycznie dokumentacja nie przewiduje takiej odpowiedzi, ale myślę, że warto, żeby paynow-php-sdk uwzględniało ten przypadek i np. wyrzucało jakiś wyjątek

(przy okazji w Payment.php jest literówka: $decpdedApiResponse)

emilleszczak2 commented 4 years ago

@jacekkarczmarczyk Dzięki za znalezienie błędu. Nie jest to błąd w SDK lecz samym API - zgłoszę do naprawienia.