Skip to content

replace Event::fire() with Event::dispatch() for L5.8#241

Open
MostafaAttia wants to merge 1 commit intothedevdojo:masterfrom
MostafaAttia:hotfix/laravel_5.8
Open

replace Event::fire() with Event::dispatch() for L5.8#241
MostafaAttia wants to merge 1 commit intothedevdojo:masterfrom
MostafaAttia:hotfix/laravel_5.8

Conversation

@MostafaAttia
Copy link
Copy Markdown

Can't create a new discussion when using Laravel 5.8:

Symfony\Component\Debug\Exception\FatalThrowableError Call to undefined method Illuminate\Events\Dispatcher::fire().

Illuminate\Events\Dispatcher::fire() changed to dispatch in Laravel 5.8

@marcinsdance
Copy link
Copy Markdown

This fix worked for me on L5.8 - please merge it.

@ChafikHadjAbdouRazack
Copy link
Copy Markdown

please merge it.

@shez1983
Copy link
Copy Markdown

i am not sure what happened but in every branch this has reverted to fire() ??
Screenshot 2019-07-23 at 14 22 45

@igolubic
Copy link
Copy Markdown

igolubic commented Sep 2, 2019

Can this be merged?

@mattglover11
Copy link
Copy Markdown

Please merge this - i just raised an issue relating to this.

@sudheerb9
Copy link
Copy Markdown

Yes, This worked for me. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants