"No item route associated with the type" exception using class inheritance

[vincentchalamon]

With class inheritance with ApiResource annotation on the parent (abstract) class, I get a No item route associated with the type... exception on the serialization of a related object through the IriConverter > RouteNameResolver.

PR in progress...

[Nek-]

#2714 fix it already isn't it?

[vincentchalamon]

@Nek- I though it too, but your PR is about ApiResource inheritance, right? In my case, it's only about routing.

[vincentchalamon]

But I agree it's linked

[Nek-]

@vincentchalamon AFAIK if the problem is fixed at the resource resolver level, it is also in routing, isn't it?

[vincentchalamon]

@Nek- I'm not sure about that, cause the RouteNameResolver compares strictly the resource class with the route resource class.

core/src/Bridge/Symfony/Routing/RouteNameResolver.php

Line 53 in df70878

if ($resourceClass === $currentResourceClass && null !== $operation && (empty($methods) || \in_array('GET', $methods, true))) {

This PR allows to compare using parent class.

[Nek-]

@vincentchalamon
I meant, by this change: https://github.com/api-platform/core/pull/2714/files#diff-2ec5bcaf896c1cc215d3c801a173c5f2R50

[vincentchalamon]

@Nek- I don't see the link with the RouteNameResolver? Or maybe it's the missing point in your PR: to call the ResourceClassResolver from the IriConverter before any call to the RouteNameConverter to be sure to handle the right resourceClass (i.e.: AbstractUser instead of InternalUser in my PR behat tests). WDYT?

[Nek-]

You're adding resource guess in the route name resolver because it's broken for your case. My PR fixes your case but unfortunately, there's still a bad usage in the AbstractItemNoramlizer after my commits. I took your tests and managed to make them pass with simple modifications using my modifications on the ResourceClassResolver.

I really think managing resource in only one place is the best.

Please see and comment if you need to: 6dcaee0

[teohhanhui]

Should be fixed by #2797. Please verify.