EventManagerAwareInterface and EventManagerAwareTrait parameters have different names

[rogervila]

Bug Report

On the EventManagerAwareInterface, the parameter for setEventManager method is called $eventManager while on EventManagerAwareTrait is called $events.

This causes Psalm errors when using the Trait. I can open a PR to use the same variable name, but we have to decide which one to use.

Q	A
Version(s)	3.10.x

Summary

Psalm triggers a ParamMismatch error when using the EventManagerAwareTrait

Current behavior

Affects Psalm analysis only

How to reproduce

check this Pull Request comment

Expected behavior

Once the parameter name remains the same, the error will disappear.

[gsteel]

Phew! It's been nearly 2 years! Thanks for the report - we don't support named parameters as part of BC guarantees, so feel free to send in a patch. I'd say that the interface should be the canonical version and the trait should be changed.