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

Update 0.15 for Laravel 5.1 #22

Closed
wants to merge 2 commits into from
Closed

Conversation

chaosfinity
Copy link
Contributor

Moved the error handler that converted responses to a function on the controller since Laravel 5 does not support adding error handlers on App any longer

Updated controllers to use the conversion method.

Updated the ServiceProvider to use new Laravel 5 configuration

$this->package('payum/payum-laravel-package');
\View::addNamespace('payum/payum', __DIR__.'/../../views');

$this->app->error(function(ReplyInterface $reply)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't it work in Laravel5?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer this way, cuz you can easily customize response just by registering another listener

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

never mind, I read you comments

@makasim
Copy link
Member

makasim commented Jun 18, 2015

And the most important, This could not be merged to 0.15 because of BC breaks. Please rebase onto master

Catch response of execute for conversion
Update PayumServiceProvder.php for Laravel 5.1

HttpRedirect is a child of HttpResponse moved the reply conversion for HttpRedirect above HttpResponse so redirects are handled as intended
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

Successfully merging this pull request may close these issues.

None yet

2 participants