Fixed removed fire method for event dispatcher

[harrygulliford]

As of Laravel 5.8, the Event::fire method has been officially removed and replaced with Event::dispatch.

https://laravel.com/docs/5.8/upgrade#events

This PR adds support for Laravel 5.8, however will mean the minimum supported version will now be 5.4+

If keeping 5.0-5.3 support is imperative, it may be possible to create a function to delegate between Event::fire and Event::dispatch based upon the value returned from App::version().

[stevebauman]

Thanks @harrygulliford!

Maybe the event() helper might be better used to support all versions of 5.X? Thoughts?

[harrygulliford]

@stevebauman Agreed! The event() helper definitely looks like the best option here.

[stevebauman]

Perfect thanks again! Are you possibly able to add the setUp(): void return type declaration for the tests to pass? I’m not at a computer at the moment unfortunately.

[stevebauman]

Whoops that was quick! Thanks! 😄

[harrygulliford]

All green! ✅