Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
jacekkarczmarczyk opened this issue May 22, 2020 · 1 comment

Comments

@jacekkarczmarczyk
Copy link

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants