-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Potential breaking change for rhumsaa/uuid #86
Comments
Thanks for alerting us to this. For this library the Uuid class is vendored in Raygun4php/Uuid.php and placed beneath its own namespace I take it you're using Composer - in your |
Exactly. I think this should avoid any collision problems, as well.
I see no other dependencies on Rhumsaa/uuid in my source, but upon digging into it more I'm not quite sure how Uuid can be causing issues in raygun4php since, like you've said, you've namespaced it yourselves to prevent specifically these kinds of issues. I'm going to go ahead and close this ticket until I can find more definitive proof that the error is actually due to Uuid as it is included with raygun4php. Thanks for the input. |
I'm getting the following error using Raygun4php:
It looks like recently the author decided to re-namespace the package. From https://github.com/ramsey/uuid:
It looks like this may be a breaking change for Raygun4Php since the library for generating UUIDs has been renamespaced out from under it.
However, upon investigating the source of raygun4php more, it looks like the Uuid file is bundled with the soruce as opposed to being loaded via composer, which is what I initially could lead to this error. If that's not the case, though, what could be leading to this error so far down in the Raygun4php source?
Thanks.
The text was updated successfully, but these errors were encountered: