-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generic bound lists #144
Comments
Thank you very much @dantheperson ! Genericity resolving is the most complicated thing in this project and I'm looking for the day where 100% of cases will be handled! I'll have a look how kongchen handled it. And I have to think about the warning / Object resolving. It's true that this error is quite ... definitive ... Thanks again ! 🙏 |
@dantheperson I know it's been a long time ... Can I abuse of your time a little more and ask you to try the current 0.0.22-SNAPSHOT (it require a local "install", like "mvn clean install -DskipTests" for example). I should have fixed your issue. |
that's on the |
Yes it is on dev. Thank you very much. 🙏 |
I have a controller endpoint that takes a Dto which extends a parentDto and bind the generic list to a concrete type.
class ParentDto<T extends Foo>{ List<T> items}
class ChildDto<ConcreteSubclassOfFoo> {}
this gives the dreaded
java.lang.RuntimeException: Type T (+TypeVariableImpl) is not supported yet.
I'm a bit stumped as to how this one should be resolved. The old plugin we are trying to migrate away from did work with this, so it should be possible.
I've created a PR with a Unit test to reproduce.
As an aside, as a temporary workaround I've changed it resolve to Object.class in this case. Perhaps this is a good default behaviour, log a warning and resolve to
Object
. Better to complete with 95% detail than to not complete at all?The text was updated successfully, but these errors were encountered: