Infinite loop when trash and eck_bundle_permissions modules are installed

To prevent the BC break and the need for a new major branch, we could allow EntityTypeInterface|string for the $entity_type_id parameter and, if the argument is an instance of EntityTypeInterface, get the entity type ID from it :)

Are you sure a method parameter type change is not a BC break? It wouldn't be for callers, but it would be for implementers. Not sure how big of a problem that would be in practice though, probably not (m)any other implementations out there.

If we really want to be sure, we can completely remove the type from the argument and replace it with an assertion on the first line of the method. And yeah, I'm also pretty sure there aren't many trash manager decorators in the wild.

I went for the union type, up to you if you want to remove the type.

That's fine, we can always remove the type if anyone reports it as problematic.

Merged into 3.x, thanks!