Skip to content

Conversation

@gavinking
Copy link
Member

@gavinking gavinking commented May 11, 2025

[Please describe here what your change is about]


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-19449

@hibernate-github-bot
Copy link

hibernate-github-bot bot commented May 11, 2025

Thanks for your pull request!

This pull request appears to follow the contribution rules.

› This message was automatically generated.

@gavinking gavinking marked this pull request as draft May 11, 2025 22:45
@gavinking gavinking force-pushed the HHH-19449 branch 2 times, most recently from cb48ca5 to 5fa4195 Compare May 12, 2025 00:27
@gavinking gavinking force-pushed the HHH-19449 branch 2 times, most recently from abbb758 to 36261a9 Compare May 12, 2025 02:42
gavinking added 7 commits May 12, 2025 06:23
seems we can just use getExpressible() which already calls getResolvedModel()
Remove BindableType.getBindableJavaType() which created all
sorts of confusion with JPAs Bindable.getBindableJavaType().
Now can just use Type.getJavaType() for this.

@Override
public ProcedureCall markAsFunctionCall(BindableType<?> typeReference) {
public ProcedureCall markAsFunctionCall(Type<?> typeReference) {

Check notice

Code scanning / CodeQL

Confusing overloading of methods Note

Method ProcedureCallImpl.markAsFunctionCall(..) could be confused with overloaded method
markAsFunctionCall
, since dispatch depends on static types.
and to reduce possible confusion with JPA Bindable
@gavinking gavinking marked this pull request as ready for review May 12, 2025 09:52
@gavinking gavinking changed the title HHH-19449 HHH-19449 allow use of static metamodel Type references to bind query parameters May 12, 2025
@gavinking gavinking merged commit 77fcc67 into hibernate:main May 12, 2025
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant