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.

Sign up for GitHub

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

Jump to bottom

Notify Controller doUnsafeAction doesn't resolve gateway name parameter #55

Open
mattiaclementi opened this issue Dec 14, 2020 · 0 comments
Open

Notify Controller doUnsafeAction doesn't resolve gateway name parameter #55

mattiaclementi opened this issue Dec 14, 2020 · 0 comments

Comments

@mattiaclementi
Copy link

As route payum_notify_do_unsafe describe, this controller should recieve a gateway_name parameter, to retrive the right gateway from payum.
Instead this method receive a Request object and cannot get the right parameter.

The test can be done easily, call route('payum_notify_do_unsafe', ['gateway_name' => 'gateway-test']) via HTTP, throws a InvalidArgumentException with messagge Gateway "" does not exist. instead of Gateway "gateway-test" does not exist.

The PR #54 fix the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mattiaclementi