Problem/Motivation

ContainerAwareEventDispatcher is deprecated

The ContainerAwareEventDispatcher is used in a type annotation, which should be updated.

Steps to reproduce

Proposed resolution

Update the type annotation to use the Symfony EventDispatcher class, following the guidance in the linked change record above.

Remaining tasks

Issue fork search_api-3524812

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

weekbeforenext created an issue. See original summary.

weekbeforenext opened merge request !236

weekbeforenext’s picture

weekbeforenext
Primary language English
Location Asheville, NC
commented
Status: Active » Needs review

Replaced the use of ContainerAwareEventDispatcher and changed existing updated type annotations to use Symfony\Component\EventDispatcher\EventDispatcherInterface instead of Symfony\Contracts\EventDispatcher\EventDispatcherInterface. That last change is more of a nitpick update because they are one and the same.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

Thanks a lot, looks great and makes sense.
Merged.
Thanks again!

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Title: Deprecate ContainerAwareEventDispatcher » Remove reference to deprecated ContainerAwareEventDispatcher

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.