[Bug]: symfony/event-dispatcher is required
netzhuffle opened this issue · 2 comments
netzhuffle commented
Is there an existing issue for this?
- I have searched the existing issues
Describe the bug
Latest release added symfony/event-dispatcher as an optional dependency (according to composer.json, it is only required for dev).
Yet the code is written in a way that symfony/event-dispatcher is required. Thus an exception is triggered when e.g. a feature flag is disabled and the autoloader cannot find the associated classes, e.g. Symfony\Contracts\EventDispatcher\Event.
To reproduce
- Run Unleash without sentry/event-dispatcher
- Check for a disabled feature flag
Sample code (optional)
No response
Version
v1.5.081
Expected behavior
isEnabled() returns false instead of throwing an error or exception.
Logs (optional)
No response
Additional context (optional)
No response
RikudouSage commented
Fixed in 1.5.181. Let me know if it works.
netzhuffle commented
Yes, can confirm it worked. Thanks!