Payum / PayumLaravelPackage

Payum offers everything you need to work with payments. From simplest use cases to very advanced ones.
https://payum.forma-pro.com/
MIT License
122 stars 38 forks source link

Call to undefined method [package] on Laravel 5 #21

Closed Zenuncl closed 9 years ago

Zenuncl commented 9 years ago

its seem newest Laravel 5 delete the method Package?

or its juts i am doing some thing wrong?

I am getting this udefined method error when i try to use Payum as payment system on Laravel 5

makasim commented 9 years ago

Are you using stable Payum extension (0.14) or master. Master is not working right now because of the changes in the Payum's Core. They are not synced yet

Zenuncl commented 9 years ago

I've tried 0.14 and 0.15-dev, neither one works. Because I am using the Newest Laravel 5 version and it's not compatible with your system.

On Dec 4 Commit

Leravel team recreate a new package handle system where delete the package() method. and in your package, its seem still using the package() method which cause the problem.

Hope the comping update will solve it.

Thanks

makasim commented 9 years ago

I looked at the commit and I see that the method was removed but I cannot find any hints on how to "migrate" package from laravel4 to laravel5 or make both version works.

Could you point me out on some docs about it?

Zenuncl commented 9 years ago

Hey, Thanks you for considering my issue. I don't really have any example but just simply take look at Laravel 5's document you may get answers http://laravel.com/docs/master/packages

Thanks again

Zenuncl commented 9 years ago

and this post give some detail https://laracasts.com/discuss/channels/general-discussion/l5-latest-commit-breaks-if-using-waygenerators-or-image-internvention/?page=2

makasim commented 9 years ago

reverted support of laravel5 from composer for now https://github.com/Payum/PayumLaravelPackage/commit/2fce92d6cdf9dea7e8476ec85c69eccf96ffd636

sdeering commented 9 years ago

So to confirm this package currently doesn't support L5?

makasim commented 9 years ago

yes it is.

makasim commented 9 years ago

I was not able to find a good tutorial or upgrade manual so I skip it for now

sdeering commented 9 years ago

I am unable to install the package using composer. Which version do I use for L5?

I've tried "0.15" and "1.0-dev". Both give me errors on a clean L5 install.

If you need screenshots of errors let me know.

makasim commented 9 years ago

IT DOES NOT WORK WITH L5

sdeering commented 9 years ago

Ok that is now clear lol ^_^

makasim commented 9 years ago

The fix https://github.com/Payum/PayumLaravelPackage/pull/22