laravel-friendships
laravel-friendships copied to clipboard
hootlex
Support for laravel 5.8
The fire method (which was deprecated in Laravel 5.4) of the Illuminate/Events/Dispatcher class has been removed. You should use the dispatch method instead.
@hootlex Server Requirements The Laravel framework has a few system requirements. All of these requirements are satisfied by the Laravel Homestead virtual machine, so it's highly recommended that you use Homestead as your local Laravel development environment.
However, if you are not using Homestead, you will need to make sure your server meets the following requirements:
PHP >= 7.1.3
OpenSSL PHP Extension
PDO PHP Extension
Mbstring PHP Extension
Tokenizer PHP Extension
XML PHP Extension
Ctype PHP Extension
JSON PHP Extension
BCMath PHP Extension
Travis CI only test by php 7.0.25, so it failed.
@hootlex I have try my best to make it work with laravel 5.8. It works in php 7.2 now.
Am I right in thinking that the only reason this is failing is because the tests are also being run against PHP 5.6 and 7? Laravel >= 5.6 only supports PHP >= 7.1.3.
So, if this plugin needs to work in the lastest versions of Laravel, then surely the PHP version needs to be upped? You'd then be in a situation where people using Laravel < 5.6 would have to stick to this package version before this was released, but I've seen other packages do that.
@GreenImp The pr only replace Event::fire to Event::dispatch. If you read the Event::fire method, you will find that Event::fire is a alias of Event::dispatch. So, it will work though using laravel <= 5.8.
So it's the addition of "beyondcode/laravel-dump-server": "^1.2", in the composer.js file, because that requires PHP >= 7.1?
Is that required for Laravel 5.8 (I'm assuming so)?
If so, it may be worth either dropping support for older versions of PHP, or drop compatibility with Laravel < 5.6 in future releases.
I realise that you're not the project owner here, I'm just voicing my thoughts. Maybe @hootlex can say whether this is something to do or not. I'm not sure how it can support Laravel >= 5.8 otherwise?
@GreenImp If you wanna run on Laravel >= 5.4, you only need to replace Event::fire to Event::dispatch. The other changes are added to pass test. I don't know php unit test, so I can not say it.
@hootlex @MoeCasts PHP 5.6 and 7.0 support should be dropped (only changing the yaml file should be fine ?). Starting Laravel 5.6, the minimum PHP version required was 7.1.3.
I also think the laravel version in the composer.json file should be explicitly defined. Only having 5.* is not really a good idea.
