Skip to content
This repository was archived by the owner on Feb 7, 2024. It is now read-only.

Fix namespace in StatisticsUpdated.php #32

Closed
wants to merge 1 commit into from
Closed

Fix namespace in StatisticsUpdated.php #32

wants to merge 1 commit into from

Conversation

Namoshek
Copy link

@Namoshek Namoshek commented Dec 6, 2018

The spelling mistake will cause the following error when the queue worker tries to unserialize the event class:

method_exists(): The script tried to execute a method or access a property of an incomplete object. Please ensure that the class definition "BeyondCode\LaravelWebsockets\Statistics\Events\StatisticsUpdated" of the object you are trying to operate on was loaded before unserialize() gets called or provide an autoloader to load the class definition

This may be a Windows specific bug, otherwise there would have been more reports I guess.

Btw. I also searched the other files for spelling mistakes in the namespaces, but everything looks fine.

@mpociot
Copy link
Member

mpociot commented Dec 6, 2018

Thank you for finding this! I merged #33 instead, as it fixed all namespace issues.

@mpociot mpociot closed this Dec 6, 2018
@Namoshek Namoshek deleted the patch-1 branch December 6, 2018 20:34
@Namoshek
Copy link
Author

Namoshek commented Dec 6, 2018

Now I'm wondering why I didn't find the other occurences with global search + match case. 😕

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants