search

gerardojbaez / laraplans

SaaS style recurring plans for Laravel.
http://laraplans.readthedocs.io
MIT License
180 stars 80 forks source link

Swapped and for Laravel 5.3 #1

Closed Gerhardt0011 closed 7 years ago

Gerhardt0011 commented 7 years ago

It seems that for Laravel 5.3 the $key and $value needs to swap, else the package returns null from these methods.

gerardojbaez commented 7 years ago

Thanks for the PR! I'll merge it and correct the code, the change should be on the function parameters.

gerardojbaez commented 7 years ago

I have replaced the first() function with the traditional foreach to be backward compatible. All tests passes using illuminate/support v5.3.