antonioribeiro / support

Support Classes
BSD 3-Clause "New" or "Revised" License
59 stars 49 forks source link

Drop laravel 4 compatibility #15

Closed fgreinus closed 5 years ago

fgreinus commented 7 years ago

Hey there,

i tried to come up with a solution for issue #13 .

The only clean way to do this (in my opinion) is to drop support for laravel 4.

I also refactored db_listen function to be able to remove the isLaravel53 function. Thing is, that when accessing Laravel::VERSION from within the package, the php artisan optimize command seem to bug. Maybe some double-loaded application stack due to the helpers file being loaded too early or so...

These changes fixed the problem.

BTW: Taylor's opinion on how to support different laravel versions https://twitter.com/taylorotwell/status/512050601115414528.

This is a reopen of PR #14