Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
As a pre-requisite of #2876675: Allow symfony/event-dispatcher 4+ to be installed in Drupal 8 any maybe other issues like #3020303: Consider using Symfony flex to allow switching between major Symfony versions, we need to change the EventSubscriber/RedirectResponseSubscriberTest.
Instead of using using the Symfony\Component\EventDispatcher\EventDispatcher
which would change in an incompatible way in Symfony >= 4.3 we must use our own Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher
as EventDispatcher;
Proposed resolution
Adjust the test.
Remaining tasks
Commit the patch.
User interface changes
none
API changes
none
Data model changes
none
Release notes snippet
none
Comment | File | Size | Author |
---|---|---|---|
#2 | 3081582_use_drupal_event_dispatcher_in_RedirectResponseSubscriberTest.patch | 2.74 KB | mkalkbrenner |
Comments
Comment #2
mkalkbrennerComment #3
mkalkbrennerComment #4
mkalkbrenner.
Comment #5
hchonovThank you.
Comment #6
catchCommitted 5c98790 and pushed to 8.8.x. Thanks!