Drop IEventListener template parameter

[provokateurin]

Summary

Split out from #37390

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[come-nc]

See vimeo/psalm#7549

[provokateurin]

I see, so the type hint will actually be useful?

[come-nc]

I see, so the type hint will actually be useful?

Hopefully some day. I never found a way to make it work properly.

[provokateurin]

Hm so we either keep it and the changes in this PR to make it work in the future, or we completely drop it now since it doesn't work. I'd go with the second option, what do you think @come-nc @ChristophWurst ?

[come-nc]

Hm so we either keep it and the changes in this PR to make it work in the future, or we completely drop it now since it doesn't work. I'd go with the second option, what do you think @come-nc @ChristophWurst ?

What happens if you keep your changes except for the ones in lib/public/EventDispatcher/IEventListener.php ?

[provokateurin]

I think that'll work and have no effect as expected.

[provokateurin]

@come-nc @ChristophWurst please review again.